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-08 #39807

Closed
github-actions bot opened this issue Apr 8, 2024 · 37 comments
Closed

Deploy Checklist: New Expensify 2024-04-08 #39807

github-actions bot opened this issue Apr 8, 2024 · 37 comments
Assignees
Labels
🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging StagingDeployCash

Comments

@github-actions
Copy link
Contributor

github-actions bot commented Apr 8, 2024

Release Version: 1.4.61-8
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

@Julesssss Julesssss added the 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging label Apr 8, 2024
@Julesssss
Copy link
Contributor

Checking off #39710 as I forgot to remove the label this morning 😵‍💫

@mountiny
Copy link
Contributor

mountiny commented Apr 8, 2024

Demoted #39827

@mountiny
Copy link
Contributor

mountiny commented Apr 8, 2024

Demoted #39841

@mountiny
Copy link
Contributor

mountiny commented Apr 8, 2024

#39834 fixed by backend change

@luacmartins
Copy link
Contributor

Demoted #39838

@mountiny
Copy link
Contributor

mountiny commented Apr 8, 2024

Demoted #39845
Demoted #39847

@marcaaron
Copy link
Contributor

Demoted #39851

@mountiny
Copy link
Contributor

mountiny commented Apr 8, 2024

demoted #39855

@jasperhuangg
Copy link
Contributor

Demoted #39853 NAB

@kavimuru
Copy link

kavimuru commented Apr 8, 2024

Regression is 84% progress.

#38887 failing with #35257, bug is not fixed and repro in all the platforms, comment
#39390 failing with #38833 issue not fixed in mweb and android, comment

@srikarparsi
Copy link
Contributor

Hey, is it okay to check off #39825, I don't think it's a deploy blocker because it's related to this feature and this item was similarly checked off. I've been working on it for about 2 hours and have left some updates on the issue. I have a little bit of other things to finish up today so just wanted to check if I can move to those before stopping work on this one.

@gijoe0295
Copy link
Contributor

gijoe0295 commented Apr 9, 2024

#39390 failing with #38833 issue not fixed in mweb and android, #39390 (comment)

Would be fixed after #39685 was merged. The issue is not constantly reproducible and does not break any functionality so I think we can demote it.

@Julesssss
Copy link
Contributor

#39825 was checked off here

@kavimuru
Copy link

kavimuru commented Apr 9, 2024

@Julesssss Regression is completed.

Unchecked PRs:
#36039 failing with #39919
#37733 - failing with #39834
#38675 - failing with #39923
#38887 failing with #35257, bug is not fixed and repro in all the platforms comment
#39038 failing with #39847
#39390 failing with #38833 issue not fixed in mweb and android - checking off as per this comment

Open Blockers:
#39824
#39831
#39833
#39836
#39837
#39864
#39904
#39907
#39919
#39920
#39923
#39925
#39926

@amyevans
Copy link
Contributor

amyevans commented Apr 9, 2024

Demoted #39926

@mountiny
Copy link
Contributor

mountiny commented Apr 9, 2024

demoted #39904

@Julesssss
Copy link
Contributor

Demoting #39864 as it isn't reproducible, and is a generic perf issue

@Julesssss
Copy link
Contributor

Checking off #39923 as we've merged a PR already and it is a minor issue

@cristipaval
Copy link
Contributor

Demoted #39907

@amyevans
Copy link
Contributor

amyevans commented Apr 9, 2024

Checked off 39833 following CP fix and retest

@kavimuru
Copy link

kavimuru commented Apr 9, 2024

#39896 failing in Desktop due to #39851. Other platforms are pass
#39938 failing in mweb with #39932

@mountiny
Copy link
Contributor

mountiny commented Apr 9, 2024

checked #39896 off as the follow up issue is being handled

@Julesssss
Copy link
Contributor

Checked off #39968 which fixed a blocker by reverting #39465

@Julesssss
Copy link
Contributor

@Julesssss
Copy link
Contributor

#39938 confirmed not a blocker here

@Julesssss
Copy link
Contributor

As discussed here in Slack, we will not mark #39971 as a blocker. It's only reproducable when triple clicking before copy/pasting and is part of a new feature.

@Julesssss
Copy link
Contributor

  • Github Status ✅
  • Firebase crashes ✅
    • Android:
      • More instances of a long running crash, with no increase -- no need to hold
      • The only new crash has one instance
    • iOS:
      • No new crashes
      • A build from last week had a spike in crashes

@Julesssss
Copy link
Contributor

App timing

A) Starting at the beginning of the month, there is a spike in homepage initial render on Desktop:
Screenshot 2024-04-10 at 10 56 59
- This is not affecting the other platforms, and at the same time we see a major improvement for iOS:
Screenshot 2024-04-10 at 10 57 13
Screenshot 2024-04-10 at 10 57 46
As the Desktop regression has also been deployed for 10 days, this shouldn't block the deploy, but I will look into this and share in relevant channels after closing this release

B) QA reported a performance regression on Android where rendering is taking an average of 20/30 seconds, which we have data for. This doesn't seem to be reproducable, but I am holding the deploy while we investigate further to be sure about this.

@Julesssss
Copy link
Contributor

As discussed, I'm going to ship as there is no real path for resolving this vague and currently unreproducable Android performance regression.

@Julesssss
Copy link
Contributor

Performance regression is being worked on here

@roryabraham
Copy link
Contributor

heads up that we just CP'd a fix to a fire room - let's make sure that PR is tested and goes out too: https://expensify.slack.com/archives/C07J32337/p1712765918676959

@rlinoz
Copy link
Contributor

rlinoz commented Apr 10, 2024

Checking off #40025

@Julesssss
Copy link
Contributor

checking off #39979, the fire exists in prior releases and although not confirmed fixed, hasn't further broken the app

@Julesssss
Copy link
Contributor

Status from earlier are still valid: #39807 (comment)

@Julesssss
Copy link
Contributor

Julesssss commented Apr 10, 2024

There's an issue with Github codespaces, which is not relevant to our deploy

Investigating - We are investigating reports of degraded performance for Codespaces
Apr 10, 2024 - 16:12 UTC

@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