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-04-17 #17546

Closed
53 tasks done
OSBotify opened this issue Apr 17, 2023 · 25 comments
Closed
53 tasks done

Deploy Checklist: New Expensify 2023-04-17 #17546

OSBotify opened this issue Apr 17, 2023 · 25 comments
Assignees
Labels
Daily KSv2 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging StagingDeployCash

Comments

@OSBotify
Copy link
Contributor

OSBotify commented Apr 17, 2023

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

@mountiny mountiny added the Daily KSv2 label Apr 17, 2023
@mountiny mountiny self-assigned this Apr 17, 2023
@mvtglobally mvtglobally added the 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging label Apr 18, 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.1-0 🚀

@mvtglobally
Copy link

@mountiny ios app is still not updated

@mountiny
Copy link
Contributor

@mvtglobally correct, it was rejected by apple, there is a CP running no which should hopefully resolve that

@mvtglobally
Copy link

Thanks for checking @mountiny . We are running other platforms for now. Checklist is enormous this time. 😜

@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.1-1 🚀

@MelvinBot
Copy link

⚠️ Looks like this issue was linked to a Deploy Blocker here

If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results.

If a regression has occurred and you are the assigned CM follow the instructions here.

If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.

@OSBotify
Copy link
Contributor Author

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

@OSBotify
Copy link
Contributor Author

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

@mountiny
Copy link
Contributor

Checking #17601 off, not a deploy blocker

#17580 revert was CPed to staging and I have confirmed it works

QAed the internalQA and confirmed it works #17448

This is also fixed in staging #17574

@OSBotify
Copy link
Contributor Author

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

@mvtglobally
Copy link

mvtglobally commented Apr 19, 2023

@mountiny regression is completed

PR #15737 is failing #17575. Not repro in prod, but as a new feature we didn't add blocker label
PR #16564 is failing #17576. Repro in PROd checking off
PR #16664 is failing #17638 repro in PROD
PR #16957 is failing #17577. Not repro in prod
PR #17132 is failing #17653, not repro in prod
PR #17138 is failing #17604. Repro in prod checking off
PR #17310 is failing #17641. PR seem to be reverted, are we ok to check off?
PR #17385 is failing #17631. Repro in prod, checking off
PR can you check it internally? This also seems like PROD QA #17406
PR #17444 is failing #17587 repro in prod & #17650 (very inconsistent can you double check)
PR #17464 is failing #17424. Repro in prod
PR #17618 is failing #17656.

@mountiny
Copy link
Contributor

PR #17310 is failing #17641. PR seem to be reverted, are we ok to check off?

Reverted, checking off

@mountiny
Copy link
Contributor

PR can you check it internally? This also seems like PROD QA #17406

Yes, helpdot is prod qa, checking off

@mountiny
Copy link
Contributor

Checking off #17444 since the failing case is repro in prod

@mountiny
Copy link
Contributor

Checking #17132 off since this is currently worked on feature behind beta.

@mountiny
Copy link
Contributor

Checking #15737 off the list, the raised issue seems to be only linked to Windows and their browser and its not a blocker.

@mountiny
Copy link
Contributor

Checked #17618 off seems like a minor design issue and we will try to address it in a follow up PR

@mountiny
Copy link
Contributor

Checking #17464 off since its repro in prod

@mountiny
Copy link
Contributor

Checking #16664 off since its repro in production

@mountiny
Copy link
Contributor

Checking #16957 off the linked issue is not a deploy blocker if its an issue at all

@mountiny
Copy link
Contributor

And checking of #17326 since the linked issue also does not look like a deploy blocker.

@mountiny
Copy link
Contributor

Graphs and crashes look good, deploying

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

5 participants