Merge lp:~fginther/cupstream2distro-config/independent-triggering into lp:cupstream2distro-config
Status: | Merged |
---|---|
Approved by: | Francis Ginther on 2013-10-09 |
Approved revision: | 843 |
Merged at revision: | 844 |
Proposed branch: | lp:~fginther/cupstream2distro-config/independent-triggering |
Merge into: | lp:cupstream2distro-config |
Diff against target: |
350 lines (+128/-52) 2 files modified
c2dconfigutils/cu2dTrigger.py (+37/-27) tests/test_cu2dTrigger.py (+91/-25) |
To merge this branch: | bzr merge lp:~fginther/cupstream2distro-config/independent-triggering |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Joe Talbott (community) | 2013-10-08 | Approve on 2013-10-09 | |
Francis Ginther | Resubmit on 2013-10-08 | ||
PS Jenkins bot | continuous-integration | Approve on 2013-10-08 | |
Review via email:
|
Commit message
Allow independent triggering of ci and autolanding jobs for entire stacks.
Description of the change
Allow independent triggering of ci and autolanding for entire stacks.
This primarily contains changes to the argument handling to change the behavior of the --trigger-ci and --trigger-
Here are examples of this in action:
http://
http://
Joe Talbott (joetalbott) wrote : | # |
L97 here trigger_types will either be ['autolanding'] or ['ci'] but in other places I see trigger_types being set to ['autolanding', 'ci']. I might be missing some context.
Francis Ginther (fginther) wrote : | # |
> L97 here trigger_types will either be ['autolanding'] or ['ci'] but in other
> places I see trigger_types being set to ['autolanding', 'ci']. I might be
> missing some context.
Good catch. This
100 + else:
should be:
if args.trigger_ci:
I want to enable both types of jobs if the arguments are there.
- 843. By Francis Ginther on 2013-10-08
-
Should be able to trigger autolanding and ci jobs, not either/or.
PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:843
http://
Executed test runs:
SUCCESS: http://
Click here to trigger a rebuild:
http://
PASSED: Continuous integration, rev:842 jenkins. qa.ubuntu. com/job/ cupstream2distr o-config- ci/547/ jenkins. qa.ubuntu. com/job/ cupstream2distr o-config- saucy-amd64- ci/345
http://
Executed test runs:
SUCCESS: http://
Click here to trigger a rebuild: 10.97.0. 26:8080/ job/cupstream2d istro-config- ci/547/ rebuild
http://