fixeria has submitted this change. ( 
https://gerrit.osmocom.org/c/python/osmo-python-tests/+/37799?usp=email )

Change subject: osmotestconfig: fix return value of write_config()
......................................................................

osmotestconfig: fix return value of write_config()

This patch fixes the following exception, if the received string does
not match. The calling function just expects an integer.

----
TypeError: '>' not supported between instances of 'tuple' and 'int'
----

Change-Id: I24adfd344937bab1ab641327e59a25bf76e18591
---
M scripts/osmotestconfig.py
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Jenkins Builder: Verified
  fixeria: Looks good to me, but someone else must approve
  osmith: Looks good to me, approved
  laforge: Looks good to me, but someone else must approve




diff --git a/scripts/osmotestconfig.py b/scripts/osmotestconfig.py
index aa926f1..c56849d 100755
--- a/scripts/osmotestconfig.py
+++ b/scripts/osmotestconfig.py
@@ -86,7 +86,7 @@
     new_config = vty.enabled_command("write")
     if not new_config.startswith("Configuration saved to "):
         print(new_config)
-        return 1, [new_config]
+        return 1
     return 0



--
To view, visit 
https://gerrit.osmocom.org/c/python/osmo-python-tests/+/37799?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: merged
Gerrit-Project: python/osmo-python-tests
Gerrit-Branch: master
Gerrit-Change-Id: I24adfd344937bab1ab641327e59a25bf76e18591
Gerrit-Change-Number: 37799
Gerrit-PatchSet: 1
Gerrit-Owner: jolly <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: osmith <[email protected]>

Reply via email to