Merge pull request #101 from hellt/sendconfig-failed

propagate multiresponse failed status to the combined response of SendConfig
scrapli · Oct 4, 2022 · 93d22ca · 93d22ca
2 parents 0474d2d + 9a24639
commit 93d22ca
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/assets/platforms/nokia_srl.yaml b/assets/platforms/nokia_srl.yaml
@@ -24,6 +24,7 @@ default:
   default-desired-privilege-level: "exec"
   failed-when-contains:
     - "Error:"
+    - "error:" # wildcard catch for errors like `Validation error:`, `Parsing error:`
   textfsm-platform: "" # ignored in go because no ntc-templates
   network-on-open:
     - operation: "acquire-priv" # targets default desired priv by default

diff --git a/driver/network/sendconfig.go b/driver/network/sendconfig.go
@@ -35,6 +35,7 @@ func (d *Driver) SendConfig(config string, opts ...util.Option) (*response.Respo
 	r.EndTime = time.Now()
 	r.ElapsedTime = r.EndTime.Sub(r.StartTime).Seconds()
 	r.Result = strings.Join(rOutputs, "\n")
+	r.Failed = m.Failed
 
 	return r, nil
 }