Commit 66522389 authored by k8s-merge-robot's avatar k8s-merge-robot Committed by GitHub

Merge pull request #28572 from atombender/master

Automatic merge from submit-queue Fixes bad heuristic when calling "tc show" to check interface `tc` sometimes returns stuff that has more than 12 words in its response. The heuristic is bad, but this at least fixes the case when `tc` is returning too much. Fixes #28571.
parents 0f20ab63 e681002d
...@@ -255,7 +255,7 @@ func (t *tcShaper) ReconcileInterface() error { ...@@ -255,7 +255,7 @@ func (t *tcShaper) ReconcileInterface() error {
return t.initializeInterface() return t.initializeInterface()
} }
fields := strings.Split(output, " ") fields := strings.Split(output, " ")
if len(fields) != 12 || fields[1] != "htb" || fields[2] != "1:" { if len(fields) < 12 || fields[1] != "htb" || fields[2] != "1:" {
if err := t.deleteInterface(fields[2]); err != nil { if err := t.deleteInterface(fields[2]); err != nil {
return err return err
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment