[MOSIP-43631] updated helm charts

[bhumi46]

Summary by CodeRabbit

• Chores
  • Optimized container resource allocation to improve deployment efficiency and system scalability while maintaining performance standards.

[coderabbitai]

Walkthrough

The pull request reduces Kubernetes resource requests and limits for a container in the Key Manager Helm chart. CPU limits decrease from 2000m to 1000m, memory limits from 6000Mi to 5000Mi, CPU requests from 2000m to 500m, and memory requests from 6000Mi to 3000Mi.

Changes

Cohort / File(s)	Summary
Key Manager Helm Configuration helm/keymanager/values.yaml	Reduced container resource requests and limits: CPU limits 2000m → 1000m, memory limits 6000Mi → 5000Mi, CPU requests 2000m → 500m, memory requests 6000Mi → 3000Mi

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

• Simple numerical configuration adjustment with no logic changes
• Single file modification affecting only resource allocation values
• No structural or behavioral changes to the system

Possibly related PRs

• [MOSIP-43631] Updated helm charts #468: Modifies the same helm/keymanager/values.yaml file with opposite resource value adjustments—directly related configuration change.

Suggested reviewers

• ckm007

Poem

🐰 Whiskers twitching with efficiency pride,
We trim the requests, the limits we guide,
Fewer CPUs, less memory to spend,
Leaner deployments—a rabbit's best friend! 🏃‍♂️✨

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	The title 'MOSIP-43631 updated helm charts' is vague and generic. While it references a specific issue, 'updated helm charts' does not clearly convey which charts were modified or what specific changes were made. The actual changes involve resource limit/request adjustments in the keymanager values.yaml, which is not evident from the title.	Revise the title to be more specific, such as 'MOSIP-43631 Reduce keymanager resource requests and limits' to clearly indicate the primary change.

✅ 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 c2665dc and 489a702.

📒 Files selected for processing (1)

• helm/keymanager/values.yaml (1 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)

• GitHub Check: build-kernel / maven-build
• GitHub Check: chart-lint-publish / chart-lint-publish

🔇 Additional comments (1)

helm/keymanager/values.yaml (1)

128-133: Verify resource reduction is validated for production workload.

The CPU request has been reduced by 75% (2000m → 500m) and memory request by 50% (6000Mi → 3000Mi). Before merging, confirm:

• Was this reduction based on production monitoring/profiling data?
• Has the application been load-tested with these reduced resources?
• Are there any performance SLO implications?

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}