Skip to content

[/DSD-9312] move image from dev to dev2 - #1409

Merged
abhishek8shankar merged 1 commit into
mosip:release-1.2.0.1from
gaganamadival:release-1.2.0.1
Nov 24, 2025
Merged

[/DSD-9312] move image from dev to dev2#1409
abhishek8shankar merged 1 commit into
mosip:release-1.2.0.1from
gaganamadival:release-1.2.0.1

Conversation

@gaganamadival

@gaganamadival gaganamadival commented Nov 24, 2025

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Chores
    • Updated container image configurations by removing legacy versions of multiple service components and introducing a new certified image, modernizing the deployment registry.

✏️ Tip: You can customize this high-level summary in your review settings.

Signed-off-by: gaganamadival <94885434+gaganamadival@users.noreply.github.com>
@coderabbitai

coderabbitai Bot commented Nov 24, 2025

Copy link
Copy Markdown

Walkthrough

This pull request updates the release configuration file by removing multiple mosipdev service image entries versioned at release-1.3.x and introducing a single new image entry for mosipdev/inji-certify-with-plugins at release-0.13.x.

Changes

Cohort / File(s) Summary
Image Configuration Update
release/vidivi/images.txt
Removed multiple mosipdev services at release-1.3.x; added mosipdev/inji-certify-with-plugins:release-0.13.x

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Verify that all removed image entries are no longer required
  • Confirm the inji-certify-with-plugins:release-0.13.x image is the correct replacement and compatible with existing deployment infrastructure

Possibly related PRs

Suggested reviewers

  • Prafulrakhade
  • ckm007

Poem

🐰 Out with the old services, release-1.3,
In hops the new certify plugin with glee,
One image replaces the multitude's way,
Simplifying deployments, hip-hop hooray! 🎉

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning The title references moving an image from 'dev to dev2', but the actual changes show removal of multiple mosipdev entries and addition of a mosipdev/inji-certify-with-plugins entry, which doesn't align with a simple 'dev to dev2' migration described in the title. Update the title to accurately reflect the actual changes, such as: 'Replace mosipdev service images with inji-certify-with-plugins' or provide details about which specific image is being moved and why.
✅ 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 5c249fc and a066acc.

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

1-1: Verify PR intent and version alignment.

The PR title references "move image from dev to dev2," but the image tag is release-0.13.x rather than a dev-labeled version. Clarify whether:

  1. The "dev to dev2" migration refers to an infrastructure/environment change (not reflected in image tags), or
  2. The image tag should use a different versioning scheme (e.g., dev2, dev2-0.13.x)

Also, confirm that removing the multiple release-1.3.x entries (mentioned in the summary) and retaining only this single release-0.13.x entry is the correct end state for this release configuration.


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.

@abhishek8shankar
abhishek8shankar merged commit 4c711c8 into mosip:release-1.2.0.1 Nov 24, 2025
2 checks 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.

2 participants