-
Notifications
You must be signed in to change notification settings - Fork 185
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
kie-issues#1285: Remove unnecessary base image from quarkus examples (quay.io/kiegroup/kogito-runtime-jvm:latest) #2457
kie-issues#1285: Remove unnecessary base image from quarkus examples (quay.io/kiegroup/kogito-runtime-jvm:latest) #2457
Conversation
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 addressing this @fantonangeli !
packages/sonataflow-operator/bddframework/pkg/framework/util.go
Outdated
Show resolved
Hide resolved
Co-authored-by: Tiago Bento <[email protected]>
Co-authored-by: Tiago Bento <[email protected]>
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.
In the referenced PR from examples I see;
registry.access.redhat.com/ubi9/openjdk-17:1.18
but in this PR I see:
registry.access.redhat.com/ubi9/openjdk-17:1.20
Please check, imho they should match
@domhanak Otherwise, we need to update the version in |
If we can upgrade to a newer version, then I think we should... @fantonangeli @domhanak... +1 for consistency too. |
@fantonangeli imho we update also examples if no issues, we should aim having this consistent |
@tiagobento @domhanak good, I will prepare the kogito-example update PR |
|
Closes: apache/incubator-kie-issues#1285
This PR is related to: