kong updates #2280

Merged

mfreeman451 merged 1 commit from refs/pull/2280/head into main

2025-10-05 18:31:44 +00:00

mfreeman451 commented

2025-10-05 18:31:07 +00:00

(Migrated from github.com)

Owner

Imported from GitHub pull request.

Original GitHub pull request: #1708
Original author: @mfreeman451
Original URL: https://github.com/carverauto/serviceradar/pull/1708
Original created: 2025-10-05T18:31:07Z
Original updated: 2025-10-05T18:32:10Z
Original head: carverauto/serviceradar:k8s/kong_oom_killed
Original base: main
Original merged: 2025-10-05T18:31:44Z by @mfreeman451

PR Type

Enhancement

Description

Increase Kong container resource limits and requests
Configure nginx worker processes for better performance

Diagram Walkthrough

flowchart LR
  A["Kong Container"] --> B["Resource Limits"]
  A --> C["Worker Processes"]
  B --> D["CPU: 250m→500m (req), 1→2 (limit)"]
  B --> E["Memory: 512Mi→1Gi (req), 1Gi→2Gi (limit)"]
  C --> F["NGINX Workers: auto→2"]

File Walkthrough

Relevant files

Enhancement

serviceradar-kong.yaml

Kong resource optimization and worker configuration

k8s/demo/base/serviceradar-kong.yaml

Added KONG_NGINX_WORKER_PROCESSES environment variable set to "2"
Increased CPU requests from 250m to 500m and limits from 1 to 2
Increased memory requests from 512Mi to 1Gi and limits to 2Gi

+6/-4

Imported from GitHub pull request. Original GitHub pull request: #1708 Original author: @mfreeman451 Original URL: https://github.com/carverauto/serviceradar/pull/1708 Original created: 2025-10-05T18:31:07Z Original updated: 2025-10-05T18:32:10Z Original head: carverauto/serviceradar:k8s/kong_oom_killed Original base: main Original merged: 2025-10-05T18:31:44Z by @mfreeman451 --- ### **PR Type** Enhancement ___ ### **Description** - Increase Kong container resource limits and requests - Configure nginx worker processes for better performance ___ ### Diagram Walkthrough ```mermaid flowchart LR A["Kong Container"] --> B["Resource Limits"] A --> C["Worker Processes"] B --> D["CPU: 250m→500m (req), 1→2 (limit)"] B --> E["Memory: 512Mi→1Gi (req), 1Gi→2Gi (limit)"] C --> F["NGINX Workers: auto→2"] ``` <details> <summary><h3> File Walkthrough</h3></summary> <table><thead><tr><th></th><th align="left">Relevant files</th></tr></thead><tbody><tr><td><strong>Enhancement</strong></td><td><table> <tr> <td> <details> <summary><strong>serviceradar-kong.yaml</strong><dd><code>Kong resource optimization and worker configuration</code>            </dd></summary> <hr> k8s/demo/base/serviceradar-kong.yaml <ul><li>Added <code>KONG_NGINX_WORKER_PROCESSES</code> environment variable set to "2"<br> <li> Increased CPU requests from 250m to 500m and limits from 1 to 2<br> <li> Increased memory requests from 512Mi to 1Gi and limits to 2Gi</ul> </details> </td> <td><a href="https://github.com/carverauto/serviceradar/pull/1708/files#diff-e1426b570f5caae8eee31f02984392f1bc68d0c329ae2f1118de3272d654856e">+6/-4</a>      </td> </tr> </table></td></tr></tr></tbody></table> </details> ___

qodo-code-review[bot] commented

2025-10-05 18:31:25 +00:00

(Migrated from github.com)

Author

Owner

Imported GitHub PR comment.

Original author: @qodo-code-review[bot]
Original URL: https://github.com/carverauto/serviceradar/pull/1708#issuecomment-3369239946
Original created: 2025-10-05T18:31:25Z

PR Compliance Guide 🔍

Below is a summary of compliance checks for this PR:

Security Compliance
🟢	No security concerns identified No security vulnerabilities detected by AI analysis. Human verification advised for critical code.
Ticket Compliance
⚪	🎫 No ticket provided Create ticket/issue
Codebase Duplication Compliance
⚪	Codebase context is not defined Follow the guide to enable codebase context checks.
Custom Compliance
⚪	No custom compliance provided Follow the guide to enable custom compliance check.

Compliance status legend

🟢 - Fully Compliant
🟡 - Partial Compliant
🔴 - Not Compliant
⚪ - Requires Further Human Verification
🏷️ - Compliance label

Imported GitHub PR comment. Original author: @qodo-code-review[bot] Original URL: https://github.com/carverauto/serviceradar/pull/1708#issuecomment-3369239946 Original created: 2025-10-05T18:31:25Z --- ## PR Compliance Guide 🔍  Below is a summary of compliance checks for this PR:<br> <table><tbody><tr><td colspan='2'><strong>Security Compliance</strong></td></tr> <tr><td>🟢</td><td><details><summary><strong>No security concerns identified</strong></summary> No security vulnerabilities detected by AI analysis. Human verification advised for critical code. </details></td></tr> <tr><td colspan='2'><strong>Ticket Compliance</strong></td></tr> <tr><td>⚪</td><td><details><summary>🎫 <strong>No ticket provided </summary></strong> - [ ] Create ticket/issue  </details></td></tr> <tr><td colspan='2'><strong>Codebase Duplication Compliance</strong></td></tr> <tr><td>⚪</td><td><details><summary><strong>Codebase context is not defined </strong></summary> Follow the <a href='https://qodo-merge-docs.qodo.ai/core-abilities/rag_context_enrichment/'>guide</a> to enable codebase context checks. </details></td></tr> <tr><td colspan='2'><strong>Custom Compliance</strong></td></tr> <tr><td>⚪</td><td><details><summary><strong>No custom compliance provided</strong></summary> Follow the <a href='https://qodo-merge-docs.qodo.ai/tools/compliance/'>guide</a> to enable custom compliance check. </details></td></tr> <tr><td align="center" colspan="2">   </td></tr></tbody></table> <details><summary>Compliance status legend</summary> 🟢 - Fully Compliant<br> 🟡 - Partial Compliant<br> 🔴 - Not Compliant<br> ⚪ - Requires Further Human Verification<br> 🏷️ - Compliance label<br> </details>

qodo-code-review[bot] commented

2025-10-05 18:32:10 +00:00

(Migrated from github.com)

Author

Owner

Imported GitHub PR comment.

Original author: @qodo-code-review[bot]
Original URL: https://github.com/carverauto/serviceradar/pull/1708#issuecomment-3369240409
Original created: 2025-10-05T18:32:10Z

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Impact
General	Use `auto` for worker process configuration Set the `KONG_NGINX_WORKER_PROCESSES` value to `auto` instead of the hardcoded `2`. This will automatically adjust the number of worker processes based on the available CPU cores, improving flexibility and maintainability. k8s/demo/base/serviceradar-kong.yaml [55-56] `- name: KONG_NGINX_WORKER_PROCESSES - value: "2" + value: "auto"` Apply / Chat Suggestion importance[1-10]: 6 __ Why: The suggestion correctly proposes using `auto` for `KONG_NGINX_WORKER_PROCESSES`, which improves maintainability by automatically aligning worker processes with the CPU limit, making the configuration more robust to future resource changes.	Low
More

Imported GitHub PR comment. Original author: @qodo-code-review[bot] Original URL: https://github.com/carverauto/serviceradar/pull/1708#issuecomment-3369240409 Original created: 2025-10-05T18:32:10Z --- ## PR Code Suggestions ✨  Explore these optional code suggestions: <table><thead><tr><td><strong>Category</strong></td><td align=left><strong>Suggestion                                                                                                                                    </strong></td><td align=center><strong>Impact</strong></td></tr><tbody><tr><td rowspan=1>General</td> <td> <details><summary>Use <code>auto</code> for worker process configuration</summary> ___ **Set the <code>KONG_NGINX_WORKER_PROCESSES</code> value to <code>auto</code> instead of the hardcoded <code>2</code>. <br>This will automatically adjust the number of worker processes based on the <br>available CPU cores, improving flexibility and maintainability.** [k8s/demo/base/serviceradar-kong.yaml [55-56]](https://github.com/carverauto/serviceradar/pull/1708/files#diff-e1426b570f5caae8eee31f02984392f1bc68d0c329ae2f1118de3272d654856eR55-R56) ```diff - name: KONG_NGINX_WORKER_PROCESSES - value: "2" + value: "auto" ``` - [ ] **Apply / Chat**  <details><summary>Suggestion importance[1-10]: 6</summary> __ Why: The suggestion correctly proposes using `auto` for `KONG_NGINX_WORKER_PROCESSES`, which improves maintainability by automatically aligning worker processes with the CPU limit, making the configuration more robust to future resource changes. </details></details></td><td align=center>Low </td></tr> <tr><td align="center" colspan="2"> - [ ] More  </td><td></td></tr></tbody></table>