feat: enable minio webhook

[ambersun1234]

PR Type

Enhancement

---

Description

• Introduced a ConfigMap for MinIO webhook settings to enable webhook notifications.
• Added a new parameter extraEnvVarsCM in the MinIO configuration to reference the ConfigMap.
• Updated the MinIO configuration documentation for clarity.

---

Changes walkthrough 📝

Relevant files

Enhancement

agh-minio-cm.yml Add ConfigMap for MinIO webhook configuration                        charts/agh3/templates/minio/agh-minio-cm.yml Added a ConfigMap for MinIO webhook configuration. ConfigMap includes settings for enabling the webhook and specifying the endpoint. +9/-0      values.yaml Update MinIO values with extraEnvVarsCM parameter                charts/agh3/values.yaml Introduced a new parameter extraEnvVarsCM for MinIO configuration. Updated documentation to reflect the new configuration option. +3/-0

---

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

[ai-themis]

PR Reviewer Guide 🔍

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

🧪 No relevant tests

🔒 No security concerns identified

⚡ Key issues to review Code Smell The MINIO_NOTIFY_WEBHOOK_ENDPOINT_PRIMARY is constructed using a hardcoded URL pattern. Consider making this more configurable to accommodate different environments or URL structures.

[ai-themis]

PR Code Suggestions ✨

Category	Suggestion	Score
Best practice	Ensure the endpoint value is properly quoted to avoid parsing issues Ensure that the MINIO_NOTIFY_WEBHOOK_ENDPOINT_PRIMARY value is properly quoted to prevent any potential parsing issues with special characters or spaces. charts/agh3/templates/minio/agh-minio-cm.yml [8] -MINIO_NOTIFY_WEBHOOK_ENDPOINT_PRIMARY: {{ printf "http://captain.%s.svc.cluster.local:8080/internal/webhook/minio" .Release.Namespace | quote }} +MINIO_NOTIFY_WEBHOOK_ENDPOINT_PRIMARY: "{{ printf "http://captain.%s.svc.cluster.local:8080/internal/webhook/minio" .Release.Namespace | quote }}" Suggestion importance[1-10]: 8 Why: Quoting the endpoint value is a good practice to avoid parsing issues, especially with special characters. However, the current implementation already uses the quote function, which mitigates the risk, hence the score is not higher.	8

[knowlet]

Plz use helm template to verify.