-
Notifications
You must be signed in to change notification settings - Fork 25
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
Install Sveltos as a Provider Template #294
Conversation
6aad700
to
87fc7dc
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sveltos chart should be referenced, not copied over. This way we doomed to copy it on every update.
Just create an empty chart and reference sveltos as the only subchart.
631a224
to
dca7f61
Compare
dca7f61
to
d610b33
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@wahabmk minor typo in version.
Rest looks good
description: A Helm chart to refer the official projectsveltos helm chart | ||
type: application | ||
version: 0.37.1 | ||
appVersion: "0.37.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Inconsistent version here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh that was intentional because the 0.37.1
chart version did not have change in app version. However, Sveltos has since released 0.38.1
so I've updated it to that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
This fixes #269.
provider
type template.projectsveltos
namespace for it to function.hmc-system
namespace but it didn't work properly, the details of which can be seen here.Manual Testing
Installed with AWS provider and waited for everything to be in running state.
Management Cluster
HelmReleases
have been successfully reconciled:projectsveltos
namespace:➜ ~ kubectl get deployments -A NAMESPACE NAME READY UP-TO-DATE AVAILABLE AGE hmc-system azureserviceoperator-controller-manager 1/1 1 1 12m hmc-system capa-controller-manager 1/1 1 1 13m hmc-system capi-controller-manager 1/1 1 1 14m hmc-system capz-controller-manager 1/1 1 1 12m hmc-system helm-controller 1/1 1 1 15m hmc-system hmc-cert-manager 1/1 1 1 15m hmc-system hmc-cert-manager-cainjector 1/1 1 1 15m hmc-system hmc-cert-manager-webhook 1/1 1 1 15m hmc-system hmc-cluster-api-operator 1/1 1 1 14m hmc-system hmc-controller-manager 1/1 1 1 15m hmc-system k0smotron-controller-manager-bootstrap 1/1 1 1 13m hmc-system k0smotron-controller-manager-control-plane 1/1 1 1 13m hmc-system k0smotron-controller-manager-infrastructure 1/1 1 1 12m hmc-system source-controller 1/1 1 1 15m kube-system coredns 2/2 2 2 16m local-path-storage local-path-provisioner 1/1 1 1 16m projectsveltos access-manager 1/1 1 1 13m projectsveltos addon-controller 1/1 1 1 13m projectsveltos classifier-manager 1/1 1 1 13m projectsveltos conversion-webhook 1/1 1 1 13m projectsveltos event-manager 1/1 1 1 13m projectsveltos hc-manager 1/1 1 1 13m projectsveltos sc-manager 1/1 1 1 13m projectsveltos shard-controller 1/1 1 1 13m projectsveltos sveltos-agent-manager 1/1 1 1 12m
➜ ~ kubectl get classifier NAME AGE default-classifier 13m
env=fv
:➜ ~ kubectl get cluster -A --show-labels NAMESPACE NAME CLUSTERCLASS PHASE AGE VERSION LABELS hmc-system wali-aws-dev Provisioned 11m app.kubernetes.io/managed-by=Helm,env=fv,helm.toolkit.fluxcd.io/name=wali-aws-dev,helm.toolkit.fluxcd.io/namespace=hmc-system,sveltos-agent=present
➜ ~ kubectl get clusterprofile NAME AGE kyverno 118m
Target Cluster
As can be seen below, Sveltos is running in
projectsveltos
namespace andkyverno
has also been installed in the target cluster.➜ ~ kubectl get deployments -A NAMESPACE NAME READY UP-TO-DATE AVAILABLE AGE kube-system calico-kube-controllers 1/1 1 1 7m33s kube-system coredns 2/2 2 2 7m37s kube-system ebs-csi-controller 2/2 2 2 7m17s kube-system metrics-server 1/1 1 1 7m28s kyverno kyverno-admission-controller 1/1 1 1 40s kyverno kyverno-background-controller 1/1 1 1 40s kyverno kyverno-cleanup-controller 1/1 1 1 40s kyverno kyverno-reports-controller 1/1 1 1 40s projectsveltos sveltos-agent-manager 1/1 1 1 7m3s