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-26 #12165

Closed
28 tasks done
OSBotify opened this issue Oct 26, 2022 · 21 comments
Closed
28 tasks done

Deploy Checklist: New Expensify 2022-10-26 #12165

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

Comments

@OSBotify
Copy link
Contributor

OSBotify commented Oct 26, 2022

Release Version: 1.2.20-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 27, 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.20-0 🚀

@Julesssss
Copy link
Contributor

I checked off #12173 #12171 which have been resolved by this PR

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

@OSBotify
Copy link
Contributor Author

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

@Julesssss
Copy link
Contributor

@Expensify/applauseleads could you please retest #12172? Thanks

@melvin-bot
Copy link

melvin-bot bot commented Oct 27, 2022

⚠️ 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.

@Julesssss
Copy link
Contributor

I checked off #12172, as it was also fixed by #12176

@mvtglobally
Copy link

@Julesssss @chiragsalian
Team is still working on the regression. Few notes

#11868 (comment)
PR #12077 is blocked on IOS and Android due to the VBA issue (see tread https://expensify.slack.com/archives/C01GTK53T8Q/p1666811104732769)
PR #12087 is failing due to #12207. Issue is repro in PROD. Checking off
PR #12113 is failing due to #12194. We are not able to check in PROD, pls double check
PR #12147 is failing #12210, repro in PROD. Checking off

@mvtglobally
Copy link

@Julesssss @chiragsalian
Regression is completed

2 Blockers
#12183
#12204

And last open issues are
#11868 (comment)
PR #12077 is blocked on IOS and Android due to the VBA issue (see tread https://expensify.slack.com/archives/C01GTK53T8Q/p1666811104732769)
PR #12113 is failing due to #12194. We are not able to check in PROD, pls double check

@chiragsalian
Copy link
Contributor

I started discussing about the points in #expensify-open-source (comment)

We just pushed a CP for #12183, once its on staging we'll ask you guys to re-test the issue.

Still working on the other points.

@chiragsalian
Copy link
Contributor

Checking off #12077 since there isn't much we can do on it right now. As long as its tested on web, mobile web and desktop we should be good to go.

@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.20-2 🚀

@chiragsalian
Copy link
Contributor

Checking off #11868 as per comment.

@OSBotify
Copy link
Contributor Author

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

@Luke9389
Copy link
Contributor

I ran a passing Web QA for 12113

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

@OSBotify
Copy link
Contributor Author

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

@OSBotify
Copy link
Contributor Author

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

@OSBotify OSBotify reopened this Oct 28, 2022
@chiragsalian
Copy link
Contributor

oops i misclicked earlier.

Anyway checking off #12192, #12216, #12183 and #12204 as its been tested by QA here and here.

@chiragsalian
Copy link
Contributor

App timing dashboard has a bit of spike on ios but its hard to say if its something introduced here. I'm checking this.

There are a number of crashes for SVGLength.java line 111, but this is only on version 1.2.20-0, and so i believe a fix was CP'd since i don't see it on a newer build. Checking it off.

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

6 participants