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-06-06 #43168

Closed
github-actions bot opened this issue Jun 6, 2024 · 27 comments
Closed

Deploy Checklist: New Expensify 2024-06-06 #43168

github-actions bot opened this issue Jun 6, 2024 · 27 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 Jun 6, 2024

Release Version: 1.4.80-17
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

@kavimuru kavimuru added the 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging label Jun 6, 2024
@OSBotify
Copy link
Contributor

OSBotify commented Jun 6, 2024

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

@rlinoz
Copy link
Contributor

rlinoz commented Jun 6, 2024

Checking off #43173, CP'd the fix to staging and it worked.

@francoisl
Copy link
Contributor

francoisl commented Jun 6, 2024

Revert fix for #43172 tested on staging, checking it off.

Edit: it also fixes #43175 and #43190, clearing those too.

Copy link

melvin-bot bot commented Jun 6, 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.

@roryabraham
Copy link
Contributor

#43181 is NAB, demoting.

@roryabraham
Copy link
Contributor

@amyevans
Copy link
Contributor

amyevans commented Jun 6, 2024

Closed #43212 as expected behavior

@luacmartins
Copy link
Contributor

Checking off #43214. Not a real bug

@kavimuru
Copy link

kavimuru commented Jun 7, 2024

Regression is 69% of progress
unchecked PRs:
#43030 production PR (help.expensify.com) checked off
#43052 - iOS validation is pending
#43027 failing with #43212
#42737 Desktop validation is pending
#42574 Web, mWeb, android and iOS validation are pending
#42554 iOS validation pending
#42498 validation pending in all the platforms
#42497 Desktop validation pending
#42387 Android validation pending
#42330 iOS validation pending

@marcaaron
Copy link
Contributor

Checking off #43215 as NAB

@MonilBhavsar
Copy link
Contributor

Checked off #43242, NAB

Copy link

melvin-bot bot commented Jun 7, 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.

@blimpich
Copy link
Contributor

blimpich commented Jun 7, 2024

Checking off #43207 because we reverted the PR that caused it.

@kavimuru
Copy link

kavimuru commented Jun 7, 2024

Regression is 10% left in the TR
Unchecked PRs:
#43027 failing with #43212
#42498 blocked in android with the issue #40978 reproduced in PROD checking it off.
#42554 failing with #43213

@luacmartins
Copy link
Contributor

Checking off #43222. Demoted to NAB.

This comment was marked as off-topic.

@luacmartins
Copy link
Contributor

Checking off #43238. Demoted to NAB

@rlinoz
Copy link
Contributor

rlinoz commented Jun 7, 2024

Checking off #43259, tested and it is fixed by a CP.

@luacmartins
Copy link
Contributor

Checking off #43243. It was demoted

@blimpich
Copy link
Contributor

blimpich commented Jun 7, 2024

Demoting #43296, NAB

@luacmartins
Copy link
Contributor

Checking off #43274. Expected behavior.

@luacmartins
Copy link
Contributor

Demoted #43318

@luacmartins
Copy link
Contributor

Checking off #43313. Fixed on staging

@melvin-bot melvin-bot bot added the Overdue label Jun 9, 2024
@mountiny
Copy link
Contributor

mountiny commented Jun 9, 2024

demoted #43335

@mountiny
Copy link
Contributor

mountiny commented Jun 9, 2024

Checked #42554 off as the raised issue was closed
Checked #43027 off for the same reason as above.

@mountiny mountiny added Daily KSv2 and removed Hourly KSv2 labels Jun 10, 2024
@melvin-bot melvin-bot bot removed the Overdue label Jun 10, 2024
@mountiny mountiny self-assigned this Jun 10, 2024
@mountiny
Copy link
Contributor

All cleared, retested the last CPed PR.

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