Correctly find the highest version number for chromedriver

[rouke-broersma]

Relates to issue https://issues.jenkins-ci.org/browse/JENKINS-33270

Perhaps there's a better way to get the latest version number, but I am not familiar with groovy, and this seems to work.

[jenkinsadmin]

Thank you for this pull request! Please check this document for how the Jenkins project handles pull requests.

[rouke-broersma]

Does anyone have any thoughts on this? Improvements, suggestions, or does it look good as it is?

[rouke-broersma]

@orrc @daniel-beck @kohsuke Any thoughts? Can this be merged? Would love to hear from anyone.

[rouke-broersma]

Ping @kohsuke

[daniel-beck]

Let's see what happens 😄

[daniel-beck]

Looks good in the build log.

[rouke-broersma]

Thanks!

[rouke-broersma]

@daniel-beck Quick question. Does this require a new jenkins release before the change is live? Or can I re-enable the jenkins plugin that uses the chrome-driver binary downloaded by this script right now?

[daniel-beck]

@Mobrockers No, this is part of the Jenkins project infrastructure. The build I was mentioning provides the metadata to http://updates.jenkins-ci.org/updates/ from which Jenkins downloads it.

Between our mirror network being slow to update, and your Jenkins caching this data for a day or so, it may take a while to show up, so in the meantime, enjoy the fixed content at the source:
New: https://ci.jenkins-ci.org/job/infra_backend_crawler/296/artifact/target/org.jenkins-ci.plugins.chromedriver.ChromeDriver.json/*view*/
Old: https://ci.jenkins-ci.org/job/infra_backend_crawler/295/artifact/target/org.jenkins-ci.plugins.chromedriver.ChromeDriver.json/*view*/

[rouke-broersma]

@daniel-beck thank you 👏