-
Notifications
You must be signed in to change notification settings - Fork 167
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
feat: support ComponentDefinition custom annotations and Component inherit Cluster annotations #6724
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #6724 +/- ##
==========================================
+ Coverage 66.25% 66.51% +0.25%
==========================================
Files 306 306
Lines 36870 36900 +30
==========================================
+ Hits 24430 24544 +114
+ Misses 10337 10269 -68
+ Partials 2103 2087 -16
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@@ -190,6 +190,13 @@ type ComponentDefinitionSpec struct { | |||
// +optional | |||
Labels map[string]string `json:"labels,omitempty"` | |||
|
|||
// Defines static annotations that will be patched to all k8s resources created for the component. | |||
// If a annotation key conflicts with any other system annotations or user-specified annotations, it will be silently ignored. | |||
// This field is immutable. |
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.
then enforce it by a validation rule?
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.
then enforce it by a validation rule?
This is to keep consistency with other fields, and validation will be added once the API is finally stable.
/approve |
No description provided.