Commit b38fcfd
authored
[show][muxcable] fix
What I did
show mux hwmode muxdirection always has rc==1.
Fixed the issue below:
get_grpc_cached_version_mux_direction_per_port has reverted TRUE/FALSE return value compared to get_hwmode_mux_direction_port. The former is used to get results for active-active ports, the latter is used to get results for active-standby ports.
Use sys.exit() instead of return rc. CLI rc is different from function return value.
Fixed show mux grpc muxdirection as well.
sign-off: Jing Zhang
How to verify it
Tested on DUTs to verify rc. Run commands for single port and all ports, with and without --json.
Passed all UTs.show mux hwmode muxdirection RC (sonic-net#2812)1 parent 7e24463 commit b38fcfd
1 file changed
Lines changed: 15 additions & 14 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1394 | 1394 | | |
1395 | 1395 | | |
1396 | 1396 | | |
1397 | | - | |
| 1397 | + | |
| 1398 | + | |
1398 | 1399 | | |
1399 | 1400 | | |
1400 | 1401 | | |
1401 | 1402 | | |
1402 | 1403 | | |
1403 | | - | |
| 1404 | + | |
1404 | 1405 | | |
1405 | 1406 | | |
1406 | 1407 | | |
| |||
1449 | 1450 | | |
1450 | 1451 | | |
1451 | 1452 | | |
1452 | | - | |
1453 | | - | |
| 1453 | + | |
| 1454 | + | |
| 1455 | + | |
1454 | 1456 | | |
1455 | 1457 | | |
1456 | 1458 | | |
| |||
1464 | 1466 | | |
1465 | 1467 | | |
1466 | 1468 | | |
1467 | | - | |
1468 | | - | |
| 1469 | + | |
1469 | 1470 | | |
1470 | 1471 | | |
1471 | 1472 | | |
| |||
2140 | 2141 | | |
2141 | 2142 | | |
2142 | 2143 | | |
2143 | | - | |
| 2144 | + | |
2144 | 2145 | | |
2145 | 2146 | | |
2146 | 2147 | | |
| |||
2182 | 2183 | | |
2183 | 2184 | | |
2184 | 2185 | | |
2185 | | - | |
| 2186 | + | |
2186 | 2187 | | |
2187 | 2188 | | |
2188 | 2189 | | |
| |||
2222 | 2223 | | |
2223 | 2224 | | |
2224 | 2225 | | |
2225 | | - | |
| 2226 | + | |
| 2227 | + | |
2226 | 2228 | | |
2227 | 2229 | | |
2228 | 2230 | | |
2229 | 2231 | | |
2230 | 2232 | | |
2231 | 2233 | | |
2232 | | - | |
| 2234 | + | |
2233 | 2235 | | |
2234 | 2236 | | |
2235 | 2237 | | |
| |||
2266 | 2268 | | |
2267 | 2269 | | |
2268 | 2270 | | |
2269 | | - | |
2270 | | - | |
| 2271 | + | |
| 2272 | + | |
2271 | 2273 | | |
2272 | 2274 | | |
2273 | 2275 | | |
| |||
2276 | 2278 | | |
2277 | 2279 | | |
2278 | 2280 | | |
2279 | | - | |
2280 | | - | |
| 2281 | + | |
2281 | 2282 | | |
2282 | 2283 | | |
2283 | 2284 | | |
| |||
0 commit comments