Auto_reboot: Enable/disable values were backwards
- Also changed comments and doc strings to wrap at 110 chars vs 79.
Change-Id: Ia4a66265be50f40468b293a5a67765aa81fd52d6
Signed-off-by: Michael Walsh <micwalsh@us.ibm.com>
diff --git a/bin/plug_ins/Auto_reboot/cp_master b/bin/plug_ins/Auto_reboot/cp_master
index b7aad55..50b3f42 100755
--- a/bin/plug_ins/Auto_reboot/cp_master
+++ b/bin/plug_ins/Auto_reboot/cp_master
@@ -42,8 +42,7 @@
def exit_function(signal_number=0,
frame=None):
r"""
- Execute whenever the program ends normally or with the signals that we
- catch (i.e. TERM, INT).
+ Execute whenever the program ends normally or with the signals that we catch (i.e. TERM, INT).
"""
dprint_executing()
@@ -57,26 +56,23 @@
def signal_handler(signal_number,
frame):
r"""
- Handle signals. Without a function to catch a SIGTERM or SIGINT, our
- program would terminate immediately with return code 143 and without
- calling our exit_function.
+ Handle signals. Without a function to catch a SIGTERM or SIGINT, our program would terminate immediately
+ with return code 143 and without calling our exit_function.
"""
- # Our convention is to set up exit_function with atexit.register() so
- # there is no need to explicitly call exit_function from here.
+ # Our convention is to set up exit_function with atexit.register() so there is no need to explicitly
+ # call exit_function from here.
dprint_executing()
- # Calling exit prevents us from returning to the code that was running
- # when we received the signal.
+ # Calling exit prevents us from returning to the code that was running when we received the signal.
exit(0)
def validate_parms():
r"""
- Validate program parameters, etc. Return True or False (i.e. pass/fail)
- accordingly.
+ Validate program parameters, etc. Return True or False (i.e. pass/fail) accordingly.
"""
get_plug_vars()
@@ -109,7 +105,9 @@
lib_file_path = init_robot_file_path("lib/utils.robot")
- keyword_string = "Set Auto Reboot ${%i}" % AUTO_REBOOT_DISABLE
+ enable_auto_reboot = 1 - AUTO_REBOOT_DISABLE
+ print_var(enable_auto_reboot)
+ keyword_string = "Set Auto Reboot ${%i}" % enable_auto_reboot
cmd_buf = create_robot_cmd_string("extended/run_keyword.robot",
OPENBMC_HOST, REST_USERNAME,