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-12-11 #32794

Closed
81 tasks done
github-actions bot opened this issue Dec 11, 2023 · 28 comments
Closed
81 tasks done

Deploy Checklist: New Expensify 2023-12-11 #32794

github-actions bot opened this issue Dec 11, 2023 · 28 comments
Assignees
Labels
🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging StagingDeployCash

Comments

@github-actions
Copy link
Contributor

github-actions bot commented Dec 11, 2023

Release Version: 1.4.11-25
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

@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.11-1 🚀

@Julesssss
Copy link
Contributor

@tgolen demoted #32807 form being a blocker, and closed #32811 as expected behaviour 👍

@luacmartins
Copy link
Contributor

Checking off #32819. We CPed a fix

@luacmartins
Copy link
Contributor

Checking off #32808 and #32809. We're CPing fixes

@luacmartins
Copy link
Contributor

Checking off #32829. We're CPing a fix

@luacmartins
Copy link
Contributor

Checking off #32836. We merged a fix and are CPing it https://expensify.slack.com/archives/C07J32337/p1702331007057339

@kavimuru
Copy link

Regression is at 94%.

#32209 failing with #32812 repro in PROD checking it off
#28618 failing with #32836
#32633 failing with KI #31822 checking it off
#31481 failing with KI #32656 checking it off
#32334 failing with KI #31816 not fixed, checking it off

@luacmartins
Copy link
Contributor

@Julesssss we went through as many blockers as we could, and we have a few fixes along the way:

@marcaaron
Copy link
Contributor

Fix here for #32882 needs review + CP

@shubham1206agra
Copy link
Contributor

Please CP this #32888

@Julesssss
Copy link
Contributor

The following 13 PRs have been resolved and cherry-picked to staging. I haven't checked these off yet as there have been a lot of regression fixes merged today. I am about to seek help testing these on build 1.4.11-16:

#32838 fixed by #32884
#32872 fixed by #32888
#32826 fixed by #32851
#32831 fixed by #32877
#32855 fixed by #32896
#32852 fixed by #32896
#32853 fixed by #32878
#32836 fixed by #32854
#32911 fixed by #32916
#32925 fixed by #32925
#32895 fixed by #32918
#32867 fixed by #32918

@puneetlath
Copy link
Contributor

Confirmed #32867 is fixed.

@luacmartins
Copy link
Contributor

Checking off #32934. Fixed in #32849

@luacmartins
Copy link
Contributor

Checking off #32855. Fixed

@luacmartins
Copy link
Contributor

Checking off #32834. Fixed by #32892

@francoisl
Copy link
Contributor

francoisl commented Dec 12, 2023

Checking off 32865 and 32844, confirmed working on native Android on v1.4.11-20.

@luacmartins
Copy link
Contributor

Checking off #32908. Fixed by #32932

@luacmartins
Copy link
Contributor

luacmartins commented Dec 12, 2023

Checking off #32924. Demoted to NAB since the prod behavior although different, is also broken.

@luacmartins
Copy link
Contributor

Checking off #32882. Fixed by #32887

@luacmartins
Copy link
Contributor

Checking off #32891. Fixed by #32943

@luacmartins
Copy link
Contributor

Checking off #32795. Fixed by #32945

@Julesssss
Copy link
Contributor

Checking off #32912, fixed by #32948

@cristipaval
Copy link
Contributor

cristipaval commented Dec 13, 2023

Checking off #32906, fixed by #32949

@Julesssss
Copy link
Contributor

@cristipaval lets wait until its successfully CP'd and tested

@Julesssss
Copy link
Contributor

Checking off these as its testing well for me

@Julesssss
Copy link
Contributor

Checking off the main issue: #32794 as all of the regressions have been handled 😓

@Julesssss
Copy link
Contributor

  • App timing nominal
  • iOS has a couple of new exceptions:
    • A) Cannot read property 'chatReportID' of null in 1.4.11-4 -- a single crash in an outdated build is not worth blocking over
    • B) EXC_BAD_ACCESS KERN_INVALID_ADDRESS 0x0000000000000000 -- ignoring as a one-off, also 20 releases out of date
    • C) ReferenceError: Property 'compose' doesn't exist, js engine: hermes -- I introduced this exception with a bad conflict resolution, fixed here
  • Github status is good

@Julesssss
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 StagingDeployCash
Projects
None yet
Development

No branches or pull requests

10 participants