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

Vineyard operator can't be deployed successfully with helm chart #1490

Closed
dashanji opened this issue Jul 25, 2023 · 7 comments · Fixed by #1492
Closed

Vineyard operator can't be deployed successfully with helm chart #1490

dashanji opened this issue Jul 25, 2023 · 7 comments · Fixed by #1492
Assignees
Labels
bug Something isn't working kubernetes upstream

Comments

@dashanji
Copy link
Member

Describe your problem

Get the following debug info.

 Warning  FailedMount  108s (x10 over 5m58s)  kubelet            MountVolume.SetUp failed for volume "cert" : secret "vineyard-operator-webhook-server-cert" not found 
 Warning  FailedMount  100s (x2 over 3m55s)   kubelet            Unable to attach or mount volumes: unmounted volumes=[cert], unattached volumes=[], failed to process volumes=[]: timed out waiting for the condition
@dashanji dashanji self-assigned this Jul 25, 2023
@dashanji dashanji added bug Something isn't working kubernetes priority:high labels Jul 25, 2023
@sighingnow
Copy link
Member

I have noticed this issue many times. There might be another cert-manager running (maybe in another namespace).

@dashanji
Copy link
Member Author

dashanji commented Jul 26, 2023

In my case, after adding the --wait flag, the vineyard operator can't be deployed successfully.

@dashanji
Copy link
Member Author

dashanji commented Jul 26, 2023

I have noticed this issue many times. There might be another cert-manager running (maybe in another namespace).

Currently, if another cert-manager exists, vineyard operator helm chart will not be created successfully as the cert-manager CRD already exist.

In my case, after adding the --wait flag, the vineyard operator can't be deployed successfully.

If we set the --wait flag, the secret vineyard-operator-webhook-server-cert will not be created which might be caused by the issue. It looks like we need to wait for the helm pr to merge. As a workaround, the users need to delete the --wait flag and wait for the operator ready explicitly.

/cc @sighingnow

@dashanji
Copy link
Member Author

@sighingnow I'm sorry I didn't find the true reason previously and made some noise.

The current chart uses the helm chart hooks to control the order of deployment as follows:

annotations:
"helm.sh/hook": post-install,post-upgrade
"helm.sh/hook-weight": "1"

annotations:
"helm.sh/hook": post-install,post-upgrade
"helm.sh/hook-weight": "2"

From the helm chart hooks, we can see the following explanation:

The library loads the resulting resources into Kubernetes. Note that if the --wait flag is set, the library will wait until all resources are in a ready state and will not run the post-install hook until they are ready.

In a word, If we set the --wait flag, the vineyard operator will not be ready as the issuer and cert never suppose to be created. Maybe we need to find a better way to control the execution order of two helm charts, or regard the deployment of cert-manager as a prerequisite.

@sighingnow
Copy link
Member

Remove --wait is ok.

Thanks!

@sighingnow
Copy link
Member

Add a record in the FAQ section of documentation.

@dashanji
Copy link
Member Author

Copy.

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

Successfully merging a pull request may close this issue.

2 participants