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-07-07 #22399

Closed
64 tasks done
OSBotify opened this issue Jul 7, 2023 · 18 comments
Closed
64 tasks done

Deploy Checklist: New Expensify 2023-07-07 #22399

OSBotify opened this issue Jul 7, 2023 · 18 comments
Assignees
Labels
🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging Monthly KSv2 StagingDeployCash

Comments

@OSBotify
Copy link
Contributor

OSBotify commented Jul 7, 2023

Release Version: 1.3.38-3
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 Jul 7, 2023
@OSBotify
Copy link
Contributor Author

OSBotify commented Jul 7, 2023

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

@melvin-bot
Copy link

melvin-bot bot commented Jul 7, 2023

⚠️ 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

Checking off #22398, after the fix was cherry-picked.

@mvtglobally
Copy link

mvtglobally commented Jul 10, 2023

Regression is still in progress. We have only 2% left in TR.

Current PR summary
#19906 Pending validation on all platforms
#21482 Pending validation on all platforms
#21781 Pending validation on all platforms
#21877 Pending validation on all platforms
#21941 is failing due to #21941
#21945 Pending validation on all platforms
#22100 is failing due to #21001
#22151 Pending validation on all platforms
#22188 is failing #22480
#22200 Pending validation on all platforms
#22244 Pending validation on all platforms
#22246 Pending validation on all platforms
#22253 Pending validation on all platforms
#22262 Pending validation on all platforms
#22269 Pending validation on all platforms
#22275 Pending validation on all platforms
#22280 Pending validation on all platforms
#22281 Pending validation on all platforms
#22327 Pending validation on all platforms

@Julesssss
Copy link
Contributor

@cristipaval I checked off #22469 as you closed it 👍

@cristipaval
Copy link
Contributor

@cristipaval I checked off #22469 as you closed it 👍

Thanks, sorry for missing to check it off here 🙇

@mvtglobally
Copy link

mvtglobally commented Jul 10, 2023

Regression in TR is completed, down to just a few validations

Current PR summary
#21781 Pending validation on all platforms
#21941 is failing due to #22554
#22100 is failing due to #21001
#22188 is failing #22480
#22253 Pending validation on all platforms
#22275 Pending validation on all platforms
#22327 Pending validation on all platforms

@Li357
Copy link
Contributor

Li357 commented Jul 10, 2023

Removing #22557 since I can reproduce on prod and proposal is being worked on here: #21881

@mvtglobally
Copy link

mvtglobally commented Jul 10, 2023

Regression is completed

Current PR summary
#21941 is failing due to #22554
#22100 is failing due to #21001
#22188 is failing #22480

@francoisl
Copy link
Contributor

francoisl commented Jul 10, 2023

Thanks @mvtglobally. We're looking into the last blockers. A fix for #22426 is being CP'ed at the moment.

In the meantime, I checked the ones that were looked into and no longer considered blockers:
#22487
#22494
#22485
#22480
#22498

@mvtglobally
Copy link

@francoisl since checklist is not automatically updated with the new builds any more, can you please tag me once the new CP is ready to test. I dont really have a way to tell

@melvin-bot melvin-bot bot added the Monthly KSv2 label Jul 10, 2023
@francoisl
Copy link
Contributor

Oh true. The fix for #22426 should be live on all platforms on staging, now. The version number will be 1.3.38-5.

Also, regarding this one:

#22188 is failing #22480

Since the issue is only with Gmail accounts, could you run the QA steps again from a private domain please (like a regular applause.expensify.com account)? We found the issue is also happening on production, so we'll treat #22480 as a regular bug.

@francoisl
Copy link
Contributor

@mvtglobally v1.3.38-6 is also available now, it fixes the blocker #22502.

@mvtglobally
Copy link

@francoisl CP #22501 is pass

@francoisl
Copy link
Contributor

francoisl commented Jul 10, 2023

  • Checking off 22502, I tested it on staging

  • A fix for 22445 is being deployed right now, I can also help test when it's fully deployed

  • Checking 22100, tested on staging and can't reproduce the issue
    image

  • Checking 21941, tested on staging - I believe the reported issue is because the email address was so long that the backend didn't accept it. A similar issue is happening on production, so no need to mark this a deploy blocker
    image

  • Checking 22445, fix tested on staging

@francoisl
Copy link
Contributor

Checklist is complete and all blockers are resolved, waiting for #21651 to be merged and we can ship this.

@francoisl
Copy link
Contributor

:shipit:

@mvtglobally
Copy link

@francoisl is pass #22584

@francoisl francoisl self-assigned this Jul 11, 2023
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

7 participants