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 2023-05-09 #18627

Closed
26 tasks done
OSBotify opened this issue May 9, 2023 · 22 comments
Closed
26 tasks done

Deploy Checklist: New Expensify 2023-05-09 #18627

OSBotify opened this issue May 9, 2023 · 22 comments
Assignees
Labels
🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging StagingDeployCash

Comments

@OSBotify
Copy link
Contributor

OSBotify commented May 9, 2023

Release Version: 1.3.12-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. More detailed instructions on this verification can be found here.
  • I checked GitHub Status and verified there is no reported incident with Actions.

cc @Expensify/applauseleads

@mvtglobally mvtglobally added the 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging label May 9, 2023
@OSBotify
Copy link
Contributor Author

OSBotify commented May 9, 2023

🚀 All staging deploys are complete, @Expensify/applauseleads please begin QA on version https://github.com/Expensify/App/releases/tag/1.3.12-0 🚀

@roryabraham
Copy link
Contributor

Lots of very important context in slack:

TL;DR 1.3.12-0 was already deployed to prod on web + desktop. The correct diff between our last intentional prod deploy and the current production branch (i.e: all the code in this checklist) is here: 1.3.11-5...1.3.12-0

@marcaaron
Copy link
Contributor

Checking off #18696 because we have a tight deadline to get some iteration of Manual Requests out the door as per #18696 (comment)

@mvtglobally
Copy link

@marcaaron we are almost done, getting summary in a few

@mvtglobally
Copy link

@roryabraham @marcaaron

Regression is completed
PR #17992 is failing with multiple issues #18649, #18647, #18660, #18659
PR #18468 seem failing, #18468 (comment)
PR #18143 pass on all but Desktop. Can you double check the Desktop?

@Beamanator
Copy link
Contributor

Ok so it seems like #18650 was marked as not a deploy blocker so I'll check that off the list

@Beamanator
Copy link
Contributor

Similar with #18682, it looks like it is also reproducible in production so i'll check that off as well

@Beamanator
Copy link
Contributor

Beamanator commented May 10, 2023

Regression is completed
PR #17992 is failing with multiple issues #18649, #18647, #18660, #18659
PR #18468 seem failing, #18468 (comment)
PR #18143 pass on all but Desktop. Can you double check the Desktop?

Looking into these now

@Beamanator
Copy link
Contributor

@mvtglobally to be clear, is the issue with #18468 the part where the form changes size slightly when data populates the fields? I don't see the labels shifting left like in the bug report (here) but maybe I'm missing something?

@Beamanator
Copy link
Contributor

I didn't see #18143 failing when I tested on Desktop, 1.3.11-5 - but @mvtglobally can you specify what exactly wasn't passing in the tests on that PR?

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

@mvtglobally
Copy link

@Beamanator sorry, I didn't mean to say it's failing on Desktop, we just didn't finish testing it on Desktop. It is pass checking off
PR #18468 more details here #18714

@Beamanator
Copy link
Contributor

Aah that's great! Thanks @mvtglobally 👍 👍

I'm currently waiting for the latest iOS staging build & deploy, then I plan to ship this version 💪

@OSBotify
Copy link
Contributor Author

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

@Beamanator
Copy link
Contributor

Alright here we go!

@Beamanator
Copy link
Contributor

:shipit:

@Beamanator
Copy link
Contributor

Checked off #17992 and #18468 so we could hopefully get the deploy out

@Beamanator
Copy link
Contributor

But there seems to be a problem with GitHub Status at the moment:

Update - For Actions we are seeing issues for a subset of customers triggering workflow runs and scheduled runs. Workflows using the GITHUB_TOKEN environment variable are also encountering issues.
May 10, 2023 - 14:05 UTC

@marcaaron
Copy link
Contributor

:shipit:

@OSBotify
Copy link
Contributor Author

This issue either has unchecked items or has not yet been marked with the :shipit: emoji of approval.
Reopening!

@OSBotify OSBotify reopened this May 11, 2023
@marcaaron
Copy link
Contributor

:shipit:

1 similar comment
@marcaaron
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