Separate reporters from triggers
Allows multiple reports per a patchset to be sent to pluggable
destinations. These are configurable per pipeline and, if not
specified, defaults to the legacy behaviour of reporting back only
to gerrit.
Having multiple reporting methods means only certain success/failure
/start parameters will apply to certain reporters. Reporters are
listed as keys under each of those actions.
This means that each key under success/failure/start is a reporter and the
dictionaries under those are sent to the reporter to deal with.
Change-Id: I80d7539772e1485d5880132f22e55751b25ec198
diff --git a/tests/fixtures/layout-delayed-repo-init.yaml b/tests/fixtures/layout-delayed-repo-init.yaml
index e0613f1..6caf622 100644
--- a/tests/fixtures/layout-delayed-repo-init.yaml
+++ b/tests/fixtures/layout-delayed-repo-init.yaml
@@ -5,9 +5,11 @@
gerrit:
- event: patchset-created
success:
- verified: 1
+ gerrit:
+ verified: 1
failure:
- verified: -1
+ gerrit:
+ verified: -1
- name: post
manager: IndependentPipelineManager
@@ -25,12 +27,15 @@
approval:
- approved: 1
success:
- verified: 2
- submit: true
+ gerrit:
+ verified: 2
+ submit: true
failure:
- verified: -2
+ gerrit:
+ verified: -2
start:
- verified: 0
+ gerrit:
+ verified: 0
precedence: high
projects:
diff --git a/tests/fixtures/layout-live-reconfiguration-functions.yaml b/tests/fixtures/layout-live-reconfiguration-functions.yaml
index f477a12..e261a88 100644
--- a/tests/fixtures/layout-live-reconfiguration-functions.yaml
+++ b/tests/fixtures/layout-live-reconfiguration-functions.yaml
@@ -11,12 +11,15 @@
approval:
- approved: 1
success:
- verified: 2
- submit: true
+ gerrit:
+ verified: 2
+ submit: true
failure:
- verified: -2
+ gerrit:
+ verified: -2
start:
- verified: 0
+ gerrit:
+ verified: 0
precedence: high
jobs:
diff --git a/tests/fixtures/layout-timer.yaml b/tests/fixtures/layout-timer.yaml
index e24fb13..9e0f66b 100644
--- a/tests/fixtures/layout-timer.yaml
+++ b/tests/fixtures/layout-timer.yaml
@@ -5,9 +5,11 @@
gerrit:
- event: patchset-created
success:
- verified: 1
+ gerrit:
+ verified: 1
failure:
- verified: -1
+ gerrit:
+ verified: -1
- name: periodic
manager: IndependentPipelineManager
diff --git a/tests/fixtures/layout.yaml b/tests/fixtures/layout.yaml
index 675d351..1cb8688 100644
--- a/tests/fixtures/layout.yaml
+++ b/tests/fixtures/layout.yaml
@@ -8,9 +8,11 @@
gerrit:
- event: patchset-created
success:
- verified: 1
+ gerrit:
+ verified: 1
failure:
- verified: -1
+ gerrit:
+ verified: -1
- name: post
manager: IndependentPipelineManager
@@ -28,12 +30,15 @@
approval:
- approved: 1
success:
- verified: 2
- submit: true
+ gerrit:
+ verified: 2
+ submit: true
failure:
- verified: -2
+ gerrit:
+ verified: -2
start:
- verified: 0
+ gerrit:
+ verified: 0
precedence: high
- name: unused
@@ -51,9 +56,11 @@
gerrit:
- event: change-restored
success:
- verified: 1
+ gerrit:
+ verified: 1
failure:
- verified: -1
+ gerrit:
+ verified: -1
- name: dup2
manager: IndependentPipelineManager
@@ -61,9 +68,11 @@
gerrit:
- event: change-restored
success:
- verified: 1
+ gerrit:
+ verified: 1
failure:
- verified: -1
+ gerrit:
+ verified: -1
- name: conflict
dequeue-on-conflict: false
@@ -75,12 +84,15 @@
approval:
- approved: 1
success:
- verified: 2
- submit: true
+ gerrit:
+ verified: 2
+ submit: true
failure:
- verified: -2
+ gerrit:
+ verified: -2
start:
- verified: 0
+ gerrit:
+ verified: 0
jobs:
- name: ^.*-merge$
diff --git a/tests/fixtures/layouts/good_layout.yaml b/tests/fixtures/layouts/good_layout.yaml
index 15be6ef..4bd5e70 100644
--- a/tests/fixtures/layouts/good_layout.yaml
+++ b/tests/fixtures/layouts/good_layout.yaml
@@ -8,9 +8,11 @@
gerrit:
- event: patchset-created
success:
- verified: 1
+ gerrit:
+ verified: 1
failure:
- verified: -1
+ gerrit:
+ verified: -1
- name: post
manager: IndependentPipelineManager
@@ -28,15 +30,18 @@
- event: comment-added
approval:
- approved: 1
- success:
- verified: 2
- code-review: 1
- submit: true
- failure:
- verified: -2
- workinprogress: true
start:
- verified: 0
+ gerrit:
+ verified: 0
+ success:
+ gerrit:
+ verified: 2
+ code-review: 1
+ submit: true
+ failure:
+ gerrit:
+ verified: -2
+ workinprogress: true
jobs:
- name: ^.*-merge$