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

stop pruning events on deploy to avoid timeouts #909

Merged
merged 2 commits into from
Nov 18, 2022

Conversation

ethanaubuchon
Copy link

What are you trying to accomplish with this PR?
Closes #908

How is this accomplished?
This PR adds Event to the deny list of resources to prune on deploy.

@ethanaubuchon ethanaubuchon requested a review from a team as a code owner November 17, 2022 21:02
@ethanaubuchon ethanaubuchon self-assigned this Nov 17, 2022
@ethanaubuchon ethanaubuchon requested review from d1egoaz and peiranliushop and removed request for a team November 17, 2022 21:02
Copy link
Contributor

@d1egoaz d1egoaz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Might want to add an entry in the Changelog in the next section to signal this PR changes to let users know that they'll need to deal with pruning events in the future.

@@ -1,5 +1,6 @@
## next

- Add `Event` to the list of resources not to be pruned on a deployment. Kubernetes should clean these up automatically. This can cause unnessessary deployment timeouts and increased deployment time.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@ethanaubuchon ethanaubuchon merged commit 763b826 into master Nov 18, 2022
@ethanaubuchon ethanaubuchon mentioned this pull request Nov 18, 2022
@shopify-shipit shopify-shipit bot temporarily deployed to rubygems November 18, 2022 15:01 Inactive
@shopify-shipit shopify-shipit bot temporarily deployed to rubygems September 26, 2023 12:55 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Stop pruning Event resources on deploy.
3 participants