-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
馃悰 Fix plugin name derivation from image name #3711
Conversation
/kind changelog-not-required |
The issue this PR fixes has been reported here: #3715. |
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.
This looks good, thanks for addressing it so quickly, @ashish-amarnath! Given that this has been reported by a community member (#3715) and is a bug, I think it should have a changelog.
31142dc
to
3cdf5e4
Compare
Signed-off-by: Ashish Amarnath <ashisham@vmware.com>
3cdf5e4
to
4728ed8
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.
Thsnkd for adding the test!
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.
Thanks!
* 馃悰 Fix plugin name derivation from image name Signed-off-by: Ashish Amarnath <ashisham@vmware.com> * changelog Signed-off-by: Ashish Amarnath <ashisham@vmware.com>
* 馃悰 Fix plugin name derivation from image name Signed-off-by: Ashish Amarnath <ashisham@vmware.com> * changelog Signed-off-by: Ashish Amarnath <ashisham@vmware.com>
* 馃悰 Fix plugin name derivation from image name Signed-off-by: Ashish Amarnath <ashisham@vmware.com> * changelog Signed-off-by: Ashish Amarnath <ashisham@vmware.com>
Signed-off-by: Ashish Amarnath ashisham@vmware.com
Thank you for contributing to Velero!
Plugin's init container name is derived from the plugin's image that is supplied in the
velero plugin add
command.This derivation is a concatenation of image's repository name, and the image name. Further replacing characters that are allowed in container images repository name but not in a valid RFC-1123 label. Specifically, the container image repository name matches the regex
[a-z0-9]+(?:[._-][a-z0-9]+)*
. However, valid RFC-1123 labels should not have_
or.
.This PR replaces
_
and.
with-
making the derived name RFC-1123 compliant.Does your change fix a particular issue?
Fixes #3715
Please indicate you've done the following:
/kind changelog-not-required
.site/content/docs/main
.