Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

sonic config bgp listen limit not effect #3025

Closed
tim-rj opened this issue Jun 18, 2019 · 1 comment
Closed

sonic config bgp listen limit not effect #3025

tim-rj opened this issue Jun 18, 2019 · 1 comment
Assignees

Comments

@tim-rj
Copy link
Contributor

tim-rj commented Jun 18, 2019

Description
图片
config bgp listen limit 2,show ip bgp su display config is effect,but in fact dynamically BGP neighbors is 3,(after execute clear bgp, still 3)
图片

Steps to reproduce the issue:

  1. sonic config dynamically BGP neighbors with DUT4。
    switch1# vtysh
    switch1# configure terminal
    switch1(config)# router bgp 65100
    switch1(config-router)# neighbor PEER peer-group
    switch1(config-router)# neighbor PEER remote-as 100
    switch1(config-router)# bgp listen range 10.0.0.0/16 peer-group PEER
    switch1(config-router)# bgp listen limit 2
  2. config bgp listen limit 2
  3. clear bgp ,rebuild bgp neighbors
  4. show ip bgp su, bgp neighbors is 3

Describe the results you received:
config bgp listen limit 2, dynamically BGP neighbors is 3

Describe the results you expected:
config bgp listen limit 2, dynamically BGP neighbors is 2

Additional information you deem important (e.g. issue happens only occasionally):

**Output of `show version`:**

```
    root@switch1:/home/admin# show version
SONiC Software Version: SONiC.201803_rj.0-9e5e2a4-20181128.173206
Distribution: Debian 8.11
Kernel: 3.16.0-5-amd64
Build commit: 9e5e2a4
Build date: Wed Nov 28 17:35:18 UTC 2018
Built by: ngcf@ubuntu01

Docker images:
REPOSITORY TAG IMAGE ID SIZE
docker-orchagent-brcm 201803_rj.0-9e5e2a4-20181128.173206 ff563f8a8a5e 364.6 MB
docker-orchagent-brcm latest ff563f8a8a5e 364.6 MB
docker-teamd 201803_rj.0-9e5e2a4-20181128.173206 2bec314c0cba 356 MB
docker-teamd latest 2bec314c0cba 356 MB
docker-fpm-frr 201803_rj.0-9e5e2a4-20181128.173206 8e8c63535800 360.9 MB
docker-fpm-frr latest 8e8c63535800 360.9 MB
docker-syncd-brcm 201803_rj.0-9e5e2a4-20181128.173206 271c843e1f4d 986.1 MB
docker-syncd-brcm latest 271c843e1f4d 986.1 MB
docker-lldp-sv2 201803_rj.0-9e5e2a4-20181128.173206 e6372670ce81 365.3 MB
docker-lldp-sv2 latest e6372670ce81 365.3 MB
docker-snmp-sv2 201803_rj.0-9e5e2a4-20181128.173206 682b40d6e204 387.9 MB
docker-snmp-sv2 latest 682b40d6e204 387.9 MB
docker-platform-monitor 201803_rj.0-9e5e2a4-20181128.173206 5482ac5f494a 367.2 MB
docker-platform-monitor latest 5482ac5f494a 367.2 MB
docker-router-advertiser 201803_rj.0-9e5e2a4-20181128.173206 f7dabf27a671 348.3 MB
docker-router-advertiser latest f7dabf27a671 348.3 MB
docker-dhcp-relay 201803_rj.0-9e5e2a4-20181128.173206 89e26a55b78d 350.1 MB
docker-dhcp-relay latest 89e26a55b78d 350.1 MB
docker-database 201803_rj.0-9e5e2a4-20181128.173206 c7238159374c 350.6 MB
docker-database latest c7238159374c 350.6 MB

GIT version:
MODULE BRANCH COMMIT_ID VERSION
sonic-swss-common 201803_rj 60a0a29dd09a
sonic-kernel 201803_rj 6cfcef24f4e0
sonic-sairedis 201803_rj 4ac34bb472e8
sonic-swss 201803_rj 3791b9405874
sonic-quagga 201803_rj b497e090787c
sonic-dbsync 201803_rj 1d0b2165e097
sonic-py-swsssdk 201803_rj ca540c9c51e2
sonic-snmpagent 201803_rj 12bc0e0c9843
sonic-ptf 201803_rj aac6efd0a78b
sonic-utilities 201803_rj d3bbeec71330
sonic-platform-common 201803_rj 0a6094670d0c
sonic-platform-daemons 201803_rj 31c007e260ba
sonic-frr 201803_rj 26473f4b87c5
sonic-platform-modules-ruijie master 064c83605b6a
sonic-sai 201803_rj c7c198d7d2cb
sonic-ptf-sairedis 201803_rj aac6efd0a78b
sonic-saithrift-ctypesgen 201803_rj 3d2d98033395
sonic-googletest 201803_rj ec44c6c1675c
sonic-brcm-sai brcm-sai-GA ad131e7a1157
sonic-initramfs-tools 201803_rj 8ec9eca084bf
sonic-libteam master 8c1cea00a219 v1.26
sonic-libteam-debian master da006f226c53 da006f2
sonic-supervisor master 85d0b206928f
sonic-isc-dhcp master 1d74837d0203
sonic-lldpd master 66f8a863e9ae
sonic-libnl3 master b77c0e49cb64 b77c0e49c
sonic-pam_tacplus master 8dddbec2940f v1.4.1
sonic-libnss-tacplus master 19008ab68d9d 19008ab

SDK version:
||/ Name Version Architecture Description
+++-==============================-============-============-==========================================================
ii libopennsl 3.5.0.4 amd64 shared library for broadcom opennsl sdk
ii libopennsl-dev 3.5.0.4 amd64 development library for broadcom opennsl sdk
ii libsaibcm 3.2.1.1 amd64 Switch Abstraction Interface sdk based on Broadcom opennsl
ii opennsl-modules-3.16.0-5-amd64 3.5.0.4 amd64 kernel modules for broadcom opennsl sdk 
```

**Attach debug file `sudo generate_dump`:**

```

config_db.json.txt

```
@pavel-shirshov
Copy link
Contributor

@tim-rj We found the issue too. We decided don't solve it in quagga for now. If this is critical for you, can you please create PR fixing the issue?

Thanks

yxieca pushed a commit that referenced this issue Nov 30, 2023
* Update sonic-utilities to master branch version

sonic-utilities was (intentionally) pointing to a commit on a fork,
since merging sonic-utilities's changes for Bookworm first onto the
master branch would result in PR checker failures. Now that
sonic-buildimage is on master branch and the Bookworm changes in
sonic-utilities have been merged into master, sonic-utilties can now
point to master.

17e77fe2 Revert "Run yang validation in unit test (#3025)" (#3055)
96dd5559 [dhcp_relay] Fix dhcp_relay counter display issue (#3054)
6dfeee69 [sflow][db_migrator] Egress Sflow support (#3020)
02a588b7 Don't collect /proc/sched_debug
d7ec3251 Fix error about having a mutable default for field headers in dataclass
0ab3ab91 Fix test execution on Bookworm (#3041)
ef8f6f83 Specify test dependencies under extra_requires
61c44e80 Update python packages
1e813105 [wol] Implement wol command line utility (#3048)
8ebc56a0 [sonic_installer]: Improve exception handling: introduce notes. (#3029)
3610ce93 [sonic-package-manager] Fix YANG validation failure on upgrade when feature has constraints in YANG model on FEATURE table (#2933)
cfd2dd39 Add container rsyslog.conf to the sys dump (#3039)
c4b07828 Support new platform in generic configuration update (#3038)
a8d236c8 [fast-reboot-filter-routes.py] Remove click and improve error reporting (#3030)
75199c0f [sonic-package-manager] insert newline in /etc/sonic/generated_services.conf (#3040)
cd855698 [VOQ][saidump] Modify generate_dump: replace save_saidump with save_saidump_by_route_size (#2972)
f1e24ae5 GCU support for Cisco-8000 features (#3010)
67e1c3dc Update GCU rsyslog validator (#3012)
253b7975 [sonic-package-manager] do not modify config_db.json (#3032)
177dd8e8 [sonic-package-manager] add generated service to /etc/sonic/generated_services.conf (#3037)
62fcd77a Configure NTP according to extended configuration (#2835)
ced09404 [dualtor_neighbor_check] Adjust zero-mac check condition (#3034)
a4eeb698 [config] config reload should generate sysinfo if missing  (#3031)
e01fc891 Run yang validation in unit test (#3025)

Signed-off-by: Saikrishna Arcot <[email protected]>
yxieca pushed a commit that referenced this issue Dec 14, 2023
…atically (#17457)

src/sonic-utilities

* 1b1402f5 - (HEAD -> 202311, origin/202311) [hash]: Add ECMP/LAG hash algorithm CLI (#3036) (9 days ago) [Nazarii Hnydyn]
* 71514ea3 - Revert "Run yang validation in unit test (#3025)" (#3055) (9 days ago) [Ying Xie]
* b5daf5d4 - [dhcp_relay] Fix dhcp_relay counter display issue (#3054) (9 days ago) [Yaqiang Zhu]
* b3172505 - [sflow][db_migrator] Egress Sflow support (#3020) (9 days ago) [Rajkumar-Marvell]
* 1e813105 - [wol] Implement wol command line utility (#3048) (3 weeks ago) [Zhijian Li]
* 8ebc56a0 - [sonic_installer]: Improve exception handling: introduce notes. (#3029) (3 weeks ago) [Nazarii Hnydyn]
* 3610ce93 - [sonic-package-manager] Fix YANG validation failure on upgrade when feature has constraints in YANG model on FEATURE table (#2933) (3 weeks ago) [Stepan Blyshchak]
* cfd2dd39 - Add container rsyslog.conf to the sys dump (#3039) (4 weeks ago) [Vivek]
* c4b07828 - Support new platform in generic configuration update (#3038) (4 weeks ago) [Stephen Sun]
* a8d236c8 - [fast-reboot-filter-routes.py] Remove click and improve error reporting (#3030) (4 weeks ago) [Stepan Blyshchak]
* 75199c0f - [sonic-package-manager] insert newline in /etc/sonic/generated_services.conf (#3040) (4 weeks ago) [Stepan Blyshchak]
* cd855698 - [VOQ][saidump] Modify generate_dump: replace save_saidump with save_saidump_by_route_size (#2972) (4 weeks ago) [JunhongMao]
* f1e24ae5 - GCU support for Cisco-8000 features (#3010) (4 weeks ago) [rbpittman]
* 67e1c3dc - Update GCU rsyslog validator (#3012) (4 weeks ago) [jingwenxie]
* 253b7975 - [sonic-package-manager] do not modify config_db.json (#3032) (5 weeks ago) [Stepan Blyshchak]
* 177dd8e8 - [sonic-package-manager] add generated service to /etc/sonic/generated_services.conf (#3037) (5 weeks ago) [Stepan Blyshchak]
* 62fcd77a - Configure NTP according to extended configuration (#2835) (5 weeks ago) [Yevhen Fastiuk]
* ced09404 - [dualtor_neighbor_check] Adjust zero-mac check condition (#3034) (5 weeks ago) [Longxiang Lyu]
* a4eeb698 - [config] config reload should generate sysinfo if missing  (#3031) (6 weeks ago) [jingwenxie]
* e01fc891 - Run yang validation in unit test (#3025) (6 weeks ago) [ganglv]
dgsudharsan added a commit to dgsudharsan/sonic-buildimage that referenced this issue Apr 5, 2024
…commits

* c96a2f84 - Revert "[acl] Add IN_PORTS qualifier for L3 table (sonic-net#3078)" (sonic-net#3092) (6 days ago) [Neetha John]
* 80e0b57d - [Copp]Refactor coppmgr tests (sonic-net#3093) (8 days ago) [Sudharsan Dhamal Gopalarathnam]
* a4647299 - [portsorch] process only updated APP_DB fields when port is already   created (sonic-net#3025) (10 days ago) [Stepan Blyshchak]
* 91bacca5 - [buffermgrd] Move switch-statement outside of if-statement in BufferMgr::doTask (sonic-net#3055) (2 weeks ago) [Amir]
* 04912ad0 - [bulker] add support for neighbor bulking (sonic-net#2768) (2 weeks ago) [Nikola Dancejic]
* 9d4a3add - [acl] Add IN_PORTS qualifier for L3 table (sonic-net#3078) (2 weeks ago) [Neetha John]
* a13e081f - [Mellanox] Fix inconsistence in the shared headroom pool initialization (sonic-net#3057) (3 weeks ago) [Stephen Sun]
* ff2b2b85 - Add basic fabric link monitoring counters and states handling. (sonic-net#2988) (3 weeks ago) [jfeng-arista]
* 0c620910 - Add port flap count and last flap timestamp to APPL_DB (sonic-net#3052) (3 weeks ago) [Prince George]
* e9931f31 - [EVPN] Skip EVPN routes with invalid VNI or router mac field (sonic-net#3073) (3 weeks ago) [Lior Avramov]
* 600d5e80 - Set HOST_TX_READY_NOTIFY attribute only after query capabilities(sonic-net#3070) (3 weeks ago) [noaOrMlnx]
liat-grozovik pushed a commit that referenced this issue Apr 7, 2024
…commits (#18576)

* c96a2f84 - Revert "[acl] Add IN_PORTS qualifier for L3 table (#3078)" (#3092) (6 days ago) [Neetha John]
* 80e0b57d - [Copp]Refactor coppmgr tests (#3093) (8 days ago) [Sudharsan Dhamal Gopalarathnam]
* a4647299 - [portsorch] process only updated APP_DB fields when port is already   created (#3025) (10 days ago) [Stepan Blyshchak]
* 91bacca5 - [buffermgrd] Move switch-statement outside of if-statement in BufferMgr::doTask (#3055) (2 weeks ago) [Amir]
* 04912ad0 - [bulker] add support for neighbor bulking (#2768) (2 weeks ago) [Nikola Dancejic]
* 9d4a3add - [acl] Add IN_PORTS qualifier for L3 table (#3078) (2 weeks ago) [Neetha John]
* a13e081f - [Mellanox] Fix inconsistence in the shared headroom pool initialization (#3057) (3 weeks ago) [Stephen Sun]
* ff2b2b85 - Add basic fabric link monitoring counters and states handling. (#2988) (3 weeks ago) [jfeng-arista]
* 0c620910 - Add port flap count and last flap timestamp to APPL_DB (#3052) (3 weeks ago) [Prince George]
* e9931f31 - [EVPN] Skip EVPN routes with invalid VNI or router mac field (#3073) (3 weeks ago) [Lior Avramov]
* 600d5e80 - Set HOST_TX_READY_NOTIFY attribute only after query capabilities(#3070) (3 weeks ago) [noaOrMlnx]
mssonicbld added a commit that referenced this issue May 1, 2024
…lly (#18838)

#### Why I did it
src/sonic-swss
```
* e1005dd6 - (HEAD -> 202305, origin/202305) [portsorch] process only updated APP_DB fields when port is already created (#3025) (19 hours ago) [Stepan Blyshchak]
```
#### How I did it
#### How to verify it
#### Description for the changelog
mssonicbld added a commit that referenced this issue May 18, 2024
…lly (#18961)

#### Why I did it
src/sonic-swss
```
* e69750d9 - (HEAD -> 202311, origin/202311) [portsorch] process only updated APP_DB fields when port is already created (#3025) (4 days ago) [Stepan Blyshchak]
* 10c6b9a2 - [VRF]Fixing vrf orch to update state_db when evpn nvo arrives late (#3140) (4 days ago) [Sudharsan Dhamal Gopalarathnam]
```
#### How I did it
#### How to verify it
#### Description for the changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants