diff options
author | Zuul <zuul@review.opendev.org> | 2022-08-15 22:59:09 +0000 |
---|---|---|
committer | Gerrit Code Review <review@openstack.org> | 2022-08-15 22:59:09 +0000 |
commit | afebbb6a6caae95c5edf56333a191d9c525d4c1d (patch) | |
tree | 3cd0704250412fbae7bc549cf8b0c32b82ac18a1 /zuul | |
parent | bb1b0d5c1108b2b4d21559535e9c120b732e99b2 (diff) | |
parent | d61b9772ff6c5af57d93414bdc0f1a9d62ae35b4 (diff) | |
download | zuul-afebbb6a6caae95c5edf56333a191d9c525d4c1d.tar.gz |
Merge "Fix zoned executor metric when unzoned is allowed"
Diffstat (limited to 'zuul')
-rw-r--r-- | zuul/executor/server.py | 4 | ||||
-rw-r--r-- | zuul/scheduler.py | 4 |
2 files changed, 6 insertions, 2 deletions
diff --git a/zuul/executor/server.py b/zuul/executor/server.py index eac7fa7e5..e00612e9e 100644 --- a/zuul/executor/server.py +++ b/zuul/executor/server.py @@ -3157,6 +3157,10 @@ class ExecutorServer(BaseMergeServer): self.allow_unzoned = get_default(self.config, 'executor', 'allow_unzoned', False) + # If this executor has no zone configured it is implicitly unzoned + if self.zone is None: + self.allow_unzoned = True + # Those attributes won't change, so it's enough to set them once on the # component info. self.component_info.zone = self.zone diff --git a/zuul/scheduler.py b/zuul/scheduler.py index 272235757..dfc922cf1 100644 --- a/zuul/scheduler.py +++ b/zuul/scheduler.py @@ -438,12 +438,12 @@ class Scheduler(threading.Thread): mergers_online = 0 for executor_component in self.component_registry.all("executor"): - if executor_component.allow_unzoned or not executor_component.zone: + if executor_component.allow_unzoned: if executor_component.state == BaseComponent.RUNNING: executors_unzoned_online += 1 if executor_component.accepting_work: executors_unzoned_accepting += 1 - else: + if executor_component.zone: zone_stats = zoned_executor_stats.setdefault( executor_component.zone, executor_stats_default.copy()) |