Skip to content

Fix: Have single source of truth for ICR urls#6739

Merged
hkantare merged 2 commits intoIBM-Cloud:masterfrom
bainsy88:issue_6737
Apr 9, 2026
Merged

Fix: Have single source of truth for ICR urls#6739
hkantare merged 2 commits intoIBM-Cloud:masterfrom
bainsy88:issue_6737

Conversation

@bainsy88
Copy link
Copy Markdown
Contributor

@bainsy88 bainsy88 commented Apr 9, 2026

Community Note

  • Please vote on this pull request by adding a 👍 reaction to the original pull request comment to help the community and maintainers prioritize this request
  • Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for pull request followers and do not help prioritize the request

Closes #6737

This PR changes the behaviour to constructing the private ICR endpoint based on the public one. This is deterministic and means that it is only maintained in one place.

Output from acceptance testing:

$ make testacc TEST=./ibm/service/registry TESTARGS='-run="TestAccIBMCr.*"'
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./ibm/service/registry -v -run="TestAccIBMCr.*" -timeout 700m 
=== RUN   TestAccIBMCrNamespacesDataSourceBasic
--- PASS: TestAccIBMCrNamespacesDataSourceBasic (37.75s)
=== RUN   TestAccIBMCrNamespaceBasic
--- PASS: TestAccIBMCrNamespaceBasic (46.64s)
=== RUN   TestAccIBMCrNamespaceAllArgs
--- PASS: TestAccIBMCrNamespaceAllArgs (69.03s)
=== RUN   TestAccIBMCrRetentionPolicyAllArgs
--- PASS: TestAccIBMCrRetentionPolicyAllArgs (38.95s)
PASS

@hkantare hkantare merged commit 8787354 into IBM-Cloud:master Apr 9, 2026
1 check passed
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.

ICR endpoint list for Private is sourced from different location to public and is not complete.

2 participants