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-10-31 #51778

Closed
github-actions bot opened this issue Oct 31, 2024 · 21 comments
Closed

Deploy Checklist: New Expensify 2024-10-31 #51778

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

Comments

@github-actions
Copy link
Contributor

github-actions bot commented Oct 31, 2024

Release Version: 9.0.56-9
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 Oct 31, 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.56-0 🚀

@grgia
Copy link
Contributor

grgia commented Oct 31, 2024

Demoted #51799

@kavimuru
Copy link

kavimuru commented Nov 1, 2024

Regression is 28% left
Unchecked PRs:
Android Hybrid app is not available yet. So all the android slots are still pending.
#51390 is reverted. Should we verify this PR?

@Beamanator
Copy link
Contributor

@kavimuru do you have access to the latest android hybrid app by now? I think i see that y'all should have access to 9.0.56.2

@Beamanator
Copy link
Contributor

checking off #51792 as expected, not a bug

@izarutskaya
Copy link

Regression is at 90 % completed

PRs:
#49536 - Failing with #51801
#51390 - KI found #51812
#51407 - Failing with #51819
#51422 - Found KI #51825
#51645 - KI #51312
#51663 - Failing with #51797

Blockers:
#51784
#51786
#51788
#51789
#51797
#51813
#51819
#51823

@kavimuru
Copy link

kavimuru commented Nov 1, 2024

@Beamanator Hybrid app is updated and we are working on android slots too.

@kavimuru
Copy link

kavimuru commented Nov 1, 2024

Regression completed

Unchecked PRs:
#49536 - Failing with #51801
#50698 - Failing with #51577 and #50194 KI is not fixed in all the platforms
#51390 failing with #50563 #51390 (comment)
#51117 - is blocked in Desktop with #10107 (comment) Ok to check it off
#51217 failing with #51872
#51407 - Failing with #51819
#51663 - Failing with #51797
Blockers:
#51784
#51786
#51788
#51789
#51797
#51813
#51823
#51853
#51854
#51858
#51860
#51871
#51872
#51885

@Beamanator
Copy link
Contributor

Checking off #51784, #51788, and #51786 as fixed by #51795

@Beamanator
Copy link
Contributor

Checkin off #51797 as fixed by #51848

@Beamanator
Copy link
Contributor

Checking off #51789 as fixed by #51943

@Julesssss
Copy link
Contributor

Checking off #51885 as not a blocker

@Julesssss
Copy link
Contributor

Checking off #51871.

@Julesssss
Copy link
Contributor

Also downgrading #51872, Invoicing is a brand new feature

@Julesssss
Copy link
Contributor

Downgrading #51860 as it's an incredibly minor issue.

@Julesssss
Copy link
Contributor

#51853 was fixed by #51879

@Julesssss
Copy link
Contributor

#51813 is also being downgraded as it is a minor UI issue, and will be fixed by a separate PR that is in progress.

@Julesssss
Copy link
Contributor

I'm just awaiting testing on this issue, which should have been fixed by a CP. Details below on the others I checked off:

#49536 - Failing with #51801

Issue fixed.

#50698 - Failing with #51577 and #50194 KI is not fixed in all the platforms

Issue was closed as will not fix.

#51390 failing with #50563 #51390 (comment)

Marked not blockers

#51117 - is blocked in Desktop with #10107 (comment) Ok to check it off

Yep I agree with closing as expected behaviour

#51217 failing with #51872

Issue was marked not a blocker

#51407 - Failing with #51819

Issue was closed as dupe

#51663 - Failing with #51797

Fixed.

@Julesssss
Copy link
Contributor

Final blocker marked not-reproducable 🎉

@Julesssss
Copy link
Contributor

  • Firebase is looking good:
    • The worse case is 14 crashes over a 7-day period, but this existed in previous releases
  • App timing nominal
  • Github ✅

@Julesssss
Copy link
Contributor

:shipit:

@Julesssss Julesssss self-assigned this Nov 6, 2024
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

7 participants