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

Separate OrphanMitigation condition? #1771

Closed
nilebox opened this issue Feb 28, 2018 · 1 comment · Fixed by #1789
Closed

Separate OrphanMitigation condition? #1771

nilebox opened this issue Feb 28, 2018 · 1 comment · Fixed by #1789

Comments

@nilebox
Copy link
Contributor

nilebox commented Feb 28, 2018

With addressing #1715 and support for retries, it becomes harder to report on the ongoing orphan mitigation.

As @staebler suggested in the comment, we can introduce a new OrphanMitigation condition, where we could report the status of orphan mitigation (reason and message) separately from Ready and Failed conditions.

@nilebox
Copy link
Contributor Author

nilebox commented Feb 28, 2018

Another issue is that after exceeding the retry limit we reset the OrphanMitigationInProgress = false, which means that after the spec is updated we will try to provision the instance instead of trying to mitigate orphan first.
Having an OphanMitigation condition could help there.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant