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-03 #22116

Closed
40 tasks done
OSBotify opened this issue Jul 3, 2023 · 22 comments
Closed
40 tasks done

Deploy Checklist: New Expensify 2023-07-03 #22116

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

Comments

@OSBotify
Copy link
Contributor

OSBotify commented Jul 3, 2023

Release Version: 1.3.36-4
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 3, 2023
@Julesssss Julesssss self-assigned this Jul 3, 2023
@Julesssss
Copy link
Contributor

Hey @mvtglobally, there was one small issue with the iOS build and checklist generation. It seems okay now, but let me know if you see anything odd!

@OSBotify
Copy link
Contributor Author

OSBotify commented Jul 3, 2023

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

@Li357
Copy link
Contributor

Li357 commented Jul 3, 2023

Fix for #22123 is here: #22132

@dangrous
Copy link
Contributor

dangrous commented Jul 3, 2023

Fix for #22130 is in review - #22135

EDIT: It pleases me how close together these numbers are

@Julesssss
Copy link
Contributor

I'm not sure why the description wasn't updated, but a fix Cp has bumped the current version to 1.3.36-2

@Julesssss
Copy link
Contributor

I cherry-picked the fixes for the remaining blockers.

Were now on 1.3.36-4 -- I'll manually update it now

@Julesssss
Copy link
Contributor

I have to sign off, and plan to deploy early tomorrow morning to give Applause plenty of time to work on the follow-up release checklist

@mvtglobally
Copy link

PR #21557 is failing #21952
PR #21700 is failing #22175. Repro in prod

@mvtglobally
Copy link

@Julesssss can you pls confirm what CPs were added for us to validate? I am not seeing the checklist updated with new PRs

@mvtglobally
Copy link

mvtglobally commented Jul 5, 2023

@Julesssss Regression is completed

#22135 - Pass
#22132 - Pass

PR #21557 is failing #21952
PR #21662 is failing #21662. repro in PROD, checking off
PR #21834 (comment). Can you check it internally?
PR #21895 is failing #22234. Repro in PROD

@Julesssss
Copy link
Contributor

Checked off the above items, and verified #21834 manually

@Julesssss
Copy link
Contributor

Julesssss commented Jul 5, 2023

@mvtglobally
Copy link

@Julesssss We have to re-test one of the TCs on Android. Will update in a few min once the retest is completed

@Julesssss
Copy link
Contributor

No problem. I'll be back online in about 45 mins

@mvtglobally
Copy link

@Julesssss We are done, no new blockers, so we are good to proceed

@Julesssss
Copy link
Contributor

Perfect. Thanks!

@Julesssss
Copy link
Contributor

Checked off #22258 after deploying the fix. @mvtglobally in order to release I'm going to verify myself, FYI

@Julesssss
Copy link
Contributor

Julesssss commented Jul 5, 2023

  • The final blocker is merged and has been tested against an AdHoc build (arguable it's not even worth blocking over)
  • timing looking good
  • raised an unrelated OldDot crash here
  • Single instance of a new ANR
  • One non-fatal iOS issue here
  • Github is up

@Julesssss
Copy link
Contributor

@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

6 participants