Skip to content

Add OTEL k8s taints integration tests#706

Open
louisall wants to merge 1 commit into
mainfrom
feat/otel-taints-integration-tests
Open

Add OTEL k8s taints integration tests#706
louisall wants to merge 1 commit into
mainfrom
feat/otel-taints-integration-tests

Conversation

@louisall

Copy link
Copy Markdown
Contributor

Summary

Adds integration tests for the k8staintsprocessor which decorates node telemetry with k8s.node.taint.<key> resource attributes.

Tests added

  • TestTaintsAppearOnDaemonSetMetrics — verifies taint attributes on DaemonSet metrics (local_mode=true)
  • TestTaintsEmptyValueSkipped — verifies empty-value taints are NOT emitted (Granite rejects blank attribute values)
  • TestTaintsAbsentOnUntaintedNodes — verifies untainted nodes have no k8s.node.taint.* attributes
  • TestTaintsAppearOnClusterScraperMetrics — verifies cluster-scraper (local_mode=false) sees taints
  • TestTaintsMultipleOnSameNode — verifies multiple taints appear together on the same node

Terraform changes

Adds null_resource.apply_test_taints that applies two test taints (ci-test.example.com/dedicated=gpu and ci-test.example.com/team=ml) to the first node in the standard node group before tests run.

Dependencies

This PR will not pass CI until the following upstream PRs are merged and deployed:

  1. amazon-contributing/opentelemetry-collector-contrib #509k8staintsprocessor implementation
  2. Agent PR (not yet opened) — registers k8staintsprocessor in components.go + bumps go.mod SHA (blocked on Integration test for Prometheus EMF Pipeline #509 merging)
  3. aws-observability/helm-charts #300 — wires k8staints processor into all Container Insights pipelines

Validated

All 5 tests pass against a live EKS cluster with a hand-built agent containing the processor.

@louisall louisall requested a review from a team as a code owner May 27, 2026 10:27
@louisall louisall force-pushed the feat/otel-taints-integration-tests branch from 0d91b2d to 2cb3960 Compare June 8, 2026 11:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant