Skip to content

[Feature] Support cross-namespace clusterclasses in cluster creation #111

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

Closed
mantis-toboggan-md opened this issue Jan 16, 2025 · 4 comments · Fixed by #161
Closed

[Feature] Support cross-namespace clusterclasses in cluster creation #111

mantis-toboggan-md opened this issue Jan 16, 2025 · 4 comments · Fixed by #161
Assignees
Labels

Comments

@mantis-toboggan-md
Copy link
Member

Turtles will soon support the use of cluster classes in different namespaces than the cluster to be provisioned. The UI needs to be updated to support a new field during cluster creation: classNamespace. If no class namespace is selected, the default is to use the same namespace as the cluster.

@mantis-toboggan-md mantis-toboggan-md self-assigned this Jan 22, 2025
@Danil-Grigorev
Copy link

kubernetes-sigs/cluster-api#5673 is fixed, this will be added in the next turtles release with CAPI 1.9.5

@anmazzotti
Copy link

We could just have a "Cluster Namespace" in the second screen of Cluster creation. The UI currently uses the Class namespace's instead. However that will then need to be passed as classNamespace field:

apiVersion: cluster.x-k8s.io/v1beta1
kind: Cluster
metadata:
  name: my-cluster
  namespace: my-namespace
spec:
  topology:
    class: azure-something
    classNamespace: my-clusterclasses

This requires CAPI core provider to be >= v1.6.5, which raises the question if we should have feature gates based on CAPI core and maybe even Turtles versions.

@gaktive
Copy link
Member

gaktive commented Apr 15, 2025

We may need an annotation to indicate what versions of CAPI a specific version of Turtles works with since this is a big can of worms in the making as the engine changes.

@mantis-toboggan-md mantis-toboggan-md removed their assignment May 8, 2025
@eva-vashkevich
Copy link
Member

For now, we decided to just check the schema to see if this feature should be enabled

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants