feat: allow changing the name of container in audit and controlle manager

[mozillazg]

What this PR does / why we need it:

Which issue(s) this PR fixes (optional, using fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when the PR gets merged):
Fixes #3756

Special notes for your reviewer:

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 47.65%. Comparing base (3350319) to head (705c76b).
Report is 221 commits behind head on master.

❗ There is a different number of reports uploaded between BASE (3350319) and HEAD (705c76b). Click for more details.

HEAD has 1 upload less than BASE

Flag	BASE (3350319)	HEAD (705c76b)
unittests	2	1

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #3759      +/-   ##
==========================================
- Coverage   54.49%   47.65%   -6.85%     
==========================================
  Files         134      236     +102     
  Lines       12329    19924    +7595     
==========================================
+ Hits         6719     9494    +2775     
- Misses       5116     9533    +4417     
- Partials      494      897     +403     

Flag	Coverage Δ
unittests	47.65% <ø> (-6.85%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[JaydipGabani]

LGTM

[NoamY8]

@CodiumAI-Agent /review

[CodiumAI-Agent]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🎫 Ticket compliance analysis ✅ 3756 - Fully compliant Fully compliant requirements: Allow overriding the container name for the audit component in the Helm chart. Allow overriding the container name for the controller manager component in the Helm chart. Not compliant requirements: []

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

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Possible Misconfiguration Ensure that the new containerName values are properly validated and do not lead to misconfigurations or conflicts in deployments. - op: replace path: /spec/template/spec/containers/0/name value: "{{ .Values.audit.containerName }}" - op: remove path: /spec/template/spec/containers/0/resources/limits - op: remove path: /spec/template/spec/containers/0/resources/requests - op: remove path: /spec/template/spec/nodeSelector/kubernetes.io~1os - op: remove path: /spec/template/spec/containers/0/image - op: remove path: /spec/template/spec/containers/0/securityContext/allowPrivilegeEscalation - op: remove path: /spec/template/spec/containers/0/securityContext/capabilities - op: remove path: /spec/template/spec/containers/0/securityContext/readOnlyRootFilesystem - op: remove path: /spec/template/spec/containers/0/securityContext/runAsGroup - op: remove path: /spec/template/spec/containers/0/securityContext/runAsNonRoot - op: remove path: /spec/template/spec/containers/0/securityContext/runAsUser - op: remove path: /spec/template/spec/containers/0/securityContext/seccompProfile - target: kind: Deployment name: gatekeeper-controller-manager patch: |- - op: replace path: /spec/template/spec/containers/0/name value: "{{ .Values.controllerManager.containerName }}" Helm Template Consistency Verify that the containerName field is correctly templated and resolves properly in all deployment scenarios. name: '{{ .Values.audit.containerName }}' Helm Template Consistency Verify that the containerName field is correctly templated and resolves properly in all deployment scenarios. name: '{{ .Values.controllerManager.containerName }}'