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-09-09 #10916

Closed
16 tasks done
OSBotify opened this issue Sep 9, 2022 · 25 comments
Closed
16 tasks done

Deploy Checklist: New Expensify 2022-09-09 #10916

OSBotify opened this issue Sep 9, 2022 · 25 comments
Assignees
Labels
🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging Monthly KSv2 StagingDeployCash

Comments

@OSBotify
Copy link
Contributor

OSBotify commented Sep 9, 2022

@Julesssss
Copy link
Contributor

@mountiny reverted to resolve the deploy blocker: #10917

@Julesssss Julesssss added the 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging label Sep 9, 2022
@OSBotify
Copy link
Contributor Author

OSBotify commented Sep 9, 2022

👏 Heads up @Expensify/applauseleads 👏
A new pull request has been 🍒 cherry-picked 🍒 to staging, and will be deployed to staging in version 1.1.99-2 🚀

@Julesssss
Copy link
Contributor

Hey @Expensify/applauseleads, I manually added the lock label because we have an important release today. Please take a look at this release checklist when ready :)

@thienlnam
Copy link
Contributor

Checking off 10855 #10855 (comment)

@mountiny
Copy link
Contributor

mountiny commented Sep 9, 2022

Checking off 10763 #10763 (comment)

@thienlnam
Copy link
Contributor

Checking off #10690 since the revert in #10920 just removed those changes

@mvtglobally
Copy link

We are 40% through regression.
Are we ok to checkoff #10908. It seems from the previous build

@Julesssss
Copy link
Contributor

@mvtglobally yeah 👍

@mvtglobally
Copy link

I am going offline for few hrs. We are at 17% left. Should be done soon.

2 Blockers right now
#10925
#10927

@melvin-bot melvin-bot bot added the Monthly KSv2 label Sep 12, 2022
@Julesssss
Copy link
Contributor

Checked off #10925 as it's not always reproducible, and isn't a major problem as users can still chat to concierge.

@Julesssss
Copy link
Contributor

I also checked off #10927 as it's not reproducible by three internal employees.

@melvin-bot
Copy link

melvin-bot bot commented Sep 12, 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

Julesssss commented Sep 12, 2022

Given that the release is pretty much clear, I was going to deploy for @yuwenmemon. This would have given Applause some time to run tests on a new release candidate before the US gets online.

Unfortunately, when checking the Firebase console I noticed that this issue is a new regression with a lot of reported crashes that we do not want to release, even if it is not always reproducible. I'm unable to pick this up this morning, but I shared it in Slack here.

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

@OSBotify
Copy link
Contributor Author

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

@Julesssss
Copy link
Contributor

A PR was just merged to resolve #10876, but the release failed 😭 Looking into 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.1.99-4 🚀

@Julesssss
Copy link
Contributor

Okay, this one seemed to be successful. Builds are in progress and should be released within an hour 🎉

@yuwenmemon yuwenmemon assigned francoisl and unassigned yuwenmemon Sep 12, 2022
@OSBotify
Copy link
Contributor Author

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

@francoisl
Copy link
Contributor

francoisl commented Sep 12, 2022

The iOS build was hanging forever so I canceled it. So in short, 1.1.99-4 is deployed everywhere except iOS. I'm rerunning the canceled action now

Stuck/Canceled iOS build: https://github.com/Expensify/App/actions/runs/3038990393/jobs/4893392499
Retry: https://github.com/Expensify/App/actions/runs/3038990393/jobs/4897356125

@mvtglobally
Copy link

@francoisl
We got the IOS build around 1.45pm EST.
Not sure if the issue is not in build not publishing, but rather system not updating it published.
IMG_5127

@francoisl
Copy link
Contributor

Interesting. The action got stuck after the build was sent to the AppStore, so it makes sense.
image

So for now, I think we just need to confirm that the regression #10876 is fixed, by running the QA steps from #10937. If those work, we should be able to check the last blocker and deploy.

@francoisl
Copy link
Contributor

The app doesn't crash on Android and iOS when testing #10937, so all the blockers are resolved now.

It's getting late for me right now and I need to go, so either @Julesssss or anyone else can :shipit: this checklist when you get online if you feel like it, otherwise I'll do it first thing tomorrow morning!

@Julesssss
Copy link
Contributor

I rechecked Firebase and closed the Crash reports linked to the final blocker. Preparing to ship

@Julesssss
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 Monthly KSv2 StagingDeployCash
Projects
None yet
Development

No branches or pull requests

8 participants