diff options
| author | Felix Edel <felix.edel@bmw.de> | 2021-07-05 13:58:01 +0200 |
|---|---|---|
| committer | James E. Blair <jim@acmegating.com> | 2021-07-13 14:57:01 -0700 |
| commit | 8f8f4f18987719b7c0a3f7c86ebf8e545d977ff9 (patch) | |
| tree | 5fccf00f4177be35e3009f7d628d57b535febd52 /releasenotes | |
| parent | 95ee8e81506f286218464567ca947c5ecfb7fedc (diff) | |
| download | zuul-8f8f4f18987719b7c0a3f7c86ebf8e545d977ff9.tar.gz | |
Switch to ZooKeeper backed NodesProvisionedEvents
This puts the NodesProvisionedEvents into ZooKeeper. With this, all
result events are now in ZooKeeper.
To make the NodesProvisionedEvent serializable, we cannot store the
whole NodeRequest object in the event anymore. Instead we are using the
request.id, job name and the buildset UUID to look up the corresponding
NodeRequest object from the buildset when the NodesProvisionedEvent is
handled.
As a result of this we have to find a way to return the NodeSet even in
case the NodeRequest or the buildset cannot be found anymore (e.g. due
to a gate reset). In this case we look up the NodeRequest directly from
ZooKeeper and provide a faked NodeSet which allows us to retrieve the
node information from Zookeeper (via the update mechanism of the
NodeRequest in the Nodepool client).
Finally, we can get rid of the local result event queue in the scheduler
as now all result events are in ZooKeeper. Together with that the
`zuul.scheduler.eventqueues.result` gauge was also removed.
Change-Id: Ib5e0f13d25a21ebad908d38f0201e92b704a1c85
Diffstat (limited to 'releasenotes')
| -rw-r--r-- | releasenotes/notes/remove-global-result-event-queue-f00bf6e4dab2720a.yaml | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/releasenotes/notes/remove-global-result-event-queue-f00bf6e4dab2720a.yaml b/releasenotes/notes/remove-global-result-event-queue-f00bf6e4dab2720a.yaml new file mode 100644 index 000000000..425527c15 --- /dev/null +++ b/releasenotes/notes/remove-global-result-event-queue-f00bf6e4dab2720a.yaml @@ -0,0 +1,3 @@ +--- +upgrade: + - The `zuul.scheduler.eventqueues.result` gauge was removed |