Skip to content

[DSD-9536] mosipdev2 to mosipqa - #1500

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

[DSD-9536] mosipdev2 to mosipqa#1500
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 20, 2025

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Chores
    • Updated container image references and version tagging to align with current release standards.

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

Copy link
Copy Markdown

Walkthrough

Configuration update to image registry references in the release manifest. Replaces two image entries from the mosipdev namespace to mosipdev2 and adjusts version tag formatting from release-1.3.x to 1.3.x in a single file.

Changes

Cohort / File(s) Change Summary
Image registry and version tag updates
release/vidivi/images.txt
Updated image references from mosipdev to mosipdev2 namespace for two images; adjusted version tags from release-1.3.x to 1.3.x format

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

— Straightforward configuration string replacements in a single file with no logic or behavioral changes.

Possibly related PRs

Suggested reviewers

  • ckm007

Poem

🐰 From mosipdev to mosipdev2 we hop,
Version tags trimmed, no more release-prop,
Images updated in neat little rows,
Configuration changes—that's how it goes! 🎯

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title mentions 'mosipdev2 to mosipqa' but the actual changes replace 'mosipdev' with 'mosipdev2', not 'mosipqa'. The title is misleading about the actual changeset. Update the title to accurately reflect the changes, such as 'DSD-9536 Replace mosipdev with mosipdev2 in release-vidivi images' or 'DSD-9536 Update image registry from mosipdev 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 9a2986c and 60f3416.

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

1-2: Verify the version tag format change.

The version tag format has changed from release-1.3.x release-1.3.x to release-1.3.x 1.3.x (removing the release- prefix from the second tag).

Confirm that this format change is intentional and aligns with the image registry's tagging conventions.


1-2: No action required. The file is correct as-is.

The images.txt format uses space-separated values where the first column specifies the source image repository and tag to pull from, and the second column specifies the destination tag to apply. The destination organization (mosipqa) is configured separately in config.yml, not in images.txt. The commit message "DSD-9536 mosipdev2 to mosipqa" correctly describes transferring images FROM the mosipdev2 registry TO the mosipqa registry, which aligns perfectly with the images.txt content.

Likely an incorrect or invalid review comment.


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 34951f9 into mosip:release-1.2.0.1 Dec 22, 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