add smoke tests for the WithResource config func

Per the OTel spec, resource attributes set via environment variable must supercede resource attributes set in code.[1] [1] https://opentelemetry.io/docs/specs/otel/resource/sdk/#specifying-resource-information-via-an-environment-variable
honeycombio · Jun 2, 2023 · d23bbf7 · d23bbf7
1 parent 6e9cda6
commit d23bbf7
Show file tree

Hide file tree

Showing 4 changed files with 42 additions and 3 deletions.
diff --git a/examples/main.go b/examples/main.go
@@ -5,20 +5,28 @@ import (
 	"log"
 
 	"go.opentelemetry.io/otel"
+	"go.opentelemetry.io/otel/attribute"
+	"go.opentelemetry.io/otel/sdk/resource"
 
 	"github.com/honeycombio/otel-config-go/otelconfig"
 )
 
 func main() {
-	otelShutdown, err := otelconfig.ConfigureOpenTelemetry()
+	otelShutdown, err := otelconfig.ConfigureOpenTelemetry(
+		otelconfig.WithResourceOption(
+			resource.WithAttributes(
+				attribute.String("resource.example_set_in_code", "CODE"),
+				attribute.String("resource.example_clobber", "CODE_WON"),
+			),
+		),
+	)
 
 	if err != nil {
 		log.Fatalf("error setting up OTel SDK - %e", err)
 	}
 
 	defer otelShutdown()
 	tracer := otel.Tracer("my-app")
-	ctx := context.Background()
-	ctx, span := tracer.Start(ctx, "doing-things")
+	_, span := tracer.Start(context.Background(), "doing-things")
 	defer span.End()
 }
diff --git a/smoke-tests/docker-compose.yml b/smoke-tests/docker-compose.yml
@@ -3,6 +3,7 @@ version: '3.0'
 x-env-base: &env_base
   OTEL_EXPORTER_OTLP_ENDPOINT: http://collector:4317
   OTEL_EXPORTER_OTLP_INSECURE: "true"
+  OTEL_RESOURCE_ATTRIBUTES: resource.example_set_in_env=ENV,resource.example_clobber=ENV_WON
   OTEL_SERVICE_NAME: "my-go-app"
   OTEL_METRICS_ENABLED: "false"
   DEBUG: "true"

diff --git a/smoke-tests/smoke-sdk-grpc.bats b/smoke-tests/smoke-sdk-grpc.bats
@@ -27,3 +27,18 @@ teardown_file() {
 	result=$(span_names_for ${TRACER_NAME})
 	assert_equal "$result" '"doing-things"'
 }
+
+@test "Resource attributes can be set via environment variable" {
+	env_result=$(spans_received | jq ".resource.attributes[] | select(.key == \"resource.example_set_in_env\") | .value.stringValue")
+	assert_equal "$env_result" '"ENV"'
+}
+
+@test "Resource attributes can be set in code" {
+	code_result=$(spans_received | jq ".resource.attributes[] | select(.key == \"resource.example_set_in_code\") | .value.stringValue")
+	assert_equal "$code_result" '"CODE"'
+}
+
+@test "Resource attributes set in environment win over matching key set in code" {
+	clobber_result=$(spans_received | jq ".resource.attributes[] | select(.key == \"resource.example_clobber\") | .value.stringValue")
+	assert_equal "$clobber_result" '"ENV_WON"'
+}
diff --git a/smoke-tests/smoke-sdk-http.bats b/smoke-tests/smoke-sdk-http.bats
@@ -27,3 +27,18 @@ teardown_file() {
 	result=$(span_names_for ${TRACER_NAME})
 	assert_equal "$result" '"doing-things"'
 }
+
+@test "Resource attributes can be set via environment variable" {
+	env_result=$(spans_received | jq ".resource.attributes[] | select(.key == \"resource.example_set_in_env\") | .value.stringValue")
+	assert_equal "$env_result" '"ENV"'
+}
+
+@test "Resource attributes can be set in code" {
+	code_result=$(spans_received | jq ".resource.attributes[] | select(.key == \"resource.example_set_in_code\") | .value.stringValue")
+	assert_equal "$code_result" '"CODE"'
+}
+
+@test "Resource attributes set in environment win over matching key set in code" {
+	clobber_result=$(spans_received | jq ".resource.attributes[] | select(.key == \"resource.example_clobber\") | .value.stringValue")
+	assert_equal "$clobber_result" '"ENV_WON"'
+}