nser77

Results 19 comments of nser77

Hi all, adding the VRRP protocol implementation in SONiC would be a great improvement for me; really good job. While reading the HLD, I left some comments (which for GitHub...

@philo-micas, many thanks for your answers. Just want you to know that I'm pointing to another scenario that we may need to add in the `#restrictionslimitations` section (or even fix...

Hi all, sorry to spam again, but I need to do it because it affects some of my previous comments (not reviews); I am only just aware that VRRP has...

Hi all, many thanks for that and for your work. I just have one major concern regarding the actual and the future architecture: why is [`vrrpmgr`](https://github.com/philo-micas/sonic-swss/blob/d58a143c67099c4d73522b01b7bc7f220797e518/cfgmgr/vrrpmgr.cpp#L134-L154) adding and deleting VIPs...

Hi, I created a PR to avoid this, take a look to: #60

Hi, I created a PR to avoid this, take a look to: #60

HI, working fine for me with this PR: #60 Send signal: ``` docker kill -s 10 keepalived ``` Result: ``` [...] Fri Apr 21 06:51:48 2023: Printing parent data for...

Hi, I created a PR to avoid this, take a look to: #60

Quite interesting topic. I personally find a lot of benefit into ```keepalived```: 1. ```keepalived``` is heavily focused on the ```vrrp protocol (v2 and v3)``` and supports many configurations around it....

> > Hi all, many thanks for that and for your work. > > I just have one major concern regarding the actual and the future architecture: why is [`vrrpmgr`](https://github.com/philo-micas/sonic-swss/blob/d58a143c67099c4d73522b01b7bc7f220797e518/cfgmgr/vrrpmgr.cpp#L134-L154)...