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-08-23 #10498

Closed
48 tasks done
OSBotify opened this issue Aug 23, 2022 · 28 comments
Closed
48 tasks done

Deploy Checklist: New Expensify 2022-08-23 #10498

OSBotify opened this issue Aug 23, 2022 · 28 comments
Assignees
Labels
🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging StagingDeployCash

Comments

@OSBotify
Copy link
Contributor

OSBotify commented Aug 23, 2022

Release Version: 1.1.89-4
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 Aug 24, 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.1.89-0 🚀

@neil-marcellini
Copy link
Contributor

I was not able to reproduce #10524 on iOS staging, so I don't think we should block on this. I'll test on Android soon.

@mvtglobally
Copy link

mvtglobally commented Aug 24, 2022

@AndrewGable @sketchydroide
Interim summary
PR #10041 is failing due to #10528. Not a blocker. Checking off
PRs #10162 & 10322 are failing due to blocker #10520
PR #10479 is failing due to #10524

@sketchydroide
Copy link
Contributor

we have set #10520 as not a blocker for now as it's behind a closed beta
is there a way to still test #10162 & #10322?

@sketchydroide
Copy link
Contributor

#10524 does not seems to be consistent, and set it as not a blocker
can you still test #10479?

@mvtglobally
Copy link

@sketchydroide We tested #10162 & #10322 and if linked issues are not blockers, I guess we can check them off. Rest of the environments are pass.

@sketchydroide
Copy link
Contributor

@mvtglobally how about
#10432
#10444
and
#10479
?

@Julesssss
Copy link
Contributor

Checked off #10535 as the deploy blocker label has been removed

@mvtglobally
Copy link

@sketchydroide PRs #10432 & #10444. Team is facing few JS Console errors when offline. PROD experience is different, but we are not sure if these are expected as user is offline, or its actually failing PR. #10538

@sketchydroide
Copy link
Contributor

I think those are expected is a user is offline

@sketchydroide
Copy link
Contributor

@cead22 just gave a good feedback on the GH, if that is what was blocking the other 2 items we can check them off

@sketchydroide
Copy link
Contributor

@mvtglobally last one #10479 🙏🏼 ?

@mvtglobally
Copy link

few more min....

@mvtglobally
Copy link

mvtglobally commented Aug 24, 2022

@sketchydroide PR #10479 is failing due to #10524. Since #10524 is no longer a blocker. Are we ok to check off?
Only Android is impacted. Other platforms are PASS

@sketchydroide
Copy link
Contributor

sketchydroide commented Aug 24, 2022

yes, let's @mvtglobally

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

@OSBotify
Copy link
Contributor Author

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

@mvtglobally
Copy link

checking New CPs. Regression is almost completed

@OSBotify
Copy link
Contributor Author

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

@OSBotify
Copy link
Contributor Author

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

@mvtglobally
Copy link

Last CP #10545 (comment)

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

@mvtglobally
Copy link

@sketchydroide CP #10567 looks good and Issue no longer repro

Recording.1626.mp4
Record_2022-08-25-19-05-40_4f9154176b47c00da84e32064abf1c48.mp4

Screenshot_20220825_120630_com expensify chat

@sketchydroide
Copy link
Contributor

:shipit:

@sketchydroide sketchydroide reopened this Aug 25, 2022
@sketchydroide
Copy link
Contributor

:shipit:

@OSBotify
Copy link
Contributor Author

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

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