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

fix config qos clear and key only update #1562

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vishnushetty
Copy link
Contributor

What I did
Fix config qos clear command.

Why I did it
Queue table update without scheduler profile.

How I verified it
swssloglevel -l DEBUG -c orchagent
show log handleQueueTable
Dec 21 17:56:57.275101 sonic DEBUG swss#orchagent: :> handleQueueTable: enter
Dec 21 17:56:57.275305 sonic DEBUG swss#orchagent: :- handleQueueTable: processing port:Ethernet8
Dec 21 17:56:57.275305 sonic DEBUG swss#orchagent: :- handleQueueTable: processing range:1-1
Dec 21 17:56:57.275305 sonic DEBUG swss#orchagent: :- handleQueueTable: processing queue:1
Dec 21 17:56:57.424246 sonic DEBUG swss#orchagent: :- handleQueueTable: Applied scheduler to port:Ethernet8 queue:1
Dec 21 17:56:57.424246 sonic DEBUG swss#orchagent: :- handleQueueTable: finished
Dec 21 17:56:57.424246 sonic DEBUG swss#orchagent: :< handleQueueTable: exit
Dec 21 17:56:57.424246 sonic DEBUG swss#orchagent: :> handleQueueTable: enter
Dec 21 17:56:57.424449 sonic DEBUG swss#orchagent: :- handleQueueTable: processing port:Ethernet8
Dec 21 17:56:57.424488 sonic DEBUG swss#orchagent: :- handleQueueTable: processing range:2-2
Dec 21 17:56:57.424609 sonic DEBUG swss#orchagent: :- handleQueueTable: processing queue:2
Dec 21 17:56:57.447642 sonic DEBUG swss#orchagent: :- handleQueueTable: Applied scheduler to port:Ethernet8 queue:2
Dec 21 17:56:57.447773 sonic DEBUG swss#orchagent: :- handleQueueTable: finished
Dec 21 17:56:57.447803 sonic DEBUG swss#orchagent: :< handleQueueTable: exit
Dec 21 17:57:09.835991 sonic DEBUG swss#orchagent: :> handleQueueTable: enter
Dec 21 17:57:09.836647 sonic DEBUG swss#orchagent: :- handleQueueTable: processing port:Ethernet8
Dec 21 17:57:09.837054 sonic DEBUG swss#orchagent: :- handleQueueTable: processing range:1-1
Dec 21 17:57:09.837188 sonic DEBUG swss#orchagent: :- handleQueueTable: processing queue:1
Dec 21 17:57:09.862697 sonic DEBUG swss#orchagent: :- handleQueueTable: Removed scheduler to port:Ethernet8 queue:1
Dec 21 17:57:09.862697 sonic DEBUG swss#orchagent: :- handleQueueTable: finished
Dec 21 17:57:09.862697 sonic DEBUG swss#orchagent: :< handleQueueTable: exit
Dec 21 17:57:09.862697 sonic DEBUG swss#orchagent: :> handleQueueTable: enter
Dec 21 17:57:09.862775 sonic DEBUG swss#orchagent: :- handleQueueTable: processing port:Ethernet8
Dec 21 17:57:09.862964 sonic DEBUG swss#orchagent: :- handleQueueTable: processing range:2-2
Dec 21 17:57:09.862964 sonic DEBUG swss#orchagent: :- handleQueueTable: processing queue:2
Dec 21 17:57:09.867682 sonic DEBUG swss#orchagent: :- handleQueueTable: Removed scheduler to port:Ethernet8 queue:2
Dec 21 17:57:09.867682 sonic DEBUG swss#orchagent: :- handleQueueTable: finished
Dec 21 17:57:09.867682 sonic DEBUG swss#orchagent: :< handleQueueTable: exit
Details if related
config load qos.json
config qos clear
qos.json
{
"QUEUE": {
"Ethernet8|1": {
"scheduler": "[SCHEDULER|port_qos_shaper@1]"
},
"Ethernet8|2": {
"scheduler": "[SCHEDULER|port_qos_shaper@2]"
}
},
"SCHEDULER": {
"port_qos_shaper@1": {
"pir": "10000",
"meter_type": "bytes"
},
"port_qos_shaper@2": {
"pir": "2500",
"meter_type": "bytes"
}
}
}

@ben-gale
Copy link
Collaborator

@lguohan - pls help with review - thx

EdenGri pushed a commit to EdenGri/sonic-swss that referenced this pull request Feb 28, 2022
+ update 'show ip interface' command in tech support collection
+ fix unbound variable issue in case command times out
@prsunny prsunny requested a review from neethajohn as a code owner September 2, 2022 23:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants