<div dir="ltr">Thanks for getting back, Ondrej. Adding a generic EAF_TYPE_STRING, and merging after the redesign both sound good. Any idea what timeframe this redesign will happen in, so that I can keep an eye on when the patch gets merged? <div><br></div><div>Thanks!</div><div>Trisha<br clear="all"><div><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><span style="color:rgb(136,136,136)">--</span><br style="color:rgb(136,136,136)"><div dir="ltr" style="color:rgb(136,136,136)"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div style="margin:0px;padding:0px;color:rgb(51,51,51);font-family:Arial,sans-serif;font-size:14px;line-height:20px"><img src="https://www.fastly.com/img/sig.png"><br></div><div style="margin:0px;padding:0px;color:rgb(51,51,51);font-family:Arial,sans-serif;font-size:14px;line-height:20px"><b>Trisha Biswas</b> | Sr. Software Engineer, Network Systems</div><div style="margin:0px;padding:0px;color:rgb(51,51,51);font-family:Arial,sans-serif;font-size:14px;line-height:20px"><a href="http://fastly.com/" rel="nofollow" style="color:rgb(59,115,175)" target="_blank">fastly.com</a> | <a href="https://twitter.com/fastly" rel="nofollow" style="color:rgb(59,115,175)" target="_blank">@fastly</a> | <a href="http://www.linkedin.com/company/fastly" rel="nofollow" style="color:rgb(59,115,175)" target="_blank">LinkedIn</a></div></div></div></div></div></div></div></div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Mar 25, 2022 at 7:45 AM Ondrej Zajicek <<a href="mailto:santiago@crfreenet.org">santiago@crfreenet.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Thu, Mar 17, 2022 at 05:32:02PM -0600, Trisha Biswas wrote:<br>
> Add support for setting the TCP congestion control algorithm per<br>
> destination by<br>
> <br>
> modifying route attributes in BIRD. Previously supported kernel route<br>
> attributes<br>
> <br>
> in BIRD were all integer values. Subsequently, this also adds string<br>
> handling of<br>
> <br>
> kernel metrics.<br>
<br>
Hi<br>
<br>
We are currently redesigning attribute types in order to have direct<br>
matching between attribute and filter types. And considering that string<br>
type is not currently implemented in attributes and it is a special case<br>
that have a different representation in both (direct ptr vs adata), we will<br>
postpone this patch after the redesign and then merge relevant parts of<br>
your patch.<br>
<br>
Otherwise, the patch is ok and the feature is worth merginh, i would just<br>
change it to add generic string type EAF_TYPE_STRING instead of<br>
EAF_TYPE_CC_ALGO.<br>
<br>
-- <br>
Elen sila lumenn' omentielvo<br>
<br>
Ondrej 'Santiago' Zajicek (email: <a href="mailto:santiago@crfreenet.org" target="_blank">santiago@crfreenet.org</a>)<br>
OpenPGP encrypted e-mails preferred (KeyID 0x11DEADC3, <a href="http://wwwkeys.pgp.net" rel="noreferrer" target="_blank">wwwkeys.pgp.net</a>)<br>
"To err is human -- to blame it on a computer is even more so."<br>
</blockquote></div>