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-23 #12976

Closed
38 tasks done
OSBotify opened this issue Nov 23, 2022 · 22 comments
Closed
38 tasks done

Deploy Checklist: New Expensify 2022-11-23 #12976

OSBotify opened this issue Nov 23, 2022 · 22 comments
Assignees
Labels
Daily KSv2 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging StagingDeployCash

Comments

@OSBotify
Copy link
Contributor

OSBotify commented Nov 23, 2022

Release Version: 1.2.31-8
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.

cc @Expensify/applauseleads

@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.2.31-3 🚀

@mvtglobally mvtglobally added the 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging label Nov 24, 2022
@OSBotify
Copy link
Contributor Author

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

@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.31-3 🚀

@luacmartins
Copy link
Contributor

#13011 is reproducible on prod, so I'm removing the blocker label

@aldo-expensify
Copy link
Contributor

#13008 is reproducible in prod, removed the blocker label

@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.2.31-6 🚀

@OSBotify
Copy link
Contributor Author

🎉 All set?…You bet! @Expensify/applauseleads https://github.com/Expensify/App/releases/tag/1.2.31-6 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.2.31-7 🚀

@OSBotify
Copy link
Contributor Author

🎉 All set?…You bet! @Expensify/applauseleads https://github.com/Expensify/App/releases/tag/1.2.31-7 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.2.31-8 🚀

@luacmartins
Copy link
Contributor

#12936 is a pass. Checking off.

@OSBotify
Copy link
Contributor Author

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

@Julesssss
Copy link
Contributor

I removed #12999 as a deploy blocker as three people are unable to reproduce.

@luacmartins
Copy link
Contributor

@mvtglobally what's the regression tests status?

@mvtglobally
Copy link

@luacmartins we are wrapping up. I'll should be able to share summary in the next hour or so

@mvtglobally
Copy link

mvtglobally commented Nov 25, 2022

@luacmartins Regression is completed

PR #12762 is failing #13017 Repro in PROD
PR #12919 is failing #13000. blocker is fixed. we are retesting
need help here #12607 (comment)
PR #12419. "Design 6 could not be tested without valid amounts when connecting VBA manually KI #12670" All buttons/links work as expected. Let me know if its ok to check it off
We are checking few last CPs. I will have update in a bit.

@luacmartins
Copy link
Contributor

luacmartins commented Nov 25, 2022

Waiting on the CP updates and the retest of #12919!

@luacmartins
Copy link
Contributor

@mvtglobally any update on the CP checks?

@mvtglobally
Copy link

@luacmartins All checked off.

#12919 Steps are pass, but there is an error in the console after refresh. Can you just double check its ok.

Recording.1781.1.mp4

@luacmartins
Copy link
Contributor

@mvtglobally it seems to be the same error as in #12943 and seems unrelated to the changes in that PR, so I think it's safe to check off. Thanks for checking!

@luacmartins
Copy link
Contributor

I see a new crash on iOS which seems to be a memory management issue with the OS. Considering that we only had one crash, and no actionable steps to fix this, I'll go ahead with 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

6 participants