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

FDB table is not cleared during cold swss start #2932

Closed
stepanblyschak opened this issue May 22, 2019 · 0 comments · Fixed by #2933
Closed

FDB table is not cleared during cold swss start #2932

stepanblyschak opened this issue May 22, 2019 · 0 comments · Fixed by #2933

Comments

@stepanblyschak
Copy link
Collaborator

Description
FDB table is not cleared during cold swss start

Steps to reproduce the issue:

  1. Run VLAN-TRUNK test
  2. config reload -y
  3. redis-cli -n 6 keys FDB

Describe the results you received:

admin@arc-switch1025:~$ redis-cli -n 6 keys *FDB*
 1) "FDB_TABLE|Vlan100:52:54:00:83:2f:8f"
 2) "FDB_TABLE|Vlan1000:52:54:00:e9:bc:6c"
 3) "FDB_TABLE|Vlan100:5a:ee:b8:10:a3:1c"
 4) "FDB_TABLE|Vlan100:ca:5d:7f:95:98:1e"
 5) "FDB_TABLE|Vlan200:e4:1d:2d:ca:c7:02"
 6) "FDB_TABLE|Vlan1000:52:54:00:f8:bb:45"
 7) "FDB_TABLE|Vlan1000:52:54:00:05:d3:94"
 8) "FDB_TABLE|Vlan200:52:54:00:88:53:31"
 9) "FDB_TABLE|Vlan1000:52:54:00:2e:7c:70"
10) "FDB_TABLE|Vlan100:e4:1d:2d:ca:c7:02"
11) "FDB_TABLE|Vlan200:5a:ee:b8:10:a3:1c"
12) "FDB_TABLE|Vlan1000:e4:1d:2d:ca:c7:14"
13) "FDB_TABLE|Vlan200:e4:1d:2d:ca:c7:01"
14) "FDB_TABLE|Vlan1000:52:54:00:45:78:f7"
15) "FDB_TABLE|Vlan200:00:22:00:00:00:02"
16) "FDB_TABLE|Vlan1000:52:54:00:ba:d3:82"
17) "FDB_TABLE|Vlan100:00:22:00:00:00:02"
18) "FDB_TABLE|Vlan200:ca:5d:7f:95:98:1e"
19) "FDB_TABLE|Vlan1000:52:54:00:96:1f:8a"
20) "FDB_TABLE|Vlan100:e4:1d:2d:ca:c7:01"
21) "FDB_TABLE|Vlan1000:52:54:00:92:26:57"
22) "FDB_TABLE|Vlan1000:e4:1d:2d:ca:c7:0a"
23) "FDB_TABLE|Vlan1000:52:54:00:22:30:8b"
admin@arc-switch1025:~$ redis-cli -n 0 keys *VLAN_TABLE*
1) "VLAN_TABLE:Vlan1000"

Describe the results you expected:

No FDB entries referencing non-existing interfaces

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

**Output of `show version`:**

```
admin@arc-switch1025:~$ show version

SONiC Software Version: SONiC.HEAD.60-bd7b96f
Distribution: Debian 9.9
Kernel: 4.9.0-8-amd64
Build commit: bd7b96f
Build date: Mon May 20 02:23:56 UTC 2019
Built by: johnar@jenkins-worker-4

Docker images:
REPOSITORY TAG IMAGE ID SIZE
docker-orchagent-mlnx HEAD.60-bd7b96f 77b2febc6eb5 286MB
docker-orchagent-mlnx latest 77b2febc6eb5 286MB
docker-syncd-mlnx HEAD.60-bd7b96f d1b37656c622 331MB
docker-syncd-mlnx latest d1b37656c622 331MB
docker-lldp-sv2 HEAD.60-bd7b96f 2d29faeaffaf 274MB
docker-lldp-sv2 latest 2d29faeaffaf 274MB
docker-dhcp-relay HEAD.60-bd7b96f a622e4e9bc81 256MB
docker-dhcp-relay latest a622e4e9bc81 256MB
docker-database HEAD.60-bd7b96f 123cfec31f3c 255MB
docker-database latest 123cfec31f3c 255MB
docker-snmp-sv2 HEAD.60-bd7b96f 9bafa58e73fa 289MB
docker-snmp-sv2 latest 9bafa58e73fa 289MB
docker-teamd HEAD.60-bd7b96f 394ebacb3c82 274MB
docker-teamd latest 394ebacb3c82 274MB
docker-router-advertiser HEAD.60-bd7b96f 5325177d1b26 254MB
docker-router-advertiser latest 5325177d1b26 254MB
docker-platform-monitor HEAD.60-bd7b96f e705c8c67822 286MB
docker-platform-monitor latest e705c8c67822 286MB
docker-fpm-quagga HEAD.60-bd7b96f 15b215afce5a 281MB
docker-fpm-quagga latest 15b215afce5a 281MB

```

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

```
(paste your output here)
```
yxieca pushed a commit that referenced this issue Nov 7, 2023
…lly (#17094)

src/sonic-swss

* 01bf3b19 - (HEAD -> 202205, origin/202205) [muxorch] Fixing updateRoute logic (#2950) (5 hours ago) [Nikola Dancejic]
* 1e264e01 - Handle Mac address 'none' (#2593) (9 hours ago) [Prince Sunny]
* dc0e29b4 - [202205][teamd]: Clean teamd process if LAG creation fails (#2888) (#2932) (4 days ago) [Lawrence Lee]
mssonicbld added a commit that referenced this issue Mar 28, 2024
…atically (#18240)

#### Why I did it
src/sonic-utilities
```
* bdc57206 - (HEAD -> master, origin/master, origin/HEAD) Revert "Fix for Switch Port Modes and VLAN CLI Enhancement (#3108)" (#3246) (89 minutes ago) [jingwenxie]
* e35452b7 - Modify "show interface transceiver status" CLI to show SW cmis state (#3238) (2 days ago) [mihirpat1]
* 04a33e1f - Add "state" field in CONFIG_DB a toggle of the fabric port monitor feature (#2932) (2 days ago) [jfeng-arista]
* 3c489ba5 - Enhance route-check for multi-asic platforms (#3216) (5 days ago) [Deepak Singhal]
* c149e48b - [chassis] Add chassis support for CLI "config qos reload" (#3233) (6 days ago) [wenyiz2021]
* d8541add - Update port2alias (#3217) (8 days ago) [abdosi]
* d4688a8f - [graceful reboot] Add the pre_reboot_hook script execution, add the watchdog arm before the reboot (#3203) (8 days ago) [Vadym Hlushko]
* 125f36f3 - [ipintutil]Handle exception in show ip interfaces command (#3182) (10 days ago) [Sudharsan Dhamal Gopalarathnam]
* 9d532017 - [chassis][show-runningconfig] Fix the show runningconfiguration all issue on the Supervisor (#3194) (2 weeks ago) [Marty Y. Lok]
* 1a9261ce - [Techsupport]Handle SAI kv pair if present in sai common profile (#3196) (2 weeks ago) [Sudharsan Dhamal Gopalarathnam]
* 7466dc4a - Skip the validation of action in acl-loader if capability table in STATE_DB is empty (#3199) (2 weeks ago) [bingwang-ms]
* b879b658 - [Bug] Fix fw_setenv illegel character issue (#3201) (3 weeks ago) [xumia]
* 0b41a560 - [config] Add YANG alerting for override (#3188) (3 weeks ago) [jingwenxie]
* 24683b0c - [show] multi-asic show running test residue (#3198) (3 weeks ago) [jingwenxie]
* 995a797a - CLI to skip polling for periodic information for a port in DomInfoUpdateTask thread (#3187) (3 weeks ago) [mihirpat1]
* 9aa9eaa5 - [config] Add Table hard dependency check (#3159) (3 weeks ago) [jingwenxie]
* 5f0ffcca - [fast/warm-reboot] Put ERR message in syslog when a failure is seen (#3186) (4 weeks ago) [Vaibhav Hemant Dixit]
* 92220dcf - Fix for Switch Port Modes and VLAN CLI Enhancement (#3108) (4 weeks ago) [Saba Akram]
```
#### 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

Successfully merging a pull request may close this issue.

1 participant