Lubuntu/britney2-ubuntu
8
0
Fork 0
mirror of https://git.launchpad.net/~ubuntu-release/britney/+git/britney2-ubuntu synced 2025-03-12 11:51:09 +00:00
Code Issues Packages Projects Releases Wiki Activity

autopkgtest, DependsPolicy: Run tests where we didn't care about the depends problem

Browse Source 
We currently skip running autopkgtests where there is an installability
problem, but in a few cases the depends policy notes these only, but
otherwise doesn't block migration on them.

In these cases, let's try to run the autopkgtests anyway. There will be
a few instances of uninstallability here, but since we migrate the
packages we should give them a chance to be tested.
This commit is contained in:
Iain Lane 2020-07-19 12:44:15 +01:00
parent 94b6ac273a
commit 7d6d4debdf
2 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
britney2/policies/autopkgtest.py
View File

@ -396,7 +396,8 @@ class AutopkgtestPolicy(BasePolicy):
verdict = PolicyVerdict.REJECTED_TEMPORARILY
self.logger.info('%s hasn''t been built on arch %s, delay autopkgtest there', source_name, arch)
excuse.add_verdict_info(verdict, "arch:%s not built yet, autopkgtest delayed there" % arch)
elif arch in excuse.unsatisfiable_on_archs:
elif (arch in excuse.unsatisfiable_on_archs and
arch not in excuse.policy_info['depends'].get('skip_dep_check', [])):
self.logger.info('%s is uninstallable on arch %s, not running autopkgtest there', source_name, arch)
excuse.addinfo("uninstallable on arch %s, not running autopkgtest there" % arch)
else:

4
britney2/policies/policy.py
View File

@ -876,7 +876,9 @@ class DependsPolicy(BasePolicy):
# ...but if the binary is allowed to become uninstallable,
# we don't care
# we still want the binary to be listed as uninstallable,
# so the autopkgtest policy knows not to try to run tests
# let the autopkgtest policy see that we did this so that
# it can run the test anyway
deps_info.setdefault('skip_dep_check', []).append(arch)
continue
verdict = PolicyVerdict.REJECTED_PERMANENTLY
excuse.add_verdict_info(verdict, "%s/%s has unsatisfiable dependency" % (