Skip to content

[DSD-9280] image move from dev to dev2 - #1401

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

abhishek8shankar merged 1 commit into
mosip:release-1.2.0.1from
gaganamadival:release-1.2.0.1

Conversation

@gaganamadival

@gaganamadival gaganamadival commented Nov 20, 2025

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Chores
    • Updated deployment service configuration.

✏️ 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 20, 2025

Copy link
Copy Markdown

Walkthrough

The pull request modifies release/vidivi/images.txt to replace two Docker image entries (mosipdev/inji-web and mosipdev/mimoto) with a single consolidated entry for mosipdev/apitest-masterdata while preserving the downstream version tag pattern.

Changes

Cohort / File(s) Change Summary
Configuration Update
release/vidivi/images.txt
Replaces two image entries with one; removes mosipdev/inji-web and mosipdev/mimoto lines, adds single mosipdev/apitest-masterdata entry with matching version tag

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

  • Single configuration file modified with straightforward line replacements
  • No logic, code, or behavioral changes
  • Repetitive substitution pattern (two lines consolidated to one)

Possibly related PRs

Poem

🐰 Two images merge as one so fine,
The rabbit hops through lines to align,
Inji and mimoto fade away,
Apitest-masterdata holds the sway,
Config files tidied, neat and divine! ✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the main change: moving Docker image entries from one development environment to another in the release configuration file.
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 6aac3f6 and dfea2e0.

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

1-1: Verify version tag alignment and consolidation impact.

The image tag shows release-1.2.1.x but the PR targets the release-1.2.0.1 branch. Additionally, the consolidation replaces two images with one—ensure this doesn't break downstream dependencies or processes that may reference the removed images (mosipdev/inji-web or mosipdev/mimoto).


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 5d05975 into mosip:release-1.2.0.1 Nov 20, 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