Drop obsolete adt-britney autopkgtest code

Now that we look at autopkgtest results from swift we can drop the
adt-britney/lp:auto-package-testing code from autopkgtest.py.

Note that we still need it for boottest.py.

autopkgtest.py

@@ -1,9 +1,10 @@
 # -*- coding: utf-8 -*-
 
 # Copyright (C) 2013 Canonical Ltd.
-# Author: Colin Watson <cjwatson@ubuntu.com>
+# Authors:
-# Partly based on code in auto-package-testing by
+#   Colin Watson <cjwatson@ubuntu.com>
-# Jean-Baptiste Lallement <jean-baptiste.lallement@canonical.com>
+#   Jean-Baptiste Lallement <jean-baptiste.lallement@canonical.com>
+#   Martin Pitt <martin.pitt@ubuntu.com>
 
 # This program is free software; you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by
@@ -17,12 +18,7 @@
 
 from __future__ import print_function
 
-from collections import defaultdict
-from contextlib import closing
 import os
-import subprocess
-import tempfile
-from textwrap import dedent
 import time
 import json
 import tarfile
@@ -37,8 +33,6 @@ import kombu
 from consts import (AUTOPKGTEST, BINARIES, RDEPENDS, SOURCE, VERSION)
 
 
-adt_britney = os.path.expanduser("~/auto-package-testing/jenkins/adt-britney")
-
 ADT_EXCUSES_LABELS = {
     "PASS": '<span style="background:#87d96c">Pass</span>',
     "ALWAYSFAIL": '<span style="background:#e5c545">Always failed</span>',
@@ -83,8 +77,6 @@ class AutoPackageTest(object):
         self.distribution = distribution
         self.series = series
         self.debug = debug
-        self.read()
-        self.rc_path = None  # for adt-britney, obsolete
         self.test_state_dir = os.path.join(britney.options.unstable,
                                            'autopkgtest')
         # map of requested tests from request()
@@ -376,92 +368,6 @@ class AutoPackageTest(object):
                                 result.add((src, arch))
         return result
 
-    #
-    # obsolete adt-britney helpers
-    #
-
-    def _ensure_rc_file(self):
-        if self.rc_path:
-            return
-        self.rc_path = os.path.expanduser(
-            "~/proposed-migration/autopkgtest/rc.%s" % self.series)
-        with open(self.rc_path, "w") as rc_file:
-            home = os.path.expanduser("~")
-            print(dedent("""\
-                release: %s
-                aptroot: ~/.chdist/%s-proposed-amd64/
-                apturi: file:%s/mirror/%s
-                components: main restricted universe multiverse
-                rsync_host: rsync://tachash.ubuntu-ci/adt/
-                datadir: ~/proposed-migration/autopkgtest/data""" %
-                         (self.series, self.series, home, self.distribution)),
-                         file=rc_file)
-
-    @property
-    def _request_path(self):
-        return os.path.expanduser(
-            "~/proposed-migration/autopkgtest/work/adt.request.%s" %
-            self.series)
-
-    @property
-    def _result_path(self):
-        return os.path.expanduser(
-            "~/proposed-migration/autopkgtest/work/adt.result.%s" %
-            self.series)
-
-    def _parse(self, path):
-        if os.path.exists(path):
-            with open(path) as f:
-                for line in f:
-                    line = line.strip()
-                    if line.startswith("Suite:") or line.startswith("Date:"):
-                        continue
-                    linebits = line.split()
-                    if len(linebits) < 2:
-                        print("W: Invalid line format: '%s', skipped" % line)
-                        continue
-                    yield linebits
-
-    def read(self):
-        '''Loads a list of results
-
-        This function loads a list of results returned by __parse() and builds
-        2 lists:
-            - a list of source package/version with all the causes that
-            triggered a test and the result of the test for this trigger.
-            - a list of packages/version that triggered a test with the source
-            package/version and result triggered by this package.
-        These lists will be used in result() called from britney.py to generate
-        excuses and now which uploads passed, caused regression or which tests
-        have always been failing
-        '''
-        self.pkglist = defaultdict(dict)
-        self.pkgcauses = defaultdict(lambda: defaultdict(list))
-        for linebits in self._parse(self._result_path):
-            (src, ver, status) = linebits[:3]
-
-            if not (src in self.pkglist and ver in self.pkglist[src]):
-                self.pkglist[src][ver] = {
-                    "status": status,
-                    "causes": {}
-                }
-
-            i = iter(linebits[3:])
-            for trigsrc, trigver in zip(i, i):
-                self.pkglist[src][ver]['causes'].setdefault(
-                    trigsrc, []).append((trigver, status))
-                self.pkgcauses[trigsrc][trigver].append((status, src, ver))
-
-    def _adt_britney(self, *args):
-        command = [
-            adt_britney,
-            "-c", self.rc_path, "-r", self.series, "-PU",
-            ]
-        if self.debug:
-            command.append("-d")
-        command.extend(args)
-        subprocess.check_call(command)
-
     #
     # Public API
     #
@@ -485,60 +391,6 @@ class AutoPackageTest(object):
                         self.log_verbose('Requesting %s/%s/%s autopkgtest to verify %s' %
                                          (src, ver, arch, ', '.join(['%s/%s' % i for i in triggers])))
 
-        # deprecated requests for old Jenkins/lp:auto-package-testing, will go
-        # away
-
-        self._ensure_rc_file()
-        request_path = self._request_path
-        if os.path.exists(request_path):
-            os.unlink(request_path)
-        with closing(tempfile.NamedTemporaryFile(mode="w")) as request_file:
-            for src, ver in packages:
-                if src in self.pkglist and ver in self.pkglist[src]:
-                    continue
-                print("%s %s" % (src, ver), file=request_file)
-            request_file.flush()
-            self._adt_britney("request", "-O", request_path, request_file.name)
-
-        # Remove packages that have been identified as invalid candidates for
-        # testing from the request file i.e run_autopkgtest = False
-        with open(request_path, 'r') as request_file:
-            lines = request_file.readlines()
-        with open(request_path, 'w') as request_file:
-            for line in lines:
-                src = line.split()[0]
-                if src not in excludes:
-                    request_file.write(line)
-                else:
-                    if self.britney.options.verbose:
-                        self.log_verbose("Requested autopkgtest for %s but "
-                                         "run_autopkgtest set to False" % src)
-
-        for linebits in self._parse(request_path):
-            # Make sure that there's an entry in pkgcauses for each new
-            # request, so that results() gives useful information without
-            # relying on the submit/collect cycle.  This improves behaviour
-            # in dry-run mode.
-            src = linebits.pop(0)
-            ver = linebits.pop(0)
-            if self.britney.options.verbose:
-                self.log_verbose("Requested autopkgtest for %s_%s (%s)" %
-                                 (src, ver, " ".join(linebits)))
-            try:
-                status = linebits.pop(0).upper()
-                while True:
-                    trigsrc = linebits.pop(0)
-                    trigver = linebits.pop(0)
-                    for status, csrc, cver in self.pkgcauses[trigsrc][trigver]:
-                        if csrc == trigsrc and cver == trigver:
-                            break
-                    else:
-                        self.pkgcauses[trigsrc][trigver].append(
-                            (status, src, ver))
-            except IndexError:
-                # End of the list
-                pass
-
     def submit(self):
         # send AMQP requests for new test requests
         # TODO: Once we support version constraints in AMQP requests, add them
@@ -580,13 +432,6 @@ class AutoPackageTest(object):
         # mark them as pending now
         self.update_pending_tests()
 
-        # deprecated requests for old Jenkins/lp:auto-package-testing, will go
-        # away
-        self._ensure_rc_file()
-        request_path = self._request_path
-        if os.path.exists(request_path):
-            self._adt_britney("submit", request_path)
-
     def collect(self, packages):
         # fetch results from swift
         try:
@@ -626,23 +471,6 @@ class AutoPackageTest(object):
             # new results remove pending requests, update the on-disk cache
             self.update_pending_tests()
 
-        # deprecated results for old Jenkins/lp:auto-package-testing, will go
-        # away
-        self._ensure_rc_file()
-        result_path = self._result_path
-        self._adt_britney("collect", "-O", result_path)
-        self.read()
-        if self.britney.options.verbose:
-            for src in sorted(self.pkglist):
-                for ver in sorted(self.pkglist[src],
-                                  cmp=apt_pkg.version_compare):
-                    for trigsrc in sorted(self.pkglist[src][ver]['causes']):
-                        for trigver, status \
-                                in self.pkglist[src][ver]['causes'][trigsrc]:
-                            self.log_verbose("Collected autopkgtest status "
-                                             "for %s_%s/%s_%s: " "%s" %
-                                             (src, ver, trigsrc, trigver, status))
-
     def results(self, trigsrc, trigver):
         '''Return test results for triggering package
 

tests/test_autopkgtest.py

@@ -43,16 +43,6 @@ class TestAutoPkgTest(TestBase):
             else:
                 sys.stdout.write(line)
 
-        # fake adt-britney script; necessary until we drop that code
-        self.adt_britney = os.path.join(
-            self.data.home, 'auto-package-testing', 'jenkins', 'adt-britney')
-        os.makedirs(os.path.dirname(self.adt_britney))
-        with open(self.adt_britney, 'w') as f:
-            f.write('''#!/bin/sh -e
-touch $HOME/proposed-migration/autopkgtest/work/adt.request.series
-echo "$@" >> /%s/adt-britney.log ''' % self.data.path)
-        os.chmod(self.adt_britney, 0o755)
-
         # add a bunch of packages to testing to avoid repetition
         self.data.add('libc6', False)
         self.data.add('libgreen1', False, {'Source': 'green',