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
Support inherited annotations for all major objects #1236
Conversation
501ef10
to
c4b1d69
Compare
|
||
return nil | ||
} | ||
|
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.
Not sure if returning nil is used anywhere, can simply return annotations.
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.
it should return nil if there are no Annotations defined in the config and on the resource. For example TestGeneratePodDisruptionBudget
fails for me if I return an empty map as this test does not define any Annotations at all. I have also added this behavior to the AnnotationsToPropagate
function, too.
👍 |
1 similar comment
👍 |
In preparation for an inherited annotations feature, similar to inherited labels, to estimate how much it would take.