tighter documentation and drop a log statement (#899)
* fix * fix documentation
This commit is contained in:
@@ -680,7 +680,8 @@ Remember that the same can be done for config maps. If such a namespace is not s
|
||||
3. from a String residing in `/var/run/secrets/kubernetes.io/serviceaccount/namespace` file
|
||||
(kubernetes default namespace path)
|
||||
4. from a designated client method call (for example fabric8's : `KubernetesClient::getNamespace`), if the client provides
|
||||
such a method.
|
||||
such a method. This, in turn, could be configured via environment properties. For example fabric8 client can be configured via
|
||||
"KUBERNETES_NAMESPACE" property; consult the client documentation for exact details.
|
||||
|
||||
Failure to find a namespace from the above steps will result in an Exception being raised.
|
||||
|
||||
|
||||
@@ -98,10 +98,6 @@ public class Fabric8AutoConfiguration {
|
||||
.withProxyPassword(or(kubernetesClientProperties.getProxyPassword(), base.getProxyPassword()))
|
||||
.withNoProxy(or(kubernetesClientProperties.getNoProxy(), base.getNoProxy())).build();
|
||||
|
||||
if (properties.getNamespace() == null || properties.getNamespace().isEmpty()) {
|
||||
LOG.warn("No namespace has been detected. Please specify "
|
||||
+ "KUBERNETES_NAMESPACE env var, or use a later kubernetes version (1.3 or later)");
|
||||
}
|
||||
return properties;
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user