-
Notifications
You must be signed in to change notification settings - Fork 40k
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
servicecontroller needs to create namespaces in target clusters #27565
Comments
Possibly related to this, the federation-apiserver doesn't recognize the namespace type at all, meaning you can't delete namespaces to clean up after a test. |
Yes this is not good, but we dont have namespaces in federation control plane yet. Users will have to ensure that namespaces exist in underlying clusters. Will fix this in 1.4 cc @kubernetes/sig-cluster-federation |
Even without namespaces in the federation control plane yet, we could in theory save users the trouble by creating the namespace in the underlying cluster if it does not exist yet, before trying to create services in that namespace. But that would make it inconsistent with the Kubernetes API (which does not automatically create missing namespaces). So what I suggest is:
|
I think that this is pretty much all done elsewhere (federated namespaces, and events). Assigning to @nikhiljindal to close as appropriate. |
Closing. |
See #27564 for the error message that lead me to find this.
If you create a service with a namespace that doesn't exist in any target cluster, (and set --v=5) the servicecontroller logs the below:
This doesn't come up in the e2e test because the test framework automatically creates the namespace with a matching name in the target cluster. It also often works fine in manual tests as long as you use the default namespace.
cc @quinton-hoole
The text was updated successfully, but these errors were encountered: