* fix a bug in utils.run() when passing stdin as an empty string
* remove the check in AFE that a metahost job request has enough hosts under 
the label.  with the new pluggable metahost handlers, it's possible for 
handlers to create hosts on the fly to satisfy metahost requests
* expand the size of the label name field in the AFE DB to support more 
powerful metahost handlers

Signed-off-by: Steve Howard <[email protected]>

--- autotest/client/common_lib/utils.py 2010-02-25 17:15:57.000000000 -0800
+++ autotest/client/common_lib/utils.py 2010-02-25 17:15:57.000000000 -0800
@@ -512,7 +512,7 @@
         read_list.append(bg_job.sp.stderr)
         reverse_dict[bg_job.sp.stdout] = (bg_job, True)
         reverse_dict[bg_job.sp.stderr] = (bg_job, False)
-        if bg_job.string_stdin:
+        if bg_job.string_stdin is not None:
             write_list.append(bg_job.sp.stdin)
             reverse_dict[bg_job.sp.stdin] = bg_job
 
--- autotest/frontend/afe/rpc_utils.py  2010-02-25 17:15:57.000000000 -0800
+++ autotest/frontend/afe/rpc_utils.py  2010-02-25 17:15:57.000000000 -0800
@@ -456,14 +456,6 @@
     dependencies = options.get('dependencies', [])
     synch_count = options.get('synch_count')
 
-    # check that each metahost request has enough hosts under the label
-    for label, requested_count in metahost_counts.iteritems():
-        available_count = label.host_set.count()
-        if requested_count > available_count:
-            error = ("You have requested %d %s's, but there are only %d."
-                     % (requested_count, label.name, available_count))
-            raise model_logic.ValidationError({'meta_hosts' : error})
-
     if atomic_group:
         check_atomic_group_create_job(
                 synch_count, host_objects, metahost_objects,
--- /dev/null   2009-12-17 12:29:38.000000000 -0800
+++ autotest/frontend/migrations/048_expand_label_name_field.py 2010-02-25 
17:15:57.000000000 -0800
@@ -0,0 +1,7 @@
+UP_SQL = """
+ALTER TABLE afe_labels MODIFY name varchar(750) default NULL;
+"""
+
+DOWN_SQL = """
+ALTER TABLE afe_labels MODIFY name varchar(255) default NULL;
+"""
_______________________________________________
Autotest mailing list
[email protected]
http://test.kernel.org/cgi-bin/mailman/listinfo/autotest

Reply via email to