Merge autopkgtest-cloud:esm-specials into autopkgtest-cloud:master

Proposed by Brian Murray
Status: Merged
Merged at revision: 7e24d99a9113ae6f02d5e482888d0a1066189bf2
Proposed branch: autopkgtest-cloud:esm-specials
Merge into: autopkgtest-cloud:master
Diff against target: 38 lines (+6/-1)
1 file modified
charms/focal/autopkgtest-cloud-worker/autopkgtest-cloud/worker/worker (+6/-1)
Reviewer Review Type Date Requested Status
Tim Andersson Approve
Skia Approve
Review via email: mp+458030@code.launchpad.net

Description of the change

This adds support for a package config file which contains a list of packages and arches whose tests should run for ESM releases even though those arches are not supported in ESM. This is useful for packages like ubuntu-advantage-tools which end up following the regular SRU process even though the release is out of standard support.

To post a comment you must log in.
Revision history for this message
Brian Murray (brian-murray) wrote :

I've already added esm_specials to autopkgtest-package-configs.

Revision history for this message
Brian Murray (brian-murray) wrote :

Once this is merged and deployed we should run some ubuntu-advantage-tools tests again.

Revision history for this message
Skia (hyask) wrote :

Link to esm_specials in autopkgtest-package-configs for other reviewers searching it: https://git.launchpad.net/~ubuntu-release/autopkgtest-cloud/+git/autopkgtest-package-configs/commit/?id=706cc06102b7dc4e1f46f4847f297551fe56b7f9

Looks far better than previous proposal!

review: Approve
Revision history for this message
Tim Andersson (andersson123) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/charms/focal/autopkgtest-cloud-worker/autopkgtest-cloud/worker/worker b/charms/focal/autopkgtest-cloud-worker/autopkgtest-cloud/worker/worker
2index fbd5383..ffae8f0 100755
3--- a/charms/focal/autopkgtest-cloud-worker/autopkgtest-cloud/worker/worker
4+++ b/charms/focal/autopkgtest-cloud-worker/autopkgtest-cloud/worker/worker
5@@ -70,6 +70,7 @@ hostname = socket.gethostname()
6 big_packages = set()
7 long_tests = set()
8 never_run = set()
9+esm_specials = set()
10
11 ARCH_RELEASE_ALLOW_MAPPING = {
12 "trusty": ["amd64", "i386"],
13@@ -291,13 +292,14 @@ def read_per_package_configs(cfg):
14 out.add(f"{package}/{arch}/{release}")
15 return out
16
17- global big_packages, long_tests, never_run
18+ global big_packages, long_tests, never_run, esm_specials
19
20 dir = cfg.get("autopkgtest", "per_package_config_dir").strip()
21
22 big_packages = read_per_package_file(os.path.join(dir, "big_packages"))
23 long_tests = read_per_package_file(os.path.join(dir, "long_tests"))
24 never_run = read_per_package_file(os.path.join(dir, "never_run"))
25+ esm_specials = read_per_package_file(os.path.join(dir, "esm_specials"))
26
27
28 def request_matches_per_package(package, arch, release, s):
29@@ -655,6 +657,9 @@ def request(msg):
30 if (
31 release.lower() in ARCH_RELEASE_ALLOW_MAPPING
32 and architecture not in ARCH_RELEASE_ALLOW_MAPPING[release.lower()]
33+ and not request_matches_per_package(
34+ pkgname, architecture, release, esm_specials
35+ )
36 ):
37 os.makedirs(out_dir)
38 # these will be written later on

Subscribers

People subscribed via source and target branches