[autotest] Nuke get_create_job_common_args
BUG=None
TEST=None
Change-Id: I689206c076461db297e72659c60bf27c875eadb6
Reviewed-on: https://chromium-review.googlesource.com/419693
Commit-Ready: Allen Li <ayatane@chromium.org>
Tested-by: Allen Li <ayatane@chromium.org>
Reviewed-by: Allen Li <ayatane@chromium.org>
diff --git a/frontend/afe/rpc_interface.py b/frontend/afe/rpc_interface.py
index 74e61c7..7f57b01 100644
--- a/frontend/afe/rpc_interface.py
+++ b/frontend/afe/rpc_interface.py
@@ -900,31 +900,30 @@
raise Exception('Extra parameters remain: %r' % parameters)
return rpc_utils.create_job_common(
- **rpc_utils.get_create_job_common_args(dict(
- name=name,
- priority=priority,
- control_type=control_type,
- hosts=hosts,
- meta_hosts=meta_hosts,
- one_time_hosts=one_time_hosts,
- atomic_group_name=atomic_group_name,
- synch_count=synch_count,
- is_template=is_template,
- timeout=timeout,
- timeout_mins=timeout_mins,
- max_runtime_mins=max_runtime_mins,
- run_verify=run_verify,
- email_list=email_list,
- dependencies=dependencies,
- reboot_before=reboot_before,
- reboot_after=reboot_after,
- parse_failed_repair=parse_failed_repair,
- hostless=hostless,
- keyvals=keyvals,
- drone_set=drone_set,
- parameterized_job=parameterized_job.id,
- run_reset=run_reset,
- require_ssp=require_ssp)))
+ name=name,
+ priority=priority,
+ control_type=control_type,
+ hosts=hosts,
+ meta_hosts=meta_hosts,
+ one_time_hosts=one_time_hosts,
+ atomic_group_name=atomic_group_name,
+ synch_count=synch_count,
+ is_template=is_template,
+ timeout=timeout,
+ timeout_mins=timeout_mins,
+ max_runtime_mins=max_runtime_mins,
+ run_verify=run_verify,
+ email_list=email_list,
+ dependencies=dependencies,
+ reboot_before=reboot_before,
+ reboot_after=reboot_after,
+ parse_failed_repair=parse_failed_repair,
+ hostless=hostless,
+ keyvals=keyvals,
+ drone_set=drone_set,
+ parameterized_job=parameterized_job.id,
+ run_reset=run_reset,
+ require_ssp=require_ssp)
except:
parameterized_job.delete()
raise
@@ -1065,33 +1064,32 @@
if image is None:
return rpc_utils.create_job_common(
- **rpc_utils.get_create_job_common_args(dict(
- name=name,
- priority=priority,
- control_type=control_type,
- control_file=control_file,
- hosts=hosts,
- meta_hosts=meta_hosts,
- one_time_hosts=one_time_hosts,
- atomic_group_name=atomic_group_name,
- synch_count=synch_count,
- is_template=is_template,
- timeout=timeout,
- timeout_mins=timeout_mins,
- max_runtime_mins=max_runtime_mins,
- run_verify=run_verify,
- email_list=email_list,
- dependencies=dependencies,
- reboot_before=reboot_before,
- reboot_after=reboot_after,
- parse_failed_repair=parse_failed_repair,
- hostless=hostless,
- keyvals=keyvals,
- drone_set=drone_set,
- parent_job_id=parent_job_id,
- test_retry=test_retry,
- run_reset=run_reset,
- require_ssp=require_ssp)))
+ name=name,
+ priority=priority,
+ control_type=control_type,
+ control_file=control_file,
+ hosts=hosts,
+ meta_hosts=meta_hosts,
+ one_time_hosts=one_time_hosts,
+ atomic_group_name=atomic_group_name,
+ synch_count=synch_count,
+ is_template=is_template,
+ timeout=timeout,
+ timeout_mins=timeout_mins,
+ max_runtime_mins=max_runtime_mins,
+ run_verify=run_verify,
+ email_list=email_list,
+ dependencies=dependencies,
+ reboot_before=reboot_before,
+ reboot_after=reboot_after,
+ parse_failed_repair=parse_failed_repair,
+ hostless=hostless,
+ keyvals=keyvals,
+ drone_set=drone_set,
+ parent_job_id=parent_job_id,
+ test_retry=test_retry,
+ run_reset=run_reset,
+ require_ssp=require_ssp)
# Translate the image name, in case its a relative build name.
ds = dev_server.ImageServer.resolve(image)
@@ -1124,34 +1122,33 @@
# match this job to our known test, and then with the
# afe_parameterized_job_parameters table to get the actual image path.
return rpc_utils.create_job_common(
- **rpc_utils.get_create_job_common_args(dict(
- name=name,
- priority=priority,
- control_type=control_type,
- control_file=control_file,
- hosts=hosts,
- meta_hosts=meta_hosts,
- one_time_hosts=one_time_hosts,
- atomic_group_name=atomic_group_name,
- synch_count=synch_count,
- is_template=is_template,
- timeout=timeout,
- timeout_mins=timeout_mins,
- max_runtime_mins=max_runtime_mins,
- run_verify=run_verify,
- email_list=email_list,
- dependencies=dependencies,
- reboot_before=reboot_before,
- reboot_after=reboot_after,
- parse_failed_repair=parse_failed_repair,
- hostless=hostless,
- keyvals=keyvals,
- drone_set=drone_set,
- parameterized_job=known_parameterized_job.id,
- parent_job_id=parent_job_id,
- test_retry=test_retry,
- run_reset=run_reset,
- require_ssp=require_ssp)))
+ name=name,
+ priority=priority,
+ control_type=control_type,
+ control_file=control_file,
+ hosts=hosts,
+ meta_hosts=meta_hosts,
+ one_time_hosts=one_time_hosts,
+ atomic_group_name=atomic_group_name,
+ synch_count=synch_count,
+ is_template=is_template,
+ timeout=timeout,
+ timeout_mins=timeout_mins,
+ max_runtime_mins=max_runtime_mins,
+ run_verify=run_verify,
+ email_list=email_list,
+ dependencies=dependencies,
+ reboot_before=reboot_before,
+ reboot_after=reboot_after,
+ parse_failed_repair=parse_failed_repair,
+ hostless=hostless,
+ keyvals=keyvals,
+ drone_set=drone_set,
+ parameterized_job=known_parameterized_job.id,
+ parent_job_id=parent_job_id,
+ test_retry=test_retry,
+ run_reset=run_reset,
+ require_ssp=require_ssp)
def abort_host_queue_entries(**filter_data):