Enable autotest client package fetching based on updated image.

server/site_autotest.py

-import os
-from autotest_lib.server import installable_object
+import os, urlparse
+from autotest_lib.client.common_lib import global_config
+from autotest_lib.server import autoserv_parser, installable_object
+
+
+parser = autoserv_parser.autoserv_parser
+
 
 class SiteAutotest(installable_object.InstallableObject):
 
     def get(self, location = None):
         if not location:
             location = os.path.join(self.serverdir, '../client')
             location = os.path.abspath(location)
         installable_object.InstallableObject.get(self, location)
         self.got = True
 
 
+    def get_fetch_location(self):
+        c = global_config.global_config

[DaleCurtis, 2011/04/21 19:16:55]

You only use this once, might as well combine with repos =.

+        repos = c.get_config_value("PACKAGES", 'fetch_location', type=list,

[DaleCurtis, 2011/04/21 19:16:55]

Single quotes.

+                                   default=[])
+        new_repos = []
+        for repo in repos[::-1]:
+            if repo.endswith('static/archive') and parser.options.image:
+                build = '/'.join(urlparse.urlparse(

[DaleCurtis, 2011/04/21 19:16:55]

No real reason for the urlparse. [-2:] will return the same thing with the full
URL. If you really want to be safe you should add a rstrip('/') to the path.
Otherwise if a image url comes in w/ a trailing slash this will fail.

+                    parser.options.image).path.split('/')[-2:])
+                repo += '/' + build + '/autotest'

[DaleCurtis, 2011/04/21 19:16:55]

'/%s/autotest' % build

+            new_repos.append(repo)
+        repos = new_repos

[DaleCurtis, 2011/04/21 19:16:55]

Just "return new_repos"

+        return repos
+
+
 class _SiteRun(object):
     pass