pip install or python setup.py install should include httpd/keystone.py

Bug #1441733 reported by Haneef Ali
16
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
Invalid
Low
Unassigned
puppet-keystone
Invalid
Undecided
Unassigned

Bug Description

Now the recommended way to install keystone is via apache. But httpd/keystone.py is not included when we do python setup.py install in keystone. It should be included

Revision history for this message
Haneef Ali (haneef) wrote :

Can we move this httpd directory under keystone? We don't need to find a location for install if we move it. Let the distro do a symlink to appropriate place

Changed in keystone:
status: New → Confirmed
importance: Undecided → Low
Changed in keystone:
assignee: nobody → Eric Brown (ericwb)
status: Confirmed → In Progress
Revision history for this message
Dolph Mathews (dolph) wrote :

I've abandoned the referenced changed due to inactivity and failing tests. If this is still an issue, please reset the status.

Changed in keystone:
status: In Progress → Incomplete
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on keystone (master)

Change abandoned by Dolph Mathews (<email address hidden>) on branch: master
Review: https://review.openstack.org/172245
Reason: Abandoning due to inactivity and failing tests.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to keystone (master)

Reviewed: https://review.openstack.org/194442
Committed: https://git.openstack.org/cgit/openstack/keystone/commit/?id=3624c2f5ad20fb3795576b35fd07ceedcc2642ba
Submitter: Jenkins
Branch: master

commit 3624c2f5ad20fb3795576b35fd07ceedcc2642ba
Author: Brant Knudson <email address hidden>
Date: Mon Jun 22 18:37:55 2015 -0500

    Use wsgi_scripts to create admin and public httpd files

    The httpd/keystone.py file needed to be copied and then
    symlinked when used by web servers to "admin" and "main".

    pbr 1.4.0 added support for wsgi_scripts that creates scripts
    for wsgi servers on install. Keystone will now specify
    wsgi_scripts so that the admin (keystone-wsgi-admin) and
    main (keystone-wsgi-public) scripts will be created on install.

    See http://httpd.apache.org/docs/2.4/upgrading.html#access for
    the apache docs with examples for the Allow/Deny/Require
    directives.

    DocImpact

    Related-Bug: #1441733
    Change-Id: Ic9c03e6c00408f3698c10012ca98cfc6ea9b6ace

Revision history for this message
Richard Megginson (rmeggins) wrote :

On RedHat family platforms, the path to the wsgi script is hard coded. Not only that, but now there are two different and apparently not interchangeable scripts, one for admin and one for public (still with this distinction without a difference anymore!!!), which means additional parameters/variables for the keystone::params and keystone::wsgi::apache

Revision history for this message
Alan Pevec (apevec) wrote :

Not only Red Hat, Debian also shipped example httpd/keystone.py wsgi script as /usr/share/keystone/wsgi.py

https://github.com/openstack/puppet-keystone/blob/master/manifests/params.pp#L17

Alan Pevec (apevec)
Changed in puppet-keystone:
status: New → Confirmed
Eric Brown (ericwb)
Changed in keystone:
assignee: Eric Brown (ericwb) → nobody
Revision history for this message
Morgan Fainberg (mdrnstm) wrote :

This has been mostly eliminated as the wsgi scripts are now generated by pbr.

Changed in keystone:
status: Incomplete → Invalid
Revision history for this message
Matt Fischer (mfisch) wrote :

Puppet stopped shipping this script, we get it from the packages or the code depending on how you install, so this bug no longer applies to us.

Changed in puppet-keystone:
status: Confirmed → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.