[DSD-9612] mosipdev2 to mosipqa - #1520
Conversation
Signed-off-by: gaganamadival <94885434+gaganamadival@users.noreply.github.com>
WalkthroughA single configuration file ( Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@release/vidivi/images.txt`:
- Around line 1-2: The images.txt entries still reference the old registry;
update the two image references "mosipdev2/inji-web:develop develop" and
"mosipdev2/mimoto:develop develop" to the intended registry (e.g., replace the
"mosipdev2" prefix with "mosipqa") or, if the commit message was wrong, correct
the commit/PR title to reflect use of mosipdev2; also search and update any
deployment manifests/Helm charts or scripts that reference those exact image
strings to ensure consistency across deployments.
📜 Review details
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
release/vidivi/images.txt
✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.
| mosipdev2/inji-web:develop develop | ||
| mosipdev2/mimoto:develop develop |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
echo "=== Checking actual content of release/vidivi/images.txt ==="
cat -n release/vidivi/images.txt
echo -e "\n=== Checking git history to understand the change ==="
git log --oneline -n 5 -- release/vidivi/images.txt
echo -e "\n=== Searching for references to new images elsewhere in codebase ==="
rg -l "mosipdev2/inji-web|mosipdev2/mimoto" --type-list | head -20
echo -e "\n=== Searching for references to the old image (mosipdev/inji-certify-with-plugins) ==="
rg -n "mosipdev/inji-certify-with-plugins" | head -20
echo -e "\n=== Finding other image list files to check format consistency ==="
find . -type f -name "*images*.txt" -o -name "*image*.txt" | grep -v ".git" | head -20Repository: mosip/release-script
Length of output: 990
🏁 Script executed:
#!/bin/bash
echo "=== Comparing format with release/checks/images.txt ==="
cat -n release/checks/images.txt
echo -e "\n=== Searching for any mosipqa references in codebase ==="
rg "mosipqa" --type-list | head -20 2>/dev/null || rg "mosipqa" | head -20
echo -e "\n=== Checking git diff for this commit to see full context ==="
git show --stat 19f77e5
echo -e "\n=== Checking if there are more changes in this commit ==="
git show 19f77e5 | head -100Repository: mosip/release-script
Length of output: 9992
🏁 Script executed:
#!/bin/bash
echo "=== Checking full git diff for release/vidivi/images.txt specifically ==="
git show 19f77e5:release/vidivi/images.txt
echo -e "\n=== Checking if there are any related issues or PRs mentioning DSD-9612 ==="
grep -r "DSD-9612" . 2>/dev/null | head -20Repository: mosip/release-script
Length of output: 282
Critical: PR title and implementation are mismatched — images still use "mosipdev2" registry, not "mosipqa".
The commit message "[DSD-9612] mosipdev2 to mosipqa" indicates images should reference the mosipqa registry, but both lines in this file reference mosipdev2 instead. This discrepancy suggests either an incomplete implementation or an incorrect commit message that needs correction before merge.
Verify that:
- These images exist in the
mosipdev2registry and are appropriate for use. - Any deployment scripts, Helm charts, or configurations that previously referenced
mosipdev/inji-certify-with-plugins:develophave been updated to use both new images.
🤖 Prompt for AI Agents
In `@release/vidivi/images.txt` around lines 1 - 2, The images.txt entries still
reference the old registry; update the two image references
"mosipdev2/inji-web:develop develop" and "mosipdev2/mimoto:develop develop" to
the intended registry (e.g., replace the "mosipdev2" prefix with "mosipqa") or,
if the commit message was wrong, correct the commit/PR title to reflect use of
mosipdev2; also search and update any deployment manifests/Helm charts or
scripts that reference those exact image strings to ensure consistency across
deployments.
Summary by CodeRabbit
Note: This release contains infrastructure updates with no direct end-user facing changes.
✏️ Tip: You can customize this high-level summary in your review settings.