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-18 #34701

Closed
31 tasks done
github-actions bot opened this issue Jan 18, 2024 · 9 comments
Closed
31 tasks done

Deploy Checklist: New Expensify 2024-01-18 #34701

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

Comments

@github-actions
Copy link
Contributor

github-actions bot commented Jan 18, 2024

Release Version: 1.4.27-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 Jan 18, 2024
@OSBotify
Copy link
Contributor

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

@Julesssss
Copy link
Contributor

Checking off #34725 & #34729 as fixed with this revert PR. Tests will be confirmed during the testing of #34733

@mountiny mountiny added the Daily KSv2 label Jan 18, 2024
@mountiny mountiny self-assigned this Jan 18, 2024
@kbecciv
Copy link

kbecciv commented Jan 18, 2024

TR is 30% left

#33691 - pending validation in IOS app
#34117 - pending validation in IOS app and Web
#34125 - pending validation in IOS app
#34231 is failing #34726, repro in prod, checking off
#34332 - pending validation in IOS app
#34333 - pending validation in Desktop
#34733 - pending validation in Desktop

@mvtglobally
Copy link

Regression is completed

PR #34125 is failing #34758

@thienlnam
Copy link
Contributor

Checklist is clear - going to kick it off!

@thienlnam
Copy link
Contributor

There's a couple crashes which seem to be affecting a decent amount of users

MainActivity.onCreate
This issue has 57 crash events affecting 33 users and is impacting the latest version of staging... Version 1.4.27-1

ReactNativeBlobUtilReq.run
This issue has 32 crash events affecting 9 users
This one is probably fine as it is not happening in the latest version

@thienlnam
Copy link
Contributor

Digging deeper into the MainActivity.onCreate

Appears this is only happening on Android the OnePlus8Pro / One phones

couldn't find DSO to load: libhermes.so caused by: couldn't find DSO to load: libjsi.so caused by: couldn't find DSO to load: libfolly_runtime.so caused by: couldn't find DSO to load: libglog.so caused by: couldn't find DSO to load: libc++_shared.so caused by: dlopen failed: "/data/data/com.expensify.chat/lib-3/libc++_shared.so" is for EM_AARCH64 (183) instead of EM_X86_64 (62) result: 0 result: 0 result: 0 result: 0 result: 0

Looks like this has happened before. However, the proposed fix which is adding com.facebook.soloader:soloader to RN build.gradle has not changed facebook/react-native#29528 (comment)

Turns out it still exists https://github.com/Expensify/react-native/blob/main/ReactAndroid/hermes-engine/build.gradle#L202

@thienlnam
Copy link
Contributor

Considering this is only impacting a single device as we can see - I think it's fine to continue deploy

@thienlnam
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

7 participants