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-05-29 #19785

Closed
44 tasks done
OSBotify opened this issue May 29, 2023 · 19 comments
Closed
44 tasks done

Deploy Checklist: New Expensify 2023-05-29 #19785

OSBotify opened this issue May 29, 2023 · 19 comments
Assignees
Labels
Daily KSv2 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging StagingDeployCash

Comments

@OSBotify
Copy link
Contributor

OSBotify commented May 29, 2023

Release Version: 1.3.20-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

@mvtglobally mvtglobally added the 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging label May 30, 2023
@OSBotify
Copy link
Contributor Author

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

@OSBotify
Copy link
Contributor Author

👏 Heads up @Expensify/applauseleads 👏
A new pull request has been 🍒 cherry-picked 🍒 to staging, and will be deployed to staging in version 1.3.20-2 🚀

@OSBotify
Copy link
Contributor Author

🎉 All set?…You bet! @Expensify/applauseleads https://github.com/Expensify/App/releases/tag/1.3.20-2 has been deployed to staging 🎉

@Julesssss
Copy link
Contributor

Checked off the only current internal QA issue: #19739

@Julesssss Julesssss self-assigned this May 30, 2023
@Julesssss Julesssss added the Daily KSv2 label May 30, 2023
@Julesssss
Copy link
Contributor

A PR was CP'd to resolve the two blockers.

@OSBotify
Copy link
Contributor Author

👏 Heads up @Expensify/applauseleads 👏
A new pull request has been 🍒 cherry-picked 🍒 to staging, and will be deployed to staging in version 1.3.20-3 🚀

@OSBotify
Copy link
Contributor Author

🎉 All set?…You bet! @Expensify/applauseleads https://github.com/Expensify/App/releases/tag/1.3.20-3 has been deployed to staging 🎉

@OSBotify
Copy link
Contributor Author

👏 Heads up @Expensify/applauseleads 👏
A new pull request has been 🍒 cherry-picked 🍒 to staging, and will be deployed to staging in version 1.3.20-4 🚀

@OSBotify
Copy link
Contributor Author

🎉 All set?…You bet! @Expensify/applauseleads https://github.com/Expensify/App/releases/tag/1.3.20-4 has been deployed to staging 🎉

@mvtglobally
Copy link

mvtglobally commented May 31, 2023

@Julesssss Regression is completed

PR #18577 is failing #19852. pls double check as we cant repro in prod.
PR #19295 is failing #19813
PR #19390 is failing #19855
PR #19491 is failing #19863. repro in PROD checking off
PR #19521 is blocked on Desktop. rest platforms are pass. OK to check it off?
PR #19579 is failing #19814
PR #19582 is failing #18918
PR #19610 is failing #19857. repro in PROD checking off
PR #19639 is failing #19848. repro in PROD checking off
PR #19818 & #18293 is failing #19856

@OSBotify
Copy link
Contributor Author

👏 Heads up @Expensify/applauseleads 👏
A new pull request has been 🍒 cherry-picked 🍒 to staging, and will be deployed to staging in version 1.3.20-5 🚀

@OSBotify
Copy link
Contributor Author

🎉 All set?…You bet! @Expensify/applauseleads https://github.com/Expensify/App/releases/tag/1.3.20-5 has been deployed to staging 🎉

@Julesssss
Copy link
Contributor

Julesssss commented May 31, 2023

Thanks @mvtglobally. I've reviewed all of these and think the only actionable issues are the three below. Could you please check my comments for these three? I think we're good to deploy but just want to confirm the following points:


Remaining questions

PR #19521 is blocked on Desktop. rest platforms are pass. OK to check it off?

A) Is testing blocked? It seems okay in the contributors video, so I think we're good ✅

PR #19579 is failing #19814

B) 🚧 - We CP'd this fix. And I can confirm it's working now on staging-- could you please retest though?

PR #19582 is failing #18918

C) ❓is that the correct link? It shows a two-week old issue


Checked items

PR #18577 is failing #19852. pls double check as we cant repro in prod.

✅ - Not a blocker, but we'll need to fix the issue you raised

PR #19295 is failing #19813

✅ - Not a blocker

PR #19390 is failing #19855

✅ - Dupe of this one, but it's also not a blocker

PR #19491 is failing #19863. repro in PROD checking off

PR #19610 is failing #19857. repro in PROD checking off

PR #19639 is failing #19848. repro in PROD checking off

PR #19818 & #18293 is failing #19856

✅ - Nice find and we'll fix this, but it's not a blocker

@Julesssss
Copy link
Contributor

  • Blockers are cleared
  • App timing shows some slow down over the past few weeks, but this is likely due to AddComment regression -- we should keep an eye on this
  • No new Firebase exceptions on iOS or Android

@mvtglobally
Copy link

@Julesssss
We are good with #19521. Testers retested and it can be checked off
#19579 - retesting
#19582 this is good to check off. Old issue still repro

@Julesssss
Copy link
Contributor

That's great, thank you

@mvtglobally
Copy link

@Julesssss #19579 is pass
#19866 is pass on all but Desktop. Can you check Desktop on your side? It will take me a bit longer to validate

@Julesssss
Copy link
Contributor

Okay, so I'm unable to verify on Desktop. But I think this is fine and not a blocker. The single desktop app issue is far less important than having the feature live in staging for our Render ATL demo.

I'm going to check off #19866 and leave a note on the issue

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