Bug 735179 - Review Request: openstack-keystone - OpenStack Identity Service
Summary: Review Request: openstack-keystone - OpenStack Identity Service
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Matt Domsch
QA Contact: Fedora Extras Quality Assurance
Depends On:
TreeView+ depends on / blocked
Reported: 2011-09-01 17:33 UTC by Matt Domsch
Modified: 2011-12-12 21:56 UTC (History)
4 users (show)

Fixed In Version: openstack-keystone-2011.3.1-2.fc16
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2011-12-12 21:56:35 UTC
Type: ---
markmc: fedora-review+
gwync: fedora-cvs+

Attachments (Terms of Use)

Description Matt Domsch 2011-09-01 17:33:29 UTC
Spec URL: http://domsch.com/fedora/openstack/keystone/openstack-keystone.spec
SRPM URL: http://domsch.com/fedora/openstack/keystone/openstack-keystone-1.0-0.1.20110901git396f0bfd.src.rpm
Keystone is a proposed independent authentication service for
OpenStack (http://www.openstack.org).

This initial proof of concept aims to address the current use cases in
Swift and Nova which are:

* REST-based, token auth for Swift
* many-to-many relationship between identity and tenant for Nova.

Comment 1 Mark McLoughlin 2011-09-02 11:44:16 UTC
Looks like a great start, here's what I came up with after a first look:

To fix before approval:

 - invalid-url Source0, we should use one of the tarballs available here: 
   Monty Taylor	looks like the contact point to	get the	tarball	generation
   job fixed:

 - python_sitelib not needed any more:

 - BR python2-devel required:

 - These deps all look to be just needed for the unit tests:
     BuildRequires: python-coverage python-webtest python-unittest2 python-pep8
   we could try running the tests in %check, but otherwise drop those BR
   I think

 - Don't need to clean buildroot in %install any more:

 - Need	to deal	with config file, log file, user/group creation, systemd units


 - http://keystone.openstack.org/ is a better URL?
 - It's	a style	preference, I guess, but I find	having only a single package 
   listed for each Requires: tag as more readable

To bugzilla once the package has been added:

 - bugzilla for	no-manual-page-for-binary
 - request LICENSE be included in the distribution
 - SELinux policy
 - uidgid reservation

I've pushed my attempt to fix these up here:


Comment 2 Matt Domsch 2011-09-09 20:30:26 UTC
I've applied Mark's patches, and updated to newer upstream tarball that incorporates what was patch1 already.



Thanks to Mark for these patches.

Comment 3 Mark McLoughlin 2011-09-13 15:09:46 UTC
Cool stuff, LGTM!

Matt - you'll bugzilla those four last items above, right?

Comment 4 Matt Domsch 2011-09-26 20:55:07 UTC
yes, I'll file bugzillas for:
- no-manual-page-for-binary
- SELinux policy
- uidgid reservation

I'll fix the spec to include the LICENSE file in %doc, it does exist in upstream.

Comment 5 Matt Domsch 2011-10-03 21:31:47 UTC
Hmm, LICENSE disappeared from upstream.  I'll bz that.

New build which is the final diablo release.  Dropped the two patches, added BR: systemd-units for _unitdir.

http://domsch.com/fedora/openstack/keystone/openstack-keystone-1.0-0.3.d4.1213.fc14.src.rpm  (builds against rawhide, not f14 due to systemd stuff)

Comment 6 Matt Domsch 2011-10-04 02:45:57 UTC
here's the scratch build against rawhide (f17)

Comment 7 Mark McLoughlin 2011-10-04 11:13:05 UTC
Matt, the package is approved - can we not just do the SCM admin request and get it imported?

Comment 8 Matt Domsch 2011-10-04 13:00:04 UTC
New Package SCM Request
Package Name: openstack-keystone
Short Description: OpenStack Identity Service
Owners: mdomsch markmc pbrady
Branches: f15 f16 el6

Comment 9 Gwyn Ciesla 2011-10-04 13:08:40 UTC
Git done (by process-git-requests).

Comment 10 Matt Domsch 2011-10-04 13:24:58 UTC
built in koji for rawhide and f16.

Comment 11 Fedora Update System 2011-10-04 13:34:36 UTC
openstack-keystone-1.0-0.3.d4.1213.fc16 has been submitted as an update for Fedora 16.

Comment 12 Mark McLoughlin 2011-10-04 20:24:19 UTC
Nice one Matt, cool stuff!

Comment 13 Fedora Update System 2011-10-04 20:48:38 UTC
openstack-keystone-1.0-0.3.d4.1213.fc16 has been pushed to the Fedora 16 testing repository.

Comment 14 Fedora Update System 2011-10-19 15:51:03 UTC
openstack-keystone-1.0-0.4.d4.1213.fc16 has been submitted as an update for Fedora 16.

Comment 15 Fedora Update System 2011-11-24 22:29:25 UTC
openstack-keystone-2011.3.1-2.fc16 has been submitted as an update for Fedora 16.

Comment 16 Fedora Update System 2011-12-12 21:56:35 UTC
openstack-keystone-2011.3.1-2.fc16 has been pushed to the Fedora 16 stable repository.  If problems still persist, please make note of it in this bug report.

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