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-15 #36558

Closed
73 tasks done
github-actions bot opened this issue Feb 15, 2024 · 29 comments
Closed
73 tasks done

Deploy Checklist: New Expensify 2024-02-15 #36558

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

Comments

@github-actions
Copy link
Contributor

github-actions bot commented Feb 15, 2024

Release Version: 1.4.42-5
Compare Changes: production...staging

This release contains changes from the following pull requests:

Internal QA:

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

@thienlnam thienlnam added the 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging label Feb 15, 2024
@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.42-0 🚀

@Julesssss
Copy link
Contributor

I verified the InternalQA issue here. Checking off.

@Beamanator
Copy link
Contributor

Reducing #36575 to NAB status

@Beamanator
Copy link
Contributor

Also demoting #36576 since there's nothing broken, and this is hopefully not a high traffic page

@Beamanator
Copy link
Contributor

Similarly, demoting #36577 b/c it's weird but not necessarily a blocker

@Julesssss
Copy link
Contributor

Checking off blocker #36595 as it's a minor issue

@Beamanator
Copy link
Contributor

Beamanator commented Feb 15, 2024

Demoting #36601 since it's a Troubleshooting action

@Beamanator
Copy link
Contributor

Also demoted #36598 as it's a very minor issue

@kavimuru
Copy link

Regression is at 70%. Unchecked PRs are pending validation.

@kavimuru
Copy link

kavimuru commented Feb 16, 2024

Regression is 6% left.
#34080 is failing in mweb chrome with #36597 and repro in PROD checking it off
#35294 is failing with #36651 in web and repro in PROD checking it off
#34719 is pending validation in all the platforms but Web
#35306 is pending validation in Desktop
#36304 is pending validation in Desktop
Open blockers:
#36574
#36599
#36603
#36643
#36645
#36646
#36647

@marcaaron
Copy link
Contributor

Checking off #36645 as it does not seem urgent

@kavimuru
Copy link

Regression is 1% left.

#34719 is failing with KI #33540 (reopened the issue) OK to check it off?
#35306 is failing with #36601
#36304 is failing with #36602 and repro in PROD checking it off
Open blockers:
#36574
#36599
#36603
#36643
#36646
#36678
#36687
#36695

@kavimuru
Copy link

kavimuru commented Feb 16, 2024

Regression is completed

#34719 is failing with KI #33540 (reopened the issue) OK to check it off?
#35306 is failing with #36601
Open blockers:
#36574
#36599
#36603
#36643
#36678
#36687
#36695
#36703
#36705

@Beamanator
Copy link
Contributor

#36687 - couldn't reproduce, so marked NAB - it's also a very tiny bug and only on the troubleshooting page

@Beamanator
Copy link
Contributor

#35306 is failing with #36601

We decided the issue isn't a blocker, so I checked off the PR too

@Beamanator
Copy link
Contributor

Calling #36703 NAB since it's a backend issue & isn't critical, more of a nice to have

@Beamanator
Copy link
Contributor

#36599 and #36695 just got fixed by #36631

@Beamanator
Copy link
Contributor

Checked off #36643 b/c it got closed as expected behavior

@melvin-bot melvin-bot bot added the Monthly KSv2 label Feb 19, 2024
@chiragsalian
Copy link
Contributor

#34719. Checking it off. From this comment it looks like we're good to proceed.

@Expensify/applauseleads

#36705 - it looks like the PR to fix the issue was CP-d. From the checklist the CP'd PR was checked off by @kbecciv so I'm assuming they tested it and it ran well and since it was linked to the issue #36705 which has the same steps as the PR i'm going to check it off the list.

@kbecciv
Copy link

kbecciv commented Feb 19, 2024

@chiragsalian QA team checked CP #36798 and it passed, checked off.

@kbecciv
Copy link

kbecciv commented Feb 19, 2024

CP #36810 is Pass, checking off

@chiragsalian
Copy link
Contributor

Look like this issue - #36574, has a PR that was CP-d. It looks like you checked off the PR so i guess you tested it. Can you check if that means we're good to check off its issue too i.e., #36574.

Let me know if there is anything else pending on our end.

@kbecciv
Copy link

kbecciv commented Feb 19, 2024

@kavimuru
Copy link

kavimuru commented Feb 19, 2024

@chiragsalian #36574 can be checked off, CP #36810 is validated and it is a pass.
Only DB is #36678

@chiragsalian
Copy link
Contributor

Yup i asked about #36678 in the earlier comment here. We are unable to reproduce it. Can you or someone else retest this to confirm if the issue still persists.

@kavimuru
Copy link

@chiragsalian checking

@kavimuru
Copy link

kavimuru commented Feb 19, 2024

@chiragsalian not able to reproduce #36678 by the tester who reported and me.
#36678 (comment)

@chiragsalian
Copy link
Contributor

Cool, i checked off #36574 and #36678

@chiragsalian
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 Monthly KSv2 StagingDeployCash
Projects
None yet
Development

No branches or pull requests

9 participants