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-08-15 #47512

Closed
74 tasks done
github-actions bot opened this issue Aug 15, 2024 · 24 comments
Closed
74 tasks done

Deploy Checklist: New Expensify 2024-08-15 #47512

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

Comments

@github-actions
Copy link
Contributor

github-actions bot commented Aug 15, 2024

Release Version: 9.0.21-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

@Beamanator Beamanator added the 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging label Aug 15, 2024
@OSBotify
Copy link
Contributor

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

@grgia
Copy link
Contributor

grgia commented Aug 16, 2024

demoted #47551 as it's internal

@mvtglobally
Copy link

Regression
Crit/High TCs - 95% completed
Med/Low TCs - 0% completed

PRs:
#46324 - Failing with #45408
#46635 is failing with #47557
#46939 is failing with KI #46816. repro in PROD, checking off
#47028 - Failing with #46336
#47091 - Pass-bug only on iOS with #47550. checking off
#47092 - Pending validation on all platforms
#47144 - Pass-bug on Android/iOS only with known issue #46228 checking off
#47243 - Pending validation on IOS
#47288 is failing #47572
#47340 - Pending validation on all platforms
#47368 & #47397 seem to be conflicting. Which one should we QA?
#47450 - Failing with original KI #44829 on Web/mWeb/Desktop. Pending validation on IOs and Android
#47527 - Pending validation on all platforms

Blockers
#47524
#47525
#47538
#47553
#47557

@Beamanator
Copy link
Contributor

#47538 was demoted

@Beamanator
Copy link
Contributor

#47557 was closed as NAB 👍

@Beamanator
Copy link
Contributor

Beamanator commented Aug 16, 2024

#47368 & #47397 seem to be conflicting. Which one should we QA?

I commented in #47397 that we should QA #47397, NOT #47368 👍 so I'll check #47368 off the list

@mvtglobally
Copy link

Regression
Crit/High TCs - 100% completed
Med/Low TCs - 70% completed

@Beamanator
Copy link
Contributor

#47525 marked NAB (probably b/c it was fixed in #47527 which was CP'd

@Beamanator
Copy link
Contributor

Checking off #46635 b/c its blocker was marked NAB

@Beamanator
Copy link
Contributor

Similarly, checking off #46324 b/c its blocker (#47538) was marked nAB

@Beamanator
Copy link
Contributor

Checking off #47288 b/c the bug it's failing with (#47572) is reproducible in prod

@mvtglobally
Copy link

Regression
Crit/High TCs - 100% completed
Med/Low TCs - 95% completed

@mvtglobally
Copy link

Regression is completed

PRs:
#47028 - Failing with #46336
#47092 - Can you pls QA internally?
#47340 - Pending validation on Web
#47555 - Pending validation on all platforms

Blockers
#47524
#47553
#47601

@Beamanator
Copy link
Contributor

#47553 fixed by #47555

  • Only checking off the deploy blocker b/c the PR is also supposed to fix a 2nd blocker

@Beamanator
Copy link
Contributor

#47524 was also fixed by #47555 - checking both off!

@Beamanator
Copy link
Contributor

Beamanator commented Aug 19, 2024

#47028 doesn't look like it caused a new bug, it just didn't completely solve the bug it was trying to fix - so we can check it off

  • I commented in the issue, asking if there was any bigger issue found, but it doesn't look like it

@Beamanator
Copy link
Contributor

Just saw that #47092 can be checked off!

@Beamanator
Copy link
Contributor

Checking off #47450 as well b/c it doesn't cause new bugs, it just didn't quite fix the original bug & notification preference structure (in Onyx) is currently being updated anyway

@Beamanator
Copy link
Contributor

QA verified in slack here that #47397 passed on all platforms! checking it off!

@Beamanator
Copy link
Contributor

Beamanator commented Aug 19, 2024

#47340 confirmed working in web & all other platforms here

@Beamanator
Copy link
Contributor

#47562 was demoted, checking it off

@luacmartins
Copy link
Contributor

Checking #47601 off. Fixed in staging!

@Beamanator
Copy link
Contributor

Looking good! shipping!

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