Skip to content

[DSD-9521] mosipdev2 to mosipqa - #1491

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

[DSD-9521] mosipdev2 to mosipqa#1491
Prafulrakhade merged 1 commit into
mosip:release-1.2.0.1from
gaganamadival:release-1.2.0.1

Conversation

@gaganamadival

@gaganamadival gaganamadival commented Dec 18, 2025

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Chores
    • Updated container image registry references for application deployment.

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

Copy link
Copy Markdown

Walkthrough

Updated container image reference in release configuration, changing the registry host from mosipdev to mosipdev2 and normalizing the version tag format for the apitest-prereg image.

Changes

Cohort / File(s) Change Summary
Registry Configuration Update
release/vidivi/images.txt
Updated apitest-prereg image reference: registry host changed from mosipdev to mosipdev2, and version tag normalized from release-1.3.x to 1.3.x

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • mosip/release-script#1464: Modifies the same file to change image hosts from mosipdev to mosipdev2 and adjusts version tag format
  • mosip/release-script#1444: Updates image references in release/vidivi/images.txt with identical registry-host and tag-fragment changes
  • mosip/release-script#1377: Modifies release/vidivi/images.txt to change image references from mosipdev registry to mosipdev2

Suggested reviewers

  • Prafulrakhade

Poem

🐰 A registry hops from dev to dev-two,
Tags trimmed and tidy, versioning anew,
apitest-prereg takes flight with care,
The image finds its home somewhere fair,
Configuration refined, deployment dreams come true! ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning The pull request title states 'mosipdev2 to mosipqa', but the actual change in the code updates 'mosipdev to mosipdev2', not 'mosipdev2 to mosipqa' as claimed. Update the title to accurately reflect the actual change: 'DSD-9521 mosipdev to mosipdev2' or 'DSD-9521 Update image registry host 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: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between de7f985 and 40b50e6.

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

1-1: PR title mismatch: Title mentions "mosipdev2 to mosipqa" but change shows transition TO mosipdev2.

The PR title indicates the target should be mosipqa, but the actual changes show the registry host changed to mosipdev2. Clarify whether the intended target is mosipdev2 or mosipqa.


1-1: No stale references remain—migration to the new registry is clean.

The codebase contains no references to the old mosipdev registry host for this image. The reference has been successfully updated to mosipdev2/apitest-prereg:release-1.3.x 1.3.x in release/vidivi/images.txt, with no conflicting old tags elsewhere.

Verify that the image exists on the mosipdev2 registry with the specified tags and that the tag normalization (release-1.3.x to 1.3.x) aligns with your release process.


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 89d2cbb into mosip:release-1.2.0.1 Dec 18, 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