Skip to content

[DSD-9196] Image transfer from mosipdev2 to mosipqa - #1361

Merged
Prafulrakhade merged 1 commit into
mosip:release-1.2.0.1from
Ivanmeneges:patch-8
Nov 10, 2025
Merged

[DSD-9196] Image transfer from mosipdev2 to mosipqa#1361
Prafulrakhade merged 1 commit into
mosip:release-1.2.0.1from
Ivanmeneges:patch-8

Conversation

@Ivanmeneges

@Ivanmeneges Ivanmeneges commented Nov 10, 2025

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Chores
    • Updated container images in release configuration by replacing legacy service images with a new certification service image.

Signed-off-by: Ivanmeneges <ivan.anil016@gmail.com>
@coderabbitai

coderabbitai Bot commented Nov 10, 2025

Copy link
Copy Markdown

Walkthrough

Three image entries (pmp-revamp-ui, partner-management-service, policy-management-service) were removed from the release manifest file and replaced with a single inji-certify-with-plugins image entry from the mosipdev2 registry.

Changes

Cohort / File(s) Summary
Release Manifest Update
release/vidivi/images.txt
Removed 3 mosipdev image references (pmp-revamp-ui, partner-management-service, policy-management-service) all versioned 1.2.2.x; Added 1 mosipdev2 image reference (inji-certify-with-plugins:release-0.13.x 0.13.x)

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Single file modified with straightforward image reference replacements
  • No logic changes, configuration-only edits
  • Clear, homogeneous pattern of image entry swaps

Possibly related PRs

Suggested reviewers

  • ckm007

Poem

🐰 Containers dance and images shift,
From mosipdev to mosipdev2, a registry drift,
Three old friends make way with grace,
For inji-certify to take their place!
- The CodeRabbit, hopping through releases 🐇

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title mentions 'Image transfer from mosipdev2 to mosipqa' but the actual changes show removal of mosipdev images and addition of a mosipdev2 image, which doesn't align with the stated direction. Update the title to accurately reflect the actual changes, such as 'DSD-9196 Replace policy/partner/pmp images with inji-certify-with-plugins' or clarify the intended image transfer direction.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 82a678c and ed13784.

📒 Files selected for processing (1)
  • release/vidivi/images.txt (1 hunks)
🔇 Additional comments (1)
release/vidivi/images.txt (1)

1-1: Verify the PR objectives against the actual changes.

The PR title states "Image transfer from mosipdev2 to mosipqa", but the change shows:

  • Images being added from mosipdev2 (not transferred to mosipqa)
  • Removal of three mosipdev-hosted images (pmp-revamp-ui, partner-management-service, policy-management-service v1.2.2.x)
  • Addition of mosipdev2/inji-certify-with-plugins:release-0.13.x (v0.13.x)

Additionally, the version transitions from 1.2.2.x to 0.13.x represent a significant downgrade, which is atypical for a patch release (release-1.2.0.1 branch).

  • Confirm whether the PR title accurately reflects the intent, or if the target branch/direction is incorrect.
  • Verify that the removal of three PMP-related images is intentional and that they are no longer required for this release.
  • Clarify the reasoning behind the version downgrade from 1.2.2.x to 0.13.x for a patch release.

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.

2 participants