Skip to content

hackbot: add a try-server push action - #6531

Draft
suhaibmujahid wants to merge 2 commits into
mozilla:masterfrom
suhaibmujahid:worktree-try-push-action-6528
Draft

hackbot: add a try-server push action#6531
suhaibmujahid wants to merge 2 commits into
mozilla:masterfrom
suhaibmujahid:worktree-try-push-action-6528

Conversation

@suhaibmujahid

@suhaibmujahid suhaibmujahid commented Aug 6, 2026

Copy link
Copy Markdown
Member

Resolves #6528

Depends on mozilla-conduit/lando#1408

Adds a configurable `user_agent` setting to `LandoSettings` (defaulting to `Lando-User/Hackbot`) and includes it as a `User-Agent` header in try-patch POST requests.
Comment on lines +23 to +34
tasks: Annotated[
list[str],
Field(
description=(
"Treeherder task labels to run, e.g. ['build-linux64/opt', "
"'test-linux2404-64/opt-mochitest-browser-chrome-1']. Only the "
"tasks that actually exercise your change: every extra label "
"spends build machine time. Must not be empty — there is no "
"'run everything' shorthand."
)
),
],

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Might be better to have the agent select tests and not tasks, or use "mach try auto" (maybe in combination with agent's selected tests).

Comment on lines +71 to +73
verify all your edits first, then call this once you are done. Calling it
records the push as a proposed action for review; nothing is pushed during
the run, so you will not see the results — a human reads them on Treeherder.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is until we have a way to have hackbot continue after a try push (or other action). Are you planning to do that after as a follow-up?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, I'm planning to do that. I will file some issues in that regard.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add a new agent action to push to try server

2 participants