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 2024-01-04 #33973

Closed
74 tasks done
github-actions bot opened this issue Jan 4, 2024 · 22 comments
Closed
74 tasks done

Deploy Checklist: New Expensify 2024-01-04 #33973

github-actions bot opened this issue Jan 4, 2024 · 22 comments
Assignees
Labels
🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging Monthly KSv2 StagingDeployCash

Comments

@github-actions
Copy link
Contributor

github-actions bot commented Jan 4, 2024

Release Version: 1.4.22-6
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

@kbecciv kbecciv added the 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging label Jan 4, 2024
@OSBotify
Copy link
Contributor

OSBotify commented Jan 4, 2024

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

@jasperhuangg
Copy link
Contributor

Checking off #34003 this appears to be a back-end issue.

@youssef-lr
Copy link
Contributor

Demoting #34018

@kbecciv
Copy link

kbecciv commented Jan 5, 2024

Regression is 1% left

#29730 failing #33986, repro in prod, Checking Off
#32562 - Validation pending
#32594 - Validation pending
#32792 - Validation pending
#32799 - Validation pending
#32902 failing #34018
#33242 - Validation pending
#33421 - Validation pending
#33730 - Validation pending
#33744 - Validation pending
#33791 - Validation pending
#33829 is blocked in Desktop with #25790 (comment), checking Off

Deploy Blockers:

#33984
#33985
#33989
#34004
#34016
#34017

@Beamanator
Copy link
Contributor

Beamanator commented Jan 5, 2024

testing #33730 myself, i meant to make it internal QA 😳

  • Looking good! Also super small note: There's this follow-up to fix the case where special characters are not showing up perfectly in NewDot, and there's already a PR for it - so NAB

@blimpich
Copy link
Contributor

blimpich commented Jan 5, 2024

Checked off #34050 as I don't think its a blocker.

@rlinoz
Copy link
Contributor

rlinoz commented Jan 5, 2024

Checking off #34053 as I don't think it is a blocker.

@kbecciv
Copy link

kbecciv commented Jan 5, 2024

Regression is completed!

#29730 failing #33986, repro in prod, Checking Off
#32562 failing in Android app and Android/Chrome #34068, repro in Prod, checking Off
#32902 failing #34018
#33791 failing #34054, repro on prod, Checking Off

Blockers:
#33984
#33985
#33989
#34004
#34016
#34017
#34043
#34045
#34047
#34051
#34063
#34052
#34055
#34058

@amyevans
Copy link
Contributor

amyevans commented Jan 5, 2024

Checked off #34058 as NAB

@marcaaron
Copy link
Contributor

Checking off #33989. Does not seem related to any new changes and is not a serious problem AFAICT.

@marcaaron
Copy link
Contributor

Checking off #34004 as it is the expected behavior.

@marcaaron
Copy link
Contributor

Closed #34017 as it looks like a dupe.

@roryabraham
Copy link
Contributor

#34006 was only CP'd on all platforms about an hour ago. I haven't looked closely at the deploy blockers, but I suspect that some of them may have been caused by that. Maybe it's worth it to just ask Applause to retest all of them and see which have been fixed?

@situchan
Copy link
Contributor

situchan commented Jan 6, 2024

#34006 was only CP'd on all platforms about an hour ago. I haven't looked closely at the deploy blockers, but I suspect that some of them may have been caused by that. Maybe it's worth it to just ask Applause to retest all of them and see which have been fixed?

#34039 has the same root cause but interesting that it was marked as reproducible in production

@marcaaron
Copy link
Contributor

Sounds good @roryabraham. Not gonna deploy today so there's no rush.

@marcaaron
Copy link
Contributor

Checking off #34063

@thienlnam
Copy link
Contributor

Marked off #33730 for #33730 (comment)

@melvin-bot melvin-bot bot added the Monthly KSv2 label Jan 8, 2024
@kbecciv
Copy link

kbecciv commented Jan 8, 2024

@thienlnam We verified PR #34030 on Web, mWeb and Desktop, the build is not ready for IOS and Android apps.

@thienlnam
Copy link
Contributor

We're currently working on fixing the iOS and Android broken builds - I can manually QA the rest of these for the deploy

@thienlnam
Copy link
Contributor

There's quite a few crashes this time from 1.4.17-7

com.google.mlkit.vision.face.internal.FaceRegistrar.getComponents
java.lang.NullPointerException - Attempt to invoke virtual method 'int java.util.AbstractCollection.size()' on a null object reference

MessageSchema.reflectField
java.lang.RuntimeException - Field packageName_ for wg.a not found. Known fields are [private int wg.a.A00, private java.lang.String wg.a.A01, priva

The user impact is limited, and I believe these are related to the issue we fixed here since these are all android crashes #34030

@thienlnam
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 Monthly KSv2 StagingDeployCash
Projects
None yet
Development

No branches or pull requests