-
Notifications
You must be signed in to change notification settings - Fork 198
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#1526: [kn-plugin-workflow] kn-workflow-plugin should persist workflow in the docker image #2725
base: main
Are you sure you want to change the base?
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.
So, are we manipulating the Quarkus auto-generated Dockerfile to add the files we need to pass the validation on the operator side?
yeap, We have to include workflow.sw into the docker image, so have to modify docker files |
@@ -0,0 +1,16 @@ | |||
FROM registry.access.redhat.com/ubi8/openjdk-17:1.18 |
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.
Could this be parametrized? just a nitpick, disregard if not
@@ -0,0 +1,11 @@ | |||
FROM registry.access.redhat.com/ubi8/ubi-minimal:8.9 |
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.
Could this be parametrized? just a nitpick, disregard if not
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.
@domhanak Actually, this is only needed for testing; there's no need to keep the current version, any version will work.
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.
@treblereel please, let's rebase to ensure we are not missing some headers. This is quite old PR
eca7fdd
to
b75df24
Compare
issue: apache/incubator-kie-issues#1526
this issue is related to apache/incubator-kie-kogito-serverless-operator#405