Skip to content

Github issue-1095 corrected helm version#1096

Merged
ckm007 merged 1 commit into
mosip:release-1.3.xfrom
kameshsr:release-1.3.x-GithubIssue-1095
May 13, 2026
Merged

Github issue-1095 corrected helm version#1096
ckm007 merged 1 commit into
mosip:release-1.3.xfrom
kameshsr:release-1.3.x-GithubIssue-1095

Conversation

@kameshsr
Copy link
Copy Markdown
Contributor

@kameshsr kameshsr commented May 12, 2026

Summary by CodeRabbit

Release Notes

  • Chores
    • Updated Helm chart version to 1.3.1-develop
    • Updated Docker image tag to 1.3.x
    • Updated Docker image repository for the pre-registration booking service

Review Change Stack

Signed-off-by: kameshsr <kameshsr1338@gmail.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 12, 2026

Walkthrough

The Helm chart for the preregistration booking service was updated with a new development version. The chart version field was incremented to 1.3.1-develop, and the deployment image repository and tag were updated to reference the mosipqa registry with a flexible 1.3.x tag.

Changes

Helm Chart Configuration

Layer / File(s) Summary
Helm chart version and image configuration
helm/prereg-booking/Chart.yaml, helm/prereg-booking/values.yaml
Chart version incremented from 1.3.0 to 1.3.1-develop and container image repository switched from mosipid/pre-registration-booking-service to mosipqa/pre-registration-booking-service with tag updated from 1.3.0 to 1.3.x.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A chart bumped up to develop state,
Image tags loosened, ready to migrate,
Mosipqa takes the helm with grace,
Version one-point-three finds its place! 📦✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'corrected helm version' is vague and doesn't clearly describe the specific changes made. The PR updates both Helm chart version and Docker image settings. Use a more specific title that clearly describes the main changes, such as 'Update Helm chart version and Docker image repository/tag for pre-registration booking service' or 'Update pre-registration-booking Helm chart to 1.3.1-develop with QA image settings'.
✅ Passed checks (4 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.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@helm/prereg-booking/values.yaml`:
- Line 50: The image tag currently set as the rolling pattern "1.3.x" in the
values.yaml `tag` key should be replaced with a concrete immutable version;
update the `tag` value (refer to the `tag` entry currently "1.3.x") to a
specific numeric release (for example "1.2.0.1" or another exact "1.3.z" once
published) to ensure reproducible deployments and avoid accidental upgrades.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: a70fb60f-93f7-4f48-bf51-0eac02b8f039

📥 Commits

Reviewing files that changed from the base of the PR and between 418e7db and e9bd75a.

📒 Files selected for processing (2)
  • helm/prereg-booking/Chart.yaml
  • helm/prereg-booking/values.yaml

Comment thread helm/prereg-booking/values.yaml
@ckm007 ckm007 merged commit 34b4d82 into mosip:release-1.3.x May 13, 2026
21 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.

3 participants