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

"node-csr-approval" is not in the list of known controllers #193

Closed
nogweii opened this issue Aug 28, 2024 · 7 comments
Closed

"node-csr-approval" is not in the list of known controllers #193

nogweii opened this issue Aug 28, 2024 · 7 comments

Comments

@nogweii
Copy link

nogweii commented Aug 28, 2024

Bug Report

Description

With the most recent chart release, talos-ccm crash loops with the error message that it doesn't recognize the controller name. I figure it's because there wasn't a version released that included the code yet, although the chart is configured for it.

Logs

"node-csr-approval" is not in the list of known controllers
E0828 19:00:49.247979       1 run.go:74] "command failed" err="\"node-csr-approval\" is not in the list of known controllers"

(I had to manually approve the node's CSR to get the logs, avoiding a catch-22 I found myself in before realizing that was an option. I thought I can't capture the logs as there is a TLS error, but the error won't be resolved as talos-ccm isn't running. 😄 )

Environment

  • Talos CCM version: 1.6
  • Talos version: 1.7.6
  • Kubernetes version: 1.30.1
  • Platform: metal
@sergelogvinov
Copy link
Collaborator

Hi, this param appears here https://github.com/siderolabs/talos-cloud-controller-manager/blob/main/docs/deploy/cloud-controller-manager-edge.yml for edge version. It has imagePullPolicy: Always policy.

How do you deploy the CCM ?

@nogweii
Copy link
Author

nogweii commented Aug 29, 2024

I'm using Helmfile to deploy, with the following configuration entry:

releases:
  - name: talos-cloud-controller-manager
    namespace: kube-system
    chart: oci://ghcr.io/siderolabs/charts/talos-cloud-controller-manager
    version: 0.4.0
    values:
      - ./chart-values/talos-ccm.yaml

@sergelogvinov
Copy link
Collaborator

Thank you for feedback. Yap not I see what append here.

I update the documentation and remove node-csr-approval controller by default #194

@sergelogvinov
Copy link
Collaborator

merged

@Rhymen
Copy link

Rhymen commented Sep 4, 2024

I deployed the helm chart 0.4.1 with the example values.yaml from the README.md

# talos-ccm.yaml

replicaCount: 2

enabledControllers:
  - cloud-node
  - node-csr-approval

# Deploy CCM only on control-plane nodes
nodeSelector:
  node-role.kubernetes.io/control-plane: ""
tolerations:
  - key: node-role.kubernetes.io/control-plane
    effect: NoSchedule

I get the same error as explained in the issue here:
"node-csr-approval" is not in the list of known controllers

Environment

  • Talos CCM Helm Chart: 0.4.1
  • Talos CCM version: 1.6.0
  • Talos version: 1.7.6
  • Kubernetes version: 1.30.3
  • Platform: metal

@sergelogvinov
Copy link
Collaborator

Hi, you force the list of controllers:

enabledControllers:
  - cloud-node
  - node-csr-approval

Documentation was updated recently for new (future) release. It will work for edge version.
We are waiting Talos release right now, to release CCM.

@Rhymen
Copy link

Rhymen commented Sep 4, 2024

I see, i was missing that the changes are only deployed on the edge version. Thank you for the clarification.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants