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-11-04 #12476

Closed
20 tasks done
OSBotify opened this issue Nov 4, 2022 · 34 comments
Closed
20 tasks done

Deploy Checklist: New Expensify 2022-11-04 #12476

OSBotify opened this issue Nov 4, 2022 · 34 comments
Assignees
Labels
🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging Monthly KSv2 StagingDeployCash

Comments

@OSBotify
Copy link
Contributor

OSBotify commented Nov 4, 2022

Release Version: 1.2.24-4
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 Nov 4, 2022
@roryabraham
Copy link
Contributor

Love these shorter checklists, let's keep up with daily deploys 🚀

@OSBotify
Copy link
Contributor Author

OSBotify commented Nov 4, 2022

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

@mvtglobally
Copy link

@roryabraham did Android build go through?

@Julesssss
Copy link
Contributor

@roryabraham did Android build go through?

Nope, but we're currently investigating.

@mvtglobally
Copy link

Let us know once the build issue is resolved. Android is the only outstanding item to close out this regression

@Julesssss
Copy link
Contributor

Hi @mvtglobally, we're working on this and you should see an automated message here once it's deployed 🤞

@OSBotify
Copy link
Contributor Author

OSBotify commented Nov 7, 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.24-1 🚀

@mvtglobally
Copy link

@Julesssss This new build did not push Android update. We still see 1.2.23-9

@mvtglobally
Copy link

@Julesssss We just got the build, but its not working. App is crashing. Need a bug or you guys already aware?

Screen_Recording_20221107-111743_Google.Play.Store.mp4

@Julesssss
Copy link
Contributor

No, that wasn't reproducible during testing. Looking into this now 😕

@Julesssss
Copy link
Contributor

Created an issue here, will keep you updated: #12522

@OSBotify
Copy link
Contributor Author

OSBotify commented Nov 7, 2022

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

@Julesssss
Copy link
Contributor

@mvtglobally another fix is incoming 🤞

@yuwenmemon should be able to help with any further issues as I'm signing off now

@OSBotify
Copy link
Contributor Author

OSBotify commented Nov 7, 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.24-2 🚀

@OSBotify
Copy link
Contributor Author

OSBotify commented Nov 7, 2022

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

@mvtglobally
Copy link

@yuwenmemon Did Android build go through Google? We are not seeing a 1.2.24-2 version yet

@yuwenmemon
Copy link
Contributor

@mvtglobally Unfortunately it did not - we're working on it.

@OSBotify
Copy link
Contributor Author

OSBotify commented Nov 7, 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.24-3 🚀

@OSBotify
Copy link
Contributor Author

OSBotify commented Nov 7, 2022

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

@OSBotify
Copy link
Contributor Author

OSBotify commented Nov 7, 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.24-4 🚀

@OSBotify
Copy link
Contributor Author

OSBotify commented Nov 7, 2022

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

@yuwenmemon
Copy link
Contributor

@mvtglobally We should be good to go now! Sorry for the delay

@mvtglobally
Copy link

we are running regression. Should have results in a bit

@mvtglobally
Copy link

@yuwenmemon @Julesssss Regression is completed

PR #12034 is blocked on IOS and Android with #12480
PR #12279 is blocked by #11796 on IOS and Android and blocked by #12308 on mWeb and Desktop
PR #12296 is failing #12492. We were not able to repro on PROD, but it seems to be there as well. pls double check
Checking 2 last CPs

@melvin-bot melvin-bot bot added the Monthly KSv2 label Nov 8, 2022
@yuwenmemon
Copy link
Contributor

@mvtglobally can we test #12487 and check it off if it passes?

@yuwenmemon
Copy link
Contributor

Checking off #12296 because I was able to reproduce on prod and everything else from that PR works fine.

@mvtglobally
Copy link

Asking team to re-test

@yuwenmemon
Copy link
Contributor

Checking off #12034 because it pertains to a non-mobile flow anyway - and #12480 is actively being worked on and also present on prod.

@yuwenmemon
Copy link
Contributor

@mvtglobally any update on how that testing went?

@mvtglobally
Copy link

@yuwenmemon We are not able to repro on Web. Checking on other platforms

@yuwenmemon
Copy link
Contributor

Nice, I'm cautiously optimistic then 😄

@mvtglobally
Copy link

@yuwenmemon issue is no longer repro across platforms

@yuwenmemon
Copy link
Contributor

Sweet! Can we close and check off that last issue?

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

6 participants