ci: fix container integration test failures#3499
Merged
tippmar-nr merged 4 commits intomainfrom Mar 23, 2026
Merged
Conversation
Add a health check to the LocalStack service in docker-compose-awssdk.yml so the test app waits for LocalStack to be ready before starting. This fixes intermittent AWS SDK container test failures (SQS, Kinesis, Firehose) caused by the test app making requests before LocalStack finished initializing. Also improve Docker cleanup in ContainerApplication.cs: - Skip manual container/network removal when compose down succeeds - Remove duplicate network cleanup in PrepareForStart - Skip network inspect during pre-start diagnostics to reduce log noise Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Add [Collection("AwsSdkTests")] to all AWS SDK container test classes
so they run sequentially instead of in parallel. Each test spins up its
own LocalStack + DynamoDB + test app via docker compose, and running 6+
LocalStack containers simultaneously on CI runners causes resource
exhaustion (LocalStack exits with code 55).
Other container tests (Kafka, Memcached, smoke tests) are unaffected
and continue to run in parallel.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The localstack/localstack:stable tag was updated on 2026-03-23 to a new CalVer release (2026.03.0) that crashes on startup with exit code 55. The LocalStack GitHub repo was archived the same day, suggesting a major licensing/model change. Pin to the last known working version (4.14.0) to restore CI stability. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Same resource contention fix as AWS SDK tests — add [Collection] to Memcached and Kafka test classes so each pair of DotNet8/DotNet10 tests runs sequentially. Prevents two instances of the same dependency service (memcached-server, kafka-broker) from competing for resources on CI. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3499 +/- ##
==========================================
+ Coverage 81.79% 81.80% +0.01%
==========================================
Files 508 508
Lines 34220 34220
Branches 4040 4040
==========================================
+ Hits 27990 27994 +4
+ Misses 5265 5262 -3
+ Partials 965 964 -1
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
nrcventura
approved these changes
Mar 23, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
stabletag was updated on 2026-03-23 to a broken release (the LocalStack open source repo was archived 3/23, new image immediately throws exit code 55 on startup)[Collection]to prevent resource contention on CI runnersContainerApplication.csTest plan
🤖 Generated with Claude Code