GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Fix of some minor bugs when using the driver manifest and update helm chart versions by elivlo · Pull Request #4 · cloudstack/cloudstack-csi-driver

Conversation

@elivlo

@elivlo elivlo commented Nov 5, 2025

Copy link
Copy Markdown

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.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
elivlo added 3 commits November 5, 2025 12:31
@elivlo
Fix manifest creation caused by missing linebreaks
ceb019d
@elivlo
Add empty option to description for metadataSource as possible entry
fcba53c
@elivlo
Bump app version to 3.0.0 of csi driver in helm chart
c22f761
@yadvr
yadvr requested a review from Pearl1594 December 21, 2025 14:28
@elivlo

elivlo commented Feb 10, 2026

Copy link
Copy Markdown
Author

@Pearl1594 @yadvr Is there any interest in further developing and maintaining the CSI driver?

I could provide some support by at least maintaining the driver (dependency updates, Go updates) and keeping it up to date with some other projects from shapeblue and leaseweb.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@yadvr

yadvr commented Feb 10, 2026

Copy link
Copy Markdown
Member

@elivlo yes of course, ACS CSI drive is an actively maintained and developed project - all contributions are welcome. We use this repo for release purposes, and which syncs from the the primary/fork repo - https://github.com/shapeblue/cloudstack-csi-driver and @Pearl1594 is our lead on this.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

Pearl1594
Pearl1594 approved these changes Feb 10, 2026
View reviewed changes
@Pearl1594

Pearl1594 commented Feb 10, 2026

Copy link
Copy Markdown
Collaborator

@elivlo Thanks for your interest, any contributions are welcome. Apologies for the delay in getting to review this PR.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@elivlo

elivlo commented Feb 25, 2026

Copy link
Copy Markdown
Author

@Pearl1594 No need to apologize! I'm not able to merge the PR. Should I create i PR also in the shapeblue repository?

I also found a bug in the code and want to open a new PR. Should I do that here or in the other repository?

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@Pearl1594
Pearl1594 merged commit 3c9de10 into cloudstack:main Feb 25, 2026
@Pearl1594

Pearl1594 commented Feb 25, 2026

Copy link
Copy Markdown
Collaborator

@elivlo you can raise the PRs against the ShapeBlue repo, which will then ultimately get synced here..

elivlo reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Pearl1594 Pearl1594 Pearl1594 approved these changes

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

3 participants

@elivlo @yadvr @Pearl1594
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402520 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.