Skip to content
This repository was archived by the owner on Apr 11, 2024. It is now read-only.

feat: deploy Nutanix CCM as an Addon #38

Merged
merged 7 commits into from
Apr 10, 2024
Merged

feat: deploy Nutanix CCM as an Addon #38

merged 7 commits into from
Apr 10, 2024

Conversation

dkoshkin
Copy link
Collaborator

@dkoshkin dkoshkin commented Apr 8, 2024

Stacked on dkoshkin/fix-nutanix-csi

This PR removes the Nutanix CCM from the templates and instead deploys it using CAAPH using the published Helm chart.

$ kubectl get pods -n kube-system
NAME                                                          READY   STATUS    RESTARTS   AGE
coredns-5dd5756b68-d8t7r                                      0/1     Running   0          83s
coredns-5dd5756b68-z6l4g                                      0/1     Running   0          83s
etcd-dkoshkin-nutanix-0508-2-w9kmm-7mkl2                      1/1     Running   0          95s
kube-apiserver-dkoshkin-nutanix-0508-2-w9kmm-7mkl2            1/1     Running   0          95s
kube-controller-manager-dkoshkin-nutanix-0508-2-w9kmm-7mkl2   1/1     Running   0          95s
kube-proxy-7x762                                              1/1     Running   0          52s
kube-proxy-hg5mq                                              1/1     Running   0          83s
kube-scheduler-dkoshkin-nutanix-0508-2-w9kmm-7mkl2            1/1     Running   0          96s
kube-vip-dkoshkin-nutanix-0508-2-w9kmm-7mkl2                  1/1     Running   0          95s
nutanix-cloud-controller-manager-694ff9d9dd-m9z92             1/1     Running   0          83s


$ kubectl get nodes -o yaml | grep providerID
    providerID: nutanix://11e638da-6aa0-45b9-8477-3836aa965842
    providerID: nutanix://db91a5ca-22a0-4818-8b5e-c79361f82877

https://d2iq.atlassian.net/browse/D2IQ-99841

@dkoshkin dkoshkin self-assigned this Apr 8, 2024
@dkoshkin dkoshkin force-pushed the dkoshkin/feat-nutanix-ccm-addon branch 2 times, most recently from 961ee09 to ef132d9 Compare April 8, 2024 17:24
@dkoshkin dkoshkin force-pushed the dkoshkin/fix-nutanix-csi branch from ab52d33 to 764761c Compare April 9, 2024 16:31
Base automatically changed from dkoshkin/fix-nutanix-csi to main April 9, 2024 19:01
@github-actions github-actions bot added feature and removed feature labels Apr 9, 2024
@dkoshkin dkoshkin force-pushed the dkoshkin/feat-nutanix-ccm-addon branch 2 times, most recently from b7bea8f to ee58bbd Compare April 9, 2024 21:45
faiq
faiq approved these changes Apr 9, 2024
View reviewed changes
Copy link

@faiq faiq left a comment

Choose a reason for hiding this comment

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

looks good

@dkoshkin dkoshkin requested a review from supershal April 10, 2024 16:19
@dkoshkin dkoshkin force-pushed the dkoshkin/feat-nutanix-ccm-addon branch from abde55d to 5e41c8f Compare April 10, 2024 16:36
supershal
supershal approved these changes Apr 10, 2024
View reviewed changes
Copy link

@supershal supershal left a comment

Choose a reason for hiding this comment

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

LGTM.

@dkoshkin dkoshkin merged commit 05b8c1a into main Apr 10, 2024
13 of 16 checks passed
@dkoshkin dkoshkin deleted the dkoshkin/feat-nutanix-ccm-addon branch April 10, 2024 16:53
@github-actions github-actions bot mentioned this pull request Apr 10, 2024
Open
jimmidyson pushed a commit that referenced this pull request Apr 11, 2024
@dkoshkin @jimmidyson
feat: deploy Nutanix CCM as an Addon (#38)
3774cad 
* build: remove nutanix CCM from examples

* build: add scripts to sync Nutanix CCM manifests

* build: add CCM addon var to Nutanix examples

* feat: deploy Nutanix CCM addon

Aligns the method of deploying the CCM with all other addons.

* fixup! build: add scripts to sync Nutanix CCM manifests

* fixup! feat: deploy Nutanix CCM addon

* build: remove unused CRS tooling for Nutanix CCM

The CRS strategy is not supported for the Nutanix CCM,
removing it until we actually need it.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@faiq faiq faiq approved these changes

@supershal supershal supershal approved these changes

@deepakm-ntnx deepakm-ntnx Awaiting requested review from deepakm-ntnx

Assignees

@dkoshkin dkoshkin

Labels
feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dkoshkin @faiq @supershal