Skip to content

[DSD-9652] injistackdev2 to injistackqa - #1527

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

[DSD-9652] injistackdev2 to injistackqa#1527
Prafulrakhade 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 internal service configurations and dependencies.

✏️ 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 manifest file to replace a single mosipdev/pmp-ui-v2:release-1.3.x image entry with two new image entries from injistackdev2 registry pointing to develop branches for inji-certify-with-plugins and apitest-inji-certify.

Changes

Cohort / File(s) Summary
Manifest Image Entry Update
release/vidivi/images.txt
Replaced mosipdev/pmp-ui-v2:release-1.3.x 1.3.x with two develop-branch image entries: injistackdev2/inji-certify-with-plugins:develop develop and injistackdev2/apitest-inji-certify:develop develop

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • Prafulrakhade
  • ckm007

Poem

🐰 Two images bloom where once was one,
From mosipdev's release to develop's run,
Inji-certify and apitest unite,
In injistackdev2's caring sight!

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The PR title mentions 'injistackdev2 to injistackqa' but the actual changes replace 'mosipdev/pmp-ui-v2' with 'injistackdev2' images, which is a partial mismatch with the stated intent. Clarify whether the PR aims to migrate from injistackdev2 to injistackqa or update pmp-ui images to injistackdev2. The title's intent doesn't fully align with the actual file 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: Update the two image entries using mutable :develop tags and
the old registry: replace injistackdev2/inji-certify-with-plugins:develop and
injistackdev2/apitest-inji-certify:develop with images hosted on injistackqa and
pinned to immutable version tags or digests (e.g.,
injistackqa/inji-certify-with-plugins:<version> or `@sha256`:<digest> and
injistackqa/apitest-inji-certify:<version> or `@sha256`:<digest>) so the release
manifest is reproducible and the registry migration is applied.

Comment thread release/vidivi/images.txt
Comment on lines +1 to +2
injistackdev2/inji-certify-with-plugins:develop develop
injistackdev2/apitest-inji-certify: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:

# First, check if the file exists and read its contents
cat -n release/vidivi/images.txt 2>/dev/null || echo "File not found or inaccessible"

Repository: mosip/release-script

Length of output: 184


🏁 Script executed:

# Check for any PR context or related files that might indicate the objective
git log --oneline -20 2>/dev/null | head -20 || echo "Git history not available"

Repository: mosip/release-script

Length of output: 111


🏁 Script executed:

# Look for any documentation or comments about the registry change
find . -type f \( -name "*.md" -o -name "*.txt" -o -name "*.yml" -o -name "*.yaml" \) -exec grep -l "injistackqa\|injistackdev2" {} \; 2>/dev/null | head -20

Repository: mosip/release-script

Length of output: 91


🏁 Script executed:

# Check for similar image files and understand the format
find release -name "*.txt" -type f | head -10

Repository: mosip/release-script

Length of output: 168


🏁 Script executed:

# Look for documentation on how these images.txt files are used
find . -type f \( -name "*.md" -o -name "*.sh" \) -exec grep -l "images.txt" {} \; 2>/dev/null | head -10

Repository: mosip/release-script

Length of output: 117


🏁 Script executed:

# Check the structure around this file to understand context
ls -la release/vidivi/ 2>/dev/null

Repository: mosip/release-script

Length of output: 543


🏁 Script executed:

# Check the README files to understand the format and purpose
cat -n release/vidivi/README.md | head -100

Repository: mosip/release-script

Length of output: 5153


🏁 Script executed:

# Also check the other images.txt for reference
cat -n release/checks/images.txt

Repository: mosip/release-script

Length of output: 3165


🏁 Script executed:

# Check config.yml to understand the context
cat -n release/vidivi/config.yml

Repository: mosip/release-script

Length of output: 445


Update registry and pin immutable tags for reproducible releases.

The file uses mutable :develop tags, which breaks reproducibility in a release manifest. Additionally, the PR objective is to migrate from injistackdev2 to injistackqa, but the registry has not been updated. Replace with immutable tags or digests (following the pattern in release/checks/images.txt, which uses specific version tags like 1.2.0).

Suggested fix pattern
-injistackdev2/inji-certify-with-plugins:develop develop
-injistackdev2/apitest-inji-certify:develop develop
+injistackqa/inji-certify-with-plugins:<immutable-tag-or-digest> <version>
+injistackqa/apitest-inji-certify:<immutable-tag-or-digest> <version>
🤖 Prompt for AI Agents
In `@release/vidivi/images.txt` around lines 1 - 2, Update the two image entries
using mutable :develop tags and the old registry: replace
injistackdev2/inji-certify-with-plugins:develop and
injistackdev2/apitest-inji-certify:develop with images hosted on injistackqa and
pinned to immutable version tags or digests (e.g.,
injistackqa/inji-certify-with-plugins:<version> or `@sha256`:<digest> and
injistackqa/apitest-inji-certify:<version> or `@sha256`:<digest>) so the release
manifest is reproducible and the registry migration is applied.

@Prafulrakhade
Prafulrakhade merged commit adcf11b into mosip:release-1.2.0.1 Jan 21, 2026
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