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-05-24 #9151

Closed
26 tasks done
OSBotify opened this issue May 24, 2022 · 15 comments
Closed
26 tasks done

Deploy Checklist: New Expensify 2022-05-24 #9151

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

Comments

@OSBotify
Copy link
Contributor

OSBotify commented May 24, 2022

Release Version: 1.1.66-1
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

@Julesssss Julesssss self-assigned this May 24, 2022
@mvtglobally mvtglobally added the 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging label May 24, 2022
@mvtglobally
Copy link

@Julesssss , its been a while and IOS build still not available. Can you check if it is looking ok?

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

@mvtglobally
Copy link

@Julesssss Regression is completed

Deploy blockers logged
#9159
#9161
#9166

PR #8980 is failing due to #9158. Pls double check if it needs to be a blocker as experiences between PROD and Staging differs.
PR #9138 is failing due to #9159

Issues logged
#9157
#9158
#9162
#9164

@Julesssss
Copy link
Contributor

Checked off 9159 which we believe should be expected after changes in other PRs.

@Julesssss
Copy link
Contributor

Commented on 9161 as I have a feeling this is expected.

@Julesssss
Copy link
Contributor

Additionally, I think 9166 has the same explanation as 9159 and is expected. But I'll await a second opinion before removing the label.

@mvtglobally
Copy link

@Julesssss, thanks for update. Hopefully we can have a new deploy today. Let us know if anything needs to be re-tested

@Julesssss
Copy link
Contributor

Julesssss commented May 25, 2022

After looking into 9158 it seems to be a regression, but this has been on prod for a few weeks now -- so is unrelated to this checklist. I've posted here and here to ensure we figure out a fix.

@Julesssss
Copy link
Contributor

Julesssss commented May 25, 2022

Checking of 9161 and 9166 and as discussed in the issues and in Slack.

@Julesssss
Copy link
Contributor

Also checked off 9146 as there is no QA to be performed.

@Julesssss
Copy link
Contributor

Checking off 9138 as the tests pass and we deemed the linked issue to not be a deployBlocker.

@Julesssss
Copy link
Contributor

After looking into 9158 it seems to be a regression, but this has been on prod for a few weeks now -- so is unrelated to this checklist. I've posted here and here to ensure we figure out a fix.

Just to clarify. I was mistaken here. THIS is the PR which I believe introduced the issue so I've made it a deployBlocker.

@chiragsalian
Copy link
Contributor

@chiragsalian
Copy link
Contributor

:shipit:

@SumitDiyora
Copy link

SumitDiyora commented May 27, 2022

Checked all PRs on Production / Staging and created accessibility issues for all PRs where accessibility issues were found.
The non-applicable PRs are - #9136, #9146
Logged Issues - #8403, #8401, #8400, #8413, #8402, #8388, #8387, #8579, #8624

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