summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJames E. Blair <jeblair@redhat.com>2018-12-03 10:52:14 -0800
committerJames E. Blair <jeblair@redhat.com>2018-12-03 10:52:14 -0800
commit50302e7255b27ddc6a356ff8c92111ecf569aab0 (patch)
tree2fa2d97af95b27a0c3de28d39370d903eaba9a69
parentf116826d2b37dddfb52056af8d80befc408a776a (diff)
Make launcher debug slightly less chatty
This reduces most of the new launcher debug messages now that the relative_priority behavior has been verified, though it retains a few additions. Change-Id: I5408105f92aab5baf78ec2ea80f8c4427a2a695b
Notes
Notes (review): Code-Review+2: Tobias Henkel <tobias.henkel@bmw.de> Code-Review+2: David Shrewsbury <shrewsbury.dave@gmail.com> Workflow+1: James E. Blair <corvus@inaugust.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 04 Dec 2018 17:34:22 +0000 Reviewed-on: https://review.openstack.org/621675 Project: openstack-infra/nodepool Branch: refs/heads/master
-rwxr-xr-xnodepool/launcher.py5
1 files changed, 1 insertions, 4 deletions
diff --git a/nodepool/launcher.py b/nodepool/launcher.py
index b6400db..6f2c39c 100755
--- a/nodepool/launcher.py
+++ b/nodepool/launcher.py
@@ -180,7 +180,6 @@ class PoolWorker(threading.Thread, stats.StatsReporter):
180 if not self.running: 180 if not self.running:
181 return True 181 return True
182 182
183 self.log.debug("Considering request %s", req.id)
184 if self.paused_handler: 183 if self.paused_handler:
185 self.log.debug("Handler is now paused") 184 self.log.debug("Handler is now paused")
186 return True 185 return True
@@ -201,19 +200,17 @@ class PoolWorker(threading.Thread, stats.StatsReporter):
201 200
202 req = self.zk.getNodeRequest(req.id) 201 req = self.zk.getNodeRequest(req.id)
203 if not req: 202 if not req:
204 self.log.debug("Request %s no longer exists", req.id)
205 continue 203 continue
206 204
207 # Only interested in unhandled requests 205 # Only interested in unhandled requests
208 if req.state != zk.REQUESTED: 206 if req.state != zk.REQUESTED:
209 self.log.debug("Request %s is in state %s", req.id, req.state)
210 continue 207 continue
211 208
212 # Skip it if we've already declined 209 # Skip it if we've already declined
213 if self.launcher_id in req.declined_by: 210 if self.launcher_id in req.declined_by:
214 self.log.debug("Request %s was already declined", req.id)
215 continue 211 continue
216 212
213 self.log.debug("Locking request %s", req.id)
217 try: 214 try:
218 self.zk.lockNodeRequest(req, blocking=False) 215 self.zk.lockNodeRequest(req, blocking=False)
219 except exceptions.ZKLockException: 216 except exceptions.ZKLockException: