Skip to content
This repository has been archived by the owner on Jun 29, 2022. It is now read-only.

feat: define MaxHistory for Helm releases #1549

Merged
merged 1 commit into from Aug 25, 2021

Conversation

surajssd
Copy link
Member

Defining the MaxHistory option during the rollback Helm action so that
the releases history is cleaned up accordingly.
Without this option the history would never been cleared which is
accumulating old configuration which can be harmful when a incorrect
rollback is applied. By cleaning up the history, we limit the potential
impact rolling back to a older version.

ipochi
ipochi previously approved these changes Aug 20, 2021
Copy link
Member

@ipochi ipochi left a comment

Choose a reason for hiding this comment

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

LGTM

@ipochi
Copy link
Member

ipochi commented Aug 20, 2021

@surajssd
#1548 was opened for the same.
Can one of this be closed please.

Defining the `MaxHistory` option during the rollback Helm action so that
the releases history is cleaned up accordingly.
Without this option the history would never been cleared which is
accumulating old configuration which can be harmful when a incorrect
rollback is applied. By cleaning up the history, we limit the potential
impact rolling back to a older version.
Copy link
Member

@knrt10 knrt10 left a comment

Choose a reason for hiding this comment

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

+1

@surajssd surajssd merged commit 0b2c14b into master Aug 25, 2021
@surajssd surajssd deleted the niels/cleanup_helm_history branch August 25, 2021 03:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants