<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<meta name="Generator" content="Microsoft Word 15 (filtered medium)">
<!--[if !mso]><style>v\:* {behavior:url(#default#VML);}
o\:* {behavior:url(#default#VML);}
w\:* {behavior:url(#default#VML);}
.shape {behavior:url(#default#VML);}
</style><![endif]--><style><!--
/* Font Definitions */
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0in;
margin-bottom:.0001pt;
font-size:11.0pt;
font-family:"Calibri",sans-serif;}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
{mso-style-priority:99;
color:purple;
text-decoration:underline;}
p.MsoListParagraph, li.MsoListParagraph, div.MsoListParagraph
{mso-style-priority:34;
margin-top:0in;
margin-right:0in;
margin-bottom:0in;
margin-left:.5in;
margin-bottom:.0001pt;
font-size:11.0pt;
font-family:"Calibri",sans-serif;}
p.msonormal0, li.msonormal0, div.msonormal0
{mso-style-name:msonormal;
mso-style-priority:99;
mso-margin-top-alt:auto;
margin-right:0in;
mso-margin-bottom-alt:auto;
margin-left:0in;
font-size:11.0pt;
font-family:"Calibri",sans-serif;}
span.EmailStyle20
{mso-style-type:personal;
font-family:"Calibri",sans-serif;
color:windowtext;}
span.EmailStyle21
{mso-style-type:personal;
font-family:"Calibri",sans-serif;
color:windowtext;}
span.EmailStyle22
{mso-style-type:personal;
font-family:"Calibri",sans-serif;
color:windowtext;}
span.EmailStyle23
{mso-style-type:personal-reply;
font-family:"Calibri",sans-serif;
color:windowtext;}
.MsoChpDefault
{mso-style-type:export-only;
font-size:10.0pt;}
@page WordSection1
{size:8.5in 11.0in;
margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
{page:WordSection1;}
/* List Definitions */
@list l0
{mso-list-id:742996091;
mso-list-type:hybrid;
mso-list-template-ids:281159400 67698703 67698713 67698715 67698703 67698713 67698715 67698703 67698713 67698715;}
@list l0:level1
{mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;}
@list l0:level2
{mso-level-number-format:alpha-lower;
mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;}
@list l0:level3
{mso-level-number-format:roman-lower;
mso-level-tab-stop:none;
mso-level-number-position:right;
text-indent:-9.0pt;}
@list l0:level4
{mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;}
@list l0:level5
{mso-level-number-format:alpha-lower;
mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;}
@list l0:level6
{mso-level-number-format:roman-lower;
mso-level-tab-stop:none;
mso-level-number-position:right;
text-indent:-9.0pt;}
@list l0:level7
{mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;}
@list l0:level8
{mso-level-number-format:alpha-lower;
mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;}
@list l0:level9
{mso-level-number-format:roman-lower;
mso-level-tab-stop:none;
mso-level-number-position:right;
text-indent:-9.0pt;}
@list l1
{mso-list-id:1340162855;
mso-list-template-ids:-802668528;}
@list l1:level2
{mso-level-number-format:alpha-lower;
mso-level-tab-stop:1.0in;
mso-level-number-position:left;
text-indent:-.25in;}
@list l2
{mso-list-id:1605963711;
mso-list-template-ids:1981581804;}
@list l3
{mso-list-id:1954705536;
mso-list-type:hybrid;
mso-list-template-ids:1930717972 67698703 67698713 67698715 67698703 67698713 67698715 67698703 67698713 67698715;}
@list l3:level1
{mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;}
@list l3:level2
{mso-level-number-format:alpha-lower;
mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;}
@list l3:level3
{mso-level-number-format:roman-lower;
mso-level-tab-stop:none;
mso-level-number-position:right;
text-indent:-9.0pt;}
@list l3:level4
{mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;}
@list l3:level5
{mso-level-number-format:alpha-lower;
mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;}
@list l3:level6
{mso-level-number-format:roman-lower;
mso-level-tab-stop:none;
mso-level-number-position:right;
text-indent:-9.0pt;}
@list l3:level7
{mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;}
@list l3:level8
{mso-level-number-format:alpha-lower;
mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;}
@list l3:level9
{mso-level-number-format:roman-lower;
mso-level-tab-stop:none;
mso-level-number-position:right;
text-indent:-9.0pt;}
ol
{margin-bottom:0in;}
ul
{margin-bottom:0in;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="EN-US" link="blue" vlink="purple">
<div class="WordSection1">
<p class="MsoNormal">Jay,<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Configured distance is already considered by Zebra/RTM and that design stays as-is. Weight/Local preference alone are not enough to address this issue. Deterministic in the sense, given a set of events, FRR stack will download same route
to RIB/FIB irrespective of sequence in which the events happened. <o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Some of the details of the discussion were captured in the bug comments. Please check for more details.
<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Thanks,<o:p></o:p></p>
<p class="MsoNormal">Mohan<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b><span style="font-size:12.0pt;color:black">From: </span></b><span style="font-size:12.0pt;color:black">Jia Chen <jchen1@paloaltonetworks.com><br>
<b>Date: </b>Monday, September 10, 2018 at 10:08 AM<br>
<b>To: </b>Mohan Singamsetty <msingamsetty@vmware.com>, Donald Sharp <sharpd@cumulusnetworks.com>, "ntriantafillis@gmail.com" <ntriantafillis@gmail.com>, FRRouting-Dev <dev@lists.frrouting.org>, Don Slice <dslice@cumulusnetworks.com>, Russ White <russ@riw.us><br>
<b>Subject: </b>Re: [dev] [FRRouting/frr] Static blackhole route does not honor distance attribute (#2230)<o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<p class="MsoNormal">Just a question, <o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">Should configured distance be considered by Zebra/RTM stage rather than inside of BGP best path selection. If BGP best path selection need to be influenced, why using weight, local preference etc not enough?<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">In another note, the deterministic behavior, are we talking about ordered/prioritized FIB downloading depending on route types?<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">Thanks,<o:p></o:p></p>
<p class="MsoNormal">Jay<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">Configured distance<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b><span style="font-size:12.0pt;color:black">From: </span></b><span style="font-size:12.0pt;color:black">dev <dev-bounces@lists.frrouting.org> on behalf of Mohan Singamsetty <msingamsetty@vmware.com><br>
<b>Date: </b>Monday, September 10, 2018 at 9:40 AM<br>
<b>To: </b>Donald Sharp <sharpd@cumulusnetworks.com>, "ntriantafillis@gmail.com" <ntriantafillis@gmail.com>, FRRouting-Dev <dev@lists.frrouting.org>, Don Slice <dslice@cumulusnetworks.com>, Russ White <russ@riw.us><br>
<b>Subject: </b>Re: [dev] [FRRouting/frr] Static blackhole route does not honor distance attribute (#2230)</span><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"> <o:p></o:p></p>
</div>
<p class="MsoNormal">We had this meeting to discuss issue #2230 and conclude on the fix.
<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal"><u>Date: </u>09-05-2018<o:p></o:p></p>
<p class="MsoNormal"><u>Attendees: </u>Donald, Nikos, Don, Mohan, Soman.<o:p></o:p></p>
<p class="MsoNormal"><u>Minutes: </u><o:p></o:p></p>
<ol style="margin-top:0in" start="1" type="1">
<li class="MsoNormal" style="mso-list:l3 level1 lfo3">Two issues were discussed. <o:p>
</o:p>
<ol style="margin-top:0in" start="1" type="a">
<li class="MsoNormal" style="mso-list:l3 level2 lfo3"><span style="color:black">FRR stack not being deterministic in downloading routes to RIB/FIB and the routes downloaded to RIB/FIB depends on sequence of commands executed or sequence of network events happened.
Ideally the routes in RIB/FIB should be deterministic and consistent at a given time after executing set of same commands.</span><o:p></o:p></li><li class="MsoNormal" style="mso-list:l3 level2 lfo3"><span style="color:black">FRR stack not honoring the admin distance configured for routes and downloading routes with higher admin distance (higher in numerical value). Ideally routes with lower admin distances
should be considered as best routes and should get downloaded to RIB/FIB.</span><o:p></o:p></li></ol>
</li><li class="MsoNormal" style="color:black;mso-list:l3 level1 lfo3"><span style="color:windowtext">Two</span> Proposals were considered.
<o:p></o:p>
<ol style="margin-top:0in" start="1" type="a">
<li class="MsoNormal" style="color:black;mso-list:l3 level2 lfo3">Consider admin distance configured for each route in BGP best bath algorithm to make FRR stack behave more deterministic and consistent with respect to downloading routes to RIB/FIB.<o:p></o:p></li><li class="MsoNormal" style="color:black;mso-list:l3 level2 lfo3">Use route maps to control the routes to be downloaded to RIB/FIB depending on the requirement and use case.<o:p></o:p></li></ol>
</li></ol>
<p class="MsoNormal"><b><span style="color:black"> </span></b><o:p></o:p></p>
<p class="MsoNormal"><u><span style="color:black">Conclusion:</span></u><o:p></o:p></p>
<ol style="margin-top:0in" start="1" type="1">
<li class="MsoNormal" style="color:black;mso-list:l0 level1 lfo7">It was concluded that the FRR stack behavior should be deterministic even without using any route maps (without any additional configuration from user).
<o:p></o:p></li><li class="MsoNormal" style="color:black;mso-list:l0 level1 lfo7">Team agreed to use proposal #1 (Using the admin distance of route/protocol inside the BGP best path algorithm to determine the best route) to move forward and address the above issues mentioned.
<o:p></o:p></li><li class="MsoNormal" style="color:black;mso-list:l0 level1 lfo7">Admin distance will be the first thing to compare in FRR best bath path algorithm. This is in addition to the other 13 checks that FRR stack currently has.<o:p></o:p></li><li class="MsoNormal" style="color:black;mso-list:l0 level1 lfo7">There will be a #def to control the default behavior from build time. i.e. either to have the old behavior where admin distance of each route is not used inside the BGP best path algorithm or
to have new behavior where admin distance is used inside BGP best path algorithm to fix the above issue. User can use this #def to build the FRR stack and get the default behavior that is needed.<o:p></o:p></li><li class="MsoNormal" style="color:black;mso-list:l0 level1 lfo7">There will be a CLI command also to control the default behavior as explained above.
<o:p></o:p></li></ol>
<p class="MsoNormal"><span style="color:black"> </span><o:p></o:p></p>
<p class="MsoNormal"><span style="color:black">We will publish the design and start implementing this soon.</span><o:p></o:p></p>
<p class="MsoNormal"><span style="color:black"> </span><o:p></o:p></p>
<p class="MsoNormal"><span style="color:black">Guys, please add more if I missed to add anything here.</span><o:p></o:p></p>
<p class="MsoNormal"><span style="color:black"> </span><o:p></o:p></p>
<p class="MsoNormal"><span style="color:black">Thanks,</span><o:p></o:p></p>
<p class="MsoNormal"><span style="color:black">Mohan</span><o:p></o:p></p>
<p class="MsoNormal"><b><span style="color:black"> </span></b><o:p></o:p></p>
<div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b><span style="font-size:12.0pt;color:black">From: </span></b><span style="font-size:12.0pt;color:black">Mohan Singamsetty <msingamsetty@vmware.com><br>
<b>Date: </b>Tuesday, September 4, 2018 at 10:15 AM<br>
<b>To: </b>Donald Sharp <sharpd@cumulusnetworks.com>, "ntriantafillis@gmail.com" <ntriantafillis@gmail.com>, FRRouting-Dev <dev@lists.frrouting.org><br>
<b>Subject: </b>Re: [FRRouting/frr] Static blackhole route does not honor distance attribute (#2230)</span><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"> <o:p></o:p></p>
</div>
<p class="MsoNormal">I am setting up a meeting to discuss this further on Wednesday (5<sup>th</sup> September) at 8 AM PST.
<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">Link for the bug and comments<o:p></o:p></p>
<p class="MsoNormal"><a href="https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_FRRouting_frr_issues_2230&d=DwMGaQ&c=V9IgWpI5PvzTw83UyHGVSoW3Uc1MFWe5J8PTfkrzVSo&r=yetdj-aXQpuqTCJGs-93hOpK3740MIRXowfUNLByeos&m=2GR-5s5J4kjmjx-7MISqLe_dGLwqLf5wbETjbxxeoxI&s=Q6hfhZOXRfX1tODyMCO4XTdy800SHHjHrUIK4Py1rJw&e=">https://github.com/FRRouting/frr/issues/2230</a><o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">Donald, Nikos, Russ, Please let me know if this time works for you.
<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">If anyone else is interested to join, please let me know and I will add you to the meeting.
<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">Thanks,<o:p></o:p></p>
<p class="MsoNormal">Mohan<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b><span style="font-size:12.0pt;color:black">From: </span></b><span style="font-size:12.0pt;color:black">David Lamparter <notifications@github.com><br>
<b>Reply-To: </b>FRRouting/frr <reply+009b114fa2e0109bec00750471786212a516d3548fe6032192cf0000000117a665b892a169ce133f1a52@reply.github.com><br>
<b>Date: </b>Tuesday, September 4, 2018 at 8:24 AM<br>
<b>To: </b>FRRouting/frr <frr@noreply.github.com><br>
<b>Cc: </b>Mohan Singamsetty <msingamsetty@vmware.com>, Comment <comment@noreply.github.com><br>
<b>Subject: </b>Re: [FRRouting/frr] Static blackhole route does not honor distance attribute (#2230)</span><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"> <o:p></o:p></p>
</div>
<p>=> meeting this week<br>
=> rough consensus on call is that deterministic behaviour is desired<o:p></o:p></p>
<p style="-webkit-text-size-adjust:none"><span style="font-size:12.0pt;color:#666666">—<br>
You are receiving this because you commented.<br>
Reply to this email directly, <a href="https://urldefense.proofpoint.com/v2/url?u=https-3A__na01.safelinks.protection.outlook.com_-3Furl-3Dhttps-253A-252F-252Fgithub.com-252FFRRouting-252Ffrr-252Fissues-252F2230-2523issuecomment-2D418410028-26data-3D02-257C01-257Cmsingamsetty-2540vmware.com-257C28b1ef01464842e1682108d6127a89cc-257Cb39138ca3cee4b4aa4d6cd83d9dd62f0-257C1-257C0-257C636716714843490106-26sdata-3Dym5mVNjS29ABjocdXpheidJkzQkWsOm-252FfmNcV0ufsqw-253D-26reserved-3D0&d=DwMGaQ&c=V9IgWpI5PvzTw83UyHGVSoW3Uc1MFWe5J8PTfkrzVSo&r=yetdj-aXQpuqTCJGs-93hOpK3740MIRXowfUNLByeos&m=2GR-5s5J4kjmjx-7MISqLe_dGLwqLf5wbETjbxxeoxI&s=gbNmQUun3oOTQQt0Sba_CJITXGHNljfgtIve2sRIH4A&e=">
view it on GitHub</a>, or <a href="https://urldefense.proofpoint.com/v2/url?u=https-3A__na01.safelinks.protection.outlook.com_-3Furl-3Dhttps-253A-252F-252Fgithub.com-252Fnotifications-252Funsubscribe-2Dauth-252FAJsRTzWjPRVYKQ-2DuSqAOgQOM-5FQeVtuZRks5uXps4gaJpZM4T-2DK0s-26data-3D02-257C01-257Cmsingamsetty-2540vmware.com-257C28b1ef01464842e1682108d6127a89cc-257Cb39138ca3cee4b4aa4d6cd83d9dd62f0-257C1-257C0-257C636716714843490106-26sdata-3DadeYV-252Bfejtz6ncdZ-252B7lUclAGFwBalaWmeDVE2ZMSaq8-253D-26reserved-3D0&d=DwMGaQ&c=V9IgWpI5PvzTw83UyHGVSoW3Uc1MFWe5J8PTfkrzVSo&r=yetdj-aXQpuqTCJGs-93hOpK3740MIRXowfUNLByeos&m=2GR-5s5J4kjmjx-7MISqLe_dGLwqLf5wbETjbxxeoxI&s=Sno6noq1ljalx4mkmzVArUJq2ntADhIeemletWtTJpI&e=">
mute the thread</a>.<span style="border:solid windowtext 1.0pt;padding:0in"><img border="0" width="1" height="1" style="width:.0104in;height:.0104in" id="_x0000_i1025" src="cid:~WRD000.jpg" alt="mage removed by sender."></span></span><o:p></o:p></p>
</div>
</body>
</html>