Skip to content

Fix container integration test tags - #2270

Open
Paamicky wants to merge 1 commit into
mainfrom
mcommey/fix-containerimage-sha-tags
Open

Fix container integration test tags#2270
Paamicky wants to merge 1 commit into
mainfrom
mcommey/fix-containerimage-sha-tags

Conversation

@Paamicky

@Paamicky Paamicky commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Description of the issue

Container integration tests (ECS OTLP, EKS Container Insights) failed with 0 metrics. The per-commit image cwagent-integration-test: could resolve to a stale agent build. The : manifest was built from mutable, non-SHA arch tags (:linux-amd64/:linux-arm64) overwritten every build, so a commit's manifest could capture another commit's arch image. The stale agent couldn't translate the opentelemetry config, crash-looped on startup, and exported nothing.

Description of changes

  • :linux-amd64 to :linux-amd64-${{ github.sha }}
  • :linux-arm64 to :linux-arm64-${{ github.sha }}
  • manifest create --amend now points at the SHA-scoped arch tags

Container image path only; host/Windows/mac and the build cache are untouched.

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

Built artifacts for the branch (build-test-artifacts with run-tests=false), verified in ECR that the sha resolves to the correct image (arch tags linux-amd64-<sha>/linux-arm64-<sha> present; OTLP integration testj passes

Requirements

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

@Paamicky
Paamicky marked this pull request as ready for review September 3, 2026 18:03
@Paamicky
Paamicky requested a review from a team as a code owner September 3, 2026 18:03
@Paamicky Paamicky added the ready for testing Indicates this PR is ready for integration tests to run label Sep 3, 2026
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.

1 participant