Skip to content

Support option --ports of config qos reload for reloading ports' QoS and buffer configuration to default#2125

Merged
neethajohn merged 5 commits intosonic-net:masterfrom
stephenxs:config-qos-update
Apr 20, 2022
Merged

Support option --ports of config qos reload for reloading ports' QoS and buffer configuration to default#2125
neethajohn merged 5 commits intosonic-net:masterfrom
stephenxs:config-qos-update

Conversation

@stephenxs
Copy link
Collaborator

What I did

CLI: config qos reload --ports <ports_list>

  • --ports <ports_list>: a set of interfaces with empty QoS and buffer configurations (typically they have just been created via DPB).
    Format: <port>{,port}, like “Ethernet0” or “Ethernet4,Ethernet5,Ethernet6,Ethernet7”
  • Each port in the list should exist in the CONFIG_DB.PORT table

The flow:

  1. Render the template qos_config.j2 and buffer_config.j2, generating a temporary json file. (This is one step in “config qos reload”).
  2. Parse the json file, extracting all the items on the ports in the port_list
  3. Apply all the extracted items into the CONFIG_DB

How I did it

How to verify it

Previous command output (if the output of a command-line utility has changed)

New command output (if the output of a command-line utility has changed)

@stephenxs stephenxs requested review from keboliu and neethajohn April 1, 2022 07:22
@liat-grozovik
Copy link
Collaborator

@neethajohn kindly reminder to review

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants