Empty the "ClusterIPs" along with "ClusterIP" when "ClusterIP" isn't …

…"None" More details please refer to vmware-tanzu#4098 Fixes vmware-tanzu#4098 Signed-off-by: Wenkai Yin(尹文开) <yinw@vmware.com>
ywk253100 · Sep 2, 2021 · 972a774 · 972a774
1 parent edeec84
commit 972a774
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/pkg/restore/service_action.go b/pkg/restore/service_action.go
@@ -55,6 +55,7 @@ func (a *ServiceAction) Execute(input *velero.RestoreItemActionExecuteInput) (*v
 
 	if service.Spec.ClusterIP != "None" {
 		service.Spec.ClusterIP = ""
+		service.Spec.ClusterIPs = nil
 	}
 
 	/* Do not delete NodePorts if restore triggered with "--preserve-nodeports" flag */

diff --git a/pkg/restore/service_action_test.go b/pkg/restore/service_action_test.go
@@ -73,13 +73,14 @@ func TestServiceActionExecute(t *testing.T) {
 		expectedRes corev1api.Service
 	}{
 		{
-			name: "clusterIP (only) should be deleted from spec",
+			name: "clusterIP/clusterIPs should be deleted from spec",
 			obj: corev1api.Service{
 				ObjectMeta: metav1.ObjectMeta{
 					Name: "svc-1",
 				},
 				Spec: corev1api.ServiceSpec{
 					ClusterIP:      "should-be-removed",
+					ClusterIPs:     []string{"should-be-removed"},
 					LoadBalancerIP: "should-be-kept",
 				},
 			},