Skip to content

feat: send Slack notification to authors after paasta rollback#4312

Open
sidtuladhar wants to merge 7 commits into
masterfrom
u/sidtuladhar/PAASTA-18881/slack-ping
Open

feat: send Slack notification to authors after paasta rollback#4312
sidtuladhar wants to merge 7 commits into
masterfrom
u/sidtuladhar/PAASTA-18881/slack-ping

Conversation

@sidtuladhar
Copy link
Copy Markdown
Contributor

@sidtuladhar sidtuladhar commented May 11, 2026

Pings the git authors whose commits are being rolled back via the service's configured Slack channel, improving visibility when a rollback occurs.

Screenshot 2026-05-11 at 15 02 55

Pings the git authors whose commits are being rolled back via the
service's configured Slack channel, improving visibility when a
rollback occurs.
@sidtuladhar sidtuladhar requested a review from a team as a code owner May 11, 2026 13:55
@sidtuladhar sidtuladhar requested a review from nemacysts May 11, 2026 14:03
@sidtuladhar
Copy link
Copy Markdown
Contributor Author

let me know if you'd like the message to be different @nemacysts

mbrankin-art
mbrankin-art previously approved these changes May 11, 2026
Comment thread paasta_tools/cli/cmds/rollback.py
Comment thread paasta_tools/cli/cmds/rollback.py Outdated
Comment thread paasta_tools/cli/cmds/rollback.py Outdated
Comment thread paasta_tools/cli/cmds/rollback.py Outdated
Comment thread paasta_tools/cli/cmds/rollback.py Outdated
Comment thread paasta_tools/cli/cmds/rollback.py Outdated
Comment thread paasta_tools/cli/cmds/rollback.py Outdated
Comment thread paasta_tools/cli/cmds/rollback.py
nemacysts
nemacysts previously approved these changes May 15, 2026
Comment thread paasta_tools/cli/cmds/rollback.py
Comment thread paasta_tools/cli/cmds/rollback.py
Comment thread paasta_tools/cli/cmds/rollback.py Outdated
Copy link
Copy Markdown
Member

@nemacysts nemacysts left a comment

Choose a reason for hiding this comment

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

one quick comment, but otherwise lgtm!

Comment thread paasta_tools/cli/cmds/rollback.py Outdated
Co-authored-by: Luis Pérez <luisp@yelp.com>
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.

3 participants