-
Notifications
You must be signed in to change notification settings - Fork 68
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
rewrite func getImageNameTag for supporting registries with port #372
Conversation
43ce79d
to
ea891f6
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.
thanks for submitting this, can you please add a unit test that verifies that we parse correctly the different formats of images?
6f16388
to
bcbb39d
Compare
i added unit test |
can you rebase so that only your 2 commits show up, and add some message? we're trying to follow https://cbea.ms/git-commit, a short description suffices |
0cdf759
to
d4a049b
Compare
Please, check |
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.
few small things, and I think this is ready :)
e667a2b
to
418bfe4
Compare
Please, check |
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.
lgtm, thanks @artem198315
@artem198315 sorry clicked the wrong button 😞, can you please rebase your local branch and push again? |
To make operator works with image's path in format: myregistry.local.com:5000/metallb/controller:v0.13.9 Signed-off-by: artem198315 <[email protected]>
Signed-off-by: artem198315 <[email protected]>
Done |
To make operator works with image's path in format myregistry.local.com:5000/metallb/controller:v0.13.9