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

doc(examples): interval requires count #2691

Closed
meeech opened this issue Apr 2, 2023 · 2 comments · Fixed by #2690
Closed

doc(examples): interval requires count #2691

meeech opened this issue Apr 2, 2023 · 2 comments · Fixed by #2690
Labels
bug Something isn't working

Comments

@meeech
Copy link
Contributor

meeech commented Apr 2, 2023

Was looking at an example. Noticed examples/cluster-analysis-templates.yaml doesn't work as is:

InvalidSpec: The Rollout "scratch-chart" is invalid: spec.strategy.canary.steps[1].analysis.templates: Invalid value: "always-pass": AnalysisTemplate always-pass has metric pass which runs indefinitely. Invalid value for count: <nil>

Checklist:

  • [ n/a ] I've included steps to reproduce the bug.
  • [ n/a ] I've included the version of argo rollouts.

Describe the bug

Example wasn't working, since interval needed a count in this case.

To Reproduce

try to use examples/cluster-analysis-templates.yaml as a step in rollout

Expected behavior

Screenshots

Version

Logs

# Paste the logs from the rollout controller

# Logs for the entire controller:
kubectl logs -n argo-rollouts deployment/argo-rollouts

# Logs for a specific rollout:
kubectl logs -n argo-rollouts deployment/argo-rollouts | grep rollout=<ROLLOUTNAME

Message from the maintainers:

Impacted by this bug? Give it a 👍. We prioritize the issues with the most 👍.

@meeech meeech added the bug Something isn't working label Apr 2, 2023
@agrawroh
Copy link
Member

agrawroh commented Apr 2, 2023

Could you provide more information on what version of ArgoRollouts you are using? I wouldn't expect it to fail as the docs clearly say: "If both interval and count are omitted, the effective count is 1. If only interval is specified, metric runs indefinitely."

I would expect the analysis to keep running indefinitely in absence of the count.

@meeech
Copy link
Contributor Author

meeech commented Apr 3, 2023

(responded/ discussion on PR)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants