[autotest] Remove old priority hack
This is from 2013, Im assuming the priorities have all been migrated
over.
The reason for doing this is that the sole purpose of this function is
to remove invalid keywords from passing locals() to create_job_common,
so this is a step toward removing locals() and this function.
BUG=None
TEST=CQ
Change-Id: Ia5b527b509863bbe12ff047628c23a988dbc32c9
Reviewed-on: https://chromium-review.googlesource.com/420178
Commit-Ready: Allen Li <ayatane@chromium.org>
Tested-by: Allen Li <ayatane@chromium.org>
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>
diff --git a/frontend/afe/rpc_utils.py b/frontend/afe/rpc_utils.py
index ceb6789..4855f68 100644
--- a/frontend/afe/rpc_utils.py
+++ b/frontend/afe/rpc_utils.py
@@ -17,7 +17,7 @@
from autotest_lib.frontend import thread_local
from autotest_lib.frontend.afe import models, model_logic
from autotest_lib.client.common_lib import control_data, error
-from autotest_lib.client.common_lib import global_config, priorities
+from autotest_lib.client.common_lib import global_config
from autotest_lib.client.common_lib import time_utils
from autotest_lib.client.common_lib.cros import dev_server
# TODO(akeshet): Replace with monarch once we know how to instrument rpc server
@@ -880,11 +880,6 @@
Returns a subset of local_args, which contains only the arguments that can
be passed in to create_job_common().
"""
- # This code is only here to not kill suites scheduling tests when priority
- # becomes an int instead of a string.
- if isinstance(local_args['priority'], str):
- local_args['priority'] = priorities.Priority.DEFAULT
- # </migration hack>
arg_names, _, _, _ = inspect.getargspec(create_job_common)
return dict(item for item in local_args.iteritems() if item[0] in arg_names)