Skip to content

Add OTEL Container Insights tests - #744

Merged
Paamicky merged 11 commits into
mainfrom
mcommey/containerinsights-test
Aug 25, 2026
Merged

Add OTEL Container Insights tests#744
Paamicky merged 11 commits into
mainfrom
mcommey/containerinsights-test

Conversation

@Paamicky

@Paamicky Paamicky commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Description of the issue

Container Insights (OTEL-native) had no e2e integration test. Need coverage that
node metrics, cluster metrics (incl. KEDA/Karpenter solutions), and CI logs reach
CloudWatch.

Description of changes

Adds test/e2e/containerinsights/ using the operator/helm/addon e2e framework:

  • TestContainerInsights with Node/Cluster/KEDA/Karpenter metric subtests + CI logs.
  • Per-run isolation: unique test.run.id stamped via opentelemetry.resource_attributes; all queries filter on @resource.test.run.id.
  • Validation via the shared otlpvalidation helper; logs via the CloudWatch Logs API.
  • resources/keda_karpenter.yaml: stub emitters so the solutions pipelines have scrape targets (applied in TestMain, deleted on -destroy).
  • Registers eks_e2e_containerinsights in the generator.

Note

EKS Addon test will be added as a follow up once Addon is released with changes

License

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Tests

@Paamicky
Paamicky requested a review from a team as a code owner August 17, 2026 19:51
@Paamicky
Paamicky force-pushed the mcommey/containerinsights-test branch from fb413ab to e7a79dd Compare August 17, 2026 19:55
@Paamicky Paamicky changed the title Mcommey/containerinsights test mcommey/containerinsights test Aug 17, 2026
@Paamicky Paamicky changed the title mcommey/containerinsights test Add OTEL Container Insights tests Aug 17, 2026
@Paamicky
Paamicky force-pushed the mcommey/containerinsights-test branch 2 times, most recently from 34c82f2 to d22b0f3 Compare August 18, 2026 19:15
@Paamicky
Paamicky force-pushed the mcommey/containerinsights-test branch from d22b0f3 to 71ea9f5 Compare August 18, 2026 19:37
@Paamicky
Paamicky force-pushed the mcommey/containerinsights-test branch from e9c443c to b8871b0 Compare August 20, 2026 14:25
@Paamicky
Paamicky force-pushed the mcommey/containerinsights-test branch from b8871b0 to 5590add Compare August 20, 2026 15:20

@movence movence left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is there a way to leverage already existing tests under https://github.com/aws/amazon-cloudwatch-agent-test/tree/main/test/otel/solutions?

Comment thread generator/test_case_generator.go
Comment thread test/e2e/containerinsights/containerinsights_test.go
@Paamicky
Paamicky merged commit d0c49a7 into main Aug 25, 2026
6 checks passed
@Paamicky
Paamicky deleted the mcommey/containerinsights-test branch August 25, 2026 15:39
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.

2 participants