Skip to content

[Cherry-pick]QoSHdrmPoolsize _tx_disable_enable for all destination ports in lag (…#12788

Merged
yxieca merged 1 commit intosonic-net:202205from
ansrajpu-git:Hdrmtx_dsb_202205
Aug 5, 2024
Merged

[Cherry-pick]QoSHdrmPoolsize _tx_disable_enable for all destination ports in lag (…#12788
yxieca merged 1 commit intosonic-net:202205from
ansrajpu-git:Hdrmtx_dsb_202205

Conversation

@ansrajpu-git
Copy link
Contributor

#11008)

  • QoSHdrmPoolsize_collecting dst ports in lag for tx_disable_enable

Description of PR

Summary:Master PR #11008
Fixes # (issue)

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Back port request

  • 201911
  • 202012
  • 202205
  • 202305
  • 202311

Approach

What is the motivation for this PR?

How did you do it?

How did you verify/test it?

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

@ansrajpu-git ansrajpu-git changed the title QoSHdrmPoolsize _tx_disable_enable for all destination ports in lag (… [Cherry-pick]QoSHdrmPoolsize _tx_disable_enable for all destination ports in lag (… May 9, 2024
@vmittal-msft vmittal-msft self-requested a review May 15, 2024 18:22
@vmittal-msft
Copy link
Contributor

@yxieca @wangxin Can you please help merge ? This is backport from master.

…onic-net#11008)

* QoSHdrmPoolsize_collecting dst ports in lag for tx_disable_enable
@yxieca
Copy link
Collaborator

yxieca commented May 15, 2024

@yxieca @wangxin Can you please help merge ? This is backport from master.

@vmittal-msft this change is not 0 diff for XGS, have you tested on XGS and get pass?

@vmittal-msft
Copy link
Contributor

We haven't tested it for XGS but all new changes are under dnx flag. Will try on XGS and update.

@vmittal-msft vmittal-msft requested a review from neethajohn May 15, 2024 21:27
@vmittal-msft vmittal-msft self-assigned this May 15, 2024
@vmittal-msft
Copy link
Contributor

@ansrajpu-git Can you please check this on sonic T0 box and let us know if it is ok ?

@ansrajpu-git
Copy link
Contributor Author

@vmittal-msft , we do not have T0 box setup for test.

@arlakshm
Copy link
Contributor

@ansrajpu-git to test on T0 testbed and update.

@ansrajpu-git
Copy link
Contributor Author

@ansrajpu-git to test on T0 testbed and update.

@vmittal-msft, Tested this PR with T1 topo. No functional issue found in test run. Please review & approve

@vmittal-msft
Copy link
Contributor

@ansrajpu-git thanks for confirming.

@yxieca @wangxin please help merge.

@yxieca yxieca merged commit c225a2b into sonic-net:202205 Aug 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants