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-09-09 #48791

Closed
github-actions bot opened this issue Sep 9, 2024 · 18 comments
Closed

Deploy Checklist: New Expensify 2024-09-09 #48791

github-actions bot opened this issue Sep 9, 2024 · 18 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 Sep 9, 2024

Release Version: 9.0.31-26
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 the 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging label Sep 9, 2024
@luacmartins luacmartins self-assigned this Sep 9, 2024
@luacmartins luacmartins added the Daily KSv2 label Sep 9, 2024
@mvtglobally
Copy link

mvtglobally commented Sep 9, 2024

@luacmartins Android and IOS builds are crashing once updated. Getting the issue in a bit. but can you check if builds deployed properly?

@luacmartins
Copy link
Contributor

@mvtglobally pinged in Slack, but this should be solved now

@luacmartins
Copy link
Contributor

Checking off #48834. Demoted.

@luacmartins
Copy link
Contributor

#48817 also happens in prod for other data types. Demoting to NAB.

@luacmartins
Copy link
Contributor

#48831 demoted to NAb

@luacmartins
Copy link
Contributor

Demoted #48808

@luacmartins
Copy link
Contributor

Demoted #48803

@mvtglobally
Copy link

Regression is 35% left

Blocker
#48835

Copy link

melvin-bot bot commented Sep 10, 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

Demoting #48873

@luacmartins
Copy link
Contributor

Demoted #48882

@mvtglobally
Copy link

mvtglobally commented Sep 10, 2024

TR Regression is Completed

Blockers
#48835
#48864
#48865
#48866
#48867
#48883
#48894
#48902

PRs:
#45264 - Pending validation on all environments
#47468 - Pending validation on mWeb, Android and iOS
#47690 - Repro #48864. Checking off.
#47798 - Pending validation on Web, Desktop
#48137 - is failing on Desktop. Logging an issue
#48382 - Pending validation on iOS
#48524 - Pending validation on all environments
#48632 - Pass-bug on Web, mWeb with #48835 checking off
#48655 - Pending validation on mWeb, Desktop, Android and iOS
#48668 - Pending validation on all environments
#48706 - Pending validation on mWeb, Android
#48768 - Pending validation on mWeb, Desktop, Android and iOS
#47577 (comment) Can you confirm whats needed

@luacmartins
Copy link
Contributor

Demoted #48910

@luacmartins
Copy link
Contributor

#47577 (comment) Can you confirm whats needed

I think regression tests should be enough here

@luacmartins
Copy link
Contributor

luacmartins commented Sep 10, 2024

Backend issue - #48916. NAB though

@mvtglobally
Copy link

mvtglobally commented Sep 10, 2024

Blockers
#48865

PRs:
#45264 - Pending validation on Desktop & failing with #48916
#47468 - is failing on mWeb and Android. #48894
#47577 (comment) - Steps are no clear. We run regular TR TC for notifications. Can I check it off?
#47798 - Pending validation on Web, Desktop
#48137 - is failing on Desktop. Logging an issue
#48524 - Pending validation on all environments
#48661 - Pending validation on all environments
#48668 - is blocked by #48701
#48722 (comment) - What URL should we use for staging?
#48781 - waiting for build

@luacmartins
Copy link
Contributor

#47468 - checked off. We CPed a fix.
#47577 (comment) - yes, checked off if the regular steps for notification passed
#48668 - we're CPing a fix.
#48668 - we're CPing a fix
#48722 (comment) - no need to test this one, it affects dev only

@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

4 participants