1
0
Fork 0
mirror of https://github.com/kyverno/kyverno.git synced 2025-03-30 19:35:06 +00:00

fix: metrics with invalid validationMode (#4198)

Signed-off-by: Ramón Berrutti <ramonberrutti@gmail.com>

Co-authored-by: shuting <shuting@nirmata.com>
This commit is contained in:
Ramón Berrutti 2022-07-20 10:06:15 +01:00 committed by GitHub
parent dafa27e928
commit 65c100566c
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 6 additions and 2 deletions

View file

@ -48,10 +48,12 @@ func registerPolicyExecutionDurationMetric(
//policy - policy related data
//engineResponse - resource and rule related data
func ProcessEngineResponse(m *metrics.MetricsConfig, policy kyvernov1.PolicyInterface, engineResponse response.EngineResponse, executionCause metrics.RuleExecutionCause, generateRuleLatencyType string, resourceRequestOperation metrics.ResourceRequestOperation) error {
name, namespace, policyType, backgroundMode, validationMode, err := metrics.GetPolicyInfos(policy)
name, namespace, policyType, backgroundMode, _, _ := metrics.GetPolicyInfos(policy)
validationMode, err := metrics.ParsePolicyValidationMode(engineResponse.GetValidationFailureAction())
if err != nil {
return err
}
resourceSpec := engineResponse.PolicyResponse.Resource
resourceKind := resourceSpec.Kind
resourceNamespace := resourceSpec.Namespace

View file

@ -43,10 +43,12 @@ func registerPolicyResultsMetric(
//policy - policy related data
//engineResponse - resource and rule related data
func ProcessEngineResponse(m *metrics.MetricsConfig, policy kyvernov1.PolicyInterface, engineResponse response.EngineResponse, executionCause metrics.RuleExecutionCause, resourceRequestOperation metrics.ResourceRequestOperation) error {
name, namespace, policyType, backgroundMode, validationMode, err := metrics.GetPolicyInfos(policy)
name, namespace, policyType, backgroundMode, _, _ := metrics.GetPolicyInfos(policy)
validationMode, err := metrics.ParsePolicyValidationMode(engineResponse.GetValidationFailureAction())
if err != nil {
return err
}
resourceSpec := engineResponse.PolicyResponse.Resource
resourceKind := resourceSpec.Kind
resourceNamespace := resourceSpec.Namespace