Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[bufferorch] : Support for buffer profiles for VoQ on chassis (#2465) #2618

Merged
merged 4 commits into from
Mar 8, 2023

Conversation

vmittal-msft
Copy link
Contributor

  • QoS changes for VoQ on chassis

What I did
Porting #2465 from master branch

Why I did it
Due to merge conflict

How I verified it

Details if related

@arlakshm
Copy link
Contributor

/Azp run Azure.sonic-swss

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

arlakshm added a commit to sonic-net/sonic-buildimage that referenced this pull request Jan 19, 2023
…ts (#12156)" (#13421)

This reverts commit 1cffbc7.

Why I did it
This PR reverts the changes done in #12156 in 202205.
The dependant swss changes in PR sonic-net/sonic-swss#2618 are not merged in 202205 yet.

This revert is to avoid issues on 202205 till the sonic-net/sonic-swss#2618 is merged in.

Once sonic-net/sonic-swss#2618 changes are merged, this change will be added back.
@vmittal-msft vmittal-msft requested a review from arlakshm January 20, 2023 20:01
@gechiang
Copy link
Contributor

gechiang commented Feb 8, 2023

@vmittal-msft can you please resolve the conflict?

@vmittal-msft vmittal-msft force-pushed the vmittal/system-port-profiles branch from 3102e36 to 531094b Compare February 23, 2023 19:40
@vmittal-msft
Copy link
Contributor Author

vmittal-msft commented Feb 23, 2023

@skbarista please review.

orchagent/portsorch.cpp Outdated Show resolved Hide resolved
orchagent/portsorch.cpp Outdated Show resolved Hide resolved
@vmittal-msft
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@vmittal-msft
Copy link
Contributor Author

dependent on #2685

@gechiang
Copy link
Contributor

gechiang commented Mar 2, 2023

dependent on #2685

This PR has a dependency on which 2685? SWSS? or other submodule of buildimage? or you meant to say 2682?

@prsunny
Copy link
Collaborator

prsunny commented Mar 7, 2023

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@prsunny
Copy link
Collaborator

prsunny commented Mar 7, 2023

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

6 participants