Skip to content

[/DSD-9695] injistackdev to injistactdev2 - #1539

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

[/DSD-9695] injistackdev to injistactdev2#1539
Prafulrakhade merged 1 commit into
mosip:release-1.2.0.1from
gaganamadival:release-1.2.0.1

Conversation

@gaganamadival

@gaganamadival gaganamadival commented Jan 28, 2026

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Chores
    • Updated service components configuration by removing one entry and adding two new service components to the deployment manifest.

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

Copy link
Copy Markdown

Walkthrough

The images list in the release configuration was updated by removing the inji-certify-with-plugins image entry and adding two new image entries for inji-verify-service and inji-verify-ui, both from the injistackdev namespace.

Changes

Cohort / File(s) Change Summary
Release Configuration Images
release/vidivi/images.txt
Removed injistackdev/inji-certify-with-plugins:develop image entry; added injistackdev/inji-verify-service:develop and injistackdev/inji-verify-ui:develop entries

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • Prafulrakhade

Poem

🐰 Out with the old cert, in with the new,
Verify service and UI too!
Images shuffled with hare-like care,
Configuration's fresh and fair! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title mentions 'injistackdev to injistactdev2' but the actual changes update Docker image references from 'injistackdev/inji-certify-with-plugins' to new entries 'injistackdev/inji-verify-service' and 'injistackdev/inji-verify-ui', not a renaming of 'injistackdev' to 'injistactdev2'. Revise the title to accurately reflect the actual changes, such as 'Update image list: replace inji-certify-with-plugins with verify-service and verify-ui' or similar.
✅ 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: In release/vidivi/images.txt update both Docker image
references that currently use the "injistackdev" namespace to the PR-intended
"injistactdev2" namespace (replace "injistackdev/inji-verify-service:develop"
and "injistackdev/inji-verify-ui:develop" with
"injistactdev2/inji-verify-service:develop" and
"injistactdev2/inji-verify-ui:develop"), then run a quick grep to confirm no
other occurrences remain.

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 | 🔴 Critical

Critical: Namespace mismatch with PR objective.

The PR title indicates the purpose is to change from "injistackdev" to "injistactdev2", but the actual code still uses "injistackdev". Both image references need to be updated to use the "injistactdev2" namespace as stated in the PR objective.

🔧 Proposed fix to align with PR objective
-injistackdev/inji-verify-service:develop develop
-injistackdev/inji-verify-ui:develop develop
+injistactdev2/inji-verify-service:develop develop
+injistactdev2/inji-verify-ui:develop develop
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
injistackdev/inji-verify-service:develop develop
injistackdev/inji-verify-ui:develop develop
injistactdev2/inji-verify-service:develop develop
injistactdev2/inji-verify-ui:develop develop
🤖 Prompt for AI Agents
In `@release/vidivi/images.txt` around lines 1 - 2, In release/vidivi/images.txt
update both Docker image references that currently use the "injistackdev"
namespace to the PR-intended "injistactdev2" namespace (replace
"injistackdev/inji-verify-service:develop" and
"injistackdev/inji-verify-ui:develop" with
"injistactdev2/inji-verify-service:develop" and
"injistactdev2/inji-verify-ui:develop"), then run a quick grep to confirm no
other occurrences remain.

@Prafulrakhade
Prafulrakhade merged commit 1155f4c into mosip:release-1.2.0.1 Jan 28, 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