Skip to content

[master] Update registry reference#494

Open
Miranlfk wants to merge 2 commits intowso2:masterfrom
Miranlfk:master-reg-up
Open

[master] Update registry reference#494
Miranlfk wants to merge 2 commits intowso2:masterfrom
Miranlfk:master-reg-up

Conversation

@Miranlfk
Copy link
Copy Markdown
Member

@Miranlfk Miranlfk commented May 7, 2026

Purpose

This pull request updates the WSO2 Identity Server Docker Compose setup to use the new Docker image registry domain. The main change is replacing references to the old docker.wso2.com domain with the new registry.wso2.com domain in both documentation and Dockerfile.

Docker image registry update:

  • Updated the base image in docker-compose/is/dockerfiles/is/Dockerfile to use registry.wso2.com/wso2is:7.2.0.0 instead of docker.wso2.com/wso2is:7.2.0.0.
  • Updated instructions in docker-compose/is/README.md to direct users to remove the registry.wso2.com/ prefix (instead of the old docker.wso2.com/ prefix) when using locally built images.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 7, 2026

Review Change Stack

📝 Walkthrough

Walkthrough

This pull request updates Docker registry references across the WSO2 Identity Server Docker configuration. The Dockerfile base image is changed from docker.wso2.com/wso2is:7.2.0.0 to registry.wso2.com/wso2is:7.2.0.0. The corresponding README documentation is updated to reflect the removal of the registry.wso2.com/ prefix instruction when using locally built Docker images instead of the previous docker.wso2.com/ reference. These changes ensure consistency between the build configuration and user-facing documentation.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The pull request description addresses the Purpose section well, providing clear context about the registry domain update. However, most other required sections from the template are missing or incomplete. Add missing sections: Goals, Approach, User stories, Release note, Documentation, Training, Certification, Marketing, Automation tests, Security checks, Samples, Related PRs, Migrations, Test environment, and Learning. At minimum, provide N/A with explanations for sections that do not apply.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: updating Docker image registry references from docker.wso2.com to registry.wso2.com.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

1 participant