Modular infrastructure providers using Sveltos templates #874
Conversation
| @@ -200,7 +201,7 @@ func (r *ReleaseReconciler) ensureManagement(ctx context.Context) error { | |||
| if err != nil { | |||
| return err | |||
| } | |||
| mgmtObj.Spec.Providers = hmc.GetDefaultProviders() | |||
| mgmtObj.Spec.Providers = providers.List() | |||
There was a problem hiding this comment.
File internal/providers/azure.go is still needed due to a need of populating this list on first start of manager.
There was a problem hiding this comment.
One of the approaches that can allow removal of mentioned file is taken in https://github.com/k0rdent/kcm/pull/833/files
Specifically:
- YAML provider loader, just without Lua parts
- Actual YAML provider definition, also ignore Lua parts here
- YAML loader in
managerstart
There was a problem hiding this comment.
Or any other way that can populate list with extra values during manager start
Or maybe we could rethink logic behind that list altogether
internal/providers/azure.go
Outdated
There was a problem hiding this comment.
| @@ -41,3 +41,57 @@ spec: | |||
| kind: AzureClusterIdentity | |||
| name: azure-cluster-identity | |||
| namespace: ${NAMESPACE} | |||
| --- | |||
There was a problem hiding this comment.
Template render errors are available from ClusterSummary
kubectl -n ${NAMESPACE} get ClusterSummary some-object-name -o yaml
There was a problem hiding this comment.
Preview of worst-case scenario:
s3rj1k
force-pushed
the
provider-modules-sveltos-templates
branch
from
ac61def to
e92e0cb
Compare
Modular infrastructure providers using Sveltos templates