Skip to content

Conversation

@vakalapa
Copy link
Contributor

Right now, network policy cache is updated even if there are errors below, this PR will make sure that cache is not updated if error occurs

neaggarwMS
neaggarwMS previously approved these changes Mar 19, 2021
Copy link
Member

@neaggarwMS neaggarwMS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@vakalapa vakalapa added the npm Related to NPM. label Mar 19, 2021
@codecov
Copy link

codecov bot commented Mar 19, 2021

Codecov Report

Merging #826 (005ec29) into master (3799358) will decrease coverage by 0.05%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #826      +/-   ##
==========================================
- Coverage   41.96%   41.90%   -0.06%     
==========================================
  Files         152      152              
  Lines       14462    14462              
==========================================
- Hits         6069     6061       -8     
- Misses       7661     7670       +9     
+ Partials      732      731       -1     

Copy link
Member

@neaggarwMS neaggarwMS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@vakalapa vakalapa merged commit 4df65da into master Mar 20, 2021
@vakalapa vakalapa deleted the vakr/fixingnetpol branch March 20, 2021 03:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

npm Related to NPM.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants