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-08-24 #25813

Closed
73 tasks done
OSBotify opened this issue Aug 24, 2023 · 42 comments
Closed
73 tasks done

Deploy Checklist: New Expensify 2023-08-24 #25813

OSBotify opened this issue Aug 24, 2023 · 42 comments
Assignees
Labels
Daily KSv2 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging StagingDeployCash

Comments

@OSBotify
Copy link
Contributor

OSBotify commented Aug 24, 2023

Release Version: 1.3.57-5
Compare Changes: production...staging

This release contains changes from the following pull requests:

Internal QA:

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 Aug 24, 2023
@roryabraham
Copy link
Contributor

The following PRs should have been [No QA], checking them off:

These PRs are for ExpensifyHelp (already in production, production QA), checking them off:

@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.3.57-0 🚀

@roryabraham
Copy link
Contributor

#25860 is NAB, checking it off

@mountiny
Copy link
Contributor

Checking #25830 off, fixed in staging

@roryabraham
Copy link
Contributor

#25797 was fixed on staging for the last checklist, I just forgot to remove the deploy blocker label. Checking it off

@luacmartins
Copy link
Contributor

Checking off #25870. It's a dupe of #25784 and not a blocker since it's a minor UI thing

@roryabraham
Copy link
Contributor

#25871 is behind a beta so NAB, checking it off

@roryabraham
Copy link
Contributor

#25879 is NAB, checking it off

@mvtglobally
Copy link

@roryabraham Can you check this one
#24034 (comment)

@mvtglobally
Copy link

Also, is it possible to QA #24535 internally?
We are not sure how to pull account IDs in Staging and team is stuck

@jasperhuangg
Copy link
Contributor

Checking off #25890 as I don't think we need to be blocking deploy on it.

@roryabraham
Copy link
Contributor

#25841 is fixed, checking it off

@roryabraham
Copy link
Contributor

Demoting #25874, checking it off

@mvtglobally
Copy link

mvtglobally commented Aug 24, 2023

@roryabraham We are down to last 4% TR and few PRs
any CPs we need to validate?

Current summary
PR #22622 is failing #25883, #25884
#23994 is failing #23994. Repro in prod, checking off
#24515 issue found when testing #25582. Repro in Prod, checking off

#24535 - Can you QA internally?

#25161 blocked on Web/mWeb due to #25696. & Issue found on Native apps #25871

#25434 is failing #25696

#25669 is failing #24372. Repro in prod, checking off

@roryabraham
Copy link
Contributor

@mvtglobally three CPs listed above: #25813 (comment)

@mvtglobally
Copy link

@roryabraham for some reason earlier messages didnt show till now. Got the CPs. working on those

@Beamanator
Copy link
Contributor

Checking off #25827 as internal review passed 👍

@mvtglobally
Copy link

mvtglobally commented Aug 25, 2023

Regression is completed.

#24535 is pass on all but IOS. We are not able to test on IOS as links are not opening in the app but in mWeb. checking off

Current summary - unchecked PRs
PR #22622 is failing #25883, #25884
#25161 blocked on Web/mWeb due to #25696. & Issue found on Native apps #25871
#25434 is failing #25696
#24034 (comment)

@cristipaval
Copy link
Contributor

Demoted #25884, checking it off

@luacmartins
Copy link
Contributor

Checking off #25889. Demoted to NAB since it's a super small UI issue and only happens in DEV.

@roryabraham
Copy link
Contributor

Using this legend:

❗ No status
🆗 Demoted
📋 Solution in progress
🏁 Solution has been merged + CP'd, needs retest
✅ Resolved


🏁 #25859, should be fixed by #25882 in 1.3.57-3
🆗 #25875
📋 #25877, might be fixed by #25955
📋 #25878, might be fixed by #25955
📋 #25883, might be fixed by #25955
🆗 #25885
📋 #25888 sounds like there might be a working solution, but it still needs to be implemented
#25893
🆗 #25895
🆗 #25896
🆗 #25898

@roryabraham
Copy link
Contributor

🆗 #25888 demoted

@mountiny
Copy link
Contributor

Checking #25966 off, demoted as its a new design

@mountiny
Copy link
Contributor

#25878 is fixed in staging, checking off

@mountiny
Copy link
Contributor

mountiny commented Aug 25, 2023

Checking #25877 off, confirmed this is not repro in staging anymore
Checking off #25883 as that was also fixed in staging, tested well
Checking #25859 off, fixed in staging

@roryabraham
Copy link
Contributor

roryabraham commented Aug 25, 2023

Going to check off #22622 because all the associated regressions are fixed or otherwise resolved

@roryabraham
Copy link
Contributor

Marking #24034 as a pass. Looks good

@roryabraham
Copy link
Contributor

#25161 is behind a beta, checking it off

@roryabraham
Copy link
Contributor

#25434 is behind a beta, checking it off

@roryabraham
Copy link
Contributor

Ok, that leaves us with just two blockers:

I've self-assigned both of these, so I'm going to triage them now

@roryabraham
Copy link
Contributor

Demoting #25893, checking it off.

@roryabraham
Copy link
Contributor

#25979 is not reproducible on staging, checking it off

@melvin-bot melvin-bot bot added the Monthly KSv2 label Aug 28, 2023
@amyevans
Copy link
Contributor

Demoted #26074 and checked it off

@luacmartins luacmartins self-assigned this Aug 28, 2023
@luacmartins luacmartins added Daily KSv2 and removed Monthly KSv2 labels Aug 28, 2023
@luacmartins
Copy link
Contributor

Checking off #25998. That happened in dev only and we merged the fix - #26045

@luacmartins
Copy link
Contributor

Checking off #26027. Fix merged and being CPed. I'll test once it hits staging.

@luacmartins
Copy link
Contributor

Tested #26055 in staging. It's a pass. Checking off.

@luacmartins
Copy link
Contributor

Down to #25921

@mvtglobally
Copy link

Checked off #25921

@luacmartins
Copy link
Contributor

Checklist is clear. Shipping it.

@luacmartins
Copy link
Contributor

:shipit:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Daily KSv2 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging StagingDeployCash
Projects
None yet
Development

No branches or pull requests

10 participants