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 <santi...@crfreenet.org <mailto:santi...@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: santi...@crfreenet.org
    <mailto:santi...@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."

Reply via email to