Skip to content

Improve the error message for AnalysisRun #2101

Description

@leoluz

Summary

Today, when AnalysisRun fails it returns the message with the string "Error" to the user. We should improve Rollouts error handling to provide a better message.

Consider the success condition bellow:

successCondition: 'result[0] <= {{args.cpu-utilization-limit-perc}}'

Sometimes the error is related to the metric success condition not being met but sometimes it is related to the metric not available causing result[0] == nil with IndexOutofBound.

Rollout should be able to distinguish the 2 conditions and provide a friendly message to the user.

Diagnostics

argo-rollouts 1.2.1

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions