Merge lp://staging/~fginther/ubuntu-ci-services-itself/lander-address-and-executor-fixes into lp://staging/ubuntu-ci-services-itself
Proposed by
Francis Ginther
| Status: | Merged | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Approved by: | Chris Johnston | ||||||||
| Approved revision: | 287 | ||||||||
| Merged at revision: | 292 | ||||||||
| Proposed branch: | lp://staging/~fginther/ubuntu-ci-services-itself/lander-address-and-executor-fixes | ||||||||
| Merge into: | lp://staging/ubuntu-ci-services-itself | ||||||||
| Diff against target: |
67 lines (+25/-0) 2 files modified
charms/precise/lander-jenkins/config.yaml (+4/-0) charms/precise/lander-jenkins/hooks/hooks.py (+21/-0) |
||||||||
| To merge this branch: | bzr merge lp://staging/~fginther/ubuntu-ci-services-itself/lander-address-and-executor-fixes | ||||||||
| Related bugs: |
|
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Chris Johnston (community) | Approve | ||
| PS Jenkins bot (community) | continuous-integration | Approve | |
|
Review via email:
|
|||
Commit message
If multiple private addresses are present for the jenkins instance, take the first one. Provide config option for specifying number of jenkins executors.
Description of the change
If multiple private addresses are present for the jenkins instance, take the first one. Provide config option for specifying number of jenkins executors.
To post a comment you must log in.
Revision history for this message
| PS Jenkins bot (ps-jenkins) wrote | # |
review:
Approve
(continuous-integration)
Revision history for this message
| Chris Johnston (cjohnston) | # |
review:
Approve
Revision history for this message
| Andy Doan (doanac) wrote | # |
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
PASSED: Continuous integration, rev:287
http://
Executed test runs:
Click here to trigger a rebuild:
http://