Merge lp://staging/~pwlars/charms/trusty/jenkins-remote-slave/jenkins-new-slave-pkg into lp://staging/~canonical-ci-engineering/charms/trusty/jenkins-remote-slave/trunk

Proposed by Paul Larson
Status: Merged
Approved by: Paul Larson
Approved revision: 23
Merged at revision: 23
Proposed branch: lp://staging/~pwlars/charms/trusty/jenkins-remote-slave/jenkins-new-slave-pkg
Merge into: lp://staging/~canonical-ci-engineering/charms/trusty/jenkins-remote-slave/trunk
Diff against target: 32 lines (+4/-3)
2 files modified
hooks/install (+2/-1)
hooks/install.d/3000-ci-tools.sh (+2/-2)
To merge this branch: bzr merge lp://staging/~pwlars/charms/trusty/jenkins-remote-slave/jenkins-new-slave-pkg
Reviewer Review Type Date Requested Status
Paul Larson Approve
Canonical Hardware Certification Pending
Review via email: mp+289857@code.staging.launchpad.net

Description of the change

This uses an updated version of the jenkins-slave package, and allows for different units to run on the same host, but use a different JENKINS_RUN directory so that the slave.jar can be different for each unit. This wasn't previously important because we had a lot of nodes connected to just a few jenkins master running the same version of jenkins. Today though, we have lots more jenkins masters that have no guarantee of keeping to the same version. We also have a lot greater chance of simultaneous restarts causing a collision when downloading the .jar file. This should prevent all of those problems.

To post a comment you must log in.
Revision history for this message
Paul Larson (pwlars) wrote :

Tested with juju local provider

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches