Skip to content

[DSD-9664] injistackdev to injistackdev2 - #1530

Merged
abhishek8shankar merged 1 commit into
mosip:release-1.2.0.1from
gaganamadival:release-1.2.0.1
Jan 21, 2026
Merged

[DSD-9664] injistackdev to injistackdev2#1530
abhishek8shankar merged 1 commit into
mosip:release-1.2.0.1from
gaganamadival:release-1.2.0.1

Conversation

@gaganamadival

@gaganamadival gaganamadival commented Jan 21, 2026

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Chores
    • Updated service dependencies to the latest versions.

✏️ 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 Jan 21, 2026

Copy link
Copy Markdown

Walkthrough

Updated the vidivi release manifest by replacing a single PMS API test image reference with two new Inji verification service image entries, both pointing to the develop branch from the injistackdev registry.

Changes

Cohort / File(s) Summary
Manifest update
release/vidivi/images.txt
Replaced mosipdev2/apitest-pms:release-1.3.x entry with injistackdev/inji-verify-service:develop and injistackdev/inji-verify-ui:develop entries

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • Prafulrakhade

Poem

🐰 Two services hop where one did stand,
From ancient test to verify's grand brand,
The develop branch now takes the lead,
Injistackdev fulfills the need!
Hop, hop—the manifest's complete!

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title claims to change 'injistackdev to injistackdev2', but the actual changes replace 'mosipdev2' image entries with 'injistackdev' entries, which is inconsistent with the stated objective. Update the PR title to accurately reflect the changes, such as 'DSD-9664 Replace mosipdev2 images with injistackdev images' or clarify the actual scope of changes.
✅ 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.

✏️ 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.

❤️ 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

🤖 Fix all issues with AI agents
In `@release/vidivi/images.txt`:
- Around line 1-2: The image manifest still references the old registry and
mutable tags: update both occurrences of
injistackdev/inji-verify-service:develop and injistackdev/inji-verify-ui:develop
to the new registry injistackdev2 and replace the mutable "develop" tag with
immutable version identifiers (e.g.,
injistackdev2/inji-verify-service:develop-v1.0 or a specific semver/release tag
or digest) so the images are pinned for reproducible releases.

Comment thread release/vidivi/images.txt
Comment on lines +1 to +2
injistackdev/inji-verify-service:develop develop
injistackdev/inji-verify-ui:develop develop

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 | 🟠 Major

🧩 Analysis chain

🏁 Script executed:

# Check the file content
cat -n release/vidivi/images.txt

Repository: mosip/release-script

Length of output: 170


🏁 Script executed:

# Search for any PR or changelog context that mentions registry changes
rg "injistackdev|injistackdev2" --type txt --type md -A 2 -B 2

Repository: mosip/release-script

Length of output: 208


🏁 Script executed:

# Check if there are other image configuration files for context
fd -e txt -e md -e yaml -e yml | grep -i "image\|manifest\|release" | head -20

Repository: mosip/release-script

Length of output: 670


🏁 Script executed:

# Check the vidivi config and documentation
cat -n release/vidivi/config.yml

Repository: mosip/release-script

Length of output: 445


🏁 Script executed:

# Check README for context about this file
cat -n release/vidivi/README.md

Repository: mosip/release-script

Length of output: 16774


🏁 Script executed:

# Check git log for recent commits to this file
git log --oneline -10 release/vidivi/images.txt 2>/dev/null || echo "No git history available"

Repository: mosip/release-script

Length of output: 112


🏁 Script executed:

# Check the checks/images.txt for comparison to understand the pattern
cat -n release/checks/images.txt

Repository: mosip/release-script

Length of output: 3165


Update registry to injistackdev2 and replace mutable develop tags with immutable versions.

The commit message confirms the migration intent from injistackdev to injistackdev2, but the manifest still references injistackdev. Additionally, develop tags are mutable and break release reproducibility—use immutable version/digest tags instead (e.g., develop → a specific version like develop-v1.0 or release version).

🤖 Prompt for AI Agents
In `@release/vidivi/images.txt` around lines 1 - 2, The image manifest still
references the old registry and mutable tags: update both occurrences of
injistackdev/inji-verify-service:develop and injistackdev/inji-verify-ui:develop
to the new registry injistackdev2 and replace the mutable "develop" tag with
immutable version identifiers (e.g.,
injistackdev2/inji-verify-service:develop-v1.0 or a specific semver/release tag
or digest) so the images are pinned for reproducible releases.

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