Skip to content

Commit 7f4a1e0

Browse files
authored
[Logging] Move error to warning for log context logs (#4758)
We're moving the logging level from error to warning for exceptions that was caught during log context. We need to do this because this errors are already handled in the uper level, but we want to log it as warning to have visibility with the strucutred logs. --------- Signed-off-by: Javan Lacerda <javanlacerda@google.com>
1 parent d18c7a5 commit 7f4a1e0

File tree

1 file changed

+5
-2
lines changed
  • src/clusterfuzz/_internal/metrics

1 file changed

+5
-2
lines changed

src/clusterfuzz/_internal/metrics/logs.py

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -795,7 +795,7 @@ def task_stage_context(stage: Stage):
795795
log_contexts.add_metadata('stage', stage)
796796
yield
797797
except Exception as e:
798-
error(message='Error during task.')
798+
warning(message='Error during task.')
799799
raise e
800800
finally:
801801
log_contexts.delete_metadata('stage')
@@ -816,7 +816,10 @@ def testcase_log_context(testcase: 'Testcase',
816816
log_contexts.add_metadata('fuzz_target', fuzz_target)
817817
yield
818818
except Exception as e:
819-
error(message='Error during testcase context.')
819+
# We've put as warning because this error will be
820+
# handled in a upper level, and we would like to have
821+
# a way to track it with the current context logs
822+
warning(message='Error during testcase context.')
820823
raise e
821824
finally:
822825
log_contexts.delete_metadata('testcase')

0 commit comments

Comments
 (0)