Skip to content

Commit

Permalink
Add LACP fast rate option to Interfaces HLD
Browse files Browse the repository at this point in the history
  • Loading branch information
Arthi Sivanantham committed May 22, 2020
1 parent 8e62ba6 commit 0e10f56
Showing 1 changed file with 44 additions and 13 deletions.
57 changes: 44 additions & 13 deletions doc/mgmt/SONiC_OC_Interface_HLD.md
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,8 @@ Provide management framework capabilities to handle:
- PortChannel creation and deletion
- Addition of ports to PortChannel
- Removal of ports from PortChannel
- MTU, min-links, admin-status, IP address and LACP fallback configuration
- MTU, admin-status and IP address configuration
- Min-links, mode, LACP fallback and interval configuration during creation of PortChannel
- Associated show commands
- Support clearing PortChannel counter values.

Expand Down Expand Up @@ -279,14 +280,17 @@ module: openconfig-interfaces
+ | | +--ro oc-lag:aggregate-id? -> /oc-if:interfaces/interface/name
+ +--rw oc-lag:aggregation
+ | +--rw oc-lag:config
| | +--rw oc-lag:lag-type? aggregation-type
+ | | +--rw oc-lag:lag-type? aggregation-type
+ | | +--rw oc-lag:min-links? uint16
+ | | +--rw if-agmnt:fallback? boolean //Augmented
+ | | +--rw if-agmnt:fast-rate? boolean //Augmented
+ | +--ro oc-lag:state
| | +--ro oc-lag:lag-type? aggregation-type
+ | | +--ro oc-lag:lag-type? aggregation-type
+ | | +--ro oc-lag:min-links? uint16
| | +--ro oc-lag:lag-speed? uint32
+ | | +--ro oc-lag:member* oc-if:base-interface-ref
+ | | +--ro if-agmnt:fallback? boolean //Augmented
+ | | +--ro if-agmnt:fast-rate? boolean //Augmented
```

#### List of yang models required for Clear Counters support:
Expand Down Expand Up @@ -383,7 +387,7 @@ sonic(conf-if-po4)# no switchport access Vlan
```
#### 3.6.2.1.2 PORTCHANNEL
#### Create a PortChannel
`interface PortChannel <channel-number> [mode <active | on>] [ min-links <value> ] [fallback] `<br>
`interface PortChannel <channel-number> [mode <active | on>] [ min-links <value> ] [fallback] [fast_rate] `<br>

This comment has been minimized.

Copy link
@Chandra-BS

Chandra-BS May 26, 2020

what is fast_rate? Could not find the description anywhere.
Is it short and long timeout of LACP?
is fast_rate (i.e. short timeout) is disabled by default?

- *Supported channel-number range: 0-9999*<br>
- *Supported Min links range: 1-255*<br>
- Default values:<br>
Expand All @@ -392,6 +396,7 @@ sonic(conf-if-po4)# no switchport access Vlan
mode - active<br>
min-links - 0<br>
fallback - disabled<br>
fast_rate - disabled<br>
```
sonic(config)# interface PortChannel 1 mode active min-links 2 fallback
```
Expand All @@ -401,6 +406,9 @@ sonic(config)# interface PortChannel 2 mode active fallback
```
sonic(config)# interface PortChannel 3 mode on min-links 3
```
```
sonic(config)# interface PortChannel 4 mode active fast_rate
```

#### Configure MTU
`mtu <mtu-val>`
Expand Down Expand Up @@ -593,6 +601,7 @@ Group PortChannel Type Protocol Member Ports
1 PortChannel1 (D) Eth LACP Ethernet56(D)
Ethernet60(U)
10 PortChannel10 (U) Eth LACP Ethernet40(D)
12 PortChannel12 (D) Eth NONE Ethernet48(D)
111 PortChannel111 (D) Eth LACP
```
Expand All @@ -602,13 +611,12 @@ Group PortChannel Type Protocol Member Ports
`show interface PortChannel <id>` - Display details about a specific PortChannel.
```
sonic# show interface PortChannel 1
PortChannel 1 is up, line protocol is down, mode LACP
PortChannel 1 is up, line protocol is up, mode LACP
Fallback: Enabled
MTU 1532 bytes, IP MTU 1500 bytes
MTU 1532 bytes
Minimum number of links to bring PortChannel up is 1
LACP mode active, interval slow, priority 65535, address 90:b1:1c:f4:a8:7e
Members in this channel: Ethernet56
selected True
LACP mode ACTIVE, interval SLOW, priority 65535, address 90:b1:1c:f4:a8:7e
Members in this channel: Ethernet56(Selected)
LACP Actor port 56 address 90:b1:1c:f4:a8:7e key 1
LACP Partner port 0 address 00:00:00:00:00:00 key 0
Last clearing of "show interface" counters: 2019-12-06 21:23:20
Expand All @@ -622,6 +630,27 @@ Output statistics:
0 error, 0 discarded
```

```
sonic# show interface PortChannel 2
PortChannel2 is up, line protocol is down, mode Static
Minimum number of links to bring PortChannel up is 1
MTU 9100
sonic#
```

```
sonic# show interface PortChannel 5
PortChannel5 is up, line protocol is down, mode LACP
Minimum number of links to bring PortChannel up is 1
Fallback: Disabled
MTU 9100
LACP mode ACTIVE interval FAST priority 65535 address 90:b1:1c:f4:aa:b2
sonic#
```

#### 3.6.2.2.3 LOOPBACK
#### Display specific LOOPBACK Information
`show interface Loopback <lo-id>`
Expand Down Expand Up @@ -664,7 +693,7 @@ N/A
#### PORTCHANNEL
**PATCH**
- Create a PortChannel: `/openconfig-interfaces:interfaces/interface={name}/config`
- Set min-links: `/openconfig-interfaces:interfaces/interface={name}/openconfig-if-aggregate:aggregation/config/min-links`
- Set min-links/mode/lacp fallback/lacp interval: `/openconfig-interfaces:interfaces/interface={name}/openconfig-if-aggregate:aggregation/config/[min-links|lag-type|openconfig-interfaces-ext:fallback|openconfig-interfaces-ext:fast_rate]`
- Set MTU/admin-status: `/openconfig-interfaces:interfaces/interface={name}/config/[admin-status|mtu]`
- Set IP: `/openconfig-interfaces:interfaces/interface={name}/subinterfaces/subinterface[index=0]/openconfig-if-ip:ipv4/addresses/address={ip}/config`
- Add member: `/openconfig-interfaces:interfaces/interface={name}/openconfig-if-ethernet:ethernet/config/openconfig-if-aggregate:aggregate-id`
Expand All @@ -677,8 +706,8 @@ N/A
Get PortChannel details:
- `/openconfig-interfaces:interfaces/interface={name}`
- `/openconfig-interfaces:interfaces/interface={name}/state/[mtu|admin-status|oper-status]`
- `/openconfig-interfaces:interfaces/interface={name}/openconfig-if-aggregate:aggregation/state/[min-links|member]`
- `/openconfig-interfaces:interfaces/interface={name}/openconfig-if-aggregate:aggregation/state/dell-intf-augments:fallback`
- `/openconfig-interfaces:interfaces/interface={name}/openconfig-if-aggregate:aggregation/state/[min-links|member|lag-type]`
- `/openconfig-interfaces:interfaces/interface={name}/openconfig-if-aggregate:aggregation/state/[dell-intf-augments:fallback|dell-intf-augments:fast_rate]`

#### LOOPBACK
**PATCH**
Expand Down Expand Up @@ -742,7 +771,9 @@ N/A
| Configure min-links | Verify min-links is configured <br> Verify error returned if min-links value out of supported range |
| Remove min-links config | Verify min-links reset to default value |
| Configure MTU, admin-status| Verify MTU, admin-status configured |
| Configure Fallback | Verify Fallback configured |
| Configure Mode | Verify Mode configured using "show interface PortChannel" command |
| Configure Fallback | Verify Fallback configured using "show interface PortChannel" command |
| Configure Fast Rate | Verify Fast Rate configured using "show interface PortChannel" command |
| Configure IPv4 address | Verify IPv4 address configured |
| Configure IPv6 address | Verify IPv6 address configured |
| Add ports to PortChannel| Verify Port added using "show PortChannel" command |
Expand Down

0 comments on commit 0e10f56

Please sign in to comment.