Merge pull request #745 from Nerja/delegate

Fix for VirtualService delegation when analysis is enabled

pkg/router/istio.go

@@ -215,6 +215,12 @@ func (ir *IstioRouter) reconcileVirtualService(canary *flaggerv1.Canary) error {
 		return fmt.Errorf("VirtualService %s.%s get query error %v", apexName, canary.Namespace, err)
 	}
 
+	if canary.Spec.Service.Delegation {
+		// delegate VirtualService requires the hosts and gateway empty.
+		virtualService.Spec.Gateways = []string{}
+		virtualService.Spec.Hosts = []string{}
+	}
+
 	// update service but keep the original destination weights and mirror
 	if virtualService != nil {
 		if diff := cmp.Diff(

test/e2e-istio-tests-delegate.sh

@@ -70,7 +70,6 @@ spec:
     targetPort: 9898
     portDiscovery: true
     delegation: true
-  skipAnalysis: true
   analysis:
     interval: 15s
     threshold: 15