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-11-19 #52733

Closed
54 tasks done
github-actions bot opened this issue Nov 19, 2024 · 23 comments
Closed
54 tasks done

Deploy Checklist: New Expensify 2024-11-19 #52733

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

Comments

@github-actions
Copy link
Contributor

github-actions bot commented Nov 19, 2024

Release Version: 9.0.64-4
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 Nov 19, 2024
@OSBotify
Copy link
Contributor

🚀 All staging deploys are complete, @Expensify/applauseleads please begin QA on version https://github.com/Expensify/App/releases/tag/9.0.64-0 🚀

@izarutskaya
Copy link

Regression is at 30% completed.
We waiting iOS build

Unchecked PRs:
#51444 - Failing on Web with original #50671
#51625 - Failing with original #50418 in offline tests

Blockers:
#52744
#52746
#52747

@grgia
Copy link
Contributor

grgia commented Nov 19, 2024

Demoted #52755

@cristipaval
Copy link
Contributor

Checked off #52744 as it's been cp'd.

@Julesssss
Copy link
Contributor

I haven't checked it off yet, but #52749 should have been fixed by the latest CP. I asked here for a retest

@kavimuru
Copy link

Regression is 25% left
iOS hybrid app is not updated yet and all the PRs are pending validation in iOS platform.

Unchecked PRs
#51444 - Failing on Web with original #50671
#51625 - Failing with original #50418 in offline tests
#51724 failing in android with #52772 repro in PROD, will check it of once verified in iOS
#52163 blocked with #52746
#52526 failing with #52756
#52653 failing with #52760 repro in PROD, will check it off once verified in iOS

@izarutskaya
Copy link

izarutskaya commented Nov 20, 2024

Regression is 22% left

Unchecked PRs:
#51444 - Failing on Web with original #50671
#51625 - Failing with original #50418 in offline tests
#51724 - Failing on Android with #52772 Pending validation on iOS
#52163 - blocked with #52746
#52021 - KI found #52747
#52309 - KI found #52748
#52354 - KI found #52747
#52526 - Failing on #52756
#52653 - Failing with #52760
#52752 - KI #52746

Blockers:
#52746
#52747
#52753
#52759
#52787

@Beamanator
Copy link
Contributor

Checked off #52759 b/c i couldn't reproduce

@rlinoz
Copy link
Contributor

rlinoz commented Nov 20, 2024

Checking off #52787 fixed here #52787 (comment)

@kavimuru
Copy link

kavimuru commented Nov 20, 2024

Regression is 16% left

Unchecked PRs:
#51440 - iOS validation is pending
#51444 - iOS validation is pending and also
failing on Web with original
#50671
#51625 - Failing with original #50418 in offline tests and iOS validation is pending
#51724 - with #52772 in iOS and Android, repro in PROD checking it off
#51926 - iOS validation is pending
#52309 - failing with #52748 repro in PROD. I will check it off once the iOS platform testing completed.
#52407 Android and iOS validations are pending
#52462 - iOS validation is pending
#52526 - Failing on #52756, iOS and Android validations are pending
#52610 - Pending validation in iOS
#52653 - Failing with #52760 and iOS validation is pending
#52656 - iOS validation is pending
#52657 - iOS validation is pending
#52660 - iOS validation is pending
#52694 - iOS validation is pending
#52533 - iOS validation is pending

@chiragsalian
Copy link
Contributor

@kavimuru can you retest - #52746 when you get the chance. We CP'd a revert to address the issue.

@kavimuru
Copy link

@chiragsalian bug is fixed in both iOS and Android.

@kavimuru
Copy link

Regression is 5% left

Unchecked PRs:
#51444 - Failing on Web with original bug #50671
#51625 - Failing with original #50418 in offline tests
#52407 blocked, user is redirected to mweb clicking on the magic code
#52526 - Failing on #52756 and iIn android and iOS this is blocked and commented here #52526 (comment)
#52653 - Failing with #52760
#52656 - blocked in android with #52199

@chiragsalian
Copy link
Contributor

#51444 - i checked it off and left a comment in the PR for author to make a follow up.
#51625 - i checked it off. Looks like there is an ongoing discussion and it doesn't sound like a blocker for us regardless.
#52407 - i don't follow what you posted @kavimuru, i don't think the feature is supposed to work on mobile native. and if it works fine on mweb that sounds good to me. Can you rephrase what the issue is? Also feel free to post on the issue for the author/reviewers to also confirm.
#52526 - Looks like there is an active discussion in #52756 and it doesn't look like a blocker.
#52653 - Looks like the failing issue will be handled separately in #52760. Seems like a fix is already out but just not CP'd so no reason to block on this. Checking it off.
#52656 - Checking it off since its working everywhere else and #52199 is currently in progress.

@kavimuru
Copy link

Regression is completed

#52407 This is the behavior in app

RPReplay_Final1732117400.MP4

@chiragsalian
Copy link
Contributor

Hmm to test #52407 you need to logout first. i don't think its intended to do anything if you are logged in.

@chiragsalian
Copy link
Contributor

I did a quick test of #52407 on web and mobile and it looks fine to me so I'm checking it off the list.

@kavimuru
Copy link

It is working in both web, desktop and mweb, but in native it is not working.

az_recorder_20241120_193749.1.mp4

@chiragsalian
Copy link
Contributor

@kavimuru, could you retest this blocker - #52744.
The PR to fix it has been CP'd.

@chiragsalian
Copy link
Contributor

It is working in both web, desktop and mweb, but in native it is not working.

Sounds good. I informed the author. I think its fine to keep it checked off in the checklist since it shouldn't be a blocker for us.

@kavimuru
Copy link

kavimuru commented Nov 21, 2024

Bug is not reproduced #52744

screenrecording_11-21-2024_07-53-14_1.mp4

@chiragsalian
Copy link
Contributor

#52526 - I'll check this off. Looks like there is an active discussion in #52756 and it doesn't look like it should be a blocker to me.

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