Skip to content

Conversation

@elivlo
Copy link

@elivlo elivlo commented Nov 5, 2025

Fix of some minor bugs when using the driver manifest or helm chart. This should also create a new helm release 3.0.1 with the current app version 3.0.0.

Description of changes:

  • The manifest.yml was missing a line break between two resources
  • Update of the app version in the helm chart to 3.0.0
  • Add a third empty option to the helm value node.metadataSource description as a feasible option (csi will take the name of the kubernetes node)

Testing performed:

I have set up a cluster in a private Cloudstack instance with Talos. This cluster works with the current version (3.0.0) of the csi plugin. I made some config overrides in the helm chart values by adjusting them manually. These overrides were done with terraform and can be reproduced:

resource "helm_release" "cloudstack_csi_driver" {
  name      = "cloudstack-csi"
  chart     = "https://github.com/cloudstack/cloudstack-csi-driver/releases/download/cloudstack-csi-2.0.3/cloudstack-csi-2.0.3.tgz"
  version   = "2.0.3"
  atomic    = true
  namespace = "kube-system"
  set = [
    {
      name  = "cloudConfigData.global.api-url"
      value = var.cloudstack_api_url
    },
    {
      name  = "secret.create"
      value = true
    },
    {
      name  = "secret.enabled"
      value = true
    },
    {
      name  = "controller.image.tag"
      value = "3.0.0"
    },
    {
      name  = "syncer.image.tag"
      value = "3.0.0"
    },
    {
      name  = "node.image.tag"
      value = "3.0.0"
    },
    {
      name = "node.metadataSource"
      value = ""
    }
  ]
  set_sensitive = [
    {
      name  = "cloudConfigData.global.api-key"
      value = var.cloudstack_api_key
    },
    {
      name  = "cloudConfigData.global.secret-key"
      value = var.cloudstack_secret_key
    }
  ]
}

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

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

Successfully merging this pull request may close these issues.

1 participant