Skip to content

[DSD-10347] 1.3.1 Packet Processing Performance Tuning release#1104

Merged
Prafulrakhade merged 2 commits into
release-1.3.xfrom
releas-branch
May 31, 2026
Merged

[DSD-10347] 1.3.1 Packet Processing Performance Tuning release#1104
Prafulrakhade merged 2 commits into
release-1.3.xfrom
releas-branch

Conversation

@ckm007
Copy link
Copy Markdown
Member

@ckm007 ckm007 commented May 31, 2026

Automated PR for release.

Summary by CodeRabbit

  • Chores
    • Updated Helm chart version to 1.3.1-rc.1
    • Updated pre-registration booking service container image configuration to reflect release candidate build

Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 31, 2026

Review Change Stack

Warning

Review limit reached

@Prafulrakhade, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 2 minutes and 5 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 4e75730d-1a38-47e3-a026-c69c93d035f8

📥 Commits

Reviewing files that changed from the base of the PR and between dc4a834 and 3f6b853.

📒 Files selected for processing (9)
  • cache-provider-hazelcast/pom.xml
  • cache-provider-redis/pom.xml
  • kernel/kernel-ref-idobjectvalidator/pom.xml
  • kernel/kernel-smsserviceprovider-msg91/pom.xml
  • kernel/kernel-virusscanner-clamav/pom.xml
  • kernel/pom.xml
  • pre-registration-booking-service/pom.xml
  • registration-processor/registration-processor-external-integration-service/pom.xml
  • registration-processor/registration-processor-external-stage/pom.xml

Walkthrough

The Helm chart for the pre-registration booking service is updated to release candidate version 1.3.1-rc.1, with the container image repository migrated from the QA Docker namespace to the production namespace and the tag pinned to the matching RC version.

Changes

Helm chart RC release configuration

Layer / File(s) Summary
Helm chart version and image configuration
helm/prereg-booking/Chart.yaml, helm/prereg-booking/values.yaml
Chart version is bumped to 1.3.1-rc.1. The pre-registration booking service image repository is switched from mosipqa/pre-registration-booking-service to mosipid/pre-registration-booking-service, and the image tag is pinned from 1.3.x to 1.3.1-rc.1.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • mosip/mosip-ref-impl#1096: Both PRs modify the same helm/prereg-booking chart metadata (Chart.yaml version) and the preregistration booking image settings in values.yaml (image.repository/image.tag), so the changes directly overlap in those Helm fields (though to different target versions).

Suggested reviewers

  • dhanendra06

Poem

🐰 A rabbit hops through helm charts with glee,
RC release, version one-point-three!
From QA to prod, the image does fly,
To mosipid namespace, reaching new sky! 🚀

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title mentions '1.3.1' and 'release' which aligns with the version update to 1.3.1-rc.1 in the changeset, but 'Packet Processing Performance Tuning' is not reflected in the actual changes, making the title partially misleading. Update the title to accurately reflect the actual changes: 'Update prereg-booking Helm chart and image to 1.3.1-rc.1 release' or similar, removing the unsubstantiated 'Packet Processing Performance Tuning' reference.
✅ 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`:
- Around line 49-50: The values.yaml currently pins the image via the repository
and tag fields (repository: mosipid/pre-registration-booking-service and tag:
1.3.1-rc.1) but the specified tag does not exist on Docker Hub; update the tag
value in the tag field to a verified existing image tag (or rebuild/push the
image under 1.3.1-rc.1) so Helm can pull the image successfully—locate the
repository and tag entries in the helm/prereg-booking values.yaml and change the
tag to a valid tag or ensure the image with tag 1.3.1-rc.1 is published before
deploying.
🪄 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: 1d5d97d4-345c-4b27-9c94-dfa71413e12a

📥 Commits

Reviewing files that changed from the base of the PR and between 6851e0f and dc4a834.

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

Comment thread helm/prereg-booking/values.yaml
Signed-off-by: Prafulrakhade <prafulrakhade02@gmail.com>
@Prafulrakhade Prafulrakhade merged commit 0a6ca1c into release-1.3.x May 31, 2026
40 of 42 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