[Notes] [Git][BuildGrid/buildgrid][master] Scheduler: fix monitoring variables initialization

[Martin Blanchard <gitlab mg gitlab com>]

Title: GitLab

Martin Blanchard pushed to branch master at BuildGrid / buildgrid

Commits:

• 4c971b76
  by Santiago Gil at 2019-02-20T10:46:59Z

  Scheduler: fix monitoring variables initialization
  
  Calling `Scheduler.activate_monitoring()` from the `Scheduler`
  constructor didn't have any effect because `self.activate_monitoring()`
  always exited early.
  
  This change fixes the boolean check before that function call so that
  the monitoring structures get initialized properly.
  

1 changed file:

• buildgrid/server/scheduler.py

Changes:

buildgrid/server/scheduler.py

---

...	...	@@ -56,9 +56,8 @@ class Scheduler:
56	56	self.__queue = []
57	57	self.__queue_lock = Lock()
58	58
59		- self._is_instrumented = monitor
60		-
61		- if self._is_instrumented:
	59	+ self._is_instrumented = False
	60	+ if monitor:
62	61	self.activate_monitoring()
63	62
64	63	# --- Public API ---

—
View it on GitLab.
You're receiving this email because of your account on gitlab.com. If you'd like to receive fewer emails, you can adjust your notification settings.