Skip to content

Add AKS Container Insights Test - #2284

Merged
Paamicky merged 11 commits into
mainfrom
AKS-ContainerInsights
Sep 25, 2026
Merged

Paamicky merged 11 commits into
mainfrom
AKS-ContainerInsights

Conversation

@Paamicky

@Paamicky Paamicky commented Sep 14, 2026 •

Copy link
Copy Markdown
Contributor

Description of the issue

The agent translates OTEL Container Insights configs at runtime, including the AKS/GKE path, but CI had no job that runs Container Insights on a real AKS cluster. Only the OTLP AKS suite existed, we needed to add a CI test to make sure the agent is able to translate and deliver metrics and logs to CloudWatch in AKS mode.

Description of changes

Adds a CI job to trigger the new AKS Container Insights integration test:

  • New AKSContainerInsightsIntegrationTest job in test-artifacts.yml, mirroring the existing AKS OTLP job, passing test_mode=containerinsights and test_dir=./test/azure/aks/containerinsights.
  • Tightened the existing AKSIntegrationTest filter from a substring match to an exact match on ./test/azure/aks, so targeting the CI subdirectory no longer also triggers the OTLP job.

Tests

  • End-to-end test for AKS Container Insights tests: AKS-container-insights
  • AKS metics verified in CloudWatch:
    • Node metrics: cadvisor, kubeletstats, node-exporter
    • Cluster metrics: kube-state-metrics, apiserver, control plane metrics (apiserver)
    • Node logs: application and host container logs
    • Solution metrics: Keda and Karpenter
Screenshot 2026-09-16 at 11 58 06 Screenshot 2026-09-16 at 12 01 01

Warning

These PR's needs to merge first for this test to pass:

Requirements

  1. Run make fmt and make fmt-sh
  2. Run make lint

License

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

Integration Tests

To run integration tests against this PR, add the ready for testing label.

@Paamicky
Paamicky force-pushed the AKS-ContainerInsights branch from 5fa3432 to 63331fc Compare September 16, 2026 02:15
@Paamicky
Paamicky marked this pull request as ready for review September 16, 2026 16:27
@Paamicky
Paamicky requested a review from a team as a code owner September 16, 2026 16:27
@Paamicky Paamicky added the ready for testing Indicates this PR is ready for integration tests to run label Sep 16, 2026
@olowosulu

Copy link
Copy Markdown
Contributor

LGTM once the merge order is settled. One condition:

@Paamicky

Copy link
Copy Markdown
Contributor Author

LGTM once the merge order is settled. One condition:

Intent is right and it's handled by merge order.

lorespiz
lorespiz previously approved these changes Sep 22, 2026
@Paamicky
Paamicky merged commit 7b574ef into main Sep 25, 2026
980 of 993 checks passed
@Paamicky
Paamicky deleted the AKS-ContainerInsights branch September 25, 2026 02:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready for testing Indicates this PR is ready for integration tests to run

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants