RESERVATION_DIR = "/tmp/reservation_dir"
+RESERVATION_NODE = "TG"
def diag_cmd(node, cmd):
:type ssh: dict
:type cmd: str
"""
- print "+", cmd
+ print('+ {cmd}'.format(cmd=cmd))
_, stdout, _ = exec_cmd(node, cmd)
- print stdout
+ print(stdout)
def main():
2. List contents of test.url file in the dir.
3. Create reservation dir.
4. Touch file according to -r option.
- 5. Put -u option string to file test.url
- From these 5 steps, 1 and 2 are performed always, their RC ignored.
+ From these 4 steps, 1 and 2 are performed always, their RC ignored.
RC of step 3 gives the overall result.
- If the result is success, steps 4-5 are executed without any output,
+ If the result is success, step 4 is executed without any output,
their RC is ignored.
- The two files in reservation dir are there for reporting
- which test run holds the reservation, so people can manually fix the testbed
- if the rest run has been aborted, or otherwise failed to unregister.
-
- The two files have different audiences.
-
- The URL content is useful for people scheduling their test runs
- and wondering why the reservation takes so long.
- For them, a URL (if available) to copy and paste into browser
- to see which test runs are blocking testbeds is the most convenient.
-
The "run tag" as a filename is useful for admins accessing the testbed
via a graphical terminal, which does not allow copying of text,
as they need less keypresses to identify the test run holding the testbed.
action="store_true")
parser.add_argument("-r", "--runtag", required=False, default="Unknown",
help="Identifier for test run suitable as filename")
- parser.add_argument("-u", "--url", required=False, default="Unknown",
- help="Identifier for test run suitable as URL")
args = parser.parse_args()
with open(args.topo, "r") as topo_file:
# we are using it, because testing shows SSH access to DUT
# during test affects its performance (bursts of lost packets).
try:
- tgn = topology["TG"]
+ node = topology[RESERVATION_NODE]
except KeyError:
- print "Topology file does not contain 'TG' node"
+ print("Topology file does not contain '{node}' node".
+ format(node=RESERVATION_NODE))
return 1
# For system reservation we use mkdir it is an atomic operation and we can
# store additional data (time, client_ID, ..) within reservation directory.
if args.cancel:
- ret, _, err = exec_cmd(tgn, "rm -r {}".format(RESERVATION_DIR))
+ ret, _, err = exec_cmd(node, "rm -r {dir}".format(dir=RESERVATION_DIR))
if ret:
- print "Cancellation unsuccessful:\n{}".format(err)
+ print("Cancellation unsuccessful:\n{err}".format(err=err))
return ret
# Before critical section, output can be outdated already.
- print "Diagnostic commands:"
+ print("Diagnostic commands:")
# -d and * are to supress "total <size>", see https://askubuntu.com/a/61190
- diag_cmd(tgn, "ls --full-time -cd '{dir}'/*".format(dir=RESERVATION_DIR))
- diag_cmd(tgn, "head -1 '{dir}/run.url'".format(dir=RESERVATION_DIR))
- print "Attempting reservation."
+ diag_cmd(node, "ls --full-time -cd '{dir}'/*".format(dir=RESERVATION_DIR))
+ print("Attempting testbed reservation.")
# Entering critical section.
- # TODO: Add optional argument to exec_cmd_no_error to make it
- # sys.exit(ret) instead raising? We do not want to deal with stacktrace.
- ret, _, err = exec_cmd(tgn, "mkdir '{dir}'".format(dir=RESERVATION_DIR))
+ ret, _, _ = exec_cmd(node, "mkdir '{dir}'".format(dir=RESERVATION_DIR))
# Critical section is over.
if ret:
- print "Already reserved by another job:\n{}".format(err)
+ _, stdo, _ = exec_cmd(node, "ls '{dir}'/*".format(dir=RESERVATION_DIR))
+ print("Testbed already reserved by:\n{stdo}".format(stdo=stdo))
return 2
# Here the script knows it is the only owner of the testbed.
- print "Success, writing test run info to reservation dir."
- # TODO: Add optional argument to exec_cmd_no_error to print message
- # to console instead raising? We do not want to deal with stacktrace.
- ret2, _, err = exec_cmd(
- tgn, "touch '{dir}/{runtag}' && ( echo '{url}' > '{dir}/run.url' )"\
- .format(dir=RESERVATION_DIR, runtag=args.runtag, url=args.url))
- if ret2:
- print "Writing test run info failed, but continuing anyway:\n{}".format(
- err)
+ print("Reservation success, writing additional info to reservation dir.")
+ ret, _, err = exec_cmd(
+ node, "touch '{dir}/{runtag}'"\
+ .format(dir=RESERVATION_DIR, runtag=args.runtag))
+ if ret:
+ print("Writing test run info failed, but continuing anyway:\n{err}".
+ format(err=err))
return 0