sonic-buildimage/dockers
pavel-shirshov 2eec3b3254 [bgpcfgd]: Dynamic BBR support (#5626)
**- Why I did it**
To introduce dynamic support of BBR functionality into bgpcfgd.
BBR is adding  `neighbor PEER_GROUP allowas-in 1' for all BGP peer-groups which points to T0
Now we can add and remove this configuration based on CONFIG_DB entry 

**- How I did it**
I introduced a new CONFIG_DB entry:
 - table name: "BGP_BBR"
 - key value: "all". Currently only "all" is supported, which means that all peer-groups which points to T0s will be updated
 - data value: a dictionary: {"status": "status_value"}, where status_value could be either "enabled" or "disabled"

Initially, when bgpcfgd starts, it reads initial BBR status values from the [constants.yml](https://github.com/Azure/sonic-buildimage/pull/5626/files#diff-e6f2fe13a6c276dc2f3b27a5bef79886f9c103194be4fcb28ce57375edf2c23cR34). Then you can control BBR status by changing "BGP_BBR" table in the CONFIG_DB (see examples below).

bgpcfgd knows what peer-groups to change fron [constants.yml](https://github.com/Azure/sonic-buildimage/pull/5626/files#diff-e6f2fe13a6c276dc2f3b27a5bef79886f9c103194be4fcb28ce57375edf2c23cR39). The dictionary contains peer-group names as keys, and a list of address-families as values. So when bgpcfgd got a request to change the BBR state, it changes the state only for peer-groups listed in the constants.yml dictionary (and only for address families from the peer-group value).

**- How to verify it**
Initially, when we start SONiC FRR has BBR enabled for PEER_V4 and PEER_V6:
```
admin@str-s6100-acs-1:~$ vtysh -c 'show run' | egrep 'PEER_V.? allowas'
  neighbor PEER_V4 allowas-in 1
  neighbor PEER_V6 allowas-in 1
```

Then we apply following configuration to the db:
```
admin@str-s6100-acs-1:~$ cat disable.json                
{
        "BGP_BBR": {
            "all": {
                "status": "disabled"
            }
        }
}


admin@str-s6100-acs-1:~$ sonic-cfggen -j disable.json -w 
```
The log output are:
```
Oct 14 18:40:22.450322 str-s6100-acs-1 DEBUG bgp#bgpcfgd: Received message : '('all', 'SET', (('status', 'disabled'),))'
Oct 14 18:40:22.450620 str-s6100-acs-1 DEBUG bgp#bgpcfgd: execute command '['vtysh', '-f', '/tmp/tmpmWTiuq']'.
Oct 14 18:40:22.681084 str-s6100-acs-1 DEBUG bgp#bgpcfgd: execute command '['vtysh', '-c', 'clear bgp peer-group PEER_V4 soft in']'.
Oct 14 18:40:22.904626 str-s6100-acs-1 DEBUG bgp#bgpcfgd: execute command '['vtysh', '-c', 'clear bgp peer-group PEER_V6 soft in']'.
```

Check FRR configuraiton and see that no allowas parameters are there:
```
admin@str-s6100-acs-1:~$ vtysh -c 'show run' | egrep 'PEER_V.? allowas' 
admin@str-s6100-acs-1:~$
```

Then we apply enabling configuration back:
```
admin@str-s6100-acs-1:~$ cat enable.json 
{
        "BGP_BBR": {
            "all": {
                "status": "enabled"
            }
        }
}

admin@str-s6100-acs-1:~$ sonic-cfggen -j enable.json -w 
```
The log output:
```
Oct 14 18:40:41.074720 str-s6100-acs-1 DEBUG bgp#bgpcfgd: Received message : '('all', 'SET', (('status', 'enabled'),))'
Oct 14 18:40:41.074720 str-s6100-acs-1 DEBUG bgp#bgpcfgd: execute command '['vtysh', '-f', '/tmp/tmpDD6SKv']'.
Oct 14 18:40:41.587257 str-s6100-acs-1 DEBUG bgp#bgpcfgd: execute command '['vtysh', '-c', 'clear bgp peer-group PEER_V4 soft in']'.
Oct 14 18:40:42.042967 str-s6100-acs-1 DEBUG bgp#bgpcfgd: execute command '['vtysh', '-c', 'clear bgp peer-group PEER_V6 soft in']'.
```


Check FRR configuraiton and see that the BBR configuration is back:
```
admin@str-s6100-acs-1:~$ vtysh -c 'show run' | egrep 'PEER_V.? allowas'
  neighbor PEER_V4 allowas-in 1
  neighbor PEER_V6 allowas-in 1
```

*** The test coverage ***
Below is the test coverage
```
---------- coverage: platform linux2, python 2.7.12-final-0 ----------
Name                             Stmts   Miss  Cover
----------------------------------------------------
bgpcfgd/__init__.py                  0      0   100%
bgpcfgd/__main__.py                  3      3     0%
bgpcfgd/config.py                   78     41    47%
bgpcfgd/directory.py                63     34    46%
bgpcfgd/log.py                      15      3    80%
bgpcfgd/main.py                     51     51     0%
bgpcfgd/manager.py                  41     23    44%
bgpcfgd/managers_allow_list.py     385     21    95%
bgpcfgd/managers_bbr.py             76      0   100%
bgpcfgd/managers_bgp.py            193    193     0%
bgpcfgd/managers_db.py               9      9     0%
bgpcfgd/managers_intf.py            33     33     0%
bgpcfgd/managers_setsrc.py          45     45     0%
bgpcfgd/runner.py                   39     39     0%
bgpcfgd/template.py                 64     11    83%
bgpcfgd/utils.py                    32     24    25%
bgpcfgd/vars.py                      1      0   100%
----------------------------------------------------
TOTAL                             1128    530    53%
```

**- Which release branch to backport (provide reason below if selected)**

- [ ] 201811
- [x] 201911
- [x] 202006
2020-10-30 08:58:27 -07:00
..
docker-base Change the dpkg default behavior in slave, and docker-base, in order to prevent prompt (#3879) 2020-02-03 15:57:33 -08:00
docker-base-stretch [docker-base-stretch]: install rsyslog from stretch-backports (#5411) 2020-09-21 02:08:30 -07:00
docker-basic_router Remove extra trailing newlines at EOF (#804) 2017-07-12 20:54:37 -07:00
docker-config-engine Remove unused packages in docker images and host (#2807) 2019-04-29 17:21:24 -07:00
docker-config-engine-stretch [build]: Improve dockerfile instructions (#3048) 2019-06-22 11:26:23 -07:00
docker-database [201911][Monit] Unmonitor processes in disabled containers (#5462) 2020-09-25 00:30:41 -07:00
docker-dhcp-relay [dockers][supervisor] Increase event buffer size for dependent-startup (#5247) 2020-09-28 16:12:53 +00:00
docker-fpm-frr [bgpcfgd]: Dynamic BBR support (#5626) 2020-10-30 08:58:27 -07:00
docker-fpm-gobgp [201911][dockers] Update critical_processes file syntax (#4854) 2020-06-26 11:37:05 -07:00
docker-fpm-quagga [201911][dockers] Update critical_processes file syntax (#4854) 2020-06-26 11:37:05 -07:00
docker-lldp-sv2 [dockers][supervisor] Increase event buffer size for dependent-startup (#5247) 2020-09-28 16:12:53 +00:00
docker-nat [dockers][supervisor] Increase event buffer size for dependent-startup (#5247) 2020-09-28 16:12:53 +00:00
docker-orchagent [ECMP][Multi-ASIC] Have different ECMP seed value on each ASIC (#5357) 2020-10-13 09:48:57 -07:00
docker-platform-monitor [dockers][supervisor] Increase event buffer size for dependent-startup (#5247) 2020-09-28 16:12:53 +00:00
docker-ptf [docker-ptf] add gnmi python client (#4928) 2020-08-27 08:05:41 -07:00
docker-ptf-saithrift [docker-ptf-*]: Use buildimage infrastructure to build docker-ptf-* (#638) 2017-05-26 17:18:11 -07:00
docker-router-advertiser [dockers][supervisor] Increase event buffer size for dependent-startup (#5247) 2020-09-28 16:12:53 +00:00
docker-sflow [dockers][supervisor] Increase event buffer size for dependent-startup (#5247) 2020-09-28 16:12:53 +00:00
docker-snmp-sv2 Fix monit complaining of snmp on 201911 branch. (#5612) 2020-10-13 17:17:43 -07:00
docker-sonic-mgmt [mgmt docker] move pycryptodome installation to the end of the docker building (#4917) 2020-07-11 09:46:51 -07:00
docker-sonic-mgmt-framework [dockers][supervisor] Increase event buffer size for dependent-startup (#5247) 2020-09-28 16:12:53 +00:00
docker-sonic-restapi Update conf DB with CA cert & rename ca_crt field (#5448) 2020-09-28 16:19:27 +00:00
docker-sonic-telemetry [dockers][supervisor] Increase event buffer size for dependent-startup (#5247) 2020-09-28 16:12:53 +00:00
docker-teamd [dockers][supervisor] Increase event buffer size for dependent-startup (#5247) 2020-09-28 16:12:53 +00:00
dockerfile-macros.j2 [build]: Improve dockerfile instructions (#3048) 2019-06-22 11:26:23 -07:00