Cannot change the type of the Service from ClusterIP to ExternalName with PATCH

[jorsol]

The PR #2472 fixed the replace method, but the patch method should also handle the update of the service to ExternalName.

It assumes a patch with ClusterIP:

kubernetes-client/kubernetes-client/src/main/java/io/fabric8/kubernetes/client/dsl/internal/core/v1/ServiceOperationsImpl.java

Lines 66 to 77 in 33a735a

	public Service patch(Service item) {
	try {
	Service old = getMandatory();
	return super.patch(new ServiceBuilder(item)
	.editSpec()
	.withClusterIP(old.getSpec().getClusterIP())
	.endSpec()
	.build());
	} catch (Exception e) {
	throw KubernetesClientException.launderThrowable(forOperationType("patch"), e);
	}
	}

[rohanKanojia]

@jorsol : Thanks a lot for reporting this, Would it be possible for you to submit a fix for this issue?

[jorsol]

Yes, I could submit a fix for this issue.

[jorsol]

Is there an ETA for 5.2.0?

[manusa]

We're planning on releasing by the end of next week or beginning of the other.