From 33974c0c39a15254025e0aae5dab29d1492b1f1e Mon Sep 17 00:00:00 2001 From: Brose Johnstone Date: Sat, 28 Apr 2018 18:11:29 -0700 Subject: [PATCH] [RUNTIME] Fix State Manager data race related to RFID (#590) For some reason, State Manager sends data directly to student code by default, which can cause data races depending on the state of the student code process. This PR unsets that flag when sending RFID codes to State Manager. --- runtime/Ansible.py | 6 +++--- runtime/runtimeUtil.py | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/runtime/Ansible.py b/runtime/Ansible.py index c545a7ef..6343d480 100644 --- a/runtime/Ansible.py +++ b/runtime/Ansible.py @@ -489,11 +489,11 @@ def unpackage(data): if unpackaged_data.header == notification_pb2.Notification.GAMECODE_TRANSMISSION: state_queue.put([SM_COMMANDS.SET_VAL, [list(unpackaged_data.gamecode_solutions), - ["gamecodes_check"]]]) + ["gamecodes_check"], False]]) state_queue.put([SM_COMMANDS.SET_VAL, - [list(unpackaged_data.gamecodes), ["gamecodes"]]]) + [list(unpackaged_data.gamecodes), ["gamecodes"], False]]) state_queue.put([SM_COMMANDS.SET_VAL, - [list(unpackaged_data.rfids), ["rfids"]]]) + [list(unpackaged_data.rfids), ["rfids"], False]]) except ConnectionResetError: bad_things_queue.put( diff --git a/runtime/runtimeUtil.py b/runtime/runtimeUtil.py index f485904e..f9e88296 100644 --- a/runtime/runtimeUtil.py +++ b/runtime/runtimeUtil.py @@ -17,7 +17,7 @@ class RUNTIME_CONFIG(Enum): TEST_OUTPUT_DIR = "test_outputs/" VERSION_MAJOR = 18 VERSION_MINOR = 2 - VERSION_PATCH = 7 + VERSION_PATCH = 8 @unique class BAD_EVENTS(Enum):