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
diff --git a/charms/focal/autopkgtest-cloud-worker/autopkgtest-cloud/worker/worker b/charms/focal/autopkgtest-cloud-worker/autopkgtest-cloud/worker/worker
index fbd5383..ffae8f0 100755
--- a/charms/focal/autopkgtest-cloud-worker/autopkgtest-cloud/worker/worker
+++ b/charms/focal/autopkgtest-cloud-worker/autopkgtest-cloud/worker/worker
@@ -70,6 +70,7 @@ hostname = socket.gethostname()
70big_packages = set()70big_packages = set()
71long_tests = set()71long_tests = set()
72never_run = set()72never_run = set()
73esm_specials = set()
7374
74ARCH_RELEASE_ALLOW_MAPPING = {75ARCH_RELEASE_ALLOW_MAPPING = {
75 "trusty": ["amd64", "i386"],76 "trusty": ["amd64", "i386"],
@@ -291,13 +292,14 @@ def read_per_package_configs(cfg):
291 out.add(f"{package}/{arch}/{release}")292 out.add(f"{package}/{arch}/{release}")
292 return out293 return out
293294
294 global big_packages, long_tests, never_run295 global big_packages, long_tests, never_run, esm_specials
295296
296 dir = cfg.get("autopkgtest", "per_package_config_dir").strip()297 dir = cfg.get("autopkgtest", "per_package_config_dir").strip()
297298
298 big_packages = read_per_package_file(os.path.join(dir, "big_packages"))299 big_packages = read_per_package_file(os.path.join(dir, "big_packages"))
299 long_tests = read_per_package_file(os.path.join(dir, "long_tests"))300 long_tests = read_per_package_file(os.path.join(dir, "long_tests"))
300 never_run = read_per_package_file(os.path.join(dir, "never_run"))301 never_run = read_per_package_file(os.path.join(dir, "never_run"))
302 esm_specials = read_per_package_file(os.path.join(dir, "esm_specials"))
301303
302304
303def request_matches_per_package(package, arch, release, s):305def request_matches_per_package(package, arch, release, s):
@@ -655,6 +657,9 @@ def request(msg):
655 if (657 if (
656 release.lower() in ARCH_RELEASE_ALLOW_MAPPING658 release.lower() in ARCH_RELEASE_ALLOW_MAPPING
657 and architecture not in ARCH_RELEASE_ALLOW_MAPPING[release.lower()]659 and architecture not in ARCH_RELEASE_ALLOW_MAPPING[release.lower()]
660 and not request_matches_per_package(
661 pkgname, architecture, release, esm_specials
662 )
658 ):663 ):
659 os.makedirs(out_dir)664 os.makedirs(out_dir)
660 # these will be written later on665 # these will be written later on

Subscribers

People subscribed via source and target branches