-
Notifications
You must be signed in to change notification settings - Fork 31
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
⚠️ Rename Catalog
to ClusterCatalog
#268
Conversation
acda61b
to
ef639c8
Compare
// +kubebuilder:validation:XValidation:rule="!has(self.source.image.pollInterval) || (self.source.image.ref.find('@sha256:') == \"\")",message="cannot specify PollInterval while using digest-based image" | ||
type CatalogSpec struct { | ||
type ClusterCatalogSpec struct { | ||
// Source is the source of a Catalog that contains Operators' metadata in the FBC format |
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.
Note for reviewers: I did not modify the "Catalog" reference here because I am interpreting it as the general umbrella term "Catalog" rather than an explicit reference to the on-cluster API. Happy to update this if necessary.
5e073d6
to
9412a01
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #268 +/- ##
=======================================
Coverage 48.84% 48.84%
=======================================
Files 8 8
Lines 434 434
=======================================
Hits 212 212
Misses 201 201
Partials 21 21 ☔ View full report in Codecov by Sentry. |
b8cb89f
to
852856b
Compare
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.
rename to config/samples/core_v1alpha1_clustercatalog.yaml
?
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.
Should be done in the latest push
Signed-off-by: everettraven <everettraven@gmail.com>
b7087fa
to
a9b78da
Compare
4f1219c
Rename Catalog to ClusterCatalog to conform with the changes in catalogd. See [issue operator-framework#268](operator-framework/catalogd/pull/268) Signed-off-by: Edmund Ochieng <ochienged@gmail.com>
No description provided.