Bug 1319873 - vdsm FCoE hook should bring fcoe-utils as dependency
Summary: vdsm FCoE hook should bring fcoe-utils as dependency
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: vdsm
Classification: oVirt
Component: Packaging.rpm
Version: 4.18.0
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: ovirt-4.0.0-rc
: 4.18.0
Assignee: Pavel Zhukov
QA Contact: Elad
URL:
Whiteboard:
Depends On:
Blocks: RHEV_FCOE 1334745 1334748 1338795
TreeView+ depends on / blocked
 
Reported: 2016-03-21 17:45 UTC by Pavel Zhukov
Modified: 2016-08-01 12:27 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-08-01 12:27:46 UTC
oVirt Team: Storage
Embargoed:
rule-engine: ovirt-4.0.0+
ylavi: planning_ack+
rule-engine: devel_ack+
acanan: testing_ack+


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 58446 0 ovirt-3.6 MERGED hooks: Fix typo in fcoe hook properties 2016-06-01 14:22:59 UTC
oVirt gerrit 58449 0 ovirt-4.0 MERGED hooks: Fix typo in fcoe hook properties 2016-06-01 14:22:53 UTC

Description Pavel Zhukov 2016-03-21 17:45:25 UTC
Description of problem:
To properly handle fcoe storages vdsm should depends on fcoe-utils. 
ovirt-node iso ships fcoe-utils and lldpad so fcoe can be easily configured but normal linux host will require to install the package manually

Version-Release number of selected component (if applicable):
vdsm-4.17.23-0.el7ev.noarch

Actual results:
fcoe-utils not installed

Comment 1 Nir Soffer 2016-03-23 17:12:15 UTC
(In reply to Pavel Zhukov from comment #0)
> Description of problem:
> To properly handle fcoe storages vdsm should depends on fcoe-utils. 
> ovirt-node iso ships fcoe-utils and lldpad so fcoe can be easily configured
> but normal linux host will require to install the package manually
> 
> Version-Release number of selected component (if applicable):
> vdsm-4.17.23-0.el7ev.noarch
> 
> Actual results:
> fcoe-utils not installed

Pavel, looking at your patch:
https://gerrit.ovirt.org/#/c/55029/11/vdsm.spec.in

The vdsm-hook-fcoe package is requiring fcoe-utils, so after installing this hook,
the package should be available on the hypervisor.

Vdsm itself does not require this package because it does not need it, and 
does not interact with any of the fcoe facilities.

Can you explain why Vdsm should require this package?

Comment 2 Pavel Zhukov 2016-03-31 06:56:01 UTC
(In reply to Nir Soffer from comment #1)
> (In reply to Pavel Zhukov from comment #0)
> > Description of problem:
> > To properly handle fcoe storages vdsm should depends on fcoe-utils. 
> > ovirt-node iso ships fcoe-utils and lldpad so fcoe can be easily configured
> > but normal linux host will require to install the package manually
> > 
> > Version-Release number of selected component (if applicable):
> > vdsm-4.17.23-0.el7ev.noarch
> > 
> > Actual results:
> > fcoe-utils not installed
> 
> Pavel, looking at your patch:
> https://gerrit.ovirt.org/#/c/55029/11/vdsm.spec.in
> 
> The vdsm-hook-fcoe package is requiring fcoe-utils, so after installing this
> hook,
> the package should be available on the hypervisor.
> 
> Vdsm itself does not require this package because it does not need it, and 
> does not interact with any of the fcoe facilities.
> 
> Can you explain why Vdsm should require this package?

Hi Nir,

I'd filed the bug before started to work on the hook. Should I attach gerrit change to this bug or is it better close it?

Comment 3 Allon Mureinik 2016-04-11 12:26:34 UTC
(In reply to Pavel Zhukov from comment #2)
> (In reply to Nir Soffer from comment #1)
> > (In reply to Pavel Zhukov from comment #0)
> > > Description of problem:
> > > To properly handle fcoe storages vdsm should depends on fcoe-utils. 
> > > ovirt-node iso ships fcoe-utils and lldpad so fcoe can be easily configured
> > > but normal linux host will require to install the package manually
> > > 
> > > Version-Release number of selected component (if applicable):
> > > vdsm-4.17.23-0.el7ev.noarch
> > > 
> > > Actual results:
> > > fcoe-utils not installed
> > 
> > Pavel, looking at your patch:
> > https://gerrit.ovirt.org/#/c/55029/11/vdsm.spec.in
> > 
> > The vdsm-hook-fcoe package is requiring fcoe-utils, so after installing this
> > hook,
> > the package should be available on the hypervisor.
> > 
> > Vdsm itself does not require this package because it does not need it, and 
> > does not interact with any of the fcoe facilities.
> > 
> > Can you explain why Vdsm should require this package?
> 
> Hi Nir,
> 
> I'd filed the bug before started to work on the hook. Should I attach gerrit
> change to this bug or is it better close it?
Let's keep it to track the fact that the hook should require the fcoe-utils package.

Comment 4 Sandro Bonazzola 2016-05-02 10:03:07 UTC
Moving from 4.0 alpha to 4.0 beta since 4.0 alpha has been already released and bug is not ON_QA.

Comment 5 Yaniv Lavi 2016-05-23 13:18:20 UTC
oVirt 4.0 beta has been released, moving to RC milestone.

Comment 6 Yaniv Lavi 2016-05-23 13:22:11 UTC
oVirt 4.0 beta has been released, moving to RC milestone.

Comment 7 Nir Soffer 2016-05-23 14:15:03 UTC
This is already merged - the author should add also this bug url.

Installing the fcoe hook will install fcoe-utils.

commit 16cb9db677c545487ee009fef5b612c0359c8d18
Author:     Pavel Zhukov <pzhukov>
AuthorDate: Mon Mar 21 19:24:59 2016 +0100
Commit:     Dan Kenigsberg <danken>
CommitDate: Mon May 16 03:43:24 2016 -0400

    hooks: Add fcoe hook
    
    This hook is used to enable fcoe on one or several network interfaces
    
    Change-Id: Iad2faed7205ca08801132df072b469dbe781318c
    Bug-Url: https://bugzilla.redhat.com/1334745
    Signed-off-by: Pavel Zhukov <pzhukov>
    Reviewed-on: https://gerrit.ovirt.org/55029
    Reviewed-by: Dan Kenigsberg <danken>
    Continuous-Integration: Jenkins CI

Comment 8 Allon Mureinik 2016-05-23 14:22:11 UTC
This in included in v4.18.0.

Comment 9 Elad 2016-07-14 08:18:32 UTC
vdsm-hook-fcoe requires fcoe-utils as a dependency:

# yum deplist vdsm-hook-fcoe
package: vdsm-hook-fcoe.noarch 4.18.5.1-1.el7ev
  dependency: /bin/env
   provider: coreutils.x86_64 8.22-15.el7_2.1
  dependency: fcoe-utils
   provider: fcoe-utils.x86_64 1.0.30-3.git91c0c8c.el7
  dependency: vdsm = 4.18.5.1-1.el7ev
   provider: vdsm.x86_64 4.18.5.1-1.el7ev


===========================================
Used:
vdsm-4.18.5.1-1.el7ev.x86_64
vdsm-hook-fcoe-4.18.5.1-1.el7ev.noarch
fcoe-utils-1.0.30-3.git91c0c8c.el7.x86_64


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