Skip to content

[DSD-9522] mosipdev to mosipdev2 - #1499

Merged
Prafulrakhade merged 1 commit into
mosip:release-1.2.0.1from
SwapnilWorks-Devops:release-1.2.0.1
Dec 19, 2025
Merged

Prafulrakhade merged 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 container image references to release-1.3.x versions for DSL orchestrator and packet creator 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

This change updates the image manifest file by removing two MOSIP-41056 branch-tagged image references and replacing them with two new release-1.3.x tagged image references, shifting the release configuration to point to updated orchestrator and packet creator images.

Changes

Cohort / File(s) Summary
Release image manifest
release/vidivi/images.txt
Removed mosipdev/usecase-compass-ui and usecase-compass entries tagged with MOSIP-41056; added dsl-orchestrator and dsl-packetcreator entries tagged with release-1.3.x

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • ckm007
  • Prafulrakhade

Poem

🐰 A nibble here, a nibble there,
Old tags fade like dandelion hair,
Release-1.3.x now takes the stage,
Dsl-orchestrator writes a new page! 🎭

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title claims a change from 'mosipdev to mosipdev2', but the actual changes involve replacing MOSIP-41056 image entries with release-1.3.x entries. This is misleading and does not accurately reflect the changeset. Update the title to accurately describe the main change, such as 'DSD-9522 Update image tags from MOSIP-41056 to release-1.3.x' or similar.
✅ 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 a51fd6e and d69fdbe.

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

1-2: Verify release version alignment and PR title accuracy.

Two concerns:

  1. Release version mismatch: This PR targets the release-1.2.0.1 branch, but the images reference release-1.3.x tags. Typically, a 1.2.0.1 release should use 1.2.x or 1.2.0.1-tagged images, not 1.3.x. Confirm this is intentional.

  2. PR title mismatch: The PR title "DSD-9522 mosipdev to mosipdev2" suggests a registry migration from mosipdev to mosipdev2, but the image references still use the mosipdev/ registry prefix, not mosipdev2/. Confirm whether the migration is incomplete or if the title is misleading.


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 9a2986c into mosip:release-1.2.0.1 Dec 19, 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