Skip to content

[DSD-9194] mosipdev2 to mosipqa - #1377

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

[DSD-9194] mosipdev2 to mosipqa#1377
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 Nov 14, 2025

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Chores
    • Updated container image registry reference to maintain infrastructure consistency.

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

coderabbitai Bot commented Nov 14, 2025

Copy link
Copy Markdown

Walkthrough

A Docker image reference in the release configuration is updated, changing the repository namespace from mosipdev to mosipdev2 while updating the associated image tag format in a single configuration file.

Changes

Cohort / File(s) Summary
Image namespace migration
release/vidivi/images.txt
Updated mosipdev/apitest-pms image reference to mosipdev2/apitest-pms; tag format changed from release-1.2.2.x release-1.2.2.x to release-1.2.2.x 1.2.2.x

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

  • Single-line configuration update with no logic changes required

Possibly related PRs

Suggested reviewers

  • ckm007

Poem

🐰 From mosipdev to mosipdev2 we hop,
Image namespaces climb without stop,
Tags dancing through the release file so neat,
Configuration migration, our update complete! 📦✨

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 'mosipdev2 to mosipqa', but the actual changeset shows a modification from 'mosipdev/apitest-pms' to 'mosipdev2/apitest-pms' - moving TO mosipdev2, not FROM it. This is the opposite of what the title suggests, making it misleading. Correct the title to accurately reflect the change: 'DSD-9194 mosipdev to mosipdev2' or 'DSD-9194 Update image namespace to mosipdev2'.
✅ 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 2cefff0 and c21f4ed.

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

1-1: Revert the review comment—the original concern is incorrect.

The PR title "DSD-9194 mosipdev2 to mosipqa" refers to the issue/epic identifier, not the change direction. The actual transfer is from mosipdev/apitest-pms:release-1.2.2.xmosipdev2/apitest-pms:release-1.2.2.x, which is verified in the logs and transfer_report.md. This aligns with the staged release path documented in README.md (mosipdev → mosipdev2 → mosipqa/mosipint → mosipid), where mosipdev2 is the correct intermediate step.

The file format (left field = full image reference with tag; right field = destination tag value) is correctly structured. No issues to address.


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 793d304 into mosip:release-1.2.0.1 Nov 14, 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