-
Notifications
You must be signed in to change notification settings - Fork 20
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
Introduce VPC controller #62
Conversation
cccf3fd
to
cb4cb2c
Compare
59d34b9
to
e723014
Compare
fd84967
to
5af0987
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.
Sorry I had prepared comments but never submitted them!
I have tested this and confirmed a VPC is created and |
Co-authored-by: Alex Vest <[email protected]>
Co-authored-by: Alex Vest <[email protected]>
@eljohnson92 @avestuk I addressedd all the review items, so could you please do the final review (one more commit) and it should be ready to merge. |
* Introduce VPC controller * Requeue non empty vpc deletion * Filter out not found in vpc controller patch * Update controller/linodemachine_controller.go Co-authored-by: Alex Vest <[email protected]> * Update controller/linodemachine_controller_helpers.go Co-authored-by: Alex Vest <[email protected]> * update CRD creation for VPC and setup VPC Controller in main.go * Remove vpc list copy before sort --------- Co-authored-by: Richard Kovacs <[email protected]> Co-authored-by: Alex Vest <[email protected]> Co-authored-by: Evan Johnson <[email protected]>
* fixup! Introduce VPC controller (#62) * feat: linodevpc: kubebuilder create webhook Scaffold a validating admission webhook for the LinodeVPC resource with Kubebuider via the command: kubebuilder create webhook --group infrastructure --version v1alpha1 --kind LinodeVPC --programmatic-validation * fixup! feat: linodevpc: kubebuilder create webhook * golangci: configure varnamelen * api: linodevpc: add create validation
This change introduces a new controller to manage VPCs on Linode. The Cluster object has an optional
vpcRef
erence field. The customer should bring them own VPC by setting thelabel
field to the custom resource to pointing to an existing VPC. The machine controller also has been updated to attach new instances to the VPC.Fixes: #4