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
Improve container image repository checks for some dev services #26624
Improve container image repository checks for some dev services #26624
Conversation
This comment was marked as resolved.
This comment was marked as resolved.
.../main/java/io/quarkus/apicurio/registry/devservice/DevServicesApicurioRegistryProcessor.java
Show resolved
Hide resolved
Also CC @ozangunalp. |
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
ce8927d
to
28e621c
Compare
I have pushed a commit to this PR that unifies the container image repository name checks in all 3 dev services that still perform them. |
This comment has been minimized.
This comment has been minimized.
…me dev services This commit improves container image name checks for these dev services: - Apicurio Registry - ActiveMQ Artemis (AMQP) - RabbitMQ These dev services used to verify that the image repository is equal to a hard-coded string. This commit changes that to verify that the image repository _ends with_ the same hard-coded string, which allows for repository namespacing in private container registries.
28e621c
to
d0e1915
Compare
@ozangunalp Could you please take another look and then I'll merge? |
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.
@Ladicek LGTM!
Thanks! |
Thanks a lot for solving this issue ;) |
Thank you for submitting the PR in the first place! :-) |
This commit improves container image name checks for these dev services:
These dev services used to verify that the image repository is equal
to a hard-coded string. This commit changes that to verify that
the image repository ends with the same hard-coded string, which
allows for repository namespacing in private container registries.
Fixes #24308