Skip to content

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

Closed

Description

@github-actions

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

Activity

  1. Julesssss commented on Apr 8, 2024

    @Julesssss
    Contributor

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

  2. mountiny commented on Apr 8, 2024

    @mountiny
    Contributor

    Demoted #39827

  3. mountiny commented on Apr 8, 2024

    @mountiny
    Contributor

    Demoted #39841

  4. mountiny commented on Apr 8, 2024

    @mountiny
    Contributor

    #39834 fixed by backend change

  5. luacmartins commented on Apr 8, 2024

    @luacmartins
    Contributor

    Demoted #39838

  6. mountiny commented on Apr 8, 2024

    @mountiny
    Contributor

    Demoted #39845
    Demoted #39847

  7. marcaaron commented on Apr 8, 2024

    @marcaaron
    Contributor

    Demoted #39851

  8. mountiny commented on Apr 8, 2024

    @mountiny
    Contributor

    demoted #39855

  9. jasperhuangg commented on Apr 8, 2024

    @jasperhuangg
    Contributor

    Demoted #39853 NAB

  10. kavimuru commented on Apr 8, 2024

    @kavimuru

    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

  11. srikarparsi commented on Apr 8, 2024

    @srikarparsi
    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.

  12. gijoe0295 commented on Apr 9, 2024

    @gijoe0295
    Contributor

    #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.

  13. 11 remaining items

  14. Julesssss commented on Apr 10, 2024

    @Julesssss
    Contributor

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

  15. Julesssss commented on Apr 10, 2024

    @Julesssss
    Contributor
  16. Julesssss commented on Apr 10, 2024

    @Julesssss
    Contributor

    #39938 confirmed not a blocker here

  17. Julesssss commented on Apr 10, 2024

    @Julesssss
    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.

  18. Julesssss commented on Apr 10, 2024

    @Julesssss
    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
  19. Julesssss commented on Apr 10, 2024

    @Julesssss
    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.

  20. Julesssss commented on Apr 10, 2024

    @Julesssss
    Contributor

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

  21. Julesssss commented on Apr 10, 2024

    @Julesssss
    Contributor

    Performance regression is being worked on here

  22. roryabraham commented on Apr 10, 2024

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

  23. rlinoz commented on Apr 10, 2024

    @rlinoz
    Contributor

    Checking off #40025

  24. Julesssss commented on Apr 10, 2024

    @Julesssss
    Contributor

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

  25. Julesssss commented on Apr 10, 2024

    @Julesssss
    Contributor

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

  26. Julesssss commented on Apr 10, 2024

    @Julesssss
    Contributor

    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

  27. Julesssss commented on Apr 10, 2024

    @Julesssss
    Contributor

    :shipit:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions