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-02-13 #36429

Closed
github-actions bot opened this issue Feb 13, 2024 · 20 comments
Closed

Deploy Checklist: New Expensify 2024-02-13 #36429

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

Comments

@github-actions
Copy link
Contributor

github-actions bot commented Feb 13, 2024

Release Version: 1.4.41-12
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

@mountiny mountiny added the 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging label Feb 13, 2024
@blimpich
Copy link
Contributor

Demoting 36434

@amyevans
Copy link
Contributor

Demoted #36437

@OSBotify
Copy link
Contributor

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

@Beamanator
Copy link
Contributor

Beamanator commented Feb 14, 2024

Marked #36456 as NAB b/c we consider it a pretty strange edge case

@Beamanator
Copy link
Contributor

Beamanator commented Feb 14, 2024

Also marked #36455 as NAB, it's not really broken - and it's a test that we'll be updating over the next few weeks

@Beamanator
Copy link
Contributor

#36458 also got marked nab as it's quite a small design bug that will be quick to fix

@kavimuru
Copy link

kavimuru commented Feb 14, 2024

Regression is 4% left.
#31055 failing with #36455
#34752 failing with #36479
#34498 failing with #36449, #36451, #36447 and blocked in desktop with KI #36368
#34303 failing with #36433 on desktop
We still have validation pending on other unchecked PRs

@Julesssss
Copy link
Contributor

Checking off #36502 as it's not a big problem. Will fix without it being a blocker

@dangrous
Copy link
Contributor

#36487 is not a blocker and is sort of a dupe of #36455. Checking off.

@kavimuru
Copy link

kavimuru commented Feb 14, 2024

Regression completed.

Unchecked PRs:
#31055 failing with #36455
#34752 failing with #36479
#34498 failing with #36449,#36451,#36447 and blocked in desktop with KI36368
#34303 failing with #36433 on desktop
#34963 blocked by #36479
#35287 blocked by #36433
#35841 failing with #36434 but closed as not reproduced, checking it off
#35853 blocked by #36433
#36204 blocked by #36433 in iOS (tester does not have an account with previous requests to validate the PR. Because PR steps does not require a new IOU)
#35645 - Looks internal testing, can I check it off?

Open Blockers:
#36433
#36346
#36438
#36454
#36457
#36479
#36481
#36483
#36490
#36510
#36517

We are validating the 2 cps now.

@rlinoz
Copy link
Contributor

rlinoz commented Feb 14, 2024

Checking off #36510 as this is already in prod.

@amyevans
Copy link
Contributor

amyevans commented Feb 14, 2024

Checked off #36454 following CP/retest

@mountiny
Copy link
Contributor

checking #36479 off, its backend blocker

@AndrewGable
Copy link
Contributor

Checked off #36517, it was fixed on staging by this CP https://github.com/Expensify/App/actions/runs/7909722028

@thienlnam
Copy link
Contributor

Only fresh crash is for
ImageMetaData.getColorSpace - which only happened for 1 user on 1.4.38-6 so it seems fine to ignore

@thienlnam
Copy link
Contributor

Kicking off the deploy so we can get this fireroom fix to prod

@thienlnam
Copy link
Contributor

:shipit:

@os-botify
Copy link

os-botify bot commented Feb 15, 2024

This issue either has unchecked items or has not yet been marked with the :shipit: emoji of approval.
Reopening!

@os-botify os-botify bot reopened this Feb 15, 2024
Copy link

melvin-bot bot commented Feb 15, 2024

📣 @os-botify[bot]! 📣
Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork.
Please follow these steps:

  1. Make sure you've read and understood the contributing guidelines.
  2. Get the email address used to login to your Expensify account. If you don't already have an Expensify account, create one here. If you have multiple accounts (e.g. one for testing), please use your main account email.
  3. Get the link to your Upwork profile. It's necessary because we only pay via Upwork. You can access it by logging in, and then clicking on your name. It'll look like this. If you don't already have an account, sign up for one here.
  4. Copy the format below and paste it in a comment on this issue. Replace the placeholder text with your actual details.
    Screen Shot 2022-11-16 at 4 42 54 PM
    Format:
Contributor details
Your Expensify account email: <REPLACE EMAIL HERE>
Upwork Profile Link: <REPLACE LINK HERE>

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