openshift / jenkins-plugin

Apache License 2.0
81 stars 50 forks source link

Issue with Jenkin's Verify Build OpenShift plug-in functionality #153

Closed mexicanopelon closed 7 years ago

mexicanopelon commented 7 years ago

We are having a recurring issue with Build Verify OpenShift functionality. The Jenkins job is able to successfully kick off the OSE builds, but for some reason it tracks the incorrect build job instead of the latest one. Therefore, even though the build job succeeds, the Jenkins job fails because it is waiting for the wrong job to succeed.

We were able to confirmed that there is a bug with the OSE plugin. It is around how it sorts the list of builds. It does it alphabetically instead of timestamp. Therefore, for example, it places build-4 before build-38.

srang commented 7 years ago

If I had to guess the issue may lie here:

gabemontero commented 7 years ago

@mexicanopelon and @srang - thanks for the report, and the analysis which zero'ed in on the issue!

I should have a PR cooking shortly, followed by a new plugin version hosted at the jenkins update center, followed by an update of the openshift jenkins centos image.

mexicanopelon commented 7 years ago

Thank you for the quick response.

gabemontero commented 7 years ago

@mexicanopelon @srang I'll cut the new version of the plugin at the jenkins download center later today or early next week, but the change is available at this repo now if you are interested in building (via mvn clean package) and install the hpi file manually.

gabemontero commented 7 years ago

v1.0.52 of the plugin is now available at the jenkins download center

gabemontero commented 7 years ago

and the openshift jenkins centos image on docker hub should be updated within the hour, assuming no network issues, etc.

srang commented 7 years ago

Woooo thanks Gabe! We'll update our plugin and test it out

abustya commented 6 years ago

Will the rhel7 image also be updated? I'm using openshift3/jenkins-2-rhel7 (from jenkins-ephemeral-template), and the plugin version included is 1.0.47.

bparees commented 6 years ago

@abustya yes, when OCP3.7 ships there will be a new jenkins rhel image available and it will include this change.