-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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 2022-09-09 #10916
Comments
👏 Heads up @Expensify/applauseleads 👏 |
Hey @Expensify/applauseleads, I manually added the lock label because we have an important release today. Please take a look at this release checklist when ready :) |
Checking off 10855 #10855 (comment) |
Checking off 10763 #10763 (comment) |
We are 40% through regression. |
@mvtglobally yeah 👍 |
Checked off #10925 as it's not always reproducible, and isn't a major problem as users can still chat to concierge. |
I also checked off #10927 as it's not reproducible by three internal employees. |
If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results. If a regression has occurred and you are the assigned CM follow the instructions here. If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future. |
Given that the release is pretty much clear, I was going to deploy for @yuwenmemon. This would have given Applause some time to run tests on a new release candidate before the US gets online. Unfortunately, when checking the Firebase console I noticed that this issue is a new regression with a lot of reported crashes that we do not want to release, even if it is not always reproducible. I'm unable to pick this up this morning, but I shared it in Slack here. |
👏 Heads up @Expensify/applauseleads 👏 |
🎉 All set?…You bet! @Expensify/applauseleads https://github.com/Expensify/App/releases/tag/1.1.99-3 has been deployed to staging 🎉 |
A PR was just merged to resolve #10876, but the release failed 😭 Looking into this now. |
👏 Heads up @Expensify/applauseleads 👏 |
Okay, this one seemed to be successful. Builds are in progress and should be released within an hour 🎉 |
🎉 All set?…You bet! @Expensify/applauseleads https://github.com/Expensify/App/releases/tag/1.1.99-4 has been deployed to staging 🎉 |
The iOS build was hanging forever so I canceled it. So in short, 1.1.99-4 is deployed everywhere except iOS. I'm rerunning the canceled action now Stuck/Canceled iOS build: https://github.com/Expensify/App/actions/runs/3038990393/jobs/4893392499 |
@francoisl |
The app doesn't crash on Android and iOS when testing #10937, so all the blockers are resolved now. It's getting late for me right now and I need to go, so either @Julesssss or anyone else can this checklist when you get online if you feel like it, otherwise I'll do it first thing tomorrow morning! |
I rechecked Firebase and closed the Crash reports linked to the final blocker. Preparing to ship |
Release Version:
1.1.99-4
Compare Changes: production...staging
This release contains changes from the following pull requests:
main
for checklist tests #10842window.location.href
for current URL" #10908Deploy Blockers:
updateWorkspaceCustomUnit
#10876Deployer verifications:
cc @Expensify/applauseleads
The text was updated successfully, but these errors were encountered: