Skip to content

Deploy Checklist: New Expensify 2025-05-13 #61917

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

Open
github-actions bot opened this issue May 13, 2025 · 18 comments
Open

Deploy Checklist: New Expensify 2025-05-13 #61917

github-actions bot opened this issue May 13, 2025 · 18 comments
Assignees
Labels
🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging StagingDeployCash

Comments

@github-actions
Copy link
Contributor

github-actions bot commented May 13, 2025

Release Version: 9.1.45-18
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 for this release version and verified that this release does not introduce any new crashes. More detailed instructions on this verification can be found here.
  • I checked Firebase Crashlytics for the previous release version and verified that the release did 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

@gleitebr gleitebr added the 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging label May 13, 2025
@francoisl francoisl self-assigned this May 13, 2025
@francoisl
Copy link
Contributor

FYI there's an issue with the Testflight again so you won't have any 9.1.45-0 available for iOS yet. We're looking into it, I'll post an update when it's resolved.

@francoisl
Copy link
Contributor

francoisl commented May 13, 2025

Testflight builds are available again.


Checking 61934/61924 - fix tested internally
#61948 dupe - closed
#61928/#61955 - fix tested on web
#61964/#61961 - fix tested
#61958 – demoted as NAB and probably backend
#61914/#61938/#61921 – fix tested on web
#61956 was demoted – happening on production too
#61967/#61915 – fix tested on staging (still repro with beta tableReportView but out of scope)

@rlinoz
Copy link
Contributor

rlinoz commented May 13, 2025

Demoted #61926

@joekaufmanexpensify
Copy link
Contributor

Checking off #61370 as it is behind beta

@blimpich
Copy link
Contributor

Demoted #61965

@blimpich
Copy link
Contributor

QA-ed #61523

@lakchote
Copy link
Contributor

Demoted #61999 for this reason.

@trjExpensify
Copy link
Contributor

Demoted the issues below for this reason.

#61987
#61997
#61998
#62001
#62006

@kavimuru
Copy link

Regression is 4% left in the TR

#61691 (comment)
#61665 (comment)
#61661 failing with #62017 repro in PROD checking it off.
#61543 failing with #61940 on android
#60505 Failing with #61954 and #61932
#60192 failing with #61921

@kavimuru
Copy link

@francoisl the following PRs are missing in the checklist now. They were in the list yesterday though.
Could you check please?

#60956
#61069
#61339
#61509
#61601
#61617
#61661
#61663
#61680

@joekaufmanexpensify
Copy link
Contributor

Checked off #61666

@mountiny

This comment has been minimized.

@arosiclair
Copy link
Contributor

Demoted #62031

@francoisl
Copy link
Contributor

francoisl commented May 14, 2025

Demoting #62021
#61932/#61989 - A revert was CPed
#62024/#61973/#62012/#62014 - Revert was CPed, fixes multiple blockers
#61940/#61984 - Fixed, revert was CPed
#62019/#61927
#61963 - same behavior happens on production
#61903/#61895/#61977 - fix confirmed on staging
#62023/#61986 - fix tested on staging
#62042 was demoted
#62004 - confirmed on staging
#61971/#61942 - tested on staging
#61975 - confirmed on staging
#61972 - no longer repro
#61970 - no longer repro

@trjExpensify
Copy link
Contributor

Demoting #62033 for this reason.

@francoisl
Copy link
Contributor

Following up on Vit's comment above (I marked it as hidden to avoid confusion because it included a few incorrect PRs), here's a list of PRs that were missing from the checklist :( I added them to the OP, but we'll unfortunately need to run QA on them before we can close the checklist

#61727
#61722
#61680
#61676
#61663
#61661
#61658
#61657
#61627
#61623
#61621
#61620
#61617
#61601
#61597
#61592
#61585
#61580
#61467
#61447
#61378
#61357
#61339
#61319
#61312
#61297
#61237
#61162
#61137
#61069
#61045
#60956
#60902
#60696
#60352
#60023
#59804
#59568
#59565

@kavimuru
Copy link

Regression is just 1 step left.

Unchecked PRs:
#60023 iOS validation is pending
#60192 failing with #61921
#60292 pending validation on all the platforms
#60505 Failing with #61954 and #61932
#60959 (comment) reverted ok to check it off?#61362 failing with #61944 and #61362 (comment)
#61376 failing with #61968
#61529 pending validation on desktop android iOS and failing on web and mweb with this bug id #55349
#61543 failing with #61940 on android
#61665 (comment)
#61691 (comment)
#61760 failing with #61942 and #61953
#61764 failing with #62036 but repro in production checking it off
#61809 pending verification on all the platforms
#61840 validation pending in all the platforms
#61661 failing with #62017 repro in PROD checking it off
#61657 failing with original bug id #61657 (comment)
#61621 validation pending on Web mweb android and iOS
#61617 pending validation on desktop
#61580 pending validation on desktop and ios
#61467 pending validation on all the platforms
#61339 pending validation on all the platforms but iOS
#61297 Pending validation on desktop and iOS
#61237 Pending validation on desktop and iOS
#61045 Pending validation on desktop and iOS and android
#60902 pending validation on all the platforms
#60696 (comment)
#60352 pending validation on desktop android and iOS
#59804 iOS validation is pending
#59568 Pending validation on desktop and iOS
#59565 Pending validation on desktop and iOS

@francoisl
Copy link
Contributor

francoisl commented May 15, 2025

#60192 failing with #61921

On what platform are you still experiencing the crash? I'm unable to repro on web and desktop


#60505 Failing with #61954 and #61932

the original PR was reverted, checking off

#60959 (comment) reverted ok to check it off?

yes – I'm checking it off

#61376 failing with #61968

Can't repro on mweb, checking off

#61543 failing with #61940 on android

Previously retested myself, and can't repro anymore after new retest, checking off

#61657 failing with original bug id #61657 (comment)

PR was reverted, checking off

--

New blockers:

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