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-04-25 #40981

Closed
53 tasks done
github-actions bot opened this issue Apr 25, 2024 · 13 comments
Closed
53 tasks done

Deploy Checklist: New Expensify 2024-04-25 #40981

github-actions bot opened this issue Apr 25, 2024 · 13 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 Apr 25, 2024

Release Version: 1.4.66-5
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

@kavimuru kavimuru added the 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging label Apr 25, 2024
@mountiny mountiny added the Daily KSv2 label Apr 25, 2024
@mountiny mountiny self-assigned this Apr 25, 2024
@rlinoz
Copy link
Contributor

rlinoz commented Apr 25, 2024

Demoted #40991

@roryabraham
Copy link
Contributor

Demoted #41017

@francoisl
Copy link
Contributor

Demoted #41031 as can't repro

@mountiny
Copy link
Contributor

demoted #41000

@kavimuru
Copy link

Regression is 2% left in the TR
#40718 failing with #40914 and #40978
#40676 failing with #40911 reproduced in production checking it off
#40532 Validation pending in mweb
#41047 - validation pending

@mountiny
Copy link
Contributor

demoted #41065

@mountiny
Copy link
Contributor

Demoted #40993 - minor UX issue

@mountiny
Copy link
Contributor

demoted #41011 - its very specific issue for limited number of devices too

@mountiny
Copy link
Contributor

checking #41014, #40989 and #40996 off as they have been fixed by a revert

@kavimuru
Copy link

Regression is completed

@thienlnam
Copy link
Contributor

There's a fresh crash

CameraDeviceDetails.getFormats

Fatal Exception: java.lang.IllegalArgumentException
lower must be less than or equal to upper
android.util.Range.<init> (Range.java:59)
com.mrousavy.camera.core.CameraDeviceDetails.getFormats (CameraDeviceDetails.java:125)
com.mrousavy.camera.core.CameraDeviceDetails.toMap (CameraDeviceDetails.java:79)
com.mrousavy.camera.CameraDevicesManager.getDevicesJson (CameraDevicesManager.kt:87)
com.mrousavy.camera.CameraDevicesManager.sendAvailableDevicesChangedEvent (CameraDevicesManager.kt:94)
com.mrousavy.camera.CameraDevicesManager$initialize$1.invokeSuspend (CameraDevicesManager.kt:71)
kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith (ContinuationImpl.kt:33)

Looks related to vision camera - going to bring up in expensify-margelo 1.4.61-8 (1001046108)

@mountiny
Copy link
Contributor

Graphs looking alright and new crash has been reported by Jack above and shared with Margelo

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