diff --git a/artifacts/workloads/canary-deployment.yaml b/artifacts/workloads/canary-deployment.yaml index 75ee7cbc..814dd9c2 100644 --- a/artifacts/workloads/canary-deployment.yaml +++ b/artifacts/workloads/canary-deployment.yaml @@ -23,7 +23,7 @@ spec: spec: containers: - name: podinfod - image: quay.io/stefanprodan/podinfo:1.2.0 + image: quay.io/stefanprodan/podinfo:1.4.0 imagePullPolicy: IfNotPresent ports: - containerPort: 9898 @@ -67,9 +67,3 @@ spec: requests: cpu: 100m memory: 16Mi - volumeMounts: - - mountPath: /data - name: data - volumes: - - emptyDir: {} - name: data diff --git a/artifacts/workloads/canary-service.yaml b/artifacts/workloads/canary-service.yaml index e858e3a5..7a09e206 100644 --- a/artifacts/workloads/canary-service.yaml +++ b/artifacts/workloads/canary-service.yaml @@ -1,10 +1,8 @@ apiVersion: v1 kind: Service metadata: - name: podinfo + name: podinfo-canary namespace: test - labels: - app: podinfo spec: type: ClusterIP selector: diff --git a/artifacts/workloads/primary-deployment.yaml b/artifacts/workloads/primary-deployment.yaml index f27fa9b7..cc17b82f 100644 --- a/artifacts/workloads/primary-deployment.yaml +++ b/artifacts/workloads/primary-deployment.yaml @@ -23,7 +23,7 @@ spec: spec: containers: - name: podinfod - image: quay.io/stefanprodan/podinfo:1.1.1 + image: quay.io/stefanprodan/podinfo:1.4.1 imagePullPolicy: IfNotPresent ports: - containerPort: 9898 diff --git a/artifacts/workloads/service.yaml b/artifacts/workloads/service.yaml new file mode 100644 index 00000000..3e98f1e0 --- /dev/null +++ b/artifacts/workloads/service.yaml @@ -0,0 +1,14 @@ +apiVersion: v1 +kind: Service +metadata: + name: podinfo + namespace: test +spec: + type: ClusterIP + selector: + app: podinfo-primary + ports: + - name: http + port: 9898 + protocol: TCP + targetPort: http diff --git a/pkg/router/kubernetes.go b/pkg/router/kubernetes.go index 5e86e20f..05681458 100644 --- a/pkg/router/kubernetes.go +++ b/pkg/router/kubernetes.go @@ -20,7 +20,7 @@ type KubernetesRouter struct { logger *zap.SugaredLogger } -// Sync creates to updates the primary and canary services +// Sync creates or updates the primary and canary services func (c *KubernetesRouter) Sync(cd *flaggerv1.Canary) error { targetName := cd.Spec.TargetRef.Name primaryName := fmt.Sprintf("%s-primary", targetName) @@ -40,7 +40,7 @@ func (c *KubernetesRouter) Sync(cd *flaggerv1.Canary) error { }, Spec: corev1.ServiceSpec{ Type: corev1.ServiceTypeClusterIP, - Selector: map[string]string{"app": targetName}, + Selector: map[string]string{"app": primaryName}, Ports: []corev1.ServicePort{ { Name: "http",