Skip to content
This repository has been archived by the owner on Jan 8, 2024. It is now read-only.

Prune old releases #3730

Merged
merged 4 commits into from
Aug 31, 2022
Merged

Conversation

paladin-devops
Copy link
Contributor

Closes #2131.

@paladin-devops paladin-devops marked this pull request as ready for review August 23, 2022 18:22
@paladin-devops paladin-devops changed the title Prune old deployments Prune old releases Aug 23, 2022
@paladin-devops paladin-devops requested a review from a team August 23, 2022 19:22
Copy link
Member

@briancain briancain left a comment

Choose a reason for hiding this comment

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

Makes sense to me! Blocking on my question for some code simplification

internal/runner/operation_release.go Outdated Show resolved Hide resolved
internal/runner/operation_release.go Outdated Show resolved Hide resolved
internal/runner/operation_release.go Outdated Show resolved Hide resolved
internal/runner/operation_release.go Outdated Show resolved Hide resolved
@paladin-devops paladin-devops merged commit 2c7aa88 into main Aug 31, 2022
@paladin-devops paladin-devops deleted the b-prune-releases-for-destroyed-deployments branch August 31, 2022 14:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Releases should be destroyed when their corresponding deployments are destroyed
3 participants