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-08-31 #26359

Closed
69 tasks done
OSBotify opened this issue Aug 31, 2023 · 12 comments
Closed
69 tasks done

Deploy Checklist: New Expensify 2023-08-31 #26359

OSBotify opened this issue Aug 31, 2023 · 12 comments
Assignees
Labels
Daily KSv2 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging StagingDeployCash

Comments

@OSBotify
Copy link
Contributor

OSBotify commented Aug 31, 2023

Release Version: 1.3.60-3
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

@luacmartins luacmartins self-assigned this Aug 31, 2023
@luacmartins luacmartins added the Daily KSv2 label Aug 31, 2023
@mvtglobally mvtglobally added the 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging label Aug 31, 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.60-1 🚀

@roryabraham
Copy link
Contributor

I see why this is happening. It's a race condition between:

  1. createNewStagingDeployCash
  2. deployStaging

Both are triggered at the same time after updateStaging finishes. If deployStaging finishes (i.e: the tag for the new staging version is pushed to the remote) before the checklist is created, then it works. If not, then we get a huge checklist like this will all the PRs that were deployed in the previous version.

Obviously not good, will need to fix this ASAP.

@luacmartins
Copy link
Contributor

Checking off #26413. Demoted to NAB since it happens in prod.

@luacmartins
Copy link
Contributor

Checking off #26421. Not one of our officially supported platforms. Demoted to NAB

@mvtglobally
Copy link

mvtglobally commented Aug 31, 2023

@luacmartins Regression is completed

#25971 is blocked on Native apps as we cant upload PDF. Ok to check off?
#26022 is failing #26407
Can we check this one off?

@luacmartins
Copy link
Contributor

Checking off #26407. Happens in prod.

@luacmartins
Copy link
Contributor

Checking off #26417 - we merged and CPed a fix.

@luacmartins
Copy link
Contributor

Checking off #26441. Demoted to NAB.

@luacmartins
Copy link
Contributor

Checklist is clear. Just waiting for the final 2 CPs to finish and test them in staging and then we're good to go.

@luacmartins
Copy link
Contributor

Ok, CPs are done and they are a pass on staging. Shipping it.

@luacmartins
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

5 participants