vmware-tanzu / sources-for-knative

VMware-related event sources for Knative.
Apache License 2.0
27 stars 24 forks source link

Remove vsphere memory limit #517

Closed joeeltgroth closed 1 year ago

joeeltgroth commented 1 year ago

It was discovered during Tanzu testing that the vsphere webhook pod was consuming more memory than the 200M limit. This PR removes the limit. A story has been created to analyze the code for opportunities to reduce memory consumption.

codecov-commenter commented 1 year ago

Codecov Report

Merging #517 (e1b1cdd) into main (c75f74f) will decrease coverage by 0.15%. The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #517      +/-   ##
==========================================
- Coverage   66.23%   66.08%   -0.15%     
==========================================
  Files          38       38              
  Lines        2079     2079              
==========================================
- Hits         1377     1374       -3     
- Misses        657      659       +2     
- Partials       45       46       +1     
Impacted Files Coverage Δ
pkg/vsphere/adapter.go 63.15% <0.00%> (-1.58%) :arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.