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 2022-03-17 #8210

Closed
31 tasks done
OSBotify opened this issue Mar 17, 2022 · 30 comments
Closed
31 tasks done

Deploy Checklist: New Expensify 2022-03-17 #8210

OSBotify opened this issue Mar 17, 2022 · 30 comments
Assignees
Labels
🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging Monthly KSv2 StagingDeployCash

Comments

@OSBotify
Copy link
Contributor

OSBotify commented Mar 17, 2022

@mvtglobally mvtglobally added the 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging label Mar 17, 2022
@OSBotify
Copy link
Contributor Author

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

@OSBotify
Copy link
Contributor Author

👏 Heads up @Expensify/applauseleads 👏
A new pull request has been 🍒 cherry-picked 🍒 to staging, and will be deployed to staging in version 1.1.44-1 🚀

@OSBotify
Copy link
Contributor Author

🎉 All set?…You bet! @Expensify/applauseleads https://github.com/Expensify/App/releases/tag/1.1.44-1 has been deployed to staging 🎉

@roryabraham
Copy link
Contributor

Checked off #8063 since it passed QA 👍

@OSBotify
Copy link
Contributor Author

👏 Heads up @Expensify/applauseleads 👏
A new pull request has been 🍒 cherry-picked 🍒 to staging, and will be deployed to staging in version 1.1.44-2 🚀

@OSBotify
Copy link
Contributor Author

🎉 All set?…You bet! @Expensify/applauseleads https://github.com/Expensify/App/releases/tag/1.1.44-2 has been deployed to staging 🎉

@roryabraham
Copy link
Contributor

Maybe a regression in our checklist logic: #8186 was deployed in the last cycle, should not have been included in this checklist.

@marcaaron
Copy link
Contributor

Just tested #8204 on staging and verified it fixes the issue reported here https://github.com/Expensify/Expensify/issues/201705

@roryabraham
Copy link
Contributor

Checking off #8215 as it was closed by @neil-marcellini

@mvtglobally
Copy link

Regression is complete
#8176 is not fixed in mWeb, but it seems Safari limitation. Checking off #8215
Issues
#8212
#8220

Validating last PR and will update shortly

@neil-marcellini
Copy link
Contributor

#8204 is not working on staging currently.

@mvtglobally
Copy link

#8204 (comment)

@MelvinBot MelvinBot added the Monthly KSv2 label Mar 21, 2022
@mvtglobally
Copy link

@AndrewGable Anything we can do to help and move this checklist forward.. #8204 is the only pending item. Does it block checklist?

@AndrewGable
Copy link
Contributor

Latest updates are here: https://github.com/Expensify/Expensify/issues/201705

Yes, I believe it needs to pass, @neil-marcellini is working on a fix.

@neil-marcellini
Copy link
Contributor

Update: I have a draft PR up as a potential fix. I'm going through the process of testing out all the flows, so that part is a little bit time consuming.

@neil-marcellini
Copy link
Contributor

My PR for #8204 is ready for review.

@mvtglobally
Copy link

@AndrewGable let's see if we can push this checklist today. QA is on standby for any validations needed

@roryabraham
Copy link
Contributor

I'm actively working on the OldDot -> NewDot transition. @mvtglobally It would be very helpful if you could link me to all the existing regression tests we have to cover this flow.

@OSBotify
Copy link
Contributor Author

👏 Heads up @Expensify/applauseleads 👏
A new pull request has been 🍒 cherry-picked 🍒 to staging, and will be deployed to staging in version 1.1.44-4 🚀

@OSBotify
Copy link
Contributor Author

🎉 All set?…You bet! @Expensify/applauseleads https://github.com/Expensify/App/releases/tag/1.1.44-4 has been deployed to staging 🎉

@marcaaron
Copy link
Contributor

marcaaron commented Mar 24, 2022

Heads up we CP'd a couple of PRs to fix the NewDot Transition. Would be good if we can get some of those flows re-tested on staging web. I tested and they seem to be working well.

The only thing that does not seem to work well is transitioning from OldDot while logged in to NewDot with a different account. While testing this flow I kept getting logged out shortly after the transition happens. That particular flow isn't really a mainline flow though it should be working. The flow works on dev for me so I have a hunch it will get sorted with the next release. But I do not see an obvious reason why it should fail.

@mvtglobally
Copy link

@AndrewGable #8257 is pass aside of this issue. Its repro in PROD so not sure if its a blocker #8295

@AndrewGable
Copy link
Contributor

If it's already in production then no it is not a deploy blocker, thank you, I will deploy shortly!

@AndrewGable
Copy link
Contributor

:shipit:

@OSBotify
Copy link
Contributor Author

This issue either has unchecked QA steps or has not yet been marked with the :shipit: emoji of approval.
Reopening!

@OSBotify OSBotify reopened this Mar 24, 2022
@AndrewGable
Copy link
Contributor

:shipit:

@mountiny
Copy link
Contributor

Just noticed that none of the PRs in this deployed got a Production deploy message from Botify, which also meant that the contributor issues were not labelled as waiting for payment. Has this been investigated? 👀 cc @AndrewGable

For example this.

@AndrewGable
Copy link
Contributor

I have not investigated this no, I am not sure if this is related to the new bot or GitHub outages or what

@mountiny
Copy link
Contributor

Thanks for confirming! I have went through all the contributor PRs and updated them accordingly so they are paid out on time 👍

Also, I checked and it seems that this was being deployed at the time of the Github outage last week. The list was closed at 16:41 GMT:

image

And Github status page says that that time they had problems
image

Let's assume it was this case since all seems to work in the next deploys :) Thanks!

@Stutikuls
Copy link

Stutikuls commented Mar 31, 2022

Checked all PRs and created accessibility issues for all PRs where accessibility issue were found.
The non-applicable PRs are - #8063, #8082, #8149, #8184, #8186, #8191, #8204 ,#8211, #8257
logged Issues - #8387 , #8388 , #8389, #8383, #8384, #8385, #8382

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 Monthly KSv2 StagingDeployCash
Projects
None yet
Development

No branches or pull requests

10 participants