Update QoS config name for Mellanox platform #1028
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Andriy Moroz <[email protected]>
stcheng
approved these changes
Oct 11, 2017
|
This needs to merged only if sonic-swss submodule is advanced to 6f29a74 or higher |
|
@andriymoroz-mlnx didn't notice that, shall we forward the submodule? or shall we revert this change? which would you prefer? |
|
this will also include commit "Use sai_serialize api to write to DB" also requires updating sonic-sairedis to 888e69c (one step only). If sairedis is not updated it will not break the build but cause incomplete functionality |
stcheng
pushed a commit
that referenced
this pull request
Oct 13, 2017
)" This reverts commit fc7246f.
jleveque
added a commit
that referenced
this pull request
Aug 9, 2020
* src/sonic-utilities 9700e45...621aad0 (9): > [Python] Migrate applications/scripts to import sonic-py-common package (#1008) > [config qos] QoS and Buffer config genration for multi ASIC platforms (#978) > [cli]: pass db connector as click context (#1029) > [PDDF] Make utilities compatible to platform API 2.0, in accordance with pddf_2.0 (#940) > [tests] Add unit tests for 'show platform ...' commands (#1021) > Add secure fast/warm-reboot support for Aboot (#994) > [sonic-installer] Create Envvars File for Incoming Image (#1011) > [config load]: do not stop/reset/reload service if it is disabled (#1028) > [tests]: move feature_test.py into tests folder (#1027)
santhosh-kt
pushed a commit
to santhosh-kt/sonic-buildimage
that referenced
this pull request
Feb 25, 2021
* src/sonic-utilities 9700e45...621aad0 (9): > [Python] Migrate applications/scripts to import sonic-py-common package (sonic-net#1008) > [config qos] QoS and Buffer config genration for multi ASIC platforms (sonic-net#978) > [cli]: pass db connector as click context (sonic-net#1029) > [PDDF] Make utilities compatible to platform API 2.0, in accordance with pddf_2.0 (sonic-net#940) > [tests] Add unit tests for 'show platform ...' commands (sonic-net#1021) > Add secure fast/warm-reboot support for Aboot (sonic-net#994) > [sonic-installer] Create Envvars File for Incoming Image (sonic-net#1011) > [config load]: do not stop/reset/reload service if it is disabled (sonic-net#1028) > [tests]: move feature_test.py into tests folder (sonic-net#1027)
stepanblyschak
pushed a commit
to stepanblyschak/sonic-buildimage
that referenced
this pull request
May 10, 2021
…nic-net#1028) also introduce fixtures to setup the cmd as well as asic context Signed-off-by: Guohan Lu <[email protected]>
DavidZagury
pushed a commit
to DavidZagury/sonic-buildimage
that referenced
this pull request
Apr 28, 2025
…s in ptf dataplane (sonic-net#1028) <!-- Please make sure you've read and understood our contributing guidelines: https://github.com/Azure/SONiC/blob/gh-pages/CONTRIBUTING.md failure_prs.log Make sure all your commits include a signature generated with `git commit -s` ** If this is a bug fix, make sure your description includes "fixes #xxxx", or "closes #xxxx" or "resolves #xxxx" Please provide the following information: --> #### Why I did it We need to count packets we care about on ptf dataplane. ##### Work item tracking - Microsoft ADO **(number only)**: #### How I did it Add a specific counter in ptf datplane, user can create counter with packet mask, so ptf dataplane can count packets that matching specific mask. #### How to verify it Tested on testbed Sent 5 packets for each 100 ports and read counters. <!-- If PR needs to be backported, then the PR must be tested against the base branch and the earliest backport release branch and provide tested image version on these two branches. For example, if the PR is requested for master, 202211 and 202012, then the requester needs to provide test results on master and 202012. --> #### Which release branch to backport (provide reason below if selected) <!-- - Note we only backport fixes to a release branch, *not* features! - Please also provide a reason for the backporting below. - e.g. - [x] 202006 --> - [ ] 201811 - [ ] 201911 - [ ] 202006 - [ ] 202012 - [ ] 202106 - [ ] 202111 - [ ] 202205 - [ ] 202211 - [ ] 202305 #### Tested branch (Please provide the tested image version) <!-- - Please provide tested image version - e.g. - [x] 20201231.100 --> - [ ] <!-- image version 1 --> - [ ] <!-- image version 2 --> #### Description for the changelog <!-- Write a short (one line) summary that describes the changes in this pull request for inclusion in the changelog: --> <!-- Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU. --> #### Link to config_db schema for YANG module changes <!-- Provide a link to config_db schema for the table for which YANG model is defined Link should point to correct section on https://github.com/Azure/sonic-buildimage/blob/master/src/sonic-yang-models/doc/Configuration.md --> #### A picture of a cute animal (not mandatory but encouraged)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Signed-off-by: Andriy Moroz [email protected]
- What I did
Updated QoS config name for Mellanox platform
- Why I did it
To match changes in sonic-swss@6f29a74
This needs to merged only if sonic-swss submodule is advanced to 6f29a74 or higher