Once again I went through the autotest tree with my automated script to check all autotest source, and discovered several places that needed permission and indentation (including trailing whitespace) fixes. I have checked the generated patch and indeed it looks OK. Will commit this shortly, as preparation for autotest 0.13.
Signed-off-by: Lucas Meneghel Rodrigues <[email protected]> Property changes on: frontend/afe/resources_test.py ___________________________________________________________________ Added: svn:executable + * Property changes on: frontend/tko/rpc_interface_unittest.py ___________________________________________________________________ Added: svn:executable + * Property changes on: frontend/tko/resources_test.py ___________________________________________________________________ Added: svn:executable + * Property changes on: frontend/planner/rpc_interface_unittest.py ___________________________________________________________________ Added: svn:executable + * Property changes on: frontend/planner/rpc_utils_unittest.py ___________________________________________________________________ Added: svn:executable + * Property changes on: frontend/planner/control_file_unittest.py ___________________________________________________________________ Added: svn:executable + * Property changes on: frontend/planner/execution_engine_unittest.py ___________________________________________________________________ Added: svn:executable + * Index: frontend/migrations/062_drone_sets_unique.py =================================================================== --- frontend/migrations/062_drone_sets_unique.py (revision 4906) +++ frontend/migrations/062_drone_sets_unique.py (working copy) @@ -41,9 +41,9 @@ 'GROUP BY drone_id HAVING COUNT(*) > 1') rows = manager.execute(query) if rows: - raise Exception('Some drones are associated with more than one drone ' - 'set. Please remove all duplicates before running this ' - 'migration.') + raise Exception('Some drones are associated with more than one drone ' + 'set. Please remove all duplicates before running this ' + 'migration.') manager.execute_script(UP_SQL) if db_utils.check_index_exists(manager, 'afe_drone_sets_drones', Property changes on: frontend/client/gwt_dir ___________________________________________________________________ Added: svn:executable + * Property changes on: tko/job_serializer_unittest.py ___________________________________________________________________ Added: svn:executable + * Property changes on: tko/job_serializer.py ___________________________________________________________________ Added: svn:executable + * Property changes on: utils/external_packages.py ___________________________________________________________________ Added: svn:executable + * Index: server/prebuild.py =================================================================== --- server/prebuild.py (revision 4906) +++ server/prebuild.py (working copy) @@ -61,4 +61,3 @@ # instantiate a client_test instance. client_test = init_test(client_test_dir) client_setup_job.setup_test(client_test) - Index: server/hosts/remote.py =================================================================== --- server/hosts/remote.py (revision 4906) +++ server/hosts/remote.py (working copy) @@ -230,8 +230,8 @@ keyvals = utils.read_keyval(keyval_path) all_labels = keyvals.get('labels', '') if all_labels: - all_labels = all_labels.split(',') - return [urllib.unquote(label) for label in all_labels] + all_labels = all_labels.split(',') + return [urllib.unquote(label) for label in all_labels] return [] Index: server/tests/netperf2/netperf2.py =================================================================== --- server/tests/netperf2/netperf2.py (revision 4906) +++ server/tests/netperf2/netperf2.py (working copy) @@ -33,7 +33,7 @@ "test_time=%d, stream_list=%s, tag='%s', ", "iterations=%d)"]) - server_control_file = template % (server.ip, client.ip, 'server', test, + server_control_file = template % (server.ip, client.ip, 'server', test, time, stream_list, test, cycles) client_control_file = template % (server.ip, client.ip, 'client', test, time, stream_list, test, cycles) Index: client/tests/kvm/scripts/allocator.py =================================================================== --- client/tests/kvm/scripts/allocator.py (revision 4906) +++ client/tests/kvm/scripts/allocator.py (working copy) @@ -12,7 +12,7 @@ PAGE_SIZE = 4096 # machine page size -TMPFS_OVERHEAD = 0.0022 # overhead on 1MB of write data +TMPFS_OVERHEAD = 0.0022 # overhead on 1MB of write data class MemFill(object): @@ -34,7 +34,7 @@ self.tmpdp = tempfile.mkdtemp() ret_code = os.system("mount -o size=%dM tmpfs %s -t tmpfs" % - ((mem+math.ceil(mem*TMPFS_OVERHEAD)), + ((mem+math.ceil(mem*TMPFS_OVERHEAD)), self.tmpdp)) if ret_code != 0: if os.getuid() != 0: Property changes on: client/tests/kvm/scripts/qemu-ifup-ipv6 ___________________________________________________________________ Added: svn:executable + * Property changes on: client/tests/kvm/scripts/virtio_guest.py ___________________________________________________________________ Added: svn:executable + * Property changes on: client/tests/kvm/scripts/join_mcast.py ___________________________________________________________________ Added: svn:executable + * Index: client/tests/kvm/tests/virtio_console.py =================================================================== --- client/tests/kvm/tests/virtio_console.py (revision 4906) +++ client/tests/kvm/tests/virtio_console.py (working copy) @@ -398,7 +398,7 @@ logging.debug("Executing '%s' on virtio_guest.py loop, vm: %s," + "timeout: %s", command, vm[0].name, timeout) vm[1].sendline(command) - (match, data) = vm[1].read_until_last_line_matches(["PASS:", + (match, data) = vm[1].read_until_last_line_matches(["PASS:", "FAIL:[Failed to execute]"], timeout) return (match, data) @@ -948,4 +948,3 @@ vm[1].close() vm[0].destroy(gracefully=False) shutil.rmtree(vm[2]) - Property changes on: client/tests/kvm/deps/finish.exe ___________________________________________________________________ Deleted: svn:executable - * Property changes on: client/tests/kvm/deps/whql_delete_machine_15.exe ___________________________________________________________________ Deleted: svn:executable - * Property changes on: client/tests/kvm/deps/rss.exe ___________________________________________________________________ Deleted: svn:executable - * Property changes on: client/tests/kvm/deps/whql_submission_15.exe ___________________________________________________________________ Deleted: svn:executable - * Index: client/tests/hackbench/hackbench.py =================================================================== --- client/tests/hackbench/hackbench.py (revision 4906) +++ client/tests/hackbench/hackbench.py (working copy) @@ -18,9 +18,9 @@ def setup(self): os.chdir(self.srcdir) if 'CC' in os.environ: - cc = '$CC' + cc = '$CC' else: - cc = 'cc' + cc = 'cc' utils.system('%s -lpthread hackbench.c -o hackbench' % cc) Index: client/tests/tracing_microbenchmark/tracers.py =================================================================== --- client/tests/tracing_microbenchmark/tracers.py (revision 4906) +++ client/tests/tracing_microbenchmark/tracers.py (working copy) @@ -56,5 +56,5 @@ cpu_key = '%s_%s' % (cpu, key) total_key = 'total_' + key results[cpu_key] = val - results[total_key] = (results.get(total_key, 0) + + results[total_key] = (results.get(total_key, 0) + results[cpu_key]) Property changes on: client/tests/iozone/iozone.py ___________________________________________________________________ Deleted: svn:executable - * Index: client/profilers/cpistat/cpistat.py =================================================================== --- client/profilers/cpistat/cpistat.py (revision 4906) +++ client/profilers/cpistat/cpistat.py (working copy) @@ -17,7 +17,7 @@ def start(self, test): cmd = os.path.join(self.bindir, 'site_cpistat') if not os.path.exists(cmd): - cmd = os.path.join(self.bindir, 'cpistat') + cmd = os.path.join(self.bindir, 'cpistat') logfile = open(os.path.join(test.profdir, "cpistat"), 'w') p = subprocess.Popen(cmd, stdout=logfile, stderr=subprocess.STDOUT) Property changes on: client/common_lib/magic.py ___________________________________________________________________ Added: svn:executable + * Index: scheduler/metahost_scheduler.py =================================================================== --- scheduler/metahost_scheduler.py (revision 4906) +++ scheduler/metahost_scheduler.py (working copy) @@ -54,16 +54,16 @@ def schedule_metahost(self, queue_entry, scheduling_utility): - """Schedule the given queue entry, if possible. + """Schedule the given queue entry, if possible. - This method should make necessary database changes culminating in - assigning a host to the given queue entry in the database. It may - take no action if no host can be assigned currently. + This method should make necessary database changes culminating in + assigning a host to the given queue entry in the database. It may + take no action if no host can be assigned currently. - @param queue_entry: a HostQueueEntry DBObject - @param scheduling_utility: a HostSchedulingUtility object - """ - raise NotImplementedError + @param queue_entry: a HostQueueEntry DBObject + @param scheduling_utility: a HostSchedulingUtility object + """ + raise NotImplementedError def recovery_on_startup(self): _______________________________________________ Autotest mailing list [email protected] http://test.kernel.org/cgi-bin/mailman/listinfo/autotest
