Skip to content

[DSD-9522] mosipdev to mosipdev2 - #1492

Closed
SwapnilWorks-Devops wants to merge 1 commit into
mosip:release-1.2.0.1from
SwapnilWorks-Devops:release-1.2.0.1
Closed

[DSD-9522] mosipdev to mosipdev2#1492
SwapnilWorks-Devops wants to merge 1 commit into
mosip:release-1.2.0.1from
SwapnilWorks-Devops:release-1.2.0.1

Conversation

@SwapnilWorks-Devops

@SwapnilWorks-Devops SwapnilWorks-Devops commented Dec 19, 2025

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Chores
    • Updated deployment image references for infrastructure components.

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

Signed-off-by: Swapnil <swapnil.mohanty@technoforte.co.in>
@coderabbitai

coderabbitai Bot commented Dec 19, 2025

Copy link
Copy Markdown

Walkthrough

The pull request replaces a single Docker image entry in release/vidivi/images.txt. The mosipdev2/apitest-prereg:release-1.3.x entry is removed and replaced with two new entries: mosipdev/dsl-orchestrator:release-1.3.x and mosipdev/dsl-packetcreator:release-1.3.x.

Changes

Cohort / File(s) Change Summary
Docker image configuration
release/vidivi/images.txt
Removed single image entry mosipdev2/apitest-prereg:release-1.3.x 1.3.x and added two new image entries: mosipdev/dsl-orchestrator:release-1.3.x release-1.3.x and mosipdev/dsl-packetcreator:release-1.3.x release-1.3.x

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • ckm007
  • Prafulrakhade

Poem

🐰 Hop along, the images all change!
From apitest to dsl they rearrange,
Two orchestrators dance and packets do fly,
In release-1.3.x beneath the clear sky! 🎉

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title indicates a change from 'mosipdev' to 'mosipdev2', but the actual changes show a replacement of 'mosipdev2/apitest-prereg' with 'mosipdev/dsl-orchestrator' and 'mosipdev/dsl-packetcreator' - the opposite direction. Update the PR title to accurately reflect the actual changes, such as 'DSD-9522 Replace apitest-prereg with dsl-orchestrator and dsl-packetcreator' or clarify the intended direction of the migration.
✅ 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: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 89d2cbb and 9bd9755.

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

1-2: Verify PR title alignment with actual changes.

The PR title states "mosipdev to mosipdev2" but the code changes show entries using the mosipdev/ registry (not mosipdev2/). Based on the AI summary, this appears to be a migration from mosipdev2/apitest-prereg to mosipdev/ entries. Please clarify the intended direction and ensure the PR title accurately reflects the registry migration.


1-2: Confirm component migration intent.

This change replaces mosipdev2/apitest-prereg:release-1.3.x with two new components (mosipdev/dsl-orchestrator and mosipdev/dsl-packetcreator). The removal of apitest-prereg is safe—no lingering references exist in the codebase. However, verify:

  1. Both new images are available in the mosipdev registry with tag release-1.3.x.
  2. The two new components provide the functionality previously served by apitest-prereg or represent a planned replacement (confirmation from the team may be necessary if this migration is not documented in the repository).

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.

1 participant