DiscoveryClient allNamespaces Property Not Used When Querying Services

This relates to #428 .

There was a change made to configure the discovery client to search across all namespaces based on a property. Inside KubernetesDiscoveryClient,getInstances the method call which queries the endpoints was updated to use this new property

this.properties.isAllNamespaces()
				? this.client.endpoints().isAllNamespaces()
						.withField("metadata.name", serviceId).list().getItems()
				: Collections
						.singletonList(this.client.endpoints().withName(serviceId).get());

However, the same change was not applied later in the same method when the services are queried. The current code is:

this.client.services().withName(serviceId).get();

As a result, when trying to use the isAllNamespaces property we are consistently getting null pointer exceptions.

It seems like the call to query the services should also use the isAllNamespaces method when the property is set.

[ryanjbaxter]

@Haybu made the changes, I am sure he will take a look 😃

[ryanjbaxter]

Fixed via d9bad61