-
Notifications
You must be signed in to change notification settings - Fork 169
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
Add maintenance state for customer action needed #3294
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Code Review December 13th:
|
Reviewed with @bennerv and @SudoBrendan. We'll not support blocking transition from "customer action needed" to "maintenance ongoing." No code changes needed - code as is supports this. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
LGTM |
/azp run ci |
Azure Pipelines successfully started running 1 pipeline(s). |
Which issue this PR addresses:
Fixes https://issues.redhat.com/browse/ARO-4365
What this PR does / why we need it:
If a maintenance operation fails, we may need the customer to take action. We will notify the customer via a resource health check signal.
This PR implements a new signal to tell the customer to take action. This PR implements arrows 11, 12 and 13 in this diagram.
Test plan for issue:
Is there any documentation that needs to be updated for this PR?
See above diagram link