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 2022-10-31 #12305

Closed
22 tasks done
OSBotify opened this issue Oct 31, 2022 · 29 comments
Closed
22 tasks done

Deploy Checklist: New Expensify 2022-10-31 #12305

OSBotify opened this issue Oct 31, 2022 · 29 comments
Assignees
Labels
🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging StagingDeployCash

Comments

@OSBotify
Copy link
Contributor

OSBotify commented Oct 31, 2022

Release Version: 1.2.22-3
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.

cc @Expensify/applauseleads

@mvtglobally mvtglobally added the 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging label Oct 31, 2022
@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.2.22-1 🚀

@mvtglobally
Copy link

We are still missing IOS build.

@Julesssss Julesssss assigned Julesssss and applausebot and unassigned applausebot Nov 1, 2022
@Julesssss
Copy link
Contributor

We are still missing IOS build.

I'm looking into this after reverting the issue that introduced a regression 👍

@OSBotify
Copy link
Contributor Author

OSBotify commented Nov 1, 2022

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

@Julesssss
Copy link
Contributor

Checking off #12312, as the tests in #12336 will confirm the regression is resolved

@OSBotify
Copy link
Contributor Author

OSBotify commented Nov 1, 2022

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

@mountiny
Copy link
Contributor

mountiny commented Nov 1, 2022

⏫ @Expensify/applauseleads can you please retest this one deploy blocker? #12312 Thanks

@Julesssss
Copy link
Contributor

Hey @Expensify/applauseleads, I just cherry-picked a fix for the failed iOS build and it should appear soon 🤞

@mvtglobally
Copy link

Great @Julesssss we will resume as soon as build is out

@OSBotify
Copy link
Contributor Author

OSBotify commented Nov 1, 2022

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

@OSBotify
Copy link
Contributor Author

OSBotify commented Nov 1, 2022

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

@luacmartins
Copy link
Contributor

@mvtglobally what is the status on the regression tests?

@mvtglobally
Copy link

@luacmartins almost done. We are wrapping last iOS slots. I will update checklist in a bit

@mvtglobally
Copy link

@luacmartins Regression is completed

PRs #11586, #11875, #11907 are blocked on Android due to #12312. We are retesting
PR #12150 is failing due to #12335. Repro in prod, checking off
PR #12195 is failing #12328. Repro in prod. Checking off

@Julesssss
Copy link
Contributor

Julesssss commented Nov 2, 2022

Hey @Expensify/applauseleads

@Julesssss
Copy link
Contributor

I checked off #11907 as the PR was reverted

@Julesssss
Copy link
Contributor

Checking off #12137 as it is not testable in staging

(test only in production).

@mvtglobally
Copy link

@Julesssss @luacmartins Checking off #11586, #11875, #11907, #12336 as retest pass
PR #12125 is fail on IOS #12355. Repro in prod checking off
We are all set

@Julesssss
Copy link
Contributor

Amazing, thank you!

@Julesssss
Copy link
Contributor

Timing dash has no new changes, we're still seeing higher times due to a new measurement method.

@Julesssss
Copy link
Contributor

I cleared out some outdated issues in Firebase. The single new issue on Firebase is a one off as far as I can tell. I am unable to reproduce it based on the stack trace. We can investigate further if/when it occurs again

@Julesssss
Copy link
Contributor

:shipit:

@Julesssss
Copy link
Contributor

Hey @Expensify/applauseleads. FYI we had an issue generating the new checklist.

@mvtglobally
Copy link

@Julesssss i was just about to ping on the same. Any idea if the fix is coming soon?

@Julesssss
Copy link
Contributor

That seems very unlikely at this point. I'll share an update tomorrow as it's near the end of my day.

@OSBotify OSBotify reopened this Nov 2, 2022
@OSBotify
Copy link
Contributor Author

OSBotify commented Nov 2, 2022

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

@Julesssss
Copy link
Contributor

Oh, I triggered this accidently, please ignore!

@OSBotify OSBotify reopened this Nov 2, 2022
@OSBotify
Copy link
Contributor Author

OSBotify commented Nov 2, 2022

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

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

7 participants