Skip to content

Use public ECR for Consul Testcontainers image#9933

Merged
ReubenBond merged 1 commit into
dotnet:mainfrom
ReubenBond:fix/consul-testcontainers-public-ecr
Feb 17, 2026
Merged

Use public ECR for Consul Testcontainers image#9933
ReubenBond merged 1 commit into
dotnet:mainfrom
ReubenBond:fix/consul-testcontainers-public-ecr

Conversation

@ReubenBond

@ReubenBond ReubenBond commented Feb 17, 2026

Copy link
Copy Markdown
Member

Summary

  • switch Consul test container image source from Docker Hub to public ECR
  • keep Consul version pinned at 1.19
  • avoid Azure DevOps pull failures when registry-1.docker.io is unreachable

Testing

  • dotnet test test\Extensions\Consul.Tests\Consul.Tests.csproj --filter "FullyQualifiedName~ConsulClusteringOptionsTests" -- -parallel none -noshadow
Microsoft Reviewers: Open in CodeFlow

Switch Consul test container pull source from Docker Hub to public ECR to avoid CI failures when registry-1.docker.io is inaccessible.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings February 17, 2026 00:39
@ReubenBond ReubenBond enabled auto-merge February 17, 2026 00:40

Copilot AI 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.

Pull request overview

This PR migrates the Consul test container image source from Docker Hub to AWS public ECR to improve reliability in Azure DevOps CI environments where Docker Hub may be unreachable. The change maintains the existing Consul version 1.19 while switching only the image registry.

Changes:

  • Updated Consul testcontainer image from hashicorp/consul:1.19 to public.ecr.aws/hashicorp/consul:1.19

@ReubenBond ReubenBond added this pull request to the merge queue Feb 17, 2026
@ReubenBond ReubenBond removed this pull request from the merge queue due to a manual request Feb 17, 2026
@ReubenBond ReubenBond merged commit ad770a7 into dotnet:main Feb 17, 2026
65 checks passed
@ReubenBond ReubenBond deleted the fix/consul-testcontainers-public-ecr branch February 17, 2026 01:23
rkargMsft pushed a commit to rkargMsft/orleans that referenced this pull request Feb 27, 2026
Use public ECR for Consul testcontainer image

Switch Consul test container pull source from Docker Hub to public ECR to avoid CI failures when registry-1.docker.io is inaccessible.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions github-actions Bot locked and limited conversation to collaborators Mar 19, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants