Skip to content

Initial implementation using kustomize and envsubst - #1

Merged
thobens merged 6 commits into
masterfrom
alde-262/implement-cloudscale-provider
Aug 25, 2026
Merged

Initial implementation using kustomize and envsubst#1
thobens merged 6 commits into
masterfrom
alde-262/implement-cloudscale-provider

Conversation

@thobens

@thobens thobens commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator

This PR provides the initial implementation of the capi-provider-cloudscale component. It utilizes kustomize to build the manifests from the upstream provider repository and then substitutes variables on the generated output. It does so using https://github.com/a8m/envsubst, which will be included in the commodore image used for CI/CD. This simulates the behavior of clusterctl so we can use project syn to set up CAPI.

See also:

Checklist

  • The PR has a meaningful title. It will be used to auto-generate the
    changelog.
    The PR has a meaningful description that sums up the change. It will be
    linked in the changelog.
  • PR contains a single logical change (to build a better changelog).
  • Update the documentation.
  • Categorize the PR by adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency
    as they show up in the changelog.
  • Link this PR to related issues or PRs.

@thobens thobens added bump:patch Create a new patch release when merging the labeled PR documentation Improvements or additions to documentation labels Aug 18, 2026
@github-actions

github-actions Bot commented Aug 18, 2026

Copy link
Copy Markdown

🚀 This PR has been released as v0.1.0

Triggering workflows Release

🛠️ Auto tagging enabled with label bump:minor

@github-actions

Copy link
Copy Markdown

🚀 Merging this PR will release v0.0.1

Merging will trigger workflows Release

🛠️ Auto tagging enabled with label bump:patch

@thobens
thobens force-pushed the alde-262/implement-cloudscale-provider branch 2 times, most recently from f9665d4 to ed35f57 Compare August 18, 2026 21:25
@thobens
thobens force-pushed the alde-262/implement-cloudscale-provider branch from ed35f57 to e2b5523 Compare August 18, 2026 23:23
Comment thread class/defaults.yml Outdated
@thobens thobens added bump:minor Create a new minor release when merging the labeled PR enhancement New feature or request and removed bump:patch Create a new patch release when merging the labeled PR documentation Improvements or additions to documentation labels Aug 19, 2026
@thobens
thobens force-pushed the alde-262/implement-cloudscale-provider branch from f3be9ea to 546f66d Compare August 19, 2026 15:13
@thobens
thobens marked this pull request as ready for review August 19, 2026 15:14
@thobens
thobens requested a review from DebakelOrakel August 24, 2026 08:17
Comment thread class/capi-provider-cloudscale.yml Outdated
Comment thread class/defaults.yml Outdated
Comment thread component/main.jsonnet Outdated
Comment thread scripts/envsubst-dir Outdated
Comment thread Makefile Outdated
@simu

simu commented Aug 24, 2026

Copy link
Copy Markdown
Member

Also why is this component in https://github.com/vshn? Afaict, there's nothing in here that would prevent it from being in https://github.com/projectsyn

@thobens
thobens force-pushed the alde-262/implement-cloudscale-provider branch from bd0eddc to 27f5016 Compare August 25, 2026 11:32
@thobens

thobens commented Aug 25, 2026

Copy link
Copy Markdown
Collaborator Author

Also why is this component in https://github.com/vshn? Afaict, there's nothing in here that would prevent it from being in https://github.com/projectsyn

Move is prepared, will do as soon as this review is complete

@thobens
thobens requested a review from simu August 25, 2026 12:19

@simu simu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should probably configure the ArgoCD app to use server-side apply, since the component deploys CRDs (probably also applies to capi-core). This can be done by adding the following in component/app.jsonnet:

local app = argocd.App('capi-provider-cloudscale', params.namespace) {
  spec+: {
    syncOptions+: [
      'ServerSideApply=true',
    ],
  }
};

Comment thread docs/modules/ROOT/pages/references/parameters.adoc Outdated
Comment thread tests/defaults.yml Outdated
@thobens
thobens force-pushed the alde-262/implement-cloudscale-provider branch from 024ba65 to 16f30e1 Compare August 25, 2026 12:42
@thobens
thobens force-pushed the alde-262/implement-cloudscale-provider branch from 16f30e1 to 655fac6 Compare August 25, 2026 12:48
@thobens
thobens requested a review from simu August 25, 2026 13:42
@thobens
thobens merged commit 7dceb6b into master Aug 25, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bump:minor Create a new minor release when merging the labeled PR enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants