Skip to content

Conversation

@neaggarwMS
Copy link
Member

Update NPM logs to use Logf statements instead of Printf as those are less CPU intensive. Also fixed few logs to avoid printing the old NetworkPolicy objects as that could potentially impact specially for large networkPolicy object. There are log lines when each rule is processed which can be used for debugging.

@codecov
Copy link

codecov bot commented Jul 6, 2020

Codecov Report

Merging #601 into master will decrease coverage by 0.14%.
The diff coverage is 38.88%.

@@            Coverage Diff             @@
##           master     #601      +/-   ##
==========================================
- Coverage   38.05%   37.90%   -0.15%     
==========================================
  Files          41       46       +5     
  Lines        4904     5021     +117     
==========================================
+ Hits         1866     1903      +37     
- Misses       2783     2859      +76     
- Partials      255      259       +4     

@neaggarwMS neaggarwMS merged commit cf25cd3 into Azure:master Jul 6, 2020
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