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

Backport: pkg/helm/controller/reconcile.go: fix conditions logic (#1321) #1370

Merged
merged 1 commit into from
May 2, 2019

Conversation

joelanford
Copy link
Member

Description of the change:
Backport #1321 into v0.7.x

Motivation for the change:
We backport bug fixes to previous release.

…ework#1321)

* pkg/helm/controller/reconcile.go: fix conditions logic

If a change is made to the CR spec that causes a release failure, a ConditionReleaseFailed is added to the status conditions. If that change is then reverted to its previous state, the operator will stop attempting the release and will resume reconciling.

In this case, we need to remove the ConditionReleaseFailed because the failing release is no longer being attempted.

* CHANGELOG.md: add bug fix operator-framework#1321
@openshift-ci-robot openshift-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label May 2, 2019
@joelanford joelanford changed the title pkg/helm/controller/reconcile.go: fix conditions logic (#1321) Backport: pkg/helm/controller/reconcile.go: fix conditions logic (#1321) May 2, 2019
Copy link
Contributor

@hasbro17 hasbro17 left a comment

Choose a reason for hiding this comment

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

LGTM

@joelanford joelanford merged commit 5c3f889 into operator-framework:v0.7.x May 2, 2019
@joelanford joelanford deleted the backport-1321 branch May 2, 2019 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants