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-22 #40702

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

Deploy Checklist: New Expensify 2024-04-22 #40702

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

Release Version: 1.4.64-6
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 22, 2024
@mountiny mountiny added the Daily KSv2 label Apr 22, 2024
@mountiny
Copy link
Contributor

@Expensify/applauseleads to confirm this is ready to start the regression testing

@mountiny
Copy link
Contributor

Also checked off the deploy blockers, which were incorrectly added from the previous build. I forgot to remove the labels.

@francoisl
Copy link
Contributor

Closed #40717, the change was intended.

@chiragsalian
Copy link
Contributor

Checking off #40719 since it doesn't feel blocker worthy imo.

@mountiny
Copy link
Contributor

demoting #40722

@kavimuru
Copy link

kavimuru commented Apr 22, 2024

Regression is 43% progress.
#35041 failing with fixed issue #34120 commented here
#39397 failing with #38577 in mweb, issue not fixed. comment
#40334 failing with #39798 in all platforms except Web. comment

@blimpich
Copy link
Contributor

Demoting #40721 as its a minor UI bug and PR is already out to fix it.

@mountiny
Copy link
Contributor

Demoting #40746

@mountiny
Copy link
Contributor

demoted #40747 minor ux issues

@mountiny
Copy link
Contributor

demoted #40749 minor ux issue

@mountiny
Copy link
Contributor

demoted #40751 minor ui issue

@mountiny
Copy link
Contributor

demoting #40752 its expected for now and we need backend change to fix it

@mountiny
Copy link
Contributor

demoting #40762 as its a new feature and not really blocking behaviour

@mountiny
Copy link
Contributor

demoted #40756

@mountiny
Copy link
Contributor

mountiny commented Apr 23, 2024

demoted #40724, it feels like a minor ux issue

@mountiny
Copy link
Contributor

demoting #40755 as taxes are not yet fully supported in the split expense flow

@mountiny
Copy link
Contributor

#40745 was demoted

@kavimuru
Copy link

kavimuru commented Apr 23, 2024

Regression is just 7 steps left to run in the TR
#35041 failing with fixed issue #34120 commented here
#35822 failing with #40717 OK to check it off since it is expected behavior?
#39397 failing with #38577 in mweb, issue not fixed. comment
#40334 failing with 39798 in all platforms except Web. comment
#40393 failing with #40725
#38543 failing with #40762
#39413 failing with #40790 repro in production, checking it off
#39951 validation pending in Web, mweb, desktop and iOS
#40176 validation pending in android and iOS
#40619 validation pending in desktop
#40659 Validation pending in android and iOS
#40298 comment

@kavimuru
Copy link

kavimuru commented Apr 23, 2024

Regression is completed
unchecked PRs:
#35041 failing with fixed issue #34120 commented #35041 (comment)
#35822 failing with #40717 OK to check it off since it is expected behavior?
#39397 failing with #38577 in mweb, issue not fixed. #39397 (comment)
#40334 failing with 39798 in all platforms except Web. #40334 (comment)
#40393 failing with #40725
#38543 failing with #40762
#40619 is failing with #40820 checking it off
#40298 #40298 (comment)

Open Blockers:
#40712
#40723
#40724
#40743
#40754
#40818

@mountiny
Copy link
Contributor

demoted #40818

@mountiny
Copy link
Contributor

checking off:

@mountiny
Copy link
Contributor

Shared one expo related crash from android with SWM

Graphs looking good

@mountiny mountiny self-assigned this Apr 23, 2024
@mountiny
Copy link
Contributor

verified #40286 on all platforms except android, but its platform agnostic change so proceeding with deploy

@mountiny
Copy link
Contributor

All clear

@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

6 participants