-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
Adding New Model onyx_qos for Configuring QoS on Onyx Switches #55127
Conversation
Hi @samerd, Please take a look on this PR and approve it if you don't have farther comments. Thanks |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Anas.
excellent module.
However, I have few minor comments:
1- rewrite_dscp and rewrite_pcp, should be both disabled by default
2- regarding the "no" commands, although your commands are working, I prefer to put the no before the qos, instead before the interface
NO_REWRITE_PCP_CMD = "interface {0} {1} no qos rewrite pcp"
NO_REWRITE_DSCP_CMD = "interface {0} {1} no qos rewrite dscp"
@anasbadaha this PR contains the following merge commits: Please rebase your branch to remove these commits. |
Hi @samerd , I have fixed all your comments, please approve if it's OK for you. Thanks |
@justjais |
Hi @justjais, Can you please merge this PR if it is OK for you? Thanks |
rebuild_merge |
@anasbadaha The PR has merge commit. Please rebase it. |
Signed-off-by: Anas Badaha <anasb@mellanox.com>
Signed-off-by: Anas Badaha <anasb@mellanox.com>
Signed-off-by: Anas Badaha <anasb@mellanox.com>
Signed-off-by: Anas Badaha <anasb@mellanox.com>
Signed-off-by: Anas Badaha <anasb@mellanox.com>
Signed-off-by: Anas Badaha <anasb@mellanox.com>
a2d714c
to
6c971ff
Compare
@anasbadaha Thank you! |
Thanks so much @ganeshrn |
Signed-off-by: Anas Badaha anasb@mellanox.com
SUMMARY
Adding New Model onyx_qos for Configuring QoS on Onyx Switches
ISSUE TYPE
COMPONENT NAME
lib/ansible/modules/network/onyx/onyx_qos.py
test/units/modules/network/onyx/fixtures/show_qos_interface_ethernet.cfg
test/units/modules/network/onyx/test_onyx_qos.py
ADDITIONAL INFORMATION