Skip to content

[DSD-9189] moved image from dev to dev2 - #1352

Merged
Prafulrakhade merged 1 commit into
mosip:release-1.2.0.1from
gaganamadival:release-1.2.0.1
Nov 7, 2025
Merged

[DSD-9189] moved image from dev to dev2#1352
Prafulrakhade merged 1 commit into
mosip:release-1.2.0.1from
gaganamadival:release-1.2.0.1

Conversation

@gaganamadival

@gaganamadival gaganamadival commented Nov 7, 2025

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Chores
    • Consolidated deployment image configuration by streamlining image references while maintaining release tag consistency.

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

coderabbitai Bot commented Nov 7, 2025

Copy link
Copy Markdown

Walkthrough

Two image references (mosipdev2/inji-web and mosipdev2/mimoto) are consolidated into a single image reference (mosipdev/inji-certify-with-plugins) in the release configuration file, maintaining the release tag across both replacements.

Changes

Cohort / File(s) Summary
Image Configuration Update
release/vidivi/images.txt
Replaces two separate image entries with a single consolidated image reference while preserving the release tag

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • [DSD-9175] mosipdev2 to mosipqa #1351: Directly removes the same mosipdev2/inji-web and mosipdev2/mimoto entries that this PR consolidates, replacing them with the mosipdev/inji-certify-with-plugins image.

Poem

🐰 Two images become one in the night,
A consolidation done just right,
From mosipdev2 to mosipdev we've grown,
The inji-certify now stands alone! ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title claims to move images from dev to dev2, but the actual changes show replacement of two dev2 image references with a single dev image reference—the opposite direction. Update the title to accurately reflect the change: 'DSD-9189 moved images from dev2 to dev' or similar wording that correctly describes replacing dev2 images with a dev image.
✅ 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 425c39f and d238391.

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

1-1: The original review comment is based on a misunderstanding of the file format and should be disregarded.

The "duplicate tag" is the correct format for this file: image:source_tag destination_tag. The entry mosipdev/inji-certify-with-plugins:release-0.13.x release-0.13.x properly maps the source image tag to the destination tag. This format is consistent with other entries in release/checks/images.txt, such as mosipid/pre-registration-ui:release-1.2.0 1.2.0.

The consolidation of two images into one is intentional and complete—old image references (mosipdev2/inji-web, mosipdev2/mimoto) appear only in execution logs and documentation, not in active code. No other configuration files require updates.

Likely an incorrect or invalid review comment.


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