Skip to content

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 <richard@weave.works>
@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