From 2124e4335d5d2d994f580014c43f1bc264aff93e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sybren=20A=2E=20St=C3=BCvel?= Date: Wed, 27 Jul 2016 11:17:37 +0200 Subject: [PATCH] refresh_links_for_backend: more consistent messages --- pillar/application/modules/file_storage.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/pillar/application/modules/file_storage.py b/pillar/application/modules/file_storage.py index efa522c1..df8c6b1c 100644 --- a/pillar/application/modules/file_storage.py +++ b/pillar/application/modules/file_storage.py @@ -490,7 +490,7 @@ def refresh_links_for_backend(backend_name, chunk_size, expiry_seconds): continue if 'file_path' not in file_doc: - log.warning("File %s malformed: missing 'file_path' property.", file_id) + log.warning("Skipping file %s, missing 'file_path' property.", file_id) continue log.debug('Refreshing links for file %s', file_id) @@ -498,8 +498,8 @@ def refresh_links_for_backend(backend_name, chunk_size, expiry_seconds): try: _generate_all_links(file_doc, now) except gcloud.exceptions.Forbidden: - log.warning('GCS forbids us access to project %s bucket for file %s.', - project_id, file_id) + log.warning('Skipping file %s, GCS forbids us access to project %s bucket.', + file_id, project_id) continue refreshed += 1