Skip to content

Initial implementation using kustomize and envsubst - #1

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

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

Conversation

@thobens

@thobens thobens commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator

This PR provides the initial implementation of the capi-provider-talos component. It utilizes kustomize to build the manifests from the upstream provider repositories for the bootstrap and control-plane provider respectively. The component 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-talos-provider branch from 6bc2f4e to 5d37018 Compare August 18, 2026 23:20
Comment thread class/defaults.yml Outdated
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-talos-provider branch from fb0ec68 to ad8e8ca Compare August 19, 2026 15:13
@thobens
thobens marked this pull request as ready for review August 19, 2026 15:14
@thobens
thobens requested review from DebakelOrakel and simu August 24, 2026 08:17
Comment thread class/capi-provider-talos.yml Outdated
Comment thread component/bootstrap-provider.jsonnet Outdated
Comment thread component/main.jsonnet Outdated
Comment thread scripts/envsubst-dir Outdated
Comment thread class/capi-provider-talos.yml Outdated
Comment thread class/capi-provider-talos.yml Outdated
Comment thread class/defaults.yml 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 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 review from simu and removed request for DebakelOrakel August 25, 2026 12:20

@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.

Same as capi-provider-cloudscale, we should switch the ArgoCD app to server-side apply.

Comment thread docs/modules/ROOT/pages/references/parameters.adoc Outdated
@thobens
thobens force-pushed the alde-262/implement-talos-provider branch from 85b38ce to 1b18ab5 Compare August 25, 2026 12:42
@thobens
thobens force-pushed the alde-262/implement-talos-provider branch from 1b18ab5 to 2a93442 Compare August 25, 2026 12:49
@thobens
thobens requested a review from simu August 25, 2026 13:42
@thobens
thobens merged commit 85ae77b 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