Hello! my ballpark guess for this redesign completion is somewhere between 2 and 3 months. Maria On 3/25/22 4:45 PM, Trisha Biswas wrote:
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?
Thanks! Trisha --
*Trisha Biswas* | Sr. Software Engineer, Network Systems fastly.com <http://fastly.com/> | @fastly <https://twitter.com/fastly> | LinkedIn <http://www.linkedin.com/company/fastly>
On Fri, Mar 25, 2022 at 7:45 AM Ondrej Zajicek <santiago@crfreenet.org <mailto:santiago@crfreenet.org>> wrote:
On Thu, Mar 17, 2022 at 05:32:02PM -0600, Trisha Biswas wrote: > Add support for setting the TCP congestion control algorithm per > destination by > > modifying route attributes in BIRD. Previously supported kernel route > attributes > > in BIRD were all integer values. Subsequently, this also adds string > handling of > > kernel metrics.
Hi
We are currently redesigning attribute types in order to have direct matching between attribute and filter types. And considering that string type is not currently implemented in attributes and it is a special case that have a different representation in both (direct ptr vs adata), we will postpone this patch after the redesign and then merge relevant parts of your patch.
Otherwise, the patch is ok and the feature is worth merginh, i would just change it to add generic string type EAF_TYPE_STRING instead of EAF_TYPE_CC_ALGO.
-- Elen sila lumenn' omentielvo
Ondrej 'Santiago' Zajicek (email: santiago@crfreenet.org <mailto:santiago@crfreenet.org>) OpenPGP encrypted e-mails preferred (KeyID 0x11DEADC3, wwwkeys.pgp.net <http://wwwkeys.pgp.net>) "To err is human -- to blame it on a computer is even more so."