Skip to content
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

fix: bump log level for autogen debug logs #5687

Merged
merged 3 commits into from
Dec 14, 2022

Conversation

eddycharly
Copy link
Member

@eddycharly eddycharly commented Dec 14, 2022

Explanation

This PR bumps log level for autogen debug logs.
Level 3 is too low, they are polluting logs too much.

Signed-off-by: Charles-Edouard Brétéché <charles.edouard@nirmata.com>
@codecov
Copy link

codecov bot commented Dec 14, 2022

Codecov Report

Merging #5687 (d2b2ee5) into main (682afc8) will not change coverage.
The diff coverage is 50.00%.

@@           Coverage Diff           @@
##             main    #5687   +/-   ##
=======================================
  Coverage   34.48%   34.48%           
=======================================
  Files         189      189           
  Lines       21113    21113           
=======================================
  Hits         7280     7280           
  Misses      13032    13032           
  Partials      801      801           
Impacted Files Coverage Δ
pkg/autogen/autogen.go 64.76% <20.00%> (ø)
pkg/autogen/rule.go 76.47% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@chipzoller
Copy link
Member

What level will they be accessible after this change?

@eddycharly
Copy link
Member Author

What level will they be accessible after this change?

5

@eddycharly eddycharly enabled auto-merge (squash) December 14, 2022 12:40
@eddycharly eddycharly merged commit 84a5b0d into kyverno:main Dec 14, 2022
@eddycharly eddycharly deleted the autogen-logs branch December 14, 2022 13:29
MdSahil-oss pushed a commit to MdSahil-oss/kyverno that referenced this pull request Dec 29, 2022
Signed-off-by: Charles-Edouard Brétéché <charles.edouard@nirmata.com>

Signed-off-by: Charles-Edouard Brétéché <charles.edouard@nirmata.com>
Signed-off-by: Md Sahil <Mohdssahil1@gmail.com>
MdSahil-oss pushed a commit to MdSahil-oss/kyverno that referenced this pull request Jan 11, 2023
Signed-off-by: Charles-Edouard Brétéché <charles.edouard@nirmata.com>

Signed-off-by: Charles-Edouard Brétéché <charles.edouard@nirmata.com>
Signed-off-by: MdSahil-oss <Mohdssahil1@gmail.com>
MdSahil-oss pushed a commit to MdSahil-oss/kyverno that referenced this pull request Jan 11, 2023
Signed-off-by: Charles-Edouard Brétéché <charles.edouard@nirmata.com>

Signed-off-by: Charles-Edouard Brétéché <charles.edouard@nirmata.com>
Signed-off-by: MdSahil-oss <Mohdssahil1@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants