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 2023-11-20 #31556

Closed
90 tasks done
os-botify bot opened this issue Nov 20, 2023 · 35 comments
Closed
90 tasks done

Deploy Checklist: New Expensify 2023-11-20 #31556

os-botify bot opened this issue Nov 20, 2023 · 35 comments
Assignees
Labels
Daily KSv2 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging StagingDeployCash

Comments

@os-botify
Copy link

os-botify bot commented Nov 20, 2023

Release Version: 1.4.1-13
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

Copy link

melvin-bot bot commented Nov 20, 2023

📣 @os-botify[bot]! 📣
Please report bugs or suggest features in the #expensify-open-source Slack channel, don't directly open issues in this repo!
Instructions here to join the channel 📖

@os-botify os-botify bot reopened this Nov 20, 2023
@os-botify
Copy link
Author

os-botify bot commented Nov 20, 2023

Sorry, only members of @Expensify/Mobile-Deployers can close deploy checklists.
Reopening!

Copy link

melvin-bot bot commented Nov 20, 2023

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

@kavimuru kavimuru added the 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging label Nov 20, 2023
@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.1-4 🚀

@luacmartins luacmartins self-assigned this Nov 20, 2023
@luacmartins luacmartins added the Daily KSv2 label Nov 20, 2023
@roryabraham
Copy link
Contributor

roryabraham commented Nov 20, 2023

#31570 is expected behavior, checking it off

edit: it's not here right now due to some bugs with our CI this morning. I'll use our new tool to refresh the checklist.

@luacmartins
Copy link
Contributor

Checking off #31567 since we CPed a fix

@mvtglobally
Copy link

mvtglobally commented Nov 20, 2023

Regression is 65% complete

PR #30568 is failing with #31570
PR #30679 is failing with #31583. Repro in prod Checking off
PR #31519 is failing with #30284. Reproducible in PrOD, checking off

@luacmartins
Copy link
Contributor

Checking off #31581. Demoted to NAB

@luacmartins
Copy link
Contributor

Checking off #31580. We reverted the offending PR.

@mvtglobally
Copy link

Regression is 5% left

PR #30568 is failing with #31570
PR #30898 is failing with #31568. Repro in prod, checking off
#31036 PROD QA. Are we ok to check off?
#31398 PROD QA. Are we ok to check off?
PR #31465 is failing with #31597 Repro in prod, checking off
PR #31522 seems to be still failing with #31595

@Julesssss
Copy link
Contributor

I manually added #31610 as a blocker as the CI seemed to fail

@kbecciv
Copy link

kbecciv commented Nov 21, 2023

Regression is 1% left

PR #30568 is failing with #31570
#31036 and #31398 PROD QA. Are we ok to check off?
PR #31522 is failing with #31595, repro in prod, check off

Open blockers:
#31578
#31592
#31610
#31612

@rlinoz
Copy link
Contributor

rlinoz commented Nov 21, 2023

Manually added #31617 as a blocker

@kbecciv
Copy link

kbecciv commented Nov 21, 2023

Regression is completed!

PR #30568 is failing with #31570, issue closed. Are we ok to check off?
#31036 and #31398 PROD QA. Are we ok to check off?
PR #31522 is failing with #31595, repro in prod, check off

Open blockers:
#31578
#31592
#31610
#31612
#31617

@amyevans
Copy link
Contributor

amyevans commented Nov 21, 2023

Checking off #31580 after CP

Edit: Looks like Carlos already checked it off 😄

@rlinoz
Copy link
Contributor

rlinoz commented Nov 21, 2023

Checking off #31617, although it is a bug it is not a blocker.

@luacmartins
Copy link
Contributor

Checking off #31610 and #31581 since we're CPing a revert.

@Julesssss
Copy link
Contributor

Checking off #31610 as we reverted and CPd the PR that introduced the issue.

@luacmartins
Copy link
Contributor

Checking off #31592. The issue exists in previous versions.

@kbecciv
Copy link

kbecciv commented Nov 21, 2023

PR #30568 is failing with #31570, issue closed. Are we ok to check off?
#31036 and #31398 PROD QA. Are we ok to check off?

Open blockers:
#31578
#31612
#31623
#31625
#31628
#31639
#31654

@luacmartins
Copy link
Contributor

Checking off #31623 since we CPed a fix

@luacmartins
Copy link
Contributor

Checking off #31612. Demoted to NAB since that's an issue with all form providers and we have a PR with a fix.

@luacmartins
Copy link
Contributor

Checking off #31639. Demoted to NAB.

@luacmartins
Copy link
Contributor

Checking off #31628. CPed a fix

@luacmartins
Copy link
Contributor

Checking off #31654. We CPed a fix.

@luacmartins
Copy link
Contributor

Checking off #31036. Prod QA.

@luacmartins
Copy link
Contributor

Checking off #31398. HelpDot copy changes

@luacmartins
Copy link
Contributor

Checking off #31658. Tested in staging and it works.

@luacmartins
Copy link
Contributor

Checking off #31578. Demoted to NAB.

@luacmartins
Copy link
Contributor

Checking off #31659. Tested in staging and it's a pass

@luacmartins
Copy link
Contributor

Checking off #31625. We CPed a fix.

@luacmartins
Copy link
Contributor

Down to the last CP now before we can deploy

@luacmartins
Copy link
Contributor

Checking off #31663. Tested on staging and it works!

@luacmartins
Copy link
Contributor

Checklist is clear. Kicking off the deploy.

@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

10 participants