This is merely a historical archive of years 2008-2021, before the migration to mailman3.
A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.
Harald Welte gerrit-no-reply at lists.osmocom.orgHarald Welte has submitted this change and it was merged. Change subject: Use random operation id ...................................................................... Use random operation id According to documentation for Control Interface Protocol <id> is "A numeric identifier, uniquely identifying this particular operation", hence it's best to be illustrated with random integer - use it as default. Fix override of id with previously used python-specific objects' id. Change-Id: I32236c067360526f4e7ee4bbdba64c5137de696d Related: OS#1646 --- M openbsc/contrib/bsc_control.py 1 file changed, 10 insertions(+), 8 deletions(-) Approvals: Harald Welte: Looks good to me, approved Jenkins Builder: Verified diff --git a/openbsc/contrib/bsc_control.py b/openbsc/contrib/bsc_control.py index 26a421d..4fc8b4d 100755 --- a/openbsc/contrib/bsc_control.py +++ b/openbsc/contrib/bsc_control.py @@ -1,6 +1,6 @@ #!/usr/bin/python -import sys,os +import sys,os, random from optparse import OptionParser import socket import struct @@ -36,15 +36,17 @@ data = prefix_ipa_ctrl_header(data) sck.send(data) -def do_set(var, value, id, sck): - setmsg = "SET %s %s %s" %(options.id, var, value) +def do_set(var, value, op_id, sck): + setmsg = "SET %s %s %s" %(op_id, var, value) send(sck, setmsg) -def do_get(var, id, sck): - getmsg = "GET %s %s" %(options.id, var) +def do_get(var, op_id, sck): + getmsg = "GET %s %s" %(op_id, var) send(sck, getmsg) if __name__ == '__main__': + random.seed() + parser = OptionParser("Usage: %prog [options] var [value]") parser.add_option("-d", "--host", dest="host", help="connect to HOST", metavar="HOST") @@ -54,7 +56,7 @@ dest="cmd_get", help="perform GET operation") parser.add_option("-s", "--set", action="store_true", dest="cmd_set", help="perform SET operation") - parser.add_option("-i", "--id", dest="id", default="1", + parser.add_option("-i", "--id", dest="op_id", default=random.randint(1, sys.maxint), help="set id manually", metavar="ID") parser.add_option("-v", "--verbose", action="store_true", dest="verbose", help="be verbose", default=False) @@ -79,12 +81,12 @@ if options.cmd_set: if len(args) < 2: parser.error("Set requires var and value arguments") - do_set(args[0], ' '.join(args[1:]), options.id, sock) + do_set(args[0], ' '.join(args[1:]), options.op_id, sock) if options.cmd_get: if len(args) != 1: parser.error("Get requires the var argument") - do_get(args[0], options.id, sock) + do_get(args[0], options.op_id, sock) data = sock.recv(1024) while (len(data)>0): -- To view, visit https://gerrit.osmocom.org/657 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I32236c067360526f4e7ee4bbdba64c5137de696d Gerrit-PatchSet: 2 Gerrit-Project: openbsc Gerrit-Branch: master Gerrit-Owner: Max <msuraev at sysmocom.de> Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org> Gerrit-Reviewer: Jenkins Builder