[pve-devel] applied: [PATCH frr] use 7.2 tag for submodule

[Alexandre DERUMIER]

>>As long as we have the features available we need?
Yes, the 7.1 have all needed features.


>>Hmm, but not that you're the only one testing newest tags, 
>>and once we switch the stable branch will be less stable too :D 

I'm hoping for a 7.1.x tag soon, the minor tag are done from the stable branch.
They are backporting only bug fix or security on stable branch, so I think it shouldn't break. 
(Currently, I never have seen breaking on stable branch with updates)


>>OK for me. 
>>There's also a stable 7.2 branch, with a few fixes since 7.2 tag[0], 
>>or is this also not yet working? 

No, still broken :/ 


----- Mail original -----
De: "Thomas Lamprecht" <t.lamprecht at proxmox.com>
À: "aderumier" <aderumier at odiso.com>
Cc: "pve-devel" <pve-devel at pve.proxmox.com>
Envoyé: Mardi 26 Novembre 2019 08:27:23
Objet: Re: applied: [pve-devel] [PATCH frr] use 7.2 tag for submodule

Hi Alexandre, 

On 11/26/19 8:08 AM, Alexandre DERUMIER wrote: 
> Hi Thomas, 
> 
> I have done test with 7.2, and it seem that it's again broken with evpn 
> I have opened an issue https://github.com/FRRouting/frr/issues/5431 
> 
> 
> I wonder if it couldn't be better to use stable/xx branch, instead tag release ? 

As long as we have the features available we need? 

> 
> This is the Thirth time I'm seen broken tag release, and it's always fixed later in the the stable branch, 
> but they never tag new minor release. 

Hmm, but not that you're the only one testing newest tags, 
and once we switch the stable branch will be less stable too :D 

> 
> Currently, the last stable/7.1 is very stable. (and they do maintenance for 1-2 years on stable branchs) 
> 

Stable and working sounds definitively good. 

> 
> what do you think about this ? (so downgrade the current package, and add stable/7.1 branch to frr_mirror). 

OK for me. 
There's also a stable 7.2 branch, with a few fixes since 7.2 tag[0], 
or is this also not yet working? 

[0]: https://github.com/FRRouting/frr/compare/frr-7.2...stable/7.2