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

Use upstream csi-cloudscale Helm chart #47

Merged
merged 6 commits into from
Aug 28, 2023
Merged

Use upstream csi-cloudscale Helm chart #47

merged 6 commits into from
Aug 28, 2023

Conversation

simu
Copy link
Member

@simu simu commented Sep 21, 2022

This is a breaking change, as we remove component parameter version.

TODO

Checklist

  • PR contains a single logical change (to build a better changelog).
  • Update the documentation.
  • Categorize the PR by setting a good title and adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency
    as they show up in the changelog.

@simu simu added the breaking A breaking change. Implies a new major version. label Sep 21, 2022
@simu simu force-pushed the feat/use-helmchart branch from 54df1bf to b285349 Compare August 25, 2023 11:51
@simu simu mentioned this pull request Aug 25, 2023
3 tasks
@simu simu force-pushed the feat/use-helmchart branch from b285349 to 77ac4cd Compare August 25, 2023 12:56
@simu simu changed the title Draft: Use csi-cloudscale Helmchart Use upstream csi-cloudscale Helm chart Aug 25, 2023
@simu simu force-pushed the feat/use-helmchart branch 2 times, most recently from c780bb7 to 0c98250 Compare August 25, 2023 13:06
simu added 3 commits August 25, 2023 15:11
…tions` parameter

We can't simply pass the parameter to the `node.tolerations` Helm value,
since that value expects an array of regular tolerations while the
component parameter is structured to allow users to overwrite
tolerations.

Note that the postprocessing filter will retain tolerations configured
through the Helm value and will simply append any tolerations configured
in `driver_daemonset_tolerations`.
@simu simu force-pushed the feat/use-helmchart branch from 0c98250 to 3e95d46 Compare August 25, 2023 13:11
@simu
Copy link
Member Author

simu commented Aug 25, 2023

The only actual change is that the CSIDriver object is deployed with the Helm chart. Upgrading on the cluster is a no-op except for that object.

simu added 3 commits August 25, 2023 15:26
We don't need it ourselves since we don't use parameter `version`
anywhere.
@simu simu force-pushed the feat/use-helmchart branch from 64dedd8 to 35ddf74 Compare August 25, 2023 13:44
@simu simu marked this pull request as ready for review August 25, 2023 13:45
@simu simu requested a review from a team August 25, 2023 13:45
@simu simu linked an issue Aug 25, 2023 that may be closed by this pull request
@simu simu merged commit 26aa9d1 into master Aug 28, 2023
@simu simu deleted the feat/use-helmchart branch August 28, 2023 07:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking A breaking change. Implies a new major version.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use csi-cloudscale Helmchart
2 participants