-
Notifications
You must be signed in to change notification settings - Fork 37
Domain resolution assumes ROOT
as parent domain
#321
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
Comments
The Kubernetes project currently lacks enough contributors to adequately respond to all issues. This bot triages un-triaged issues according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
Is this still an issue @vignesh-goutham ? |
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues. This bot triages un-triaged issues according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle rotten |
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs. This bot triages issues according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /close not-planned |
@k8s-triage-robot: Closing this issue, marking it as "Not Planned". In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/kind bug
What steps did you take and what happened:
Domain resolution is done at multiple places in each reconcile loop to resolve the domain parameters with the any of the input domain identifying details. These domain identifying details can be domain path, domain id, or domain name.
When domain name is used to resolve, there is an assumption in the controller code that assumes the parent domain is
ROOT
and hence the path of the domain starts withROOT/
. This assumption is not true and must be fixed. Further, CAPC controllers should be verified that they do not pass in domain name as an alternative for domain path while resolving domains.Code with assumption is here
The text was updated successfully, but these errors were encountered: