diff options
author | Alexander Kanavin <alexander.kanavin@linux.intel.com> | 2017-03-16 15:19:04 +0200 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2017-03-17 16:53:04 +0000 |
commit | 2b77735c72cb5100dbd850e79d4f639e82629842 (patch) | |
tree | 859dde4d0a69b2392ad36d8278dc92d1d8b23554 | |
parent | cb05db0d034494ab2dfba6797225d18ac7b6a105 (diff) | |
download | poky-2b77735c72cb5100dbd850e79d4f639e82629842.tar.gz |
rpm: add support for remote package feeds via PACKAGE_FEED_URIS variable
I've used a previous patch (which was never merged) by
Humberto Ibarra <humberto.ibarra.lopez@intel.com> as a model
for how to do runtime testing of this feature (e.g. we need to boot
an image, run dnf on it, and check that it is indeed able to
access the remote repo over http). Here's his original commit message:
=====
Testing that feeds specified with PACKAGE_FEED_URIS var are set
correctly has two parts. First a build with this var set is required,
and then smart update needs to be issued in the running taget.
The previous is not a common selftest practice because this is a
simple test, but requires building and running a specific image,
which takes a lot of time. testimage is not a good fit either,
since the images tested there do not have the PACKAGE_FEED_URIS
var set.
For this test, the runtime-test module is being used, which is a
selftest module but runs a testimage command. The var and test
environment were set in runtime-perf.py and the actual test is
done in a new testcase added to meta-selftest layer.
=====
[YOCTO #10872]
(From OE-Core rev: 3a9e2fdef9316e24b52ce99ac355fc2b09786c72)
Signed-off-by: Alexander Kanavin <alexander.kanavin@linux.intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
-rw-r--r-- | meta-selftest/lib/oeqa/runtime/cases/selftest.py | 42 | ||||
-rw-r--r-- | meta/lib/oe/package_manager.py | 12 | ||||
-rw-r--r-- | meta/lib/oeqa/selftest/runtime-test.py | 7 |
3 files changed, 60 insertions, 1 deletions
diff --git a/meta-selftest/lib/oeqa/runtime/cases/selftest.py b/meta-selftest/lib/oeqa/runtime/cases/selftest.py index 19de740623..e4985a6edd 100644 --- a/meta-selftest/lib/oeqa/runtime/cases/selftest.py +++ b/meta-selftest/lib/oeqa/runtime/cases/selftest.py | |||
@@ -1,5 +1,7 @@ | |||
1 | from oeqa.runtime.case import OERuntimeTestCase | 1 | from oeqa.runtime.case import OERuntimeTestCase |
2 | from oeqa.core.decorator.depends import OETestDepends | 2 | from oeqa.core.decorator.depends import OETestDepends |
3 | from oeqa.runtime.cases.dnf import DnfTest | ||
4 | from oeqa.utils.httpserver import HTTPService | ||
3 | 5 | ||
4 | class Selftest(OERuntimeTestCase): | 6 | class Selftest(OERuntimeTestCase): |
5 | 7 | ||
@@ -29,3 +31,43 @@ class Selftest(OERuntimeTestCase): | |||
29 | 31 | ||
30 | (status, output) = self.target.run("socat -V") | 32 | (status, output) = self.target.run("socat -V") |
31 | self.assertNotEqual(status, 0, msg="socat is still installed") | 33 | self.assertNotEqual(status, 0, msg="socat is still installed") |
34 | |||
35 | |||
36 | class DnfSelftest(DnfTest): | ||
37 | |||
38 | @classmethod | ||
39 | def setUpClass(cls): | ||
40 | cls.repo_server = HTTPService(os.path.join(cls.tc.td['WORKDIR'], 'oe-rootfs-repo'), | ||
41 | cls.tc.target.server_ip) | ||
42 | cls.repo_server.start() | ||
43 | |||
44 | @classmethod | ||
45 | def tearDownClass(cls): | ||
46 | cls.repo_server.stop() | ||
47 | |||
48 | @OETestDepends(['ssh.SSHTest.test_ssh']) | ||
49 | def test_verify_package_feeds(self): | ||
50 | """ | ||
51 | Summary: Check correct setting of PACKAGE_FEED_URIS var | ||
52 | Expected: 1. Feeds were correctly set for dnf | ||
53 | 2. Update recovers packages from host's repo | ||
54 | Author: Humberto Ibarra <humberto.ibarra.lopez@intel.com> | ||
55 | Author: Alexander Kanavin <alexander.kanavin@intel.com> | ||
56 | """ | ||
57 | # When we created an image, we had to supply fake ip and port | ||
58 | # for the feeds. Now we can patch the real ones into the config file. | ||
59 | import tempfile | ||
60 | temp_file = tempfile.TemporaryDirectory(prefix="oeqa-remotefeeds-").name | ||
61 | self.tc.target.copyFrom("/etc/yum.repos.d/oe-remote-repo.repo", temp_file) | ||
62 | fixed_config = open(temp_file, "r").read().replace("bogus_ip", self.tc.target.server_ip).replace("bogus_port", str(self.repo_server.port)) | ||
63 | open(temp_file, "w").write(fixed_config) | ||
64 | self.tc.target.copyTo(temp_file, "/etc/yum.repos.d/oe-remote-repo.repo") | ||
65 | |||
66 | import re | ||
67 | output_makecache = self.dnf('makecache') | ||
68 | self.assertTrue(re.match(r".*Metadata cache created", output_makecache, re.DOTALL) is not None, msg = "dnf makecache failed: %s" %(output_makecache)) | ||
69 | |||
70 | output_repoinfo = self.dnf('repoinfo') | ||
71 | matchobj = re.match(r".*Repo-pkgs\s*:\s*(?P<n_pkgs>[0-9]+)", output_repoinfo, re.DOTALL) | ||
72 | self.assertTrue(matchobj is not None, msg = "Could not find the amount of packages in dnf repoinfo output: %s" %(output_repoinfo)) | ||
73 | self.assertTrue(int(matchobj.group('n_pkgs')) > 0, msg = "Amount of remote packages is not more than zero: %s\n" %(output_repoinfo)) | ||
diff --git a/meta/lib/oe/package_manager.py b/meta/lib/oe/package_manager.py index d51609189d..b016bc32dc 100644 --- a/meta/lib/oe/package_manager.py +++ b/meta/lib/oe/package_manager.py | |||
@@ -533,10 +533,20 @@ class RpmPM(PackageManager): | |||
533 | bb.utils.unlockfile(lf) | 533 | bb.utils.unlockfile(lf) |
534 | 534 | ||
535 | def insert_feeds_uris(self, feed_uris, feed_base_paths, feed_archs): | 535 | def insert_feeds_uris(self, feed_uris, feed_base_paths, feed_archs): |
536 | from urllib.parse import urlparse | ||
537 | |||
536 | if feed_uris == "": | 538 | if feed_uris == "": |
537 | return | 539 | return |
538 | 540 | ||
539 | raise NotImplementedError("Adding remote dnf feeds not yet supported.") | 541 | bb.utils.mkdirhier(oe.path.join(self.target_rootfs, "etc", "yum.repos.d")) |
542 | remote_uris = self.construct_uris(feed_uris.split(), feed_base_paths.split()) | ||
543 | for uri in remote_uris: | ||
544 | repo_name = "oe-remote-repo" + "-".join(urlparse(uri).path.split("/")) | ||
545 | if feed_archs is not None: | ||
546 | repo_uris = [uri + "/" + arch for arch in feed_archs] | ||
547 | else: | ||
548 | repo_uris = [uri] | ||
549 | open(oe.path.join(self.target_rootfs, "etc", "yum.repos.d", repo_name + ".repo"), 'w').write("[%s]\nbaseurl=%s\n" % (repo_name, " ".join(repo_uris))) | ||
540 | 550 | ||
541 | def _prepare_pkg_transaction(self): | 551 | def _prepare_pkg_transaction(self): |
542 | os.environ['D'] = self.target_rootfs | 552 | os.environ['D'] = self.target_rootfs |
diff --git a/meta/lib/oeqa/selftest/runtime-test.py b/meta/lib/oeqa/selftest/runtime-test.py index e8b483d7f8..171a37329f 100644 --- a/meta/lib/oeqa/selftest/runtime-test.py +++ b/meta/lib/oeqa/selftest/runtime-test.py | |||
@@ -108,14 +108,21 @@ class TestImage(oeSelfTest): | |||
108 | Summary: Check install packages functionality for testimage/testexport. | 108 | Summary: Check install packages functionality for testimage/testexport. |
109 | Expected: 1. Import tests from a directory other than meta. | 109 | Expected: 1. Import tests from a directory other than meta. |
110 | 2. Check install/uninstall of socat. | 110 | 2. Check install/uninstall of socat. |
111 | 3. Check that remote package feeds can be accessed | ||
111 | Product: oe-core | 112 | Product: oe-core |
112 | Author: Mariano Lopez <mariano.lopez@intel.com> | 113 | Author: Mariano Lopez <mariano.lopez@intel.com> |
114 | Author: Alexander Kanavin <alexander.kanavin@intel.com> | ||
113 | """ | 115 | """ |
114 | if get_bb_var('DISTRO') == 'poky-tiny': | 116 | if get_bb_var('DISTRO') == 'poky-tiny': |
115 | self.skipTest('core-image-full-cmdline not buildable for poky-tiny') | 117 | self.skipTest('core-image-full-cmdline not buildable for poky-tiny') |
116 | 118 | ||
117 | features = 'INHERIT += "testimage"\n' | 119 | features = 'INHERIT += "testimage"\n' |
118 | features += 'TEST_SUITES = "ping ssh selftest"\n' | 120 | features += 'TEST_SUITES = "ping ssh selftest"\n' |
121 | # We don't yet know what the server ip and port will be - they will be patched | ||
122 | # in at the start of the on-image test | ||
123 | features += 'PACKAGE_FEED_URIS = "http://bogus_ip:bogus_port"\n' | ||
124 | features += 'EXTRA_IMAGE_FEATURES += "package-management"\n' | ||
125 | features += 'PACKAGE_CLASSES = "package_rpm"' | ||
119 | self.write_config(features) | 126 | self.write_config(features) |
120 | 127 | ||
121 | # Build core-image-sato and testimage | 128 | # Build core-image-sato and testimage |