sonic-swss icon indicating copy to clipboard operation
sonic-swss copied to clipboard

add remove queue port map function

Open vishnushetty opened this issue 4 years ago • 2 comments

What I did Added removeQueueMapPerPort() counter part of generateQueueMapPerPort()

Why I did it Need this cleanup as part of port remove.

How I verified it Verified the Queue counter map before and after port remove and create

Details if related Used these command to verify the cleanup redis-cli -n 2 hgetall COUNTERS_QUEUE_NAME_MAP > q_name_map.txt redis-cli -n 2 hgetall COUNTERS_QUEUE_PORT_MAP > q_port_map.txt redis-cli -n 2 hgetall COUNTERS_QUEUE_TYPE_MAP > q_type_map.txt redis-cli -n 2 hgetall COUNTERS_QUEUE_INDEX_MAP > q_index_map.txt

vishnushetty avatar Dec 17 '20 17:12 vishnushetty

This pull request introduces 1 alert when merging e6c9ee6330e33dcfcae716adf9e1967de1d04f8f into e1dd32788fc9f7985761135dbc8bb6c985b3cff9 - view on LGTM.com

new alerts:

  • 1 for FIXME comment

lgtm-com[bot] avatar Dec 17 '20 18:12 lgtm-com[bot]

@lguohan - pls help with review - thx

ben-gale avatar Dec 31 '20 20:12 ben-gale