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

Helm version param doesn't exist in released CRD #4468

Closed
3 tasks done
epinzur opened this issue Oct 1, 2020 · 7 comments
Closed
3 tasks done

Helm version param doesn't exist in released CRD #4468

epinzur opened this issue Oct 1, 2020 · 7 comments
Labels
answered question Issue is a question or reach for support

Comments

@epinzur
Copy link

epinzur commented Oct 1, 2020

If you are trying to resolve an environment-specific issue or have a one-off question about the edge case that does not require a feature then please consider asking a
question in argocd slack channel.

Checklist:

  • I've searched in the docs and FAQ for my answer: https://bit.ly/argocd-faq.
  • I've included steps to reproduce the bug.
  • I've pasted the output of argocd version.

Describe the bug

The documentation states that you can specify the Helm version to use in the declarative setup for an Application. However, the version flag is stripped by Kubernetes on deploy.

In the latest release, the CRD does not have the version parameter: https://github.com/argoproj/argo-cd/blob/v1.7.7/manifests/crds/application-crd.yaml#L198-L208

But in the master version of the CRD, it does exist: https://github.com/argoproj/argo-cd/blob/master/manifests/crds/application-crd.yaml#L198-L211

Due to this, if I set the parameter version on the application (with no other helm parameters set), Kubernetes strips the field from the manifest. But Argo sees that it exists in source control, so it re-applies the manifest. And then Kubernetes strips it again. And repeat... in a constant sync loop.

See related PR where the version parameter was added: #4111

Expected behavior

The released CRD should allow a version parameter for helm, to match docs.

Version

argocd: v1.7.7+33c93ae
  BuildDate: 2020-09-29T04:54:55Z
  GitCommit: 33c93aea0b9ee3d02fb9703cd82cecce3540e954
  GitTreeState: clean
  GoVersion: go1.14.1
  Compiler: gc
  Platform: linux/amd64
@epinzur epinzur added the bug Something isn't working label Oct 1, 2020
@jessesuen
Copy link
Member

This because the feature only exists in master and is not yet released. v1.7 does not have this feature.

@jessesuen jessesuen added answered question Issue is a question or reach for support and removed bug Something isn't working labels Oct 1, 2020
@mars64
Copy link

mars64 commented Oct 1, 2020

This because the feature only exists in master and is not yet released. v1.7 does not have this feature.

How is a user supposed to know this?

@epinzur
Copy link
Author

epinzur commented Oct 2, 2020

Things that aren't released should not be in docs!

Is there any timeline on when this will be released?

@jannfis
Copy link
Member

jannfis commented Oct 2, 2020

How is a user supposed to know this?

I agree that we have some serious debt regarding documentation. We were already thinking of migrating the docs to a system that actually can handle versioned docs (such as readthedocs.org), so that users will be able to use the documentation for the specific version of Argo CD they are using. Currently, we are pretty much cluttering the docs with version information (available since vX, or only before vY etc).

Personally, I think the debt goes further than that, and that the docs could also need some serious love and restructuring. However, this is obviously a lot of work, and everyone is invited to contribute.

Oops, that is due to the fact that indeed the link is pointing to the wrong location. Correct link would be https://argoproj.github.io/argo-cd/user-guide/, I've submitted a fix with #4477. Thanks for bringing that to our attention.

@mars64
Copy link

mars64 commented Oct 2, 2020

I agree that we have some serious debt regarding documentation. We were already thinking of migrating the docs to a system that actually can handle versioned docs (such as readthedocs.org), so that users will be able to use the documentation for the specific version of Argo CD they are using. Currently, we are pretty much cluttering the docs with version information (available since vX, or only before vY etc).

Personally, I think the debt goes further than that, and that the docs could also need some serious love and restructuring. However, this is obviously a lot of work, and everyone is invited to contribute.

Huge thank you for the acknowledgement here, I think I'm not alone in my expectations of the docs site and knowing that the group at least has it on the radar is of much value to me. Perhaps I'll take a look at that contribution guide! (subtle plug eh ;) )

Oops, that is due to the fact that indeed the link is pointing to the wrong location. Correct link would be https://argoproj.github.io/argo-cd/user-guide/, I've submitted a fix with #4477. Thanks for bringing that to our attention.

Thank you!

@Stephan3555
Copy link

Is there any timeline when and in which version this feature will be released?

@mcwienczek
Copy link

mcwienczek commented Dec 22, 2020

I think it was released with version 1.8.0:
https://github.com/argoproj/argo-cd/releases/tag/v1.8.0

Configurable Helm Versions #4111

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
answered question Issue is a question or reach for support
Projects
None yet
Development

No branches or pull requests

6 participants