Skip to content
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

Deploy Checklist: New Expensify 2023-07-21 #23362

Closed
OSBotify opened this issue Jul 21, 2023 · 13 comments
Closed

Deploy Checklist: New Expensify 2023-07-21 #23362

OSBotify opened this issue Jul 21, 2023 · 13 comments
Assignees
Labels
Daily KSv2 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging StagingDeployCash

Comments

@OSBotify
Copy link
Contributor

OSBotify commented Jul 21, 2023

Release Version: 1.3.44-0
Compare Changes: production...staging

This release contains changes from the following pull requests:

Internal QA:

Deploy Blockers:

Deployer verifications:

  • I checked the App Timing Dashboard and verified this release does not cause a noticeable performance regression.
  • I checked Firebase Crashlytics and verified that this release does not introduce any new crashes. More detailed instructions on this verification can be found here.
  • I checked GitHub Status and verified there is no reported incident with Actions.

cc @Expensify/applauseleads

@mvtglobally mvtglobally added the 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging label Jul 21, 2023
@OSBotify
Copy link
Contributor Author

🚀 All staging deploys are complete, @Expensify/applauseleads please begin QA on version https://github.com/Expensify/App/releases/tag/1.3.44-0 🚀

@jasperhuangg jasperhuangg self-assigned this Jul 21, 2023
@mvtglobally
Copy link

Regression is almost completed

Will update the summary in the morning

PR #22851 is failing #23407
PR #22891 (comment) needs assistance
PR #22917 prod QA are we ok to check it off?
PR #23246 is failing #23410

@mountiny
Copy link
Contributor

Checking the internalQA #23110 off as Anu has tested this with the C+ payments

@mountiny
Copy link
Contributor

checking #23189 and #23250 off as the CPes have been deployed to staging and the issues are not reproducible anymore

@mvtglobally
Copy link

Regression is completed. No additional comments

@marcaaron marcaaron self-assigned this Jul 24, 2023
@marcaaron
Copy link
Contributor

Checking off #22891 because passwordless is 100% rolled out and I think the code from that PR is basically "dead" now.

@marcaaron
Copy link
Contributor

#22851 is on prod so we can check it off

@marcaaron
Copy link
Contributor

PR #22917 prod QA are we ok to check it off?

yes - checked off

@marcaaron
Copy link
Contributor

marcaaron commented Jul 24, 2023

Just waiting for #23487 to get tested and then we can:

Edit: We just decided to drop the blocker for this one. Reasoning in this thread.

@jasperhuangg jasperhuangg removed their assignment Jul 24, 2023
@marcaaron
Copy link
Contributor

Removed the blocker label from #23412. It seems like a performance regression, but inconsistently reproducible for everyone.

@melvin-bot melvin-bot bot added the Monthly KSv2 label Jul 24, 2023
@marcaaron marcaaron added Daily KSv2 and removed Monthly KSv2 labels Jul 24, 2023
@arosiclair
Copy link
Contributor

#23358 passes on staging and fixes #23416

@marcaaron
Copy link
Contributor

Just down to the [Deleted message] bug now. Ready to send it after that.

@marcaaron
Copy link
Contributor

:shipit:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Daily KSv2 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging StagingDeployCash
Projects
None yet
Development

No branches or pull requests

7 participants