Skip to content

Update operator api resources with a status to indicate solver success/failure #73

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

Closed
awgreene opened this issue Dec 7, 2022 · 0 comments · Fixed by #91
Closed

Update operator api resources with a status to indicate solver success/failure #73

awgreene opened this issue Dec 7, 2022 · 0 comments · Fixed by #91
Assignees

Comments

@awgreene
Copy link
Member

awgreene commented Dec 7, 2022

User Story:

As a user, if I create an Operator CR that cannot be fulfilled for any reason, I would like the reason for the failure to be surfaced in the Operator CR's status.

Open Questions:

Acceptance Criteria:

  • Unmet constraints are surfaced in the Operator CR's status.
@awgreene awgreene added this to OLM v1 Dec 7, 2022
@awgreene awgreene converted this from a draft issue Dec 7, 2022
@awgreene awgreene moved this to Todo in OLM v1 Dec 7, 2022
@tmshort tmshort moved this from Todo to In Progress in OLM v1 Jan 5, 2023
tmshort added a commit to tmshort/operator-controller that referenced this issue Jan 11, 2023
* Adds an initial Condition: "Ready" with a "False" status.
* Fixed operator-framework#73
* Include unit-tests

Signed-off-by: Todd Short <[email protected]>
tmshort added a commit that referenced this issue Jan 17, 2023
* Add Conditions to status
* Adds an initial Condition: "Ready" with a "False" status.
* Fixed #73
* Include unit-tests

Signed-off-by: Todd Short <[email protected]>
@github-project-automation github-project-automation bot moved this from In Progress to Done in OLM v1 Jan 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants