argoproj-labs / argocd-operator

A Kubernetes operator for managing Argo CD clusters.
https://argocd-operator.readthedocs.io
Apache License 2.0
637 stars 722 forks source link

[ Operator Redesign ] fix: container ResourceRequirements reconciliation #1431

Closed anandrkskd closed 3 months ago

anandrkskd commented 3 months ago

What type of PR is this? /kind bug

What does this PR do / why we need it: Before this PR, container resource didn't reconcile due to the type mismatch for func GetValueOrDefault, This change returns the resource requirement if it's not nil, And return empty/default resource requirement if nil.

Have you updated the necessary documentation?

Which issue(s) this PR fixes:

Fixes #?

How to test changes / Special notes to the reviewer: