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-05 #22282

Closed
58 tasks done
OSBotify opened this issue Jul 5, 2023 · 18 comments
Closed
58 tasks done

Deploy Checklist: New Expensify 2023-07-05 #22282

OSBotify opened this issue Jul 5, 2023 · 18 comments
Assignees
Labels
🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging StagingDeployCash

Comments

@OSBotify
Copy link
Contributor

OSBotify commented Jul 5, 2023

Release Version: 1.3.37-7
Compare Changes: production...staging

This release contains changes from the following pull requests:

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 5, 2023
@OSBotify
Copy link
Contributor Author

OSBotify commented Jul 5, 2023

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

@mvtglobally
Copy link

#21962 is failing with #21678 . Checking off

@Julesssss Julesssss self-assigned this Jul 6, 2023
@Julesssss
Copy link
Contributor

Closed #22302 as a dupe of #22289

@Beamanator
Copy link
Contributor

Checking off #21874 since it got reverted in #22303

@Julesssss
Copy link
Contributor

#22289 was fixed by this PR, which I am about to cherry-pick

@Julesssss
Copy link
Contributor

Checked off #22297 because it is not a blocker and also isn't always reproducible

@mvtglobally
Copy link

@Julesssss seeing comments about CPs, but checklist is not updating. Can you pls tag me if we need to QA any specific CPs

Regression is at 60% complete. still running through TC and PRs

@Julesssss
Copy link
Contributor

Julesssss commented Jul 6, 2023

Yeah, I think this is going to be a persistent problem in the short-term. I'll update once the build is complete.

And to clarify, are these the manual steps we should be doing to make your job simpler?

  • Bump the version after the build is available
  • Add the Cherry-picked PR to the bottom of the checklist
  • Add a comment to document the change

@Julesssss
Copy link
Contributor

Fix is in progress for #22300

@Julesssss
Copy link
Contributor

#22308 was also resolved by the #22323 CP

@Julesssss
Copy link
Contributor

Julesssss commented Jul 6, 2023

Checking off #22340 as it's not worth blocking the release on. A fix PR was just merged, but CPing it feels unnecessary due to the low importance and the delay it adds (preventing me from a hypothetical release this afternoon)

@Julesssss
Copy link
Contributor

  • Located one new Android exception that might have been introduced here. It's a wild guess though and is not a blocker
  • No new iOS exceptions 🎉
  • Timing looks nominal

@mvtglobally
Copy link

Regression is completed

#20186 is repro old KI while validating #16696. Checking off
#21312 is repro old KI #20370. Checking off
#21857 (comment)

@Julesssss
Copy link
Contributor

Julesssss commented Jul 7, 2023

Checking off two blockers (22351 & 22183), with the introduction of these PRs:

@Julesssss
Copy link
Contributor

Checked off #21857 👍

The final CP has been completed and I'm now reviewing the CP'd PRs myself manually (everything at bottom of the PRs list)

@Julesssss
Copy link
Contributor

I finished checking off the CP'd fixes and re-ran the developer verififications ✅

@Julesssss
Copy link
Contributor

:shipit:

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

No branches or pull requests

5 participants