Skip to content

[DSD-9497] image move from dev to dev2 - #1482

Merged
Prafulrakhade merged 1 commit into
mosip:release-1.2.0.1from
gaganamadival:release-1.2.0.1
Dec 16, 2025
Merged

[DSD-9497] image move from dev to dev2#1482
Prafulrakhade merged 1 commit into
mosip:release-1.2.0.1from
gaganamadival:release-1.2.0.1

Conversation

@gaganamadival

@gaganamadival gaganamadival commented Dec 16, 2025

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Chores
    • Updated image references in the release configuration to accurately reflect the current development status and overall project pipeline requirements. Removed one component's previous release version reference and added development version references for two additional components. These configuration updates help maintain proper alignment with active development, testing, and deployment infrastructure needs.

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

Copy link
Copy Markdown

Walkthrough

The PR updates the image registry file for the Vidivi release, removing a partner-onboarder entry and adding two new image references for inji-web and mimoto, both pointing to develop branches.

Changes

Cohort / File(s) Summary
Image Registry Update
release/vidivi/images.txt
Removed mosipdev/partner-onboarder:release-1.3.x 1.3.x; added mosipdev/inji-web:develop develop and mosipdev/mimoto:develop develop

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • Prafulrakhade

Poem

🐰 Registry entries hop and skip,
Old partner-onboarder takes a trip,
Inji-web and mimoto arrive with cheer,
Develop branches bloom so clear!
Images dance through the file so bright,

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning The title refers to moving images between environments (dev to dev2), which does not clearly match the actual change of updating image entries in the vidivi/images.txt file. Update the title to accurately reflect the change, such as 'DSD-9497 Update vidivi image entries' or 'DSD-9497 Replace partner-onboarder with inji-web and mimoto images'.
✅ 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 314239d and e0d4127.

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

1-2: Verify branch targeting aligns with PR intent and release requirements.

The PR title mentions moving images "from dev to dev2", but the provided entries reference develop branches instead of dev2. Additionally, using develop-branch images in a release version (1.2.0.1) warrants confirmation that this is intentional and appropriate for your deployment strategy.

Please clarify:

  1. Is develop the intended target, or should this be dev2 per the PR title?
  2. Are develop-branch images suitable for this release deployment, or should these reference stable release/main branches?

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.

@Prafulrakhade
Prafulrakhade merged commit 5e04a81 into mosip:release-1.2.0.1 Dec 16, 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