Skip to content

[MOSIP-44014] Updated images.txt - #1472

Merged
ckm007 merged 1 commit into
mosip:release-1.2.0.1from
Mohanraj209:release-1.2.0.1
Dec 11, 2025
Merged

[MOSIP-44014] Updated images.txt#1472
ckm007 merged 1 commit into
mosip:release-1.2.0.1from
Mohanraj209:release-1.2.0.1

Conversation

@Mohanraj209

@Mohanraj209 Mohanraj209 commented Dec 11, 2025

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Chores
    • Updated infrastructure dependencies to latest versions for improved stability.

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

Signed-off-by: Mohan E <mohanraj1715@gmail.com>
@coderabbitai

coderabbitai Bot commented Dec 11, 2025

Copy link
Copy Markdown

Walkthrough

Single line replacement in release manifest: PostgreSQL upgrade image reference replaced with ActiveMQ Artemis image. Changes image repository, name, and version tag while maintaining manifest format consistency.

Changes

Cohort / File(s) Change Summary
Image manifest update
release/vidivi/images.txt
Replaced mosipdev/postgres-upgrade:release-1.3.x 1.3x with mosipdev/activemq-artemis:2.39.0 2.39.0

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Verify that ActiveMQ Artemis 2.39.0 is the intended image for this release configuration
  • Confirm the version tag transition aligns with vidivi deployment requirements

Possibly related PRs

Suggested reviewers

  • ckm007

Poem

🐰 A rabbit hops through configs bright,
PostgreSQL bids its fond goodbye,
Artemis arrives with spring-like grace,
2.39.0 takes its place,
Oh what a seamless, shifting sight! 🎯

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'Updated images.txt' is too vague and generic. It does not convey the specific change made (switching from postgres-upgrade to activemq-artemis image). Improve the title to be more specific about the actual change, such as 'MOSIP-44014 Replace postgres-upgrade with activemq-artemis image in images.txt'.
✅ 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.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 06cd067 and f98a47b.

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

1-1: Verify the intentionality of this image replacement against MOSIP-44014.

This change replaces the PostgreSQL upgrade image with ActiveMQ Artemis in the VIDIVI release manifest. While no other references to the postgres-upgrade image were found in the codebase, clarify whether this replacement aligns with the intended release configuration and requirements.

Is mosipdev/activemq-artemis:2.39.0 the correct image for this release?


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.

@ckm007
ckm007 merged commit 5e507a7 into mosip:release-1.2.0.1 Dec 11, 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