Bug 732692

Summary:	Get correct version of python-xattr included in Fedora for glance image caching
Product:	[Fedora] Fedora	Reporter:	Mark McLoughlin <markmc>
Component:	openstack-glance	Assignee:	Russell Bryant <rbryant>
Status:	CLOSED UPSTREAM	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	16	CC:	kevin, markmc, mszpak, rbryant
Target Milestone:	---	Flags:	markmc: fedora_requires_release_note- markmc: needinfo-
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2012-01-25 21:03:03 UTC	Type:	---
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:	781838, 782622
Bug Blocks:

Description Mark McLoughlin 2011-08-23 10:26:03 UTC

Fedora's pyxattr package packages this upstream:

http://pyxattr.sourceforge.net/

For its image caching support (which is disabled by default), glance requires this version:

http://pypi.python.org/pypi/xattr

The only complication is that our pyxattr includes these lines:

  #python-xattr name was used in DAG repository
  Provides:       python-xattr = %{version}-%{release}
  Obsoletes:      python-xattr <= %{version}-%{release}

Comment 1 Marcin Zajaczkowski 2011-08-23 20:24:39 UTC

I think we can safely remove those lines - DAG's repository was merged into rpmfusion in 2005 and doesn't provide that package for Fedora 10+ anymore.

Nevertheless there would be still a problem to provide xattr (pypi) as python-xattr. I didn't make a test, but that new package could try to replace pyxattr (has higher version number) in case of the systems that are not updated to pyxattr without mentioned lines.

But maybe there is some clever solution which allows to mitigate that problem.

Comment 2 Russell Bryant 2012-01-04 23:50:38 UTC

Would this be an acceptable approach?

1) Update pyxattr to remove the Provides/Obsoletes lines for python-xattr.  This would be in version X of the pyxattr package.

2) Create a new package, python-xattr, which Conflicts with pyxattr <= X.

Comment 3 Marcin Zajaczkowski 2012-01-05 00:24:39 UTC

Haven't tested that solution, but seems logical for me.

> 2) Create a new package, python-xattr, which Conflicts with pyxattr <= X.

Did you mean "pyxattr < X"?

Comment 4 Mark McLoughlin 2012-01-05 07:52:06 UTC

The question is whether, given that pyxattr is already installed, yum's depsolver will take a transaction which includes python-xattr and resolve that to include latest pyxattr. We're assuming it will look at latest python-xattr, see the Conflicts and resolve those by updating pyxattr.

One thing that might happen is that 'yum install python-xattr', or 'yum install foo' where foo has 'Requires: python-xattr', would not result in newer python-xattr being installed because older pyxattr is already installed. That's not a big problem, though, we would just make sure that glance has 'Requires: python-xattr >= 0.6.2'

So, yes - I think it'll work too :)

Comment 5 Russell Bryant 2012-01-05 13:29:00 UTC

(In reply to comment #3)
> Haven't tested that solution, but seems logical for me.
> 
> > 2) Create a new package, python-xattr, which Conflicts with pyxattr <= X.
> 
> Did you mean "pyxattr < X"?

Yes.  :-)

Comment 6 Russell Bryant 2012-01-05 13:29:59 UTC

Thanks for the feedback.  I'll proceed down this path as time permits.

Comment 7 Russell Bryant 2012-01-15 16:21:39 UTC

I have a proposed python-xattr ready to submit for review, pending the resolution of bug 781838.

  http://fedorapeople.org/~russellb/python-xattr/

Comment 8 Russell Bryant 2012-01-25 17:57:06 UTC

Thanks to a tip from pbrady, I took a look at this issue from a different angle today.  All of what we have discussed here will no longer be necessary if this patch goes in:

https://review.openstack.org/#change,3414

Comment 9 Russell Bryant 2012-01-25 21:03:03 UTC

That patch has gone into glance, so at least as of Essex, glance will work with the version of pyxattr already in Fedora.