Add support for an --image flag to atest.

frontend/afe/models.py

Index: frontend/afe/models.py
diff --git a/frontend/afe/models.py b/frontend/afe/models.py
index 33ffb8ca785e979c3e170b157ded4e8be33a1610..ad9d739b4b60e12d0f7a1d4f76112899406140d6 100644
--- a/frontend/afe/models.py
+++ b/frontend/afe/models.py
@@ -993,9 +993,15 @@ class Job(dbmodels.Model, model_logic.ModelExtensions):
 
         control_file = options.get('control_file')
         parameterized_job = options.get('parameterized_job')
-        cls.check_parameterized_job(control_file=control_file,
-                                    parameterized_job=parameterized_job)
 
+        # The current implementation of parameterized jobs requires that only
+        # control files or parameterized jobs are used. Using the image
+        # parameter on autoupdate_ParameterizedJob doesn't mix pure
+        # parameterized jobs and control files jobs, it does muck enough with
+        # normal jobs by adding a parameterized id to them that this check will
+        # fail. So for now we just skip this check.
+        # cls.check_parameterized_job(control_file=control_file,
+        #                             parameterized_job=parameterized_job)
         user = User.current_user()
         if options.get('reboot_before') is None:
             options['reboot_before'] = user.get_reboot_before_display()
@@ -1032,8 +1038,14 @@ class Job(dbmodels.Model, model_logic.ModelExtensions):
 
 
     def save(self, *args, **kwargs):
-        self.check_parameterized_job(control_file=self.control_file,
-                                     parameterized_job=self.parameterized_job)
+        # The current implementation of parameterized jobs requires that only
+        # control files or parameterized jobs are used. Using the image
+        # parameter on autoupdate_ParameterizedJob doesn't mix pure
+        # parameterized jobs and control files jobs, it does muck enough with
+        # normal jobs by adding a parameterized id to them that this check will
+        # fail. So for now we just skip this check.
+        # cls.check_parameterized_job(control_file=control_file,
+        #                             parameterized_job=parameterized_job)

[truty1, 2011/02/01 01:21:31]

I'm going to point out something I noticed when patching.  You've commented out
a line that had 'self' and replaced it with a line that has 'cls'. I'm guessing
from the earlier update. Maybe this will be confusing for someone undoing this
at a later point.

         super(Job, self).save(*args, **kwargs)