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-07-08 #44947

Closed
86 tasks done
github-actions bot opened this issue Jul 8, 2024 · 34 comments
Closed
86 tasks done

Deploy Checklist: New Expensify 2024-07-08 #44947

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

Comments

@github-actions
Copy link
Contributor

github-actions bot commented Jul 8, 2024

Release Version: 9.0.5-13
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

@Julesssss Julesssss self-assigned this Jul 8, 2024
@mountiny mountiny added the 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging label Jul 8, 2024
@mountiny
Copy link
Contributor

mountiny commented Jul 8, 2024

Not sure why this is listed #43036 as the PR was reverted in the last release

@mountiny
Copy link
Contributor

mountiny commented Jul 8, 2024

checked off #44801 and #44818 as they were old deploy blockers

@Julesssss
Copy link
Contributor

Not sure why this is listed #43036 as the PR was reverted in the last release

Yeah odd.

@OSBotify
Copy link
Contributor

OSBotify commented Jul 8, 2024

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

@Julesssss
Copy link
Contributor

Wow, big delay on that message 😕 We're up to 9.0.5-3 already

Copy link

melvin-bot bot commented Jul 8, 2024

⚠️ Looks like this issue was linked to a Deploy Blocker here

If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results.

If a regression has occurred and you are the assigned CM follow the instructions here.

If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.

@Julesssss
Copy link
Contributor

#44959 resolved with #44973

@Julesssss
Copy link
Contributor

Checking off #44975 as not-a-blocker

@kbecciv
Copy link

kbecciv commented Jul 8, 2024

TR is 45% left!

#44947 is pending verification on Desktop and IOS
#43036 - left comment #43036 (comment)
#43153 is failing with #42552 on IOS app, left comment #43153 (comment)
#43822 is failing with #44965
#44221 is pending verification on Desktop and IOS app
#44229 is pending verification on IOS app
#44391 is pending verification on IOS app
#44529 is pending verification on Desktop and IOS
#44711 is failing with #44197, left comment #44711 (comment)
#44734 is pending verification on IOS app
#44743 is pending verification on Desktop and IOS
#44808 is pending verification on IOS
#44819 is pending verification on IOS
#44893 is failing with #44977, checking off per comment #44977 (comment)
#44902 is pending verification on IOS app
#44944 asked #44944 (comment)
CP #44981 is pending verification

Deploy Blockers:
#44957
#44960
#44962
#44963
#44966
#44969
#44991
#44992
#45001

@blimpich
Copy link
Contributor

blimpich commented Jul 8, 2024

Demoting #44991

@mountiny
Copy link
Contributor

mountiny commented Jul 8, 2024

demoted #45010

@mountiny
Copy link
Contributor

mountiny commented Jul 8, 2024

@thienlnam demoted #45009

@Beamanator
Copy link
Contributor

#45049 CP'd which confirmed fixed blockers #44969 and #44966, checking those 3 off

@Julesssss
Copy link
Contributor

#44988 fixes #44963

@Julesssss
Copy link
Contributor

#44962 resolved by a different fix, checking off

@rlinoz
Copy link
Contributor

rlinoz commented Jul 9, 2024

checking off #45062, can't repro in latest version

@kbecciv
Copy link

kbecciv commented Jul 9, 2024

TR is completed!

#41962 is failing with #45004
#43036 left comment #43036 (comment)
#43153 is failing with #42552 on IOS app, left comment #43153 (comment)
#43822 is failing with #44965
#44221 is failing with #43851, left comment #44221 (comment)
#44612 - left comment #44612 (comment)
#44711 is failing with #44197, left comment #44711 (comment)
#44893 is failing with #44977, checking off per comment #44977 (comment)
#44944 asked #44944 (comment)

Deploy Blockers:
#44960
#44992
#45013
#45060

@rlinoz
Copy link
Contributor

rlinoz commented Jul 9, 2024

Unchecking #45062 because we have new reproduction steps #45062 (comment)

@Julesssss
Copy link
Contributor

Hey @kbecciv, could you please retest #45056? It should be resolved in version 9.0.5-8

@kbecciv
Copy link

kbecciv commented Jul 9, 2024

@Julesssss Checking #45056

@Julesssss
Copy link
Contributor

I am demoting #44992 due to reasons discussed here

@Julesssss
Copy link
Contributor

#45081 Cp'd to resolve #45062

@kbecciv
Copy link

kbecciv commented Jul 9, 2024

@Julesssss Issue isn't reproducible, left comment #45056 (comment)

@Julesssss
Copy link
Contributor

Checking off #41962 as the failure isn't a blocker

@Julesssss
Copy link
Contributor

Checking off #44612

@Julesssss
Copy link
Contributor

@Julesssss
Copy link
Contributor

#45078 is not a blocker

Four blockers remaining. I have bumped each assignee in Slack

@rlinoz
Copy link
Contributor

rlinoz commented Jul 9, 2024

Checking off ##44960, tested and it is fixed by #45084

@luacmartins
Copy link
Contributor

Checked off #45013. Demoted to NAB.

@Julesssss
Copy link
Contributor

Julesssss commented Jul 10, 2024

@Julesssss
Copy link
Contributor

^ the above iOS crash has occured for many previous versions (I had filtered for the current version only). Shipping

@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

9 participants