Skip to content

[DSD-9446] Image transfer from dev to devint. - #1498

Merged
Prafulrakhade merged 1 commit into
mosip:release-1.2.0.1from
abhishek8shankar:release-1.2.0.1
Dec 19, 2025
Merged

[DSD-9446] Image transfer from dev to devint.#1498
Prafulrakhade merged 1 commit into
mosip:release-1.2.0.1from
abhishek8shankar:release-1.2.0.1

Conversation

@abhishek8shankar

@abhishek8shankar abhishek8shankar commented Dec 19, 2025

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • Chores
    • Updated container image references in deployment configuration to reflect current infrastructure requirements.

✏️ Tip: You can customize this high-level summary in your review settings.

Signed-off-by: Abhishek S <127825992+abhishek8shankar@users.noreply.github.com>
@coderabbitai

coderabbitai Bot commented Dec 19, 2025

Copy link
Copy Markdown

Walkthrough

The pull request updates the release image configuration file by removing four deprecated image entries for esignet and signup components and adding two new entries for the usecase-compass component.

Changes

Cohort / File(s) Change Summary
Image Configuration Update
release/vidivi/images.txt
Removed four image entries (mosipdev/esignet-with-plugins, mosipdev/signup-with-plugins, mosipdev/signup-ui, mosipdev/apitest-esignet-signup) and added two new entries (mosipdev/usecase-compass-ui, mosipdev/usecase-compass) with MOSIP-41056 tag

Estimated Code Review Effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly Related PRs

Suggested Reviewers

  • ckm007

Poem

🐰 A rabbit hops through configs with glee,
Old images fade, new ones we see,
Compass points true to the MOSIP way,
Updated and bundled, hooray, hooray!

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The pull request title accurately describes the main change: transferring Docker image entries from dev to devint environment in the images.txt file.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 87bd4b7 and f62bb80.

📒 Files selected for processing (1)
  • release/vidivi/images.txt (1 hunks)
🔇 Additional comments (1)
release/vidivi/images.txt (1)

1-2: Verify image existence and tag format consistency.

The added entries use a ticket ID (MOSIP-41056) as the Docker image tag, which differs from the removed entries that used semantic versioning (e.g., 1.7.1, 1.3.1). Additionally, both the tag and version fields contain identical values.

Please verify:

  1. That images mosipdev/usecase-compass-ui:MOSIP-41056 and mosipdev/usecase-compass:MOSIP-41056 exist in the repository.
  2. Whether using a ticket ID as a Docker tag aligns with your project's image tagging conventions (semantic versioning, commit hashes, or branch names are more typical).
  3. Whether the duplication of the tag and version fields is intentional or should be distinct values.

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.

@Prafulrakhade
Prafulrakhade merged commit a51fd6e into mosip:release-1.2.0.1 Dec 19, 2025
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