Use Update() instead of Patch() to update svc

Fixes hashicorp#23

kubernetes/resource_kubernetes_service.go

@@ -10,7 +10,6 @@ import (
 	api "k8s.io/api/core/v1"
 	"k8s.io/apimachinery/pkg/api/errors"
 	meta_v1 "k8s.io/apimachinery/pkg/apis/meta/v1"
-	pkgApi "k8s.io/apimachinery/pkg/types"
 )
 
 func resourceKubernetesService() *schema.Resource {
@@ -228,35 +227,30 @@ func resourceKubernetesServiceRead(d *schema.ResourceData, meta interface{}) err
 func resourceKubernetesServiceUpdate(d *schema.ResourceData, meta interface{}) error {
 	conn := meta.(*kubernetesProvider).conn
 
-	namespace, name, err := idParts(d.Id())
+	namespace, _, err := idParts(d.Id())
 	if err != nil {
 		return err
 	}
 
-	ops := patchMetadata("metadata.0.", "/metadata/", d)
-	if d.HasChange("spec") {
-		serverVersion, err := conn.ServerVersion()
-		if err != nil {
-			return err
-		}
-		diffOps, err := patchServiceSpec("spec.0.", "/spec/", d, serverVersion)
-		if err != nil {
-			return err
-		}
-		ops = append(ops, diffOps...)
+	metadata := expandMetadata(d.Get("metadata").([]interface{}))
+	spec := expandServiceSpec(d.Get("spec").([]interface{}))
+
+	if metadata.Namespace == "" {
+		metadata.Namespace = "default"
 	}
-	data, err := ops.MarshalJSON()
-	if err != nil {
-		return fmt.Errorf("Failed to marshal update operations: %s", err)
+
+	service := &api.Service{
+		ObjectMeta: metadata,
+		Spec:       spec,
 	}
-	log.Printf("[INFO] Updating service %q: %v", name, string(data))
-	out, err := conn.CoreV1().Services(namespace).Patch(name, pkgApi.JSONPatchType, data)
+
+	out, err := conn.CoreV1().Services(namespace).Update(service)
 	if err != nil {
 		return fmt.Errorf("Failed to update service: %s", err)
 	}
 	log.Printf("[INFO] Submitted updated service: %#v", out)
-	d.SetId(buildId(out.ObjectMeta))
 
+	d.SetId(buildId(out.ObjectMeta))
 	return resourceKubernetesServiceRead(d, meta)
 }
 

kubernetes/resource_kubernetes_service_test.go

@@ -541,6 +541,7 @@ resource "kubernetes_service" "test" {
 			target_port = 81
 		}
 		type = "LoadBalancer"
+		load_balancer_ip = "100.24.12.6" // New line added
 	}
 }`, name, name)
 }

kubernetes/structures.go

@@ -47,6 +47,9 @@ func expandMetadata(in []interface{}) metav1.ObjectMeta {
 	if v, ok := m["namespace"]; ok {
 		meta.Namespace = v.(string)
 	}
+	if v, ok := m["resource_version"]; ok {
+		meta.ResourceVersion = v.(string)
+	}
 
 	return meta
 }