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

[OLM] Fix operator UI and add new fields for operator #902

Merged
merged 3 commits into from
Jul 17, 2024
Merged

[OLM] Fix operator UI and add new fields for operator #902

merged 3 commits into from
Jul 17, 2024

Conversation

jankalinic
Copy link
Contributor

This PR fixes errors with operator display page/sidebar in openshift UI. Also it adds new fields such as creation timestamp, support, certified, categories, repository link, operator image sha etc. The procedure of creating a catalog image with such fields is following -> First a CSV yaml file is created from CSVMetadata annotation using operator-sdk. Then it's placeholder values are replaced using yq and skopeo (both need to be installed) and a catalog image is created.

Makefile Outdated Show resolved Hide resolved
@jankalinic
Copy link
Contributor Author

I think this time it's not related to this PR. it fails on PW test

// Click on the "Connection details" button within the dropdown menu
    > 23 |     await page.click(
         |                ^
      24 |       'button[class="pf-v5-c-menu__item"] span[class="pf-v5-c-menu__item-text"]:has-text("Connection details")'
      25 |     );
      ```

@MikeEdgar MikeEdgar added this to the 0.2.1 milestone Jul 17, 2024
Signed-off-by: jkalinic <[email protected]>
Copy link

Copy link
Member

@MikeEdgar MikeEdgar left a comment

Choose a reason for hiding this comment

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

LGTM, thank you @jankalinic .

@MikeEdgar MikeEdgar merged commit da42c17 into streamshub:main Jul 17, 2024
6 checks passed
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

Successfully merging this pull request may close these issues.

2 participants