refs platform/3099: update ingress-nginx to 1.11.2

[Stevesibilia]

PR Type

Enhancement

---

Description

• Updated Ingress-Nginx chart version from 4.10.1 to 4.11.2 to address CVE-2024-7646 (kubernetes issue #126744)
• Modified variables.tf to reflect the new default chart version
• Updated CHANGELOG.md with new version 0.7.0 and details about the security fix
• Updated README.md to show the new default chart version in the inputs table
• Updated reference URL in files/values.yaml.tftpl to point to the new chart version

---

Changes walkthrough 📝

	Relevant files
Enhancement	variables.tf Update Ingress-Nginx chart version                                              variables.tf Updated default chart_version from "4.10.1" to "4.11.2" +1/-1
Documentation	CHANGELOG.md Update CHANGELOG with new version and security fix              CHANGELOG.md Added new entry for version 0.7.0 Documented update to Ingress-nginx 1.11.2 Mentioned fix for CVE-2024-7646 +8/-0      README.md Update README with new chart version                                          README.md Updated default value for chart_version input variable from "4.10.1" to "4.11.2" +1/-1      values.yaml.tftpl Update reference URL for values.yaml                                          files/values.yaml.tftpl Updated reference URL for values.yaml from helm-chart-4.10.1 to helm-chart-4.11.2 +1/-1

---

💡 PR-Agent usage:
Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

[sparkfabrik-ai-bot]

PR Reviewer Guide 🔍

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪

🧪 No relevant tests

🔒 No security concerns identified

⚡ No key issues to review

[sparkfabrik-ai-bot]

PR Code Suggestions ✨

Category	Suggestion	Score
Enhancement	Expand the changelog entry with more details about the changes in the new version Consider adding more details about the changes introduced in version 0.7.0. For example, you could mention any specific improvements or new features that come with the updated Ingress-nginx version, beyond just addressing the CVE. CHANGELOG.md [15-17] ### Changed -- Update to use Ingress-nginx 1.11.2 to solve CVE-2024-7646 (https://github.com/kubernetes/kubernetes/issues/126744) +- Update to Ingress-nginx 1.11.2: + - Addresses CVE-2024-7646 (https://github.com/kubernetes/kubernetes/issues/126744) + - Improves [specific feature or performance aspect] + - Adds support for [new capability, if applicable] Suggestion importance[1-10]: 7 Why: The suggestion provides valuable additional context about the update, which can be helpful for users. However, it's not crucial for the functionality of the code.	7
Maintainability	Add a descriptive comment to explain the purpose and usage of the file Consider adding a brief comment explaining the purpose of this file and how it's used in the context of the module. This can help other developers understand its role more quickly. files/values.yaml.tftpl [1] -# https://github.com/kubernetes/ingress-nginx/blob/helm-chart-4.11.2/charts/ingress-nginx/values.yaml +# This file contains the default values for the Ingress-nginx Helm chart. +# It's used as a template for customizing the chart installation. +# Reference: https://github.com/kubernetes/ingress-nginx/blob/helm-chart-4.11.2/charts/ingress-nginx/values.yaml Suggestion importance[1-10]: 6 Why: The suggestion improves code documentation and maintainability, which is beneficial but not critical for the code's functionality.	6

[Monska85]

LGTM 👍