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

App crashes on startup with v27.214 #5778

Closed
albertlarsan68 opened this issue Oct 7, 2023 · 6 comments
Closed

App crashes on startup with v27.214 #5778

albertlarsan68 opened this issue Oct 7, 2023 · 6 comments
Labels
P0 critical Highest priority severe: crash The app quits, or stops responding.

Comments

@albertlarsan68
Copy link

Since around a day, I need at least 10 tries in order to be able to read the messages on Zulip.
The version reported by the Settings on my phone is 27.214.
The app crashes on startup, I can try to get more information if you guide me on how to get them

@sevmonster
Copy link

sevmonster commented Oct 10, 2023

Probably dupe of #5741

@albertlarsan68
Copy link
Author

Maybe, but I'm on Android 13. I have a Samsung S20 FE, with One UI v5.1. It was fixed by going back to the stable version.

@gnprice gnprice changed the title App crashes on startup App crashes on startup with v27.214 Oct 10, 2023
@gnprice
Copy link
Member

gnprice commented Oct 10, 2023

Thanks @albertlarsan68 for the report. That is concerning. It's potentially the same issue as #5741, but also potentially the same as #5757.

Good to know that going back to v27.213 (that's the current stable version) fixed it.

Unfortunately the biggest change in v27.214 is one that we are required to make soon, by Google. So I will be very interested in teasing apart whether the crashes are brought on by that change, and if so then trying to identify what we can do to make that change without crashes.

Would you make an account on chat.zulip.org and say hello in the #mobile stream? That will be the best venue for debugging.

@gnprice gnprice added severe: crash The app quits, or stops responding. P0 critical Highest priority labels Oct 10, 2023
@gnprice
Copy link
Member

gnprice commented Oct 10, 2023

Further context, from the data the Google Play Console gives us (with links for those with access):

We're seeing a number of crashes on v27.214. They look like the signature of a memory-corruption issue — they're all SIGSEGV and SIGABRT and SIGBUS, with a bunch of distinct stack traces (15 of them), across a number of different areas of native code.

These add up to a shockingly high crash rate of roughly 5-15% for beta users in the three days of data so far since we rolled out v27.214 to beta. (That's in the measure Google highlights: the daily "user-perceived crash rate", defined as "the percentage of your daily active users who experienced at least one user-perceived crash", meaning basically a crash in the foreground.)

By comparison our crash rate in July and August, before the emergence of #5757, was a bit under 0.3%. It rose in the last week of August, then spiked in mid-September; but since the rollout of v27.213, our crash rate for all users has been around 0.9-1.0% (same link). That leaves a sharp contrast with v27.214.

So @albertlarsan68 is very much not alone, and we can't roll out to users in general a version that has such a high rate of crashes. But I've been using the app regularly on my Android 13 device, and haven't seen the crash. So help from someone who is seeing it will be very useful in narrowing down the cause.

gnprice added a commit that referenced this issue Oct 10, 2023
This reverts commit f792c39, and effectively reinstates
commit e8c57e7.  It's aimed as a workaround for #5778,
which appears to revive #5757.  See chat thread:
  https://chat.zulip.org/#narrow/stream/48-mobile/topic/Android.20crash.20.23M5778/near/1659732
@gnprice
Copy link
Member

gnprice commented Oct 11, 2023

Chat thread is here: https://chat.zulip.org/#narrow/stream/48-mobile/topic/Android.20crash.20.23M5778/near/1659548

We made a v27.215 release and then a v27.216 release, both to alpha. Both of them stopped the crashes for @albertlarsan68.

  • v27.215 was atop v27.213 but adds the targetSdkVersion 33 upgrade, #M5761.
  • v27.216 was atop v27.214 (on main) but disables Sentry again on Android entirely, just like we did for v27.213.

I plan to send v27.216 to beta tomorrow. Hopefully that will bring the overall crash rate in beta back down to the same 0.9-1.0% we're seeing with v27.213.

@gnprice
Copy link
Member

gnprice commented Oct 18, 2023

I plan to send v27.216 to beta tomorrow. Hopefully that will bring the overall crash rate in beta back down

Indeed, thankfully, it did, and I've now sent that version to all users.

Thanks again @albertlarsan68 for the report! Very glad that we didn't send 214 to production.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P0 critical Highest priority severe: crash The app quits, or stops responding.
Projects
None yet
Development

No branches or pull requests

3 participants