Skip to content

Conversation

alexmt
Copy link
Collaborator

@alexmt alexmt commented Nov 18, 2021

Signed-off-by: Alexander Matyushentsev AMatyushentsev@gmail.com

The #7274 reduced controller CPU usage by introducing project caching. This PR adds project cache uses in getResourceTree method - it was forgotten in #7274

Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
@codecov
Copy link

codecov bot commented Nov 18, 2021

Codecov Report

Merging #7747 (f01b57f) into master (8bbf888) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #7747   +/-   ##
=======================================
  Coverage   41.43%   41.43%           
=======================================
  Files         163      163           
  Lines       22096    22096           
=======================================
  Hits         9156     9156           
  Misses      11635    11635           
  Partials     1305     1305           
Impacted Files Coverage Δ
controller/appcontroller.go 53.03% <100.00%> (ø)
util/settings/settings.go 46.83% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8bbf888...f01b57f. Read the comment docs.

@alexmt alexmt merged commit 0c0bc1f into argoproj:master Nov 18, 2021
@alexmt alexmt deleted the proj-cache-resource-tree branch November 18, 2021 23:45
alexmt pushed a commit that referenced this pull request Nov 19, 2021
Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants