Skip to content

Slack notifications for test repair agent - #6523

Merged
evgenyrp merged 5 commits into
mozilla:masterfrom
evgenyrp:slack-notifications
Aug 7, 2026
Merged

Slack notifications for test repair agent#6523
evgenyrp merged 5 commits into
mozilla:masterfrom
evgenyrp:slack-notifications

Conversation

@evgenyrp

@evgenyrpevgenyrp commented Aug 5, 2026•
edited
Loading

Copy link
Copy Markdown
Contributor
  • add support for runtime following Phab pattern
  • send notifications to sheriffs' channel from the build repair agent

fixes#6510

@evgenyrpevgenyrp changed the title Slack notifications[WIP] Slack notificationsAug 5, 2026
suhaibmujahid
suhaibmujahid reviewed Aug 5, 2026
View reviewed changes

@suhaibmujahidsuhaibmujahid left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks nice and clean! Thank you!

Comment threadagents/test-repair/hackbot_agents/test_repair/__main__.py Outdated
Comment threadlibs/hackbot-runtime/hackbot_runtime/actions/handlers/slack_handler.py Outdated
Comment threadlibs/hackbot-runtime/hackbot_runtime/actions/handlers/slack_handler.py Outdated
@evgenyrp
evgenyrpforce-pushed the slack-notifications branch from e2ad0a6 to b6eec17CompareAugust 6, 2026 22:06
@evgenyrpevgenyrp mentioned this pull request Aug 6, 2026
Open
@evgenyrp
evgenyrpforce-pushed the slack-notifications branch from f2bf13e to b3c58c9CompareAugust 7, 2026 18:51
@evgenyrp
evgenyrpforce-pushed the slack-notifications branch from b3c58c9 to efffec1CompareAugust 7, 2026 20:02
@evgenyrpevgenyrp changed the title [WIP] Slack notificationsSlack notifications for test repair agentAug 7, 2026
@evgenyrp
evgenyrp marked this pull request as ready for review August 7, 2026 20:03
@evgenyrp
evgenyrp requested a review from a team as a code ownerAugust 7, 2026 20:03
suhaibmujahid
suhaibmujahid approved these changes Aug 7, 2026
View reviewed changes

@suhaibmujahidsuhaibmujahid left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good! Just minor comments.

Comment threadlibs/hackbot-runtime/hackbot_runtime/actions/handlers/slack_handler.py Outdated
Comment threadlibs/hackbot-runtime/hackbot_runtime/actions/slack.py Outdated
@evgenyrp

evgenyrp commented Aug 7, 2026

Copy link
Copy Markdown
ContributorAuthor

Done, I also changed the prompt so that the summary message is smaller

@evgenyrp
evgenyrp enabled auto-merge (squash) August 7, 2026 21:02
@evgenyrp
evgenyrp merged commit 12b6ec3 into mozilla:masterAug 7, 2026
8 checks passed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@suhaibmujahidsuhaibmujahidsuhaibmujahid approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Support Slack notifications

2 participants

@evgenyrp@suhaibmujahid