Skip to content

Commit

Permalink
fixes deploy goal for maven canary release (#417)
Browse files Browse the repository at this point in the history
With respect to removal of content repository issue
openshiftio/openshift.io#3895
maven canary release target has changed from deploy to install
which is breaking other workflows in the fabric8 CD.
This patch fixes the issue by reverting maven target to deploy.

Fixes openshiftio/openshift.io#4151
  • Loading branch information
Hrishikesh Shinde authored and rupalibehera committed Aug 9, 2018
1 parent b72f826 commit d733ee0
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion vars/mavenCI.groovy
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ def call(body) {
stage('Build + Unit test') {
// set a unique temp version so we can download artifacts from nexus and run acceptance tests
sh "mvn -U versions:set -DnewVersion=${version}"
sh "mvn clean -B -e -U install -Dmaven.test.skip=${skipTests} -P openshift"
sh "mvn clean -B -e -U deploy -Dmaven.test.skip=${skipTests} -P openshift"
}

def s2iMode = utils.supportsOpenShiftS2I()
Expand Down
2 changes: 1 addition & 1 deletion vars/mavenCanaryRelease.groovy
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ def call(body) {
}
}

sh "mvn clean -B -e -U install -Dmaven.test.skip=${skipTests} ${spaceLabelArg} -P openshift"
sh "mvn clean -B -e -U deploy -Dmaven.test.skip=${skipTests} ${spaceLabelArg} -P openshift"


junitResults(body);
Expand Down

0 comments on commit d733ee0

Please sign in to comment.