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 2024-03-12 #38162

Closed
84 tasks done
github-actions bot opened this issue Mar 12, 2024 · 24 comments
Closed
84 tasks done

Deploy Checklist: New Expensify 2024-03-12 #38162

github-actions bot opened this issue Mar 12, 2024 · 24 comments
Assignees
Labels
Daily KSv2 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging StagingDeployCash

Comments

@github-actions
Copy link
Contributor

github-actions bot commented Mar 12, 2024

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

@luacmartins luacmartins added 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging Daily KSv2 labels Mar 12, 2024
@luacmartins luacmartins self-assigned this Mar 12, 2024
@luacmartins
Copy link
Contributor

Checking off #32336. This PR was reverted.

@OSBotify
Copy link
Contributor

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

@kavimuru
Copy link

#38057 is a PROD PR can we check it off?

@luacmartins
Copy link
Contributor

Yes, checked it off, thanks!

@luacmartins
Copy link
Contributor

Checking off #38172. Also happens in prod.

@luacmartins
Copy link
Contributor

Checking off #38185. Demoted to NAB

@luacmartins
Copy link
Contributor

Checking off #38182. Tested the CP on staging and it fixed all blockers it was supposed to.

@luacmartins
Copy link
Contributor

Checking off #38183. Demoted to NAB

@luacmartins
Copy link
Contributor

Checking off #38174. Demoted to NAB

@Tony-MK
Copy link
Contributor

Tony-MK commented Mar 13, 2024

Issue #38171 is handled and a fix has been deployed.

We should presumably check it off. 🤔

@luacmartins
Copy link
Contributor

Checking off #38171. Tested in staging and it's fixed.

@mountiny
Copy link
Contributor

#37998 QAed internally

@kavimuru
Copy link

@luacmartins Regression is down to 8%
unchecked PRs:
#37629 failing with #37629
#37664 failing with #38212 repro in PROD, checking it off
#37113 failing with #38201
#37755 failing with #38188 closed with this comment, checking it off
#38032 web and desktop pending validation and other platforms are pass
#37595 being validated

@luacmartins
Copy link
Contributor

Checking off #37405. Demoted to NAB

@kavimuru
Copy link

kavimuru commented Mar 13, 2024

Regression is 2% left in the TR

unchecked PRs:
#37629 failing with #37629
#37113 failing with #38201
#38032 failing with #38241 and reprod in PROD, cheecking it off

Open Blockers:
#38213 and #38214

@luacmartins
Copy link
Contributor

Checking off #37629, since we'll tackle that in #38214.

@kavimuru is this link correct? #37113 that PR was closed and I don't see it in the checklist

@mkhutornyi
Copy link
Contributor

is this link correct? #37113 that PR was closed and I don't see it in the checklist

#37713 is the correct PR

@kavimuru
Copy link

@luacmartins, @mkhutornyi was correct, Sorry it was my mistake

@tgolen
Copy link
Contributor

tgolen commented Mar 13, 2024

Removed #38213 from being a blocker and checked it off.

@luacmartins
Copy link
Contributor

Ah cool, so I'm checking off #37713 since we confirmed that's the expected behavior, we just need to update the error message.

@luacmartins
Copy link
Contributor

@kavimuru is regression done?

@kavimuru
Copy link

@luacmartins Yes we are done.

@luacmartins
Copy link
Contributor

Checklist is clear. 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

8 participants