-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
lowered logging level thats flooding logs #54
base: develop
Are you sure you want to change the base?
Conversation
Signed-off-by: Neil South <[email protected]>
Codecov Report
@@ Coverage Diff @@
## develop #54 +/- ##
=============================================
Coverage 97.47191% 97.47191%
=============================================
Files 12 12
Lines 356 356
Branches 37 37
=============================================
Hits 347 347
Misses 7 7
Partials 2 2
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report in Codecov by Sentry.
|
Signed-off-by: Neil South <[email protected]>
@dependabot recreate |
@@ -20,7 +20,7 @@ namespace Monai.Deploy.WorkflowManager.Logging | |||
{ | |||
public static partial class Log | |||
{ | |||
[LoggerMessage(EventId = 500000, Level = LogLevel.Information, Message = "Bypass authentication.")] | |||
[LoggerMessage(EventId = 500000, Level = LogLevel.None, Message = "Bypass authentication.")] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we want to set this to debug instead of none?
Description
Fixes # .
A few sentences describing the changes proposed in this pull request.
Status
Ready/Work in progress/Hold
Types of changes