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

feat: add CAPX API #13

Closed
wants to merge 6 commits into from
Closed

feat: add CAPX API #13

wants to merge 6 commits into from

Conversation

dkoshkin
Copy link
Collaborator

I wanted to propose this while we working on #6

This PR adds the minimum API that is needed to create a CAPX cluster, making it easier to review just this part and would unblock the CCM work I'm working on.

  • The first commit grabs the API directory from feat: Add support for CAPX #6
  • The second commit simplifies it by reusing alot of existing copy and pasted APIs from the external CAPX package, clusterv1.APIEndpoint and corev1.LocalObjectReference
  • The 3rd commit removes the fields that have not been implemented in ttps://github.com/feat: Add support for CAPX #6 (hopefully making it easier to review)

I've verified I didn't break the existing PR by making the required changes in 3360d27 and running the unit tests.

@dkoshkin dkoshkin requested review from faiq and deepakm-ntnx March 29, 2024 20:56
@dkoshkin dkoshkin self-assigned this Mar 29, 2024
@dkoshkin dkoshkin force-pushed the dkoshkin/feat-add-capx-api branch 2 times, most recently from 1bc4015 to 1970b45 Compare March 29, 2024 21:11
deepakm-ntnx
deepakm-ntnx previously approved these changes Mar 29, 2024
Copy link

@deepakm-ntnx deepakm-ntnx left a comment

Choose a reason for hiding this comment

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

Thanks @dkoshkin

@deepakm-ntnx
Copy link

deepakm-ntnx commented Mar 29, 2024

@dkoshkin can we first just get the things as is merged from #6 so that we can reduce the churn in other PR. would be easier to reiterate once we have one pass working as is

@dkoshkin
Copy link
Collaborator Author

dkoshkin commented Apr 1, 2024

@deepakm-ntnx I also set required fields for the nodes based on the type, PTAL d53ee24

@dkoshkin dkoshkin mentioned this pull request Apr 1, 2024
@dkoshkin dkoshkin force-pushed the dkoshkin/feat-add-capx-api branch from b45d67f to 06c094f Compare April 1, 2024 18:17
Copied this from an existing PR.
@dkoshkin dkoshkin force-pushed the dkoshkin/feat-add-capx-api branch from 06c094f to 46f7893 Compare April 1, 2024 19:13
@dkoshkin dkoshkin force-pushed the dkoshkin/feat-add-capx-api branch from 8380c67 to 870884b Compare April 1, 2024 22:12
@@ -94,6 +94,7 @@ require (
github.com/modern-go/concurrent v0.0.0-20180306012644-bacd9c7ef1dd // indirect
github.com/modern-go/reflect2 v1.0.2 // indirect
github.com/munnerz/goautoneg v0.0.0-20191010083416-a7dc8b61c822 // indirect
github.com/nutanix-cloud-native/prism-go-client v0.3.4 // indirect
Copy link

Choose a reason for hiding this comment

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

where is this used?

@dkoshkin
Copy link
Collaborator Author

dkoshkin commented Apr 2, 2024

Closing in-favor of #6

@dkoshkin dkoshkin closed this Apr 2, 2024
@dkoshkin dkoshkin deleted the dkoshkin/feat-add-capx-api branch April 5, 2024 18:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants