Closed fabriziopandini closed 5 years ago
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: fabriziopandini
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/lgtm
What this PR does / why we need it: A user now has several places to define some settings. One the cluster/machine object and one on the kubeadm config object.
Which issue(s) this PR fixes: Fixes # https://github.com/kubernetes-sigs/cluster-api-bootstrap-provider-kubeadm/issues/78
Special notes for your reviewer: This PR handle conflicts in the same ways CABPK is already handling similar situations for bootstrap tokens, certificates, controlplaneEnpoint:
Release note:
/assign @detiber @chuckha