diff options
author | Tobias Henkel <tobias.henkel@bmw.de> | 2019-01-22 21:32:53 +0100 |
---|---|---|
committer | Tobias Henkel <tobias.henkel@bmw.de> | 2019-01-26 08:11:15 +0100 |
commit | 81adf8c6e65e1525bf86316fbf213b83cfb5cf6d (patch) | |
tree | 571892cf1b713647d929e6d4139b0ff6164dcadc /tests/fixtures/layouts/broken-double-gate.yaml | |
parent | 6fccffe49bd9265721c9d091a69f9e08e986e938 (diff) | |
download | zuul-81adf8c6e65e1525bf86316fbf213b83cfb5cf6d.tar.gz |
Raise an error if pipeline is defined twice
When pipelines are defined multiple times zuul currently silently uses
the last one which is defined. This is unsafe. Instead stick with the
first found pipeline and add a configuration error. While at it add
the missing start_mark to the pipeline object as it's needed when
throwing an error.
Change-Id: I98637ecd7d4978a803d6131b7cd61da0a74e669f
Diffstat (limited to 'tests/fixtures/layouts/broken-double-gate.yaml')
-rw-r--r-- | tests/fixtures/layouts/broken-double-gate.yaml | 47 |
1 files changed, 47 insertions, 0 deletions
diff --git a/tests/fixtures/layouts/broken-double-gate.yaml b/tests/fixtures/layouts/broken-double-gate.yaml new file mode 100644 index 000000000..c5048204f --- /dev/null +++ b/tests/fixtures/layouts/broken-double-gate.yaml @@ -0,0 +1,47 @@ +- pipeline: + name: gate + manager: dependent + trigger: + gerrit: + - event: comment-added + approval: + - Approved: 1 + success: + gerrit: + Verified: 2 + submit: true + failure: + gerrit: + Verified: -2 + start: + gerrit: + Verified: 0 + +- pipeline: + name: gate + manager: dependent + trigger: + gerrit: + - event: comment-added + approval: + - Approved: 1 + success: + gerrit: + Verified: 2 + submit: true + failure: + gerrit: + Verified: -2 + start: + gerrit: + Verified: 0 + +- job: + name: base + parent: null + +- project: + name: org/project + gate: + jobs: + - base |