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

[CI] Always clean up release channels #7130

Merged
merged 1 commit into from
Oct 30, 2019
Merged

Conversation

christophermaier
Copy link
Contributor

Previously, we deleted release channels at the end of successful
release pipeline runs. However, failed and canceled runs would leave
channels behind. Over time, this can accumulate a lot of extra
channels, which can (with the current implementation of things) slow
down build operations.

Here, we remove this responsibility from the Buildkite pipeline, and
instead allow Expeditor to handle it. This allows us to remove the
offending channel all the time, regardless of what happens in the
pipeline.

Signed-off-by: Christopher Maier cmaier@chef.io

Copy link
Contributor

@scotthain scotthain left a comment

Choose a reason for hiding this comment

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

The only thing to note is this means the scripts will be run on the expeditor machine, not in a pipeline step.

That being said, I think the var name is wrong?

Previously, we deleted release channels at the end of successful
release pipeline runs. However, failed and canceled runs would leave
channels behind. Over time, this can accumulate a lot of extra
channels, which can (with the current implementation of things) slow
down build operations.

Here, we remove this responsibility from the Buildkite pipeline, and
instead allow Expeditor to handle it. This allows us to remove the
offending channel all the time, regardless of what happens in the
pipeline.

Signed-off-by: Christopher Maier <cmaier@chef.io>
@christophermaier christophermaier merged commit d3a04a9 into master Oct 30, 2019
@chef-expeditor chef-expeditor bot deleted the cm/cleanup-channels branch October 30, 2019 13:59
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.

2 participants