-
Notifications
You must be signed in to change notification settings - Fork 79
kie-issues#755: Fix kogito operator nightly deploy pipelines #1535
Conversation
…checkout and add default container engine
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.
.ci/jenkins/Jenkinsfile.deploy
Outdated
@@ -55,7 +56,8 @@ pipeline { | |||
} | |||
|
|||
// Verify we have the connectivity to Openshift | |||
openshift.loginOpenshift() | |||
// TODO: Uncomment this line when we have the Openshift credentials |
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.
What do we need to get them? Or is this in progress in any way?
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.
AFAIK the OpenShift Instance is only used to run the tests. I'm not sure who has the credentials but we need to figure that out to add those credentials to Apache Jenkins.
@porcelli Do you know who can help us with that?
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.
we don't have a OpenShift instance available, so better disable this
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!
@porcelli I've made some changes to remove the code that publishes the operator image to the OpenShift registry and running the tests that require an OpenShift instance. The build is now passing as you can see in this test job: We can revert it back if we have an available OpenShift instance in the future. |
ISSUES:
kogito-operator
disable tests and get CI stable incubator-kie-issues#807Initial fixes for kogito operator nightly pipeline