Bug 1274052 - RepoRequires dependencies are not recursively fulfilled
Summary: RepoRequires dependencies are not recursively fulfilled
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Restraint
Classification: Retired
Component: general
Version: 0.1.21
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: 0.1.25
Assignee: Artem Savkov
QA Contact: tools-bugs
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-10-21 19:09 UTC by Don Zickus
Modified: 2016-04-21 00:08 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Enhancement
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-03-24 10:22:39 UTC
Embargoed:


Attachments (Terms of Use)

Description Don Zickus 2015-10-21 19:09:23 UTC
Description of problem:

I installed a test through restraint kernel/kdump/setup-bare-metal

inside that testinfo.desc is a RepoRequires: kdump/include

restraint happily adds kdump/include to the machine under test.
However, inside kdump/include's testinfo.desc is a Requires: kexec-tools.

kexec-tools is _not_ installed and thus causes issues during testing under restraint.

Under beah this works because everything is rpm based and the rpm deps are solved by yum (which includes fetch kexec-tools) correctly.




Version-Release number of selected component (if applicable):


How reproducible:
Always

Steps to Reproduce:
1.fetch_url git://....#kdump/setup-bare-metal
2.test aborts because kdumpctl is not installed (from package kexec-tools)
3.

Actual results:


Expected results:
kexec-tools is installed correctly

Additional info:

Comment 1 Artem Savkov 2016-03-21 09:57:25 UTC
Proposed patchset fixing this issue:

   http://gerrit.beaker-project.org/4743 Move metadata processing to metadata.c
   http://gerrit.beaker-project.org/4744 Get rid of global session in fetch_http.
   http://gerrit.beaker-project.org/4745 Fix rt_sigaction valgrind suppression.
   http://gerrit.beaker-project.org/4746 Prepare test_dependency for recursive repodeps.
   http://gerrit.beaker-project.org/4747 Add common 'test' to test-data.
   http://gerrit.beaker-project.org/4748 Recursively check repodeps.
   http://gerrit.beaker-project.org/4749 Add recursive dependency tests.

Comment 2 Artem Savkov 2016-03-24 10:22:39 UTC
merged


Note You need to log in before you can comment on or make changes to this bug.