Skip to content

Conversation

@tamilmani1989
Copy link
Member

What this PR does / why we need it:
Disable RA for interfaces created by CNI to address potential security issues.

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #

Special notes for your reviewer:

Release note:

@codecov
Copy link

codecov bot commented May 26, 2020

Codecov Report

Merging #567 into master will decrease coverage by 9.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #567      +/-   ##
==========================================
- Coverage   56.89%   47.85%   -9.05%     
==========================================
  Files          28       27       -1     
  Lines        3438     3281     -157     
==========================================
- Hits         1956     1570     -386     
+ Misses       1456     1434      -22     
- Partials       26      277     +251     

Copy link

@chandanAggarwal chandanAggarwal left a comment

Choose a reason for hiding this comment

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

:shipit:

@tamilmani1989 tamilmani1989 merged commit 6515fae into Azure:master May 26, 2020
@tamilmani1989 tamilmani1989 deleted the disable_ra branch May 26, 2020 22:22
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