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

Operator doesn't show the status for Kepler #315

Open
vprashar2929 opened this issue Nov 30, 2023 · 4 comments
Open

Operator doesn't show the status for Kepler #315

vprashar2929 opened this issue Nov 30, 2023 · 4 comments

Comments

@vprashar2929
Copy link
Collaborator

We don't see any exporter status when deploying Kepler using the latest bundle quay.io/sustainable_computing_io/kepler-operator-bundle:v1alpha1 on the OpenShift console.

Below attached is the screenshot for reference:

Screenshot 2023-11-30 at 7 04 23 PM

@sunya-ch
Copy link
Collaborator

sunya-ch commented Dec 8, 2023

There is a bug. I found it during implementing the model server integration #322
Mentioned here: #322 (comment)

@sthaha If the PR #322 needs more time to merge, I can create a separated PRs to those fix two bugs first.

Please ignore the above comment. It turns out that I mixed the issue.
I got the same problem since I assigned the .namespace in KeplerInternal to openshift-kepler-operator which is not recognized since the default one is now go to kepler-operator namespace. Now sure that you are the same case or not.

It will be helpful to run oc describe kepler kepler. It will tell you more the error of status update.

@KaiyiLiu1234
Copy link
Collaborator

So the issue regarding why the status is not shown is because the OLM does not seem to handle nested fields very well. Conditions is nested inside of ExporterStatus in the KeplerStatus struct. Note KeplerStatus only contains one field which is ExporterStatus. One strategy is to flatten ExporterStatus and pull out the Conditions. The Conditions can then be populated by ExporterStatus or other Statuses in the future like ModelServer. This may not be necessary because KeplerInternal should handle this? Is this strategy ok? I am not sure if there is another way.

@KaiyiLiu1234
Copy link
Collaborator

Below is attached showing the status being filled. I replaced it with a dummy Condition but I can transfer exporter status conditions over as well.
part_1_success
part_2_success

@sthaha
Copy link
Collaborator

sthaha commented Feb 26, 2024

@KaiyiLiu1234 , thanks for the update. I think issue doesn't make it clear what we are trying to achieve. The object of this task is to explore if UI honours - https://github.com/openshift/console/blob/master/frontend/packages/operator-lifecycle-manager/src/components/descriptors/reference/reference.md#3-conditions and we can specify the path to status.exporter.conditions to render it.

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

No branches or pull requests

4 participants