[DSD-9409] mosipdev to mosipdev2 - #1436
Conversation
Signed-off-by: Swapnil <swapnil.mohanty@technoforte.co.in>
WalkthroughThis PR updates Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes
Possibly related PRs
Poem
Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
| mosipdev2/resident-service:release-1.3.x 1.3.x | ||
| mosipdev2/websub-service:release-1.3.x 1.3.x | ||
| mosipdev2/consolidator-websub-service:release-1.3.x 1.3.x | ||
| mosipdev/apitest-inji-certify:release-0.13.x release-0.13.x |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
# First, check if the file exists and get its current content
ls -la release/vidivi/images.txt 2>&1 || echo "File not found"Repository: mosip/release-script
Length of output: 138
🏁 Script executed:
# Get git history for the file
cd release/vidivi 2>/dev/null && git log --oneline images.txt 2>&1 | head -20Repository: mosip/release-script
Length of output: 122
🏁 Script executed:
# Get full git diff to see what changed
cd release/vidivi 2>/dev/null && git diff HEAD~5..HEAD -- images.txt 2>&1 || git log -p --follow images.txt 2>&1 | head -100Repository: mosip/release-script
Length of output: 46
🏁 Script executed:
# Check current git status and branch info
git status 2>&1
git branch -a 2>&1 | grep -E "release-1.2.0.1|release-0.13|release-1.3"Repository: mosip/release-script
Length of output: 228
🏁 Script executed:
# Search for any documentation or context about the mosipdev entries
git log --all --oneline --grep="mosipdev" 2>&1 | head -20
git log --all --oneline --grep="DSD-9409" 2>&1 | head -20Repository: mosip/release-script
Length of output: 145
🏁 Script executed:
# Get the specific commit details
git show 2e8a419 --statRepository: mosip/release-script
Length of output: 437
🏁 Script executed:
# Get the actual diff for that commit
git show 2e8a419 -- release/vidivi/images.txtRepository: mosip/release-script
Length of output: 3767
🏁 Script executed:
# Read the full current file content
cat -n release/vidivi/images.txtRepository: mosip/release-script
Length of output: 3615
🏁 Script executed:
# Check the previous version of the file
git show 2e8a419^:release/vidivi/images.txt 2>/dev/null | head -30Repository: mosip/release-script
Length of output: 1631
Commit message is backwards: should be "mosipdev2 to mosipdev", not "mosipdev to mosipdev2".
The git diff confirms the actual changes contradict the commit title:
- Removed: 55 entries from
mosipdev2registry, predominantly usingrelease-1.3.xtags - Added: 1 entry from
mosipdevregistry:mosipdev/apitest-inji-certify:release-0.13.x
This is a complete replacement of all mosipdev2 services with a single apitest-inji-certify service, not a version downgrade. The version difference (1.3.x vs 0.13.x) reflects different images/services, not a regression of the same service.
The commit message must be corrected to accurately reflect the direction of the migration: mosipdev2 → mosipdev.
🤖 Prompt for AI Agents
In release/vidivi/images.txt around lines 1 to 1, the commit title is backwards;
update the commit message to reflect that images were migrated from mosipdev2 to
mosipdev (many mosipdev2 entries removed and one
mosipdev/apitest-inji-certify:release-0.13.x added). Amend the commit message
(git commit --amend) or create a follow-up commit that uses a clear title like
"mosipdev2 → mosipdev: replace multiple mosipdev2 images with
mosipdev/apitest-inji-certify" and in the body list the removed mosipdev2
entries count and the added mosipdev entry for accurate history.
2 participants
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.