-
Notifications
You must be signed in to change notification settings - Fork 99
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
Automatically set dns service address #468
Comments
The problem with inferenced defaults is on If you specify a different Service CIDR with no DNS Service IPs, the defaults will be placed. To avoid these kinds of misconfiguration, I would propose a validation to check if those IPs are in the CIDR block, by reporting an error so the user can fix it. We have safe defaults, I think it's a good trade-off for some customizations on the user side upon values changes: I'm open for discussion. |
Hey @prometherion the DNS server address are always 10th address in servicesCIDR, I think it would be better to calculate dynamic default address and use it if user have not specified another |
Another problem I see, is that user might want to specify custom DNSes for his cluster |
I get your point, magic autofill is always good, the problem is the OAPIv3 default values.
Supporting different DNS rather than |
Ah got it. Then it probably should be done by the controller |
I've merged the referenced PR but not marking this as close, since it would require a change in the specification impacting the user experience of Kamaji. I want to keep this issue open not in the form of a stale issue, but rather to gather interest from the community if it's something we need to address: I don't have a strong opinion on convention over configuration rather than configuration over convention, I'd like to approach to these cum grano salis. Happy to change my idea and get the implementation the community is expecting: magic defaults are nice, but too much magic could bring issues to the software maintainability. |
I also had a similar issue. DNS resolution didn’t work because of the misconfiguration on some of tenant clusters. |
As referenced here, we need to change the I can take care of that but it would require a bit of time, of course, any contribution here would be really appreciated as usual! |
Kamaji specifies coredns address
10.96.0.10
by default, I think ifserviceCidr
specified this address should be automatically calculated from the serviceCidr + 10ref and workaround aenix-io/cozystack#147
The text was updated successfully, but these errors were encountered: