Add sidecar.istio.io/inject=false annotation to all-in-one, agent (da… (

jaegertracing#67) * Add sidecar.istio.io/inject=false annotation to all-in-one, agent (daemonset), query and collector Signed-off-by: Gary Brown <gary@brownuk.com>
andream16 · Oct 17, 2018 · c676e73 · c676e73
1 parent 07e6146
commit c676e73
Show file tree

Hide file tree

Showing 8 changed files with 15 additions and 1 deletion.
diff --git a/pkg/deployment/agent.go b/pkg/deployment/agent.go
@@ -62,6 +62,7 @@ func (a *Agent) Get() *appsv1.DaemonSet {
 	annotations := map[string]string{
 		prometheusScrapeKey: prometheusScrapeValue,
 		prometheusPortKey:   "5778",
+		"sidecar.istio.io/inject": "false",
 	}
 
 	return &appsv1.DaemonSet{

diff --git a/pkg/deployment/agent_test.go b/pkg/deployment/agent_test.go
@@ -56,5 +56,9 @@ func TestGetDaemonSetDeployment(t *testing.T) {
 	jaeger := v1alpha1.NewJaeger("TestGetDaemonSetDeployment")
 	jaeger.Spec.Agent.Strategy = daemonSetStrategy
 	agent := NewAgent(jaeger)
-	assert.NotNil(t, agent.Get())
+
+	ds := agent.Get()
+	assert.NotNil(t, ds)
+
+	assert.Equal(t, "false", ds.Spec.Template.Annotations["sidecar.istio.io/inject"])
 }
diff --git a/pkg/deployment/all-in-one.go b/pkg/deployment/all-in-one.go
@@ -40,6 +40,7 @@ func (a *AllInOne) Get() *appsv1.Deployment {
 	annotations := map[string]string{
 		prometheusScrapeKey: prometheusScrapeValue,
 		prometheusPortKey:   "16686",
+		"sidecar.istio.io/inject": "false",
 	}
 
 	return &appsv1.Deployment{

diff --git a/pkg/deployment/all-in-one_test.go b/pkg/deployment/all-in-one_test.go
@@ -36,6 +36,8 @@ func TestDefaultAllInOneImage(t *testing.T) {
 		},
 	}
 	assert.Equal(t, envvars, d.Spec.Template.Spec.Containers[0].Env)
+
+	assert.Equal(t, "false", d.Spec.Template.ObjectMeta.Annotations["sidecar.istio.io/inject"])
 }
 
 func TestAllInOneHasOwner(t *testing.T) {

diff --git a/pkg/deployment/collector.go b/pkg/deployment/collector.go
@@ -42,6 +42,7 @@ func (c *Collector) Get() *appsv1.Deployment {
 	annotations := map[string]string{
 		prometheusScrapeKey: prometheusScrapeValue,
 		prometheusPortKey:   "14268",
+		"sidecar.istio.io/inject": "false",
 	}
 
 	return &appsv1.Deployment{

diff --git a/pkg/deployment/collector_test.go b/pkg/deployment/collector_test.go
@@ -68,4 +68,6 @@ func TestDefaultCollectorImage(t *testing.T) {
 		},
 	}
 	assert.Equal(t, envvars, containers[0].Env)
+
+	assert.Equal(t, "false", dep.Spec.Template.ObjectMeta.Annotations["sidecar.istio.io/inject"])
 }
diff --git a/pkg/deployment/query.go b/pkg/deployment/query.go
@@ -45,6 +45,7 @@ func (q *Query) Get() *appsv1.Deployment {
 	annotations := map[string]string{
 		prometheusScrapeKey: prometheusScrapeValue,
 		prometheusPortKey:   "16686",
+		"sidecar.istio.io/inject": "false",
 
 		// note that we are explicitly using a string here, not the value from `inject.Annotation`
 		// this has two reasons:

diff --git a/pkg/deployment/query_test.go b/pkg/deployment/query_test.go
@@ -44,6 +44,8 @@ func TestDefaultQueryImage(t *testing.T) {
 
 	assert.Len(t, containers, 1)
 	assert.Equal(t, "org/custom-query-image:123", containers[0].Image)
+
+	assert.Equal(t, "false", dep.Spec.Template.ObjectMeta.Annotations["sidecar.istio.io/inject"])
 }
 
 func TestQueryPodName(t *testing.T) {