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

Dynamic port configuration - add port buffer cfg to the port ref counter

Open dprital opened this issue 2 years ago • 150 comments

What I did

This PR replace PR #2022

Added increasing/decreasing to the port ref counter each time a port buffer configuration is added or removed Implemented according to the - https://github.com/Azure/SONiC/pull/900

Why I did it In order to avoid cases where a port is removed before the buffer configuration on this port were removed also

How I verified it VS Test was added in order to test it. we remove a port with buffer configuration and the port is not removed. only after all buffer configurations on this port were removed - this port will be removed.

Details if related

dprital avatar Mar 23 '22 17:03 dprital

/azpw run

dprital avatar Mar 23 '22 22:03 dprital

/AzurePipelines run

mssonicbld avatar Mar 23 '22 22:03 mssonicbld

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Mar 23 '22 22:03 azure-pipelines[bot]

/azpw run

dprital avatar Mar 24 '22 14:03 dprital

/azp run Azure.sonic-swss

liat-grozovik avatar Mar 24 '22 14:03 liat-grozovik

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Mar 24 '22 14:03 azure-pipelines[bot]

/azp run Azure.sonic-swss

dprital avatar Mar 24 '22 21:03 dprital

Commenter does not have sufficient privileges for PR 2194 in repo Azure/sonic-swss

azure-pipelines[bot] avatar Mar 24 '22 21:03 azure-pipelines[bot]

/azpw run

dprital avatar Mar 27 '22 08:03 dprital

/AzurePipelines run

mssonicbld avatar Mar 27 '22 08:03 mssonicbld

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Mar 27 '22 08:03 azure-pipelines[bot]

/azpw run

dprital avatar Mar 27 '22 18:03 dprital

/AzurePipelines run

mssonicbld avatar Mar 27 '22 18:03 mssonicbld

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Mar 27 '22 18:03 azure-pipelines[bot]

/azpw run

dprital avatar Mar 28 '22 08:03 dprital

/AzurePipelines run

mssonicbld avatar Mar 28 '22 08:03 mssonicbld

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Mar 28 '22 08:03 azure-pipelines[bot]

/azpw run

dprital avatar Mar 28 '22 16:03 dprital

/AzurePipelines run

mssonicbld avatar Mar 28 '22 16:03 mssonicbld

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Mar 28 '22 16:03 azure-pipelines[bot]

/azpw run

dprital avatar Mar 30 '22 20:03 dprital

/AzurePipelines run

mssonicbld avatar Mar 30 '22 20:03 mssonicbld

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Mar 30 '22 20:03 azure-pipelines[bot]

/azpw run

dprital avatar Mar 31 '22 05:03 dprital

/AzurePipelines run

mssonicbld avatar Mar 31 '22 05:03 mssonicbld

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Mar 31 '22 05:03 azure-pipelines[bot]

/azpw run

dprital avatar Mar 31 '22 17:03 dprital

/AzurePipelines run

mssonicbld avatar Mar 31 '22 17:03 mssonicbld

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Mar 31 '22 17:03 azure-pipelines[bot]

/azpw run

dprital avatar Apr 03 '22 06:04 dprital