✨ Add default, validation and conversion webhooks #250
Conversation
/cc @kashifest |
/test-v1a3-integration |
545bddb
to
429078f
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.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jan-est, maelk The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
- add conversion-gen output for v1alpha2 - add conversion functions for v1alpha2 - add webhooks for v1alpha2
adds the webhooks deployment
Now runs a controller for webhooks and a controller for reconciliation Use Go 1.13
and update kustomize and kubebuilder install scripts
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.
/lgtm
@@ -36,10 +36,10 @@ import ( | |||
"k8s.io/utils/pointer" | |||
|
|||
bmh "github.com/metal3-io/baremetal-operator/pkg/apis/metal3/v1alpha1" | |||
capbm "github.com/metal3-io/cluster-api-provider-baremetal/api/v1alpha3" |
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.
I think we should use infrav1 in all places to be more consistent.
c.Default() | ||
} | ||
|
||
func TestBareMetalMachineTemplateValidation(t *testing.T) { |
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.
Should we only check Image for validation? No other fields to check?
This PR adds for BareMetalCluster, BareMetalMachine and BareMetalMachineTemplate:
It also upgrade the CRD version to v1, and controller-gen to v0.2.5