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-03-04 #37698

Closed
97 tasks done
github-actions bot opened this issue Mar 4, 2024 · 30 comments
Closed
97 tasks done

Deploy Checklist: New Expensify 2024-03-04 #37698

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

Comments

@github-actions
Copy link
Contributor

github-actions bot commented Mar 4, 2024

Release Version: 1.4.47-10
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 Mar 4, 2024
@OSBotify
Copy link
Contributor

OSBotify commented Mar 4, 2024

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

@rlinoz
Copy link
Contributor

rlinoz commented Mar 5, 2024

Checking off #37658 as it was CP'd and tested yesterday, just forgot to remove the deploy blocker label.

@Julesssss
Copy link
Contributor

Checked off #37756 as it is a bit of an edge case with a simple work around.

@luacmartins
Copy link
Contributor

luacmartins commented Mar 5, 2024

Demoted #37748, feature is still WIP

@grgia
Copy link
Contributor

grgia commented Mar 5, 2024

demoted #37757, same as above

@kavimuru
Copy link

kavimuru commented Mar 5, 2024

Regression is 10% left in the TR
#36920 - seems does not require QA checking off
#37202 failing with #37710
#37213 failing with #37764

@roryabraham
Copy link
Contributor

checked off #37742 because @puneetlath closed it

@luacmartins
Copy link
Contributor

Checked off #37766. Can't reproduce, demoted.

@roryabraham
Copy link
Contributor

#37749 and #37738 were fixed by #37778. Checking all three of those off

@kavimuru
Copy link

kavimuru commented Mar 5, 2024

Regression is 1% left.
#37202 failing with #37710
#37213 failing with #37764
#37584 - blocked in desktop, couldn't anonymously initiate deeplink with #room to open it in desktop (OK to check it off)?
#36278 - #36278 (comment)
PRs yet to be validated:
#36591 - to be validated in android platform
#36764 - to be validated in all the platforms
#37213 - to be validated in desktop
#37585 - to be validated in desktop
#37589 - to be validated in mweb desktop and android
#37620 - to be validated in desktop
#37663 - to be validated in desktop

@kavimuru
Copy link

kavimuru commented Mar 5, 2024

Regression is completed.

unchecked PRs:
#37202 failing with #37710
#37213 failing with #37764
#36278 -#36278 (comment)
#37584 - blocked in desktop, couldn't anonymously initiate deeplink with #room to open it in desktop

2 cps and 1 PR #36764 to be validated in all the platforms.
I will post the final summary when all the PRs and Cps are done.

@kavimuru
Copy link

kavimuru commented Mar 6, 2024

Regression is completed.
We still have the following PRs to be validated:
#36764 - mWeb, desktop and iOS validations are pending
#37786 - mweb, and native apps validations are pending

@mvtglobally
Copy link

Checking off
#37772
#37786

Following look resolved now
#37746 (comment)
#37743 (comment)

Last blocker doesn't seem to be fully fixed.User is not staying on Sign in, but need to refresh to navigate to chats
#37752 (comment)

@mvtglobally
Copy link

@roryabraham

Latest Summary
TR is completed

Unchecked PRs
#36278 - We are blocked. Can you validate internally? #36278 (comment)
#36764 - Web/Android pass. Working to check 3 other platforms
#37202 - failing with #37710
#37213 - failing with #37764
#37584 - Are we ok to check this one off? We are blocked to validate on Desktop due to deeplinking limitations. Rest platforms pass.
#37804 - 2 blockers pass, 1 needs validation if good to check off.

Still open blockers
#37289
#37750
#37752
#37758
#37764
#37779

@roryabraham
Copy link
Contributor

#37746 is resolved, checking it off

@roryabraham
Copy link
Contributor

checking off #37584

@roryabraham
Copy link
Contributor

slack threads for summary and action plan: https://expensify.slack.com/archives/C01GTK53T8Q/p1709705296107569

@grgia
Copy link
Contributor

grgia commented Mar 6, 2024

Checking off QA for #36278 (added screenshots to PR thread)

@kavimuru
Copy link

kavimuru commented Mar 6, 2024

#37758 is fixed in the 1.4.47-7. Checked it off.

@kavimuru
Copy link

kavimuru commented Mar 6, 2024

@roryabraham TR is completed.

unchecked PRs:
#37202 failing with #37710
#37213 failing with #37764
#36764 - iOS platform is being validated and other platforms are a pass

Open Blockers:
#37289
#37750
#37752
#37764
#37779

@luacmartins
Copy link
Contributor

Checked off #37764. Demoted to NAB since the fix is in Auth/Web-E and it's an UI glitch.

@kavimuru
Copy link

kavimuru commented Mar 6, 2024

@roryabraham Regression is completed.

unchecked PRs:
#37202 failing with #37710
#37213 failing with #37764 (can we check it off as per this comment)

Open Blockers:
#37289
#37750
#37752
#37779

@roryabraham
Copy link
Contributor

#37289 NAB, checking this off

@luacmartins
Copy link
Contributor

Checking off #37779. Tested in staging and it's a pass.

@roryabraham
Copy link
Contributor

we just CP'd a fix to the last blocker but it's VERY IMPORTANT we do not ship this checklist yet. We want to include #37839 to fix a fire in production as fast as possible

@roryabraham
Copy link
Contributor

CPing #37839 in 1.4.47-10 to fix a fire

@roryabraham
Copy link
Contributor

#37202 wasn't directly merged, but is shown as merged because #37840 includes the same commits plus some and was merged. Since #37840 is checked off, I'm also checking off #37202

@roryabraham
Copy link
Contributor

verified #37839 on staging on all platforms

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

9 participants