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-15 #12753

Closed
31 tasks done
OSBotify opened this issue Nov 15, 2022 · 16 comments
Closed
31 tasks done

Deploy Checklist: New Expensify 2022-11-15 #12753

OSBotify opened this issue Nov 15, 2022 · 16 comments
Assignees
Labels
🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging StagingDeployCash

Comments

@OSBotify
Copy link
Contributor

OSBotify commented Nov 15, 2022

Release Version: 1.2.28-2
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

@mvtglobally mvtglobally added the 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging label Nov 15, 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.28-0 🚀

@dangrous
Copy link
Contributor

Working on internal QA for #12429 - we may need some edits - I've posted a comment on the PR.

@arosiclair
Copy link
Contributor

Thanks for taking over QA on that one. Just saw this now

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

@OSBotify
Copy link
Contributor Author

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

@mvtglobally
Copy link

@roryabraham We are blocked on Desktop due to #12760

@roryabraham
Copy link
Contributor

A fix for #12760 is being CP'd to staging presently

@roryabraham roryabraham added Reviewing Has a PR in review and removed Reviewing Has a PR in review labels Nov 16, 2022
@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.28-2 🚀

@roryabraham
Copy link
Contributor

#12785 has been deployed on all platforms except iOS, so you should be able to retest #12760 at any time and resume regressions

@mvtglobally
Copy link

thanks @roryabraham we will resume

@roryabraham
Copy link
Contributor

Confirmed #12760 is resolved, checking it off

@mvtglobally
Copy link

Regression is completed

PR #12226 is failing #12777. Repro in PROD
PR #12334 is failing #12789. Repro in PROD
PR #12488 is failing #12779 & #12792. Repro in PROD
PR #12572 doesn't have tooltip on mweb and Native aps. Pass on Desktop/Web. Ok to checkoff?
PR #12588 is blocked by #12776
PR #12601 #12695 & #12697 are PROD QA of Help pages. Ok to check off?
PR #12785 is god on Desktop. checking other platforms just in case

@roryabraham
Copy link
Contributor

PR #12601 #12695 & #12697 are PROD QA of Help pages. Ok to check off?

While they are PROD QA, keep in mind that help pages are deployed immediately so can be QA'd at any time. Usually this will be very quick so for now while we don't have a prodQA checklist we should just do them during the main checklist. There's also no scenario where we'd have a help site deploy blocker, so there's not much risk involved with these..

#12785

Confirmed myself that this works so we should be good to check it off.

@roryabraham
Copy link
Contributor

I am seeing a spike in homepage initial render, but it seems probably that this could be attributed to the process change ensuring that people test on high traffic accounts.

@roryabraham
Copy link
Contributor

There is a new crash too, but I think it's only happening in UIAutomator, which unless I'm mistaken is only related to E2E automated tests, which aren't live yet.

image

@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

6 participants