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-19 #11979

Closed
36 tasks done
OSBotify opened this issue Oct 19, 2022 · 62 comments
Closed
36 tasks done

Deploy Checklist: New Expensify 2022-10-19 #11979

OSBotify opened this issue Oct 19, 2022 · 62 comments
Assignees
Labels
🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging Monthly KSv2 StagingDeployCash

Comments

@OSBotify
Copy link
Contributor

OSBotify commented Oct 19, 2022

Release Version: 1.2.18-10
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 19, 2022
@sketchydroide
Copy link
Contributor

#10585 checked, to old to be blocker over a month and a half, weird.

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

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

@OSBotify
Copy link
Contributor Author

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

@puneetlath puneetlath added the Bug Something is broken. Auto assigns a BugZero manager. label Oct 19, 2022
@mvtglobally
Copy link

Team is still working on the regression.

Summary so far
Blockers
#11983
#11995
#12000

PRs
PR #11640 is blocked on IOS and Android due to #11796
PR #11699 is failing due to #12000
PR #11956 is failing #11867. Checking off

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

@OSBotify
Copy link
Contributor Author

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

@mvtglobally
Copy link

@sketchydroide
Regression is completed

Blockers
#11633
#11983
#11995
#12000
#12009
#12012

PRs
PR #11640 is blocked on IOS and Android due to #11796
PR #11699 is failing due to #12000

We are checking new CP #12013

@sketchydroide
Copy link
Contributor

checking #11633, it's already in prod so not a blocker.

@sketchydroide
Copy link
Contributor

checking #11983, hard to reproduce, maybe connection issue. not a blocker

@sketchydroide
Copy link
Contributor

checking #12000, was reproduced in Prod by @ctkochan22.
@mvtglobally how is this blocking #11699? Let me know if that is still a problem

@sketchydroide
Copy link
Contributor

@thienlnam removed the blocker for #12012 so checking that one as well

@mvtglobally
Copy link

@sketchydroide
#11699 Is not blocked. I just mentioned the blocker issue we found while executing that PR. Team wasn't able to repro in PROD. But if you can repro. I will check it off.

@mvtglobally
Copy link

Also, The NEW CP #12035 seem to break #12035

@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.18-4 🚀

@sketchydroide
Copy link
Contributor

Also, The NEW CP #12035 seem to break #12035

Did you get those right @mvtglobally, they are both the same, and seems to be about a Web change

@mvtglobally
Copy link

mvtglobally commented Oct 20, 2022

My bad, not enough ☕ ☕ ☕ I meant to say CP #12013 broke #12035
This issue wasnt repro in earlier build

@OSBotify
Copy link
Contributor Author

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

@mountiny
Copy link
Contributor

Checking #12035 and #12037 off as @trjExpensify confirmed that the fixed worked.

@PauloGasparSv
Copy link
Contributor

Should #11796 be tagged as deployBlocker and added to the deployBlockerList since we mentioned it is blocking this #11640?

I'm just confused because #11640 says to connect with "Chase" via Plaid and not "Fidelity".

cc @mvtglobally

@mvtglobally
Copy link

@PauloGasparSv the PR was created a long time ago when we used Chase to test. Now Chase is blocked for a while, so we use Fidelity.
I am not sure this should be a Deploy blocker, since it was there for few releases already.. The main issues is, that we can't verify PR #11640 on IOS/and Android due to this #11796

@sketchydroide
Copy link
Contributor

#11995 was closed, we can't replicate it anymore, maybe it was a backend issue.

@sketchydroide
Copy link
Contributor

#12009 was also closed, we can't replicate it anymore. checking it off

@sketchydroide
Copy link
Contributor

@mvtglobally
What is missing from the last 3 items that need checking in the list?

@mvtglobally
Copy link

@sketchydroide

We cant validate #11640 on Android and IOS due to this #11796. Are we ok to check it off?

PR #12013 was failing. Re-testing right now

@sketchydroide
Copy link
Contributor

We cant validate #11640 on Android and IOS due to this #11796. Are we ok to check it off?

yeah lets

@roryabraham
Copy link
Contributor

@Expensify/applauseleads we decided to CP #11944 because it fixes some critical bugs and we wanted to get it out over the weekend. Let's make sure we complete QA for that before proceeding to deploy production

@mvtglobally
Copy link

We are on it

@OSBotify
Copy link
Contributor Author

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

@mvtglobally
Copy link

@roryabraham @sketchydroide Seeing few issues with new PR. Team is checking if any blockers and logging in a bit

@mountiny
Copy link
Contributor

@roryabraham @sketchydroide I think iOS build failed there so the fix is not in testflight for iOS yet (I cannot test on android unfortunately)

@roryabraham
Copy link
Contributor

Retrying the iOS deploy now

@mvtglobally
Copy link

@sketchydroide @roryabraham
While we are waiting for IOS

#12013 is only failing with #12061. Since thats not a blocker, I guess we can check it off.

PR. #11944 is failing with #11867. Its an older KI.

@mountiny Android Build is updated

@mountiny
Copy link
Contributor

#12013 is only failing with #12061. Since thats not a blocker, I guess we can check it off.

Correct

PR. #11944 is failing with #11867. Its an older KI.

@mvtglobally this is also ok, the issue mentioned should not be solved by this change. We thought it might and but it has a different root cause, updated the PR body there.

@mvtglobally
Copy link

Still no IOS build

@roryabraham
Copy link
Contributor

Working on it, just put up #12068 to try and fix it

@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.18-6 🚀

@roryabraham
Copy link
Contributor

Still working on iOS deploys, another PR here

@OSBotify
Copy link
Contributor Author

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

@roryabraham
Copy link
Contributor

Yet another deploy running here, hopefully this one works 🤞🏼

@OSBotify
Copy link
Contributor Author

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

@mvtglobally
Copy link

Unfortunately, still no IOS

@melvin-bot melvin-bot bot added the Monthly KSv2 label Oct 24, 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.18-9 🚀

@OSBotify
Copy link
Contributor Author

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

@mvtglobally
Copy link

@roryabraham @chiragsalian
We got a new Build

PR #11944 is pass on IOS. #11867 & #10290 Not blockers, so checking off.

Asked team to double check this #12067. Only 1 out of 5 testers was able to repro originally.

@chiragsalian
Copy link
Contributor

okay thank you

@mvtglobally
Copy link

@chiragsalian
We are only able to repro on Realme 6 / Android 11 #12067

Record_2022-10-24-21-37-54_4f9154176b47c00da84e32064abf1c48.mp4

@chiragsalian
Copy link
Contributor

Strange, yeah i was just able to reproduce the issue using sahil's credentials posted here. And i can reproduce only on staging and not on production. Investigating.

@chiragsalian
Copy link
Contributor

We think we found the fix (or rather revert) to address the deploy blocker. We're waiting for it to hit staging and deploy to devices before re-testing.

@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.18-10 🚀

@OSBotify
Copy link
Contributor Author

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

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

No branches or pull requests

9 participants