two new major features:
(1) added test and job dependencies
-added M2M relationship between tests and labels and between jobs and labels, for tracking the labels on which a test/job depends
-modified test_importer to read the DEPENDENCIES field and create the right M2M relationships
-modified generate_control_file() RPC to compute and return the union of test dependencies. since generate_control_file now returns four pieces of information, i converted its return type from tuple to dict, and changed clients accordingly.
-modified job creation clients (GWT and CLI) to pass this dependency list to the create_job() RPC
-modified the create_job() RPC to check that hosts satisfy job dependencies, and to create M2M relationships
-modified the scheduler to check dependencies when scheduling jobs
-modified JobDetailView to show a job's dependencies
(2) added "only_if_needed" bit to labels; if true, a machine with this label can only be used if the label is requested (either by job dependencies or by the metahost label)
-added boolean field to Labels
-modified CLI label creation/viewing to support this new field
-made create_job() RPC and scheduler check for hosts with such a label that was not requested, and reject such hosts
also did some slight refactoring of other code in create_job() to simplify it while I was changing things there.
a couple notes:
-an only_if_needed label can be used if either the job depends on the label or it's a metahost for that label. we assume that if the user specifically requests the label in a metahost, then it's OK, even if the job doesn't depend on that label.
-one-time-hosts are assumed to satisfy job dependencies.
Signed-off-by: Steve Howard <showard@google.com>
git-svn-id: http://test.kernel.org/svn/autotest/trunk@2215 592f7852-d20e-0410-864c-8624ca9c26a4
diff --git a/cli/label.py b/cli/label.py
index f8932e7..a4d6ac2 100755
--- a/cli/label.py
+++ b/cli/label.py
@@ -132,9 +132,9 @@
elif not self.all:
results = [label for label in results
if not label['platform']]
- keys = ['name', 'invalid']
+ keys = ['name', 'only_if_needed', 'invalid']
else:
- keys = ['name', 'platform', 'invalid']
+ keys = ['name', 'platform', 'only_if_needed', 'invalid']
super(label_list, self).output(results, keys)
@@ -147,12 +147,17 @@
help='To create this label as a platform',
default=False,
action='store_true')
+ self.parser.add_option('-o', '--only_if_needed',
+ help='To mark the label as "only use if needed',
+ default=False,
+ action='store_true')
def parse(self):
(options, leftover) = super(label_create, self).parse()
self.data_item_key = 'name'
self.data['platform'] = options.platform
+ self.data['only_if_needed'] = options.only_if_needed
return (options, leftover)