Skip to content
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

feat: added cluster and machine api definitions #15

Merged
merged 1 commit into from
Nov 17, 2021
Merged

Conversation

richardcase
Copy link
Member

@richardcase richardcase commented Nov 5, 2021

What this PR does / why we need it:
The initial api definitions for the capi infrastructure implementations of cluster, machine and machinetemplate have been added.

The types follow the capi requirements:

The api types are based on a simplified version of the flintlock grpc api. In this capi provider we will make some assumptions/defaults.

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #10

Special notes for your reviewer:

The main parts for review are api/v1alpha1/*.go

There is a lot of auto-generated configuration from the go code that doesn't need to be reviewed.

Checklist:

  • squashed commits into logical changes
  • adds unit tests

@richardcase richardcase added area/api Indicates an issue or PR relates to the APIs kind/feature New feature or request labels Nov 5, 2021
@github-actions github-actions bot added the size/l label Nov 5, 2021
@richardcase richardcase changed the title feat: added cluster and machine api definitions WIP feat: added cluster and machine api definitions Nov 5, 2021
@github-actions github-actions bot added size/m and removed size/l labels Nov 5, 2021
@richardcase richardcase changed the title WIP feat: added cluster and machine api definitions [WIP] feat: added cluster and machine api definitions Nov 5, 2021
@richardcase richardcase changed the title [WIP] feat: added cluster and machine api definitions feat: added cluster and machine api definitions Nov 11, 2021
@richardcase richardcase marked this pull request as ready for review November 11, 2021 07:57
The initial api definitions for the capi infrastructure implementations
of cluster, machine and machinetemplate have been added.

The type follow the capi requirements:
*[cluster](https://cluster-api.sigs.k8s.io/developer/providers/cluster-infrastructure.html)
*[machine](https://cluster-api.sigs.k8s.io/developer/providers/machine-infrastructure.html)

The api types are based on a simplified version of the flintlock grpc
api. In capi provider we will make some assumptions/defaults.

Signed-off-by: Richard Case <[email protected]>
@richardcase richardcase merged commit b57edd7 into main Nov 17, 2021
@richardcase richardcase deleted the api_types branch November 17, 2021 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/api Indicates an issue or PR relates to the APIs kind/feature New feature or request size/m
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create API definitions for Cluster and Machine
2 participants