Skip to content

[Buffer manager] Remove "TODO" comment which is no longer relevant#2645

Merged
liat-grozovik merged 1 commit intosonic-net:masterfrom
stephenxs:eliminate-todo
Feb 7, 2023
Merged

[Buffer manager] Remove "TODO" comment which is no longer relevant#2645
liat-grozovik merged 1 commit intosonic-net:masterfrom
stephenxs:eliminate-todo

Conversation

@stephenxs
Copy link
Collaborator

@stephenxs stephenxs commented Jan 30, 2023

What I did
Remove "TODO" comment which is no longer relevant.

Why I did it

How I verified it

Running CI.

Details if related

@stephenxs stephenxs marked this pull request as ready for review January 30, 2023 08:30
@stephenxs stephenxs requested a review from neethajohn as a code owner January 30, 2023 08:30
@liat-grozovik
Copy link
Collaborator

/azp run Azure.sonic-swss

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@prsunny
Copy link
Collaborator

prsunny commented Jan 30, 2023

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@stephenxs
Copy link
Collaborator Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@stephenxs
Copy link
Collaborator Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liat-grozovik liat-grozovik merged commit 7d223d3 into sonic-net:master Feb 7, 2023
dprital added a commit to dprital/sonic-buildimage that referenced this pull request Feb 7, 2023
Update sonic-swss submodule pointer to include the following:
* 7d223d3 Remove TODO comment which is no longer relevant ([sonic-net#2645](sonic-net/sonic-swss#2645))
* 02c2267 [test_mux] add sleep in test_NH ([sonic-net#2648](sonic-net/sonic-swss#2648))
* 8de52bf [EVPN]Handling race condition when remote VNI arrives before tunnel map entry ([sonic-net#2642](sonic-net/sonic-swss#2642))
* e99e2e4 [voq][chassis] Remove created ports from the default vlan. ([sonic-net#2607](sonic-net/sonic-swss#2607))

Signed-off-by: dprital <[email protected]>
@stephenxs stephenxs deleted the eliminate-todo branch February 8, 2023 01:01
liat-grozovik pushed a commit to sonic-net/sonic-buildimage that referenced this pull request Feb 8, 2023
Update sonic-swss submodule pointer to include the following:
* 7d223d3 Remove TODO comment which is no longer relevant ([#2645](sonic-net/sonic-swss#2645))
* 02c2267 [test_mux] add sleep in test_NH ([#2648](sonic-net/sonic-swss#2648))
* 8de52bf [EVPN]Handling race condition when remote VNI arrives before tunnel map entry ([#2642](sonic-net/sonic-swss#2642))
* e99e2e4 [voq][chassis] Remove created ports from the default vlan. ([#2607](sonic-net/sonic-swss#2607))

Signed-off-by: dprital <[email protected]>
Janetxxx pushed a commit to Janetxxx/sonic-swss that referenced this pull request Nov 10, 2025
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 this pull request may close these issues.

6 participants