Skip to content

[DSD-9238] mosipdev2 to mosipqa - #1389

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

[DSD-9238] mosipdev2 to mosipqa#1389
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 17, 2025

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Chores
    • Updated service image versions in release configuration files.

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

coderabbitai Bot commented Nov 17, 2025

Copy link
Copy Markdown

Walkthrough

The pull request updates a container image reference in the release configuration file, replacing the mosipdev2/apitest-esignet-signup:release-1.3.x image with mosipdev2/uitest-web:release-0.15.x in the tracked image manifest for the Vidivi release pipeline.

Changes

Cohort / File(s) Change Summary
Release Image Configuration
release/vidivi/images.txt
Replaced container image reference from mosipdev2/apitest-esignet-signup:release-1.3.x 1.3.x to mosipdev2/uitest-web:release-0.15.x 0.15.x

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • ckm007

Poem

🐰 A rabbit hops through Docker lanes,
Swapping images like window panes,
From apitest to uitest it goes,
Release version numbers in tidy rows,
Configurations dance, the pipeline flows! 🚀

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning The title claims to change 'mosipdev2 to mosipqa', but the actual change replaces a mosipdev2 image with a different mosipdev2 image, not mosipqa. Update the title to accurately reflect the change: replace with a title like 'Update image reference from apitest-esignet-signup to uitest-web in release/vidivi/images.txt' or clarify what mosipqa change is intended.
✅ 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

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.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7dbd461 and 15f4272.

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

Comment thread release/vidivi/images.txt
@@ -1 +1 @@
mosipdev2/apitest-esignet-signup:release-1.3.x 1.3.x
mosipdev2/uitest-web:release-0.15.x 0.15.x

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🔴 Critical

🧩 Analysis chain

⚠️ Title-to-content mismatch: PR title suggests "mosipdev2 to mosipqa" but image reference still uses mosipdev2 registry.

The PR title "[DSD-9238] mosipdev2 to mosipqa" indicates an intended change to the registry prefix, but the actual change on line 1 retains the mosipdev2/ prefix. This is either a misleading title or an incomplete change.

Verify that:

  • The change aligns with the stated objective (if the registry prefix should change to mosipqa/, apply the correction)
  • The old line being replaced was indeed mosipdev2/apitest-esignet-signup:release-1.3.x
  • The new image mosipdev2/uitest-web:release-0.15.x is the correct replacement

🏁 Script executed:

#!/bin/bash
# Description: Inspect the full images.txt file and git history to verify the change scope

# Show the full current file content
echo "=== Current file content ==="
cat release/vidivi/images.txt

# Show the git diff for this file to see old vs. new line
echo ""
echo "=== Git diff for this file ==="
git diff HEAD~1 release/vidivi/images.txt || git diff release/vidivi/images.txt

# Show git log for this file
echo ""
echo "=== Recent commits affecting this file ==="
git log -n 3 --oneline -- release/vidivi/images.txt

Length of output: 577


Change is incomplete: Registry prefix should be mosipqa/ not mosipdev2/

The PR title "[DSD-9238] mosipdev2 to mosipqa" explicitly directs a registry change, but the actual change on line 1 retains mosipdev2/ prefix. The line should read:

mosipqa/uitest-web:release-0.15.x 0.15.x

not:

mosipdev2/uitest-web:release-0.15.x 0.15.x

Update the registry prefix to complete the intended migration.

🤖 Prompt for AI Agents
In release/vidivi/images.txt around line 1, the registry prefix is still
mosipdev2/; update that entry to use the mosipqa/ prefix so the line reads the
registry with mosipqa/uitest-web:release-0.15.x 0.15.x instead of
mosipdev2/uitest-web:release-0.15.x 0.15.x—simply replace the mosipdev2/ prefix
with mosipqa/ and save the file.

@Prafulrakhade
Prafulrakhade merged commit 58b2c1b into mosip:release-1.2.0.1 Nov 17, 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