Merge lp://qastaging/~kissiel/checkbox/bootstrapping-include into lp://qastaging/checkbox
Proposed by
Maciej Kisielewski
| Status: | Merged |
|---|---|
| Approved by: | Zygmunt Krynicki |
| Approved revision: | 4004 |
| Merged at revision: | 4000 |
| Proposed branch: | lp://qastaging/~kissiel/checkbox/bootstrapping-include |
| Merge into: | lp://qastaging/checkbox |
| Diff against target: |
325 lines (+187/-7) 6 files modified
checkbox-touch/checkbox-touch.qml (+1/-0) plainbox/docs/changelog.rst (+18/-2) plainbox/docs/manpages/plainbox-test-plan-units.rst (+72/-0) plainbox/plainbox/impl/session/assistant.py (+10/-5) plainbox/plainbox/impl/unit/test_testplan.py (+28/-0) plainbox/plainbox/impl/unit/testplan.py (+58/-0) |
| To merge this branch: | bzr merge lp://qastaging/~kissiel/checkbox/bootstrapping-include |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Zygmunt Krynicki (community) | Approve | ||
|
Review via email:
|
|||
Description of the change
This MR brings proper bootstrapping to the Session Assistant and therefore Checkbox-Converged.
d186e7b checkbox-touch: enable local jobs
efcc34c plainbox:
2f97c46 plainbox:
eb37f39 plainbox:
95438b4 plainbox:
09bb92d plainbox:
0fdfd59 plainbox:
b00f39f plainbox:docs: update changelog
To post a comment you must log in.
Revision history for this message
| Zygmunt Krynicki (zyga) wrote | # |
Revision history for this message
| Zygmunt Krynicki (zyga) wrote | # |
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
12:44 <@zyga> kissiel: bootstrap include is not a list of patterns
12:44 <@zyga> kissiel: construct a list of qualifiers that match against IDs
The reason for this is so that there is no ambiguity in what is selected and what the boostrap include section contains. We can perfectly validate that each entry corresponds to a job or that it is invalid.