Merge pull request #15 from krzko/feat-add-tempo-bridge

feat: apply o11y network to tempo
krzko · May 28, 2023 · 87cef38 · 87cef38
2 parents bcf7b7f + 9979593
commit 87cef38
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/internal/cli/start.go b/internal/cli/start.go
@@ -138,9 +138,9 @@ func addExternalNetwork(filePath string) error {
 			}
 			service["networks"] = []string{"default"}
 			name, _ := nAny.(string)
-			if slices.Contains([]string{"otel-collector"}, name) {
-				// inject o11y network only to otel-collector service.
-				// other services like mini-o11y-stack, grafana, etc. should not be exposed.
+			// inject o11y network only to otel-collector and tempo service.
+			// other services like mini-o11y-stack, grafana, etc. should not be exposed.
+			if slices.Contains([]string{"otel-collector"}, name) || slices.Contains([]string{"tempo"}, name) {
 				service["networks"] = []string{"o11y", "default"}
 			}
 		}