[DSD-10586] mosipdev to mosipqa - #1777
Conversation
Signed-off-by: Swapnil <swapnil.mohanty@technoforte.co.in>
WalkthroughThe change replaces DSL orchestrator and packet creator image references with PMP UI, partner management service, and policy management service images. All new references use release and version ChangesViDiVi image mappings
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@release/vidivi/images.txt`:
- Around line 1-3: Update all three image mappings in the manifest from the
mosipdev registry namespace to mosipqa, preserving each repository name and tag
unchanged.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro Plus
Run ID: f2171c15-8af1-4a06-9767-3ccaa2e788de
📒 Files selected for processing (1)
release/vidivi/images.txt
| mosipdev/pmp-ui-v2:release-1.3.x 1.3.x | ||
| mosipdev/partner-management-service:release-1.3.x 1.3.x | ||
| mosipdev/policy-management-service:release-1.3.x 1.3.x |
There was a problem hiding this comment.
🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win
Complete the mosipdev to mosipqa registry migration.
The PR objective states mosipdev to mosipqa, but Lines 1-3 still use mosipdev. The published diff confirms that all three mappings retain this namespace. (github.com)
If this manifest drives the QA deployment, it will continue selecting development images. Change all three references to the approved mosipqa namespace.
Proposed fix
-mosipdev/pmp-ui-v2:release-1.3.x 1.3.x
-mosipdev/partner-management-service:release-1.3.x 1.3.x
-mosipdev/policy-management-service:release-1.3.x 1.3.x
+mosipqa/pmp-ui-v2:release-1.3.x 1.3.x
+mosipqa/partner-management-service:release-1.3.x 1.3.x
+mosipqa/policy-management-service:release-1.3.x 1.3.x📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| mosipdev/pmp-ui-v2:release-1.3.x 1.3.x | |
| mosipdev/partner-management-service:release-1.3.x 1.3.x | |
| mosipdev/policy-management-service:release-1.3.x 1.3.x | |
| mosipqa/pmp-ui-v2:release-1.3.x 1.3.x | |
| mosipqa/partner-management-service:release-1.3.x 1.3.x | |
| mosipqa/policy-management-service:release-1.3.x 1.3.x |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@release/vidivi/images.txt` around lines 1 - 3, Update all three image
mappings in the manifest from the mosipdev registry namespace to mosipqa,
preserving each repository name and tag unchanged.
Summary by CodeRabbit