Use DNS-based values for address rather than cluster IP #2145
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What this PR does / why we need it:
myservice.mynamespace.svc.cluster.local
), the UI will receive a resolvable address from the operator and will be able to retrieve healthcheck information.Which issue(s) this PR fixes: Supports #2144. Not sure if it fixes it and it definitely warrants further discussion.
Please reference the issue this PR will close: #2144
Special notes for your reviewer: I'm new to this and want to make certain my theoretical understanding is correct first. I'm happy to build and deploy this locally in my environment though may need a small bit of guidance on how to get the container out there etc. (I have a container registry I can deploy to as necessary).
Does this PR introduce a user-facing change?: If for some reason users were relying on the operator to specifically use ClusterIP -- though that use case seems unlikely to me -- they would see this as a change, though likely not a breaking change, given the way I understand Kubernetes DNS to work.
Please make sure you've completed the relevant tasks for this PR, out of the following list: