Bug 1525418

Summary: [rhv] adjust cockpit-storaged's dependencies to the storaged → udisks2 rename
Product: Red Hat Enterprise Linux 7 Reporter: Jiri Belka <jbelka>
Component: cockpitAssignee: Peter <pvolpe>
Status: CLOSED DUPLICATE QA Contact: qe-baseos-daemons
Severity: urgent Docs Contact:
Priority: unspecified    
Version: 7.5CC: mpitt, pvolpe, sbonazzo, yselkowi
Target Milestone: rcKeywords: Regression
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-12-14 20:45:09 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1477736    

Description Jiri Belka 2017-12-13 09:56:17 UTC
Description of problem:

# for i in obsoletes provides ; do rpm -q --${i} udisks2; echo ; done
storaged

udisks2 = 2.7.3-4.el7
udisks2(x86-64) = 2.7.3-4.el7


Originally found when trying install EL 7.5 as RHV 4.2 host:

--->%---
I was trying to install rhv 4.2 on EL 7.5 but it fails:

- cockpit-storaged-155-1.el7.noarch.rpm

  depends on storaged but this package got obsoleted by
  udisks2 and thus version comparison does not match

...
--> Processing Dependency: storaged(x86-64) = 2.5.2-4.el7 for package: storaged-iscsi-2.5.2-4.el7.x86_64
Package storaged is obsoleted by udisks2, but obsoleting package does not provide for requirements
---> Package storaged-lvm2.x86_64 0:2.5.2-4.el7 will be installed
--> Processing Dependency: storaged(x86-64) = 2.5.2-4.el7 for package: storaged-lvm2-2.5.2-4.el7.x86_64
Package storaged is obsoleted by udisks2, but obsoleting package does not provide for requirements
...
--> Finished Dependency Resolution
Error: Package: cockpit-storaged-155-1.el7.noarch (rhv-4.2.0)
           Requires: storaged >= 2.1.1
           Available: storaged-2.5.2-4.el7.x86_64 (rhv-4.2.0)
               storaged = 2.5.2-4.el7
Error: Package: cockpit-storaged-155-1.el7.noarch (rhv-4.2.0)
           Requires: storaged-lvm2 >= 2.1.1
           Available: storaged-lvm2-2.5.2-4.el7.x86_64 (rhv-4.2.0)
               storaged-lvm2 = 2.5.2-4.el7
Error: Package: cockpit-storaged-155-1.el7.noarch (rhv-4.2.0)
           Requires: storaged-iscsi >= 2.1.1
           Available: storaged-iscsi-2.5.2-4.el7.x86_64 (rhv-4.2.0)
               storaged-iscsi = 2.5.2-4.el7
 You could try using --skip-broken to work around the problem
 You could try running: rpm -Va --nofiles --nodigest


# rpm -qRp cockpit-storaged-155-1.el7.noarch.rpm  | grep ^storaged
storaged >= 2.1.1
storaged-iscsi >= 2.1.1
storaged-lvm2 >= 2.1.1

# rpm -q --provides --obsoletes udisks2
udisks2 = 2.7.3-4.el7
udisks2(x86-64) = 2.7.3-4.el7
storaged

My current workaround: install all deps via yum and install cockpit-storaged via rpm
with --nodeps.
---<%---

Version-Release number of selected component (if applicable):
udisks2-2.7.3-4.el7.x86_64

How reproducible:
100%

Steps to Reproduce:
1. install EL 7.5 and try to install ovirt-host package from RHV 4.2(beta)
2.
3.

Actual results:
installation ovirt-host as its dependency cockpit-storaged needs specific storaged package which is obsoletes with udisks2 but udisks2 does not provide 'storaged >= 2.1.1'

Expected results:
udisks2 should has 'provides' for storaged >= 2.1.1

Additional info:

Comment 3 Martin Pitt 2017-12-14 19:49:47 UTC
I disagree - udisks2 does not Provide storage in any meaningful way. Both the D-Bus API and the CLI are different. They were designed to run in parallel with each other, not be drop-in replacements. I thought we already agreed to instead update cockpit-storage's dependencies back to udisks2?

This has happened upstream a fair while ago already. Landing this in RHEL 7.5 Extras has just been blocked by bug 1503100 (udisks2-{lvm2,iscsi} not being available), but recently there has been progress.

Comment 4 Martin Pitt 2017-12-14 19:50:31 UTC
Does anyone object if I reassign this to cockpit instead and update the title accordingly?

Comment 5 Peter 2017-12-14 19:59:40 UTC
We are on the same page. Sorry I didn't word my comment clearer. I meant bring the cockpit-storaged spec file up to date.

Comment 6 Yaakov Selkowitz 2017-12-14 20:35:35 UTC
This is a partial dup to bug 1510667.  The extras-rhel-7.5 branch of cockpit needs to be synced wrt conditionals in <http://pkgs.devel.redhat.com/cgit/rpms/cockpit/tree/cockpit.spec?h=rhel-7.5#n411>.  Please note that the Extras 7.5 Beta compose is imminent, so this needs to be done very quickly in order to make it.

Comment 7 Martin Pitt 2017-12-14 20:45:09 UTC

*** This bug has been marked as a duplicate of bug 1510667 ***

Comment 8 Jiri Belka 2018-04-20 11:55:12 UTC
Just note for RHV users - they pushed udisks2-(iscsi,lvm2) into optional but we don't require it for RHV hosts. Thus it needs to be solved differently - push to RHV channels per host probably.