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
Copy Markdown
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

Signed-off-by: Stephen Sun <stephens@nvidia.com>
@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
Copy Markdown
Collaborator

/azp run Azure.sonic-swss

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@prsunny
Copy link
Copy Markdown
Collaborator

prsunny commented Jan 30, 2023

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@stephenxs
Copy link
Copy Markdown
Collaborator Author

/azpw run

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@stephenxs
Copy link
Copy Markdown
Collaborator Author

/azpw run

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link
Copy Markdown

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 <drorp@nvidia.com>
@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 <drorp@nvidia.com>
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