From d7eaad2f86de19454964693c1b1b22228dea8e4f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sybren=20A=2E=20St=C3=BCvel?= <sybren@stuvel.eu> Date: Mon, 30 Jan 2017 09:49:56 +0100 Subject: [PATCH] Server: reduced log level when depsgraph is empty --- packages/flamenco/flamenco/managers/api.py | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/packages/flamenco/flamenco/managers/api.py b/packages/flamenco/flamenco/managers/api.py index 45880114..6a204dfd 100644 --- a/packages/flamenco/flamenco/managers/api.py +++ b/packages/flamenco/flamenco/managers/api.py @@ -254,6 +254,7 @@ def get_depsgraph(manager_id, request_json): ) job_ids = [job['_id'] for job in jobs] if not job_ids: + log.debug('Returning empty depsgraph') return '', 204 # empty response log.debug('Requiring jobs to be in %s', job_ids) @@ -270,9 +271,13 @@ def get_depsgraph(manager_id, request_json): cursor = tasks_coll.find(task_query) depsgraph = list(cursor) - log.info('Returning depsgraph of %i tasks', len(depsgraph)) - if modified_since is not None and len(depsgraph) == 0: - return '', 304 # Not Modified + is_empty = len(depsgraph) == 0 + if is_empty: + log.debug('Returning empty depsgraph') + if modified_since is not None: + return '', 304 # Not Modified + else: + log.info('Returning depsgraph of %i tasks', len(depsgraph)) # Update the task status in the database to move queued tasks to claimed-by-manager. task_query['status'] = u'queued' -- GitLab