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

feat: propagate context to the metrics package #5479

Merged
merged 5 commits into from
Nov 28, 2022

Conversation

eddycharly
Copy link
Member

Explanation

This PR propagates context to the metrics package.

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

codecov bot commented Nov 25, 2022

Codecov Report

Merging #5479 (7296619) into main (08447c1) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main    #5479   +/-   ##
=======================================
  Coverage   36.44%   36.44%           
=======================================
  Files         171      171           
  Lines       19082    19082           
=======================================
  Hits         6955     6955           
  Misses      11332    11332           
  Partials      795      795           
Impacted Files Coverage Δ
pkg/policy/existing.go 0.00% <0.00%> (ø)
pkg/webhooks/handlers/metrics.go 0.00% <0.00%> (ø)
pkg/webhooks/resource/generation/generation.go 0.00% <0.00%> (ø)
pkg/webhooks/resource/updaterequest.go 15.38% <0.00%> (ø)
pkg/webhooks/utils/metrics.go 0.00% <0.00%> (ø)

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

realshuting
realshuting previously approved these changes Nov 28, 2022
@realshuting realshuting self-assigned this Nov 28, 2022
@realshuting realshuting enabled auto-merge (squash) November 28, 2022 08:55
realshuting and others added 2 commits November 28, 2022 16:55
Signed-off-by: Charles-Edouard Brétéché <charles.edouard@nirmata.com>
@realshuting realshuting merged commit dfded5c into kyverno:main Nov 28, 2022
@eddycharly eddycharly deleted the metrics-ctx branch November 28, 2022 11:00
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