-
Notifications
You must be signed in to change notification settings - Fork 67
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
[DNM] Modifying Java Jenkifiles to support the osio pipeline library #46
base: master
Are you sure you want to change the base?
[DNM] Modifying Java Jenkifiles to support the osio pipeline library #46
Conversation
maven/Release/Jenkinsfile
Outdated
|
||
// processing openshift template present in .openshiftio/application.yaml | ||
def resources = processTemplate(params: [ | ||
release_version: "1.0.${env.BUILD_NUMBER}" |
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.
@rupalibehera could you make it as RELEASE_VERSION
(all uppercase)
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.
done.
maven/Release/Jenkinsfile
Outdated
} | ||
} | ||
integrationTestCmd = "mvn verify integration-test -Dnamespace.use.current=false -Dnamespace.use.existing=${testNamespace()} -Dit.test=*IT -DfailIfNoTests=false -DenableImageStreamDetection=true -Popenshift,openshift-it" | ||
runTest commands: integrationTestCmd |
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 should revert this change and make use of spawn and env('build')
1. Modifying integration test command with unit test command 2. using spawn instead of runTest
This PR will enable the Java support with osio pipeline library
closes openshiftio/openshift.io#4501
This PR cannot be merged until openshiftio/openshift.io#4500 is closed.