-
Notifications
You must be signed in to change notification settings - Fork 79
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
Addons #7
Addons #7
Conversation
// +kubebuilder:default={enabled: true} | ||
CoreDNS AddonSpec `json:"coreDNS,omitempty"` | ||
// +kubebuilder:default={enabled: true} | ||
KubeProxy AddonSpec `json:"kubeProxy,omitempty"` |
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.
Are these defaults required although specified in TenantControlPlaneSpec.Addons
struct key?
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.
It is a still WIP and I am figuring out how to deal with the default values.
3d1c314
to
3512f27
Compare
0d9fdf3
to
5108b0c
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.
Could you rebase over master
branch? I see some duplicated commits.
5108b0c
to
d0ad534
Compare
d0ad534
to
7abf13b
Compare
This PR will try to customize how to allow addons (Kubeadm addons) for a provisioned tenant control plane.