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-28 #12223

Closed
20 tasks done
OSBotify opened this issue Oct 28, 2022 · 14 comments
Closed
20 tasks done

Deploy Checklist: New Expensify 2022-10-28 #12223

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

Comments

@OSBotify
Copy link
Contributor

OSBotify commented Oct 28, 2022

@mvtglobally mvtglobally added the 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging label Oct 28, 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.2.21-4 🚀

@mountiny
Copy link
Contributor

mountiny commented Oct 28, 2022

Checking #12193 off since it is also noQA

Same for #12030

@mvtglobally
Copy link

mvtglobally commented Oct 28, 2022

Interim summary

PR #11784 failing with #12265. repro in PROD checking off
PR #12124 is failing due to #12249. It is PR fail, so we did not add DeployBlocker label. Pls double check
PR #12136 looks PROD QA. Are we ok to check off?

@mvtglobally
Copy link

Regression is completed

Deploy Blocker
#12275

Open PRs
PR #12124 is failing due to #12249. It is PR fail, so we did not add DeployBlocker label. Pls double check
PR #12136 looks PROD QA. Are we ok to check off?

@melvin-bot melvin-bot bot added the Monthly KSv2 label Oct 31, 2022
@Julesssss Julesssss self-assigned this Oct 31, 2022
@Julesssss
Copy link
Contributor

Hi @mvtglobally. I agree with #12249 being a bug, but not a deploy blocker. I've checked off #12124 as the PR test steps are passing (at least on the first use).

@Julesssss
Copy link
Contributor

#12136 was checked off as it's untestable outside of prod.

@Julesssss
Copy link
Contributor

#12275 is now reproducable on prod, so this is not a mobile regression. Continueing to move forward once the last regression is resolved

@Julesssss
Copy link
Contributor

I checked off the final blocker, #12281 as it was not reproducible for me on web or iOS

@Julesssss
Copy link
Contributor

Switch Report is looking better in the App performance timing dashboard. Homepage initial render is worse, but we are aligned on the fact that this is due to a change in the timing logic and doesn't accurately represent a regression.

@Julesssss
Copy link
Contributor

The only new crashlytics issues were from the SVG library crash that was resolved in the last deploy. I cleared up the console while I was there.

@Julesssss
Copy link
Contributor

Okay, all is good except this bug which was raised against an old issue. I have asked for help with testing as I'm unable to proceed past the 2nd step.

@Julesssss
Copy link
Contributor

Hi, @Expensify/applauseleads. I saw that this crash was raised, which could well be a deploy blocker. I closed the original issue because it has a different underlying issue, but it would be great if you could open a new issue if it is still occurring and please test on both staging and prod.

Thanks!

@Julesssss Julesssss added Daily KSv2 and removed Monthly KSv2 labels Oct 31, 2022
@Julesssss
Copy link
Contributor

@chiragsalian kindly tested the issue above and wasn't able to reproduce it. @Expensify/applauseleads if you are still able to reliably reproduce this, please open a new issue and I'll investigate further:

image (35)

@Julesssss
Copy link
Contributor

:shipit:

@Julesssss Julesssss removed the Daily KSv2 label Nov 1, 2022
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

5 participants