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

Fix Hidden property of linux-polska/ezd-crd upstream.yaml and annotations on its latest chart version #1075

Open
wants to merge 1 commit into
base: main-source
Choose a base branch
from

Conversation

adamkpickering
Copy link
Collaborator

There was a bug in partner-charts-ci (see rancher/partner-charts-ci#30) that resulted in the latest version of the ezd-crd chart not having the catalog.cattle.io/hidden annotation. This PR fixes that. It also adds Hidden: true to the upstream.yaml of linux-polska/ezd-crd.

@adamkpickering adamkpickering requested a review from a team as a code owner October 9, 2024 22:39
@@ -1,6 +1,7 @@
annotations:
catalog.cattle.io/certified: partner
catalog.cattle.io/display-name: CRDs for LP Backend
catalog.cattle.io/hidden: "true"
Copy link
Collaborator

Choose a reason for hiding this comment

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

What is the meaning of this annotation?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It causes the chart to not be shown by the UI. In the partner charts repository it is used in only the linux-polska/ezd-crd package (this one). I think it is used here because it is effectively a CRD chart for linux-polska/ezd-backend.

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