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

improve doc for k8s cluster. #151

Open
vladimir4862 opened this issue Jun 2, 2021 · 4 comments
Open

improve doc for k8s cluster. #151

vladimir4862 opened this issue Jun 2, 2021 · 4 comments
Labels
k8s-stack k8s-stack helm chart related issue

Comments

@vladimir4862
Copy link
Contributor

this line from README.md in victoria-metrics-k8s-stack doesn't work, If you run this on new cluster (without victoria CRDs)

Test the installation with command:

helm install [RELEASE_NAME] vm/victoria-metrics-k8s-stack -f values.yaml -n NAMESPACE --debug --dry-run

output

Error: unable to build kubernetes objects from release manifest: [unable to recognize "": no matches for kind "VMAgent" in version "operator.victoriametrics.com/v1beta1", unable to recognize "": no matches for kind "VMAlert" in version "operator.victoriametrics.com/v1beta1.....",

so maybe need add some additional steps in "Prerequisites"

@f41gh7
Copy link
Collaborator

f41gh7 commented Jun 8, 2021

Looks like bug. Which helm version are you using?

@vladimir4862
Copy link
Contributor Author

Hello.
version.BuildInfo{Version:"v3.5.3", GitCommit:"041ce5a2c17a58be0fcd5f5e16fb3e7e95fea622", GitTreeState:"dirty", GoVersion:"go1.16"}

@f41gh7
Copy link
Collaborator

f41gh7 commented Jun 9, 2021

It was surprising for me, but according to documentation of helm, it cannot render charts with --dry-run, if it contains crd. So, you have to install chart first.

https://helm.sh/docs/chart_best_practices/custom_resource_definitions/#method-1-let-helm-do-it-for-you

@denisgolius denisgolius added the k8s-stack k8s-stack helm chart related issue label Aug 6, 2021
@b-a-t
Copy link
Contributor

b-a-t commented Apr 1, 2022

Being newbie for all EKS/k8s/helm shebang I hit the same error following the instructions and got frustrated and stuck for quite a while, until risked to go farther despite the failure with --dry-run.

So please, add a note that for the initial run while CRDs are not created --dry-run have to be skipped.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
k8s-stack k8s-stack helm chart related issue
Projects
None yet
Development

No branches or pull requests

4 participants