-
Notifications
You must be signed in to change notification settings - Fork 3.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[No QA] Only post to slack when job was successful #6256
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
NAB: just a doubt, there is no message on slack on failure, right?
@aldo-expensify There is a message for the failure, yeah. It's higher up in the file: App/.github/workflows/platformDeploy.yml Lines 350 to 372 in 300decb
|
Cool, I missed it!, thanks |
I didn't see this show up when the successful message was displayed: Are we sure the fail message posts in #deployer? |
Well, according to that action, the fail message posts to #announce. |
✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release. |
🚀 Deployed to staging by @roryabraham in version: 1.1.14-5 🚀
|
🚀 Deployed to production by @AndrewGable in version: 1.1.15-15 🚀
|
Fixed Issues
$ https://github.com/Expensify/Expensify/issues/183630
Tests
The next time there is a deploy failure, we can verify there was no success message posted
QA Steps
None
cc @roryabraham