API endpoint default URL's are unversioned

Bug #1287520 reported by Mark T. Voelker
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
puppet-keystone
Fix Released
Undecided
Mark T. Voelker
Havana
Fix Released
Undecided
Unassigned

Bug Description

In the current version of the keystone::init class, the default values for keystone endpoint URLs are unversioned, which can lead to problems connecting to the endpoint if the user doesn't provide a value. The callsign is errors like this:

Error: /Stage[main]/Keystone::Roles::Admin/Keystone_role[_member_]: Could not evaluate: Execution of '/usr/bin/keystone --os-endpoint http://192.168.255.133:35357 role-list' returned 1: The resource could not be found. (HTTP 404)

Note the URL in the error message: it should be something like "http://192.168.255.133:35357/v2.0/".

Changed in puppet-keystone:
status: New → In Progress
assignee: nobody → Mark T. Voelker (mvoelker)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to puppet-keystone (master)

Fix proposed to branch: master
Review: https://review.openstack.org/77785

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to puppet-keystone (stable/havana)

Fix proposed to branch: stable/havana
Review: https://review.openstack.org/78419

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

Reviewed: https://review.openstack.org/77785
Committed: https://git.openstack.org/cgit/stackforge/puppet-keystone/commit/?id=b72da2814d67ff4cdb90cfef47c6319e9137c9c1
Submitter: Jenkins
Branch: master

commit b72da2814d67ff4cdb90cfef47c6319e9137c9c1
Author: Mark T. Voelker <email address hidden>
Date: Mon Mar 3 23:37:36 2014 -0500

    Don't set keystone endpoint by default

    In the current version of the keystone::init class, the default
    values for keystone endpoint URLs are unversioned, which can lead
    to problems connecting to the endpoint. This patch reverts to
    the old behavior of not setting them at all by default, and includes
    sample values with versioned URL's both in the header and tests
    so that users know to use versioned URL's if they choose to
    override the defaults. It also updates the documentation in the
    header accordingly and tidies it up a bit.

    Change-Id: Ib14edf152c1d208418f101ed48cdc38b18f840a2
    Closes-Bug: #1287520

Changed in puppet-keystone:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-keystone (stable/havana)

Reviewed: https://review.openstack.org/78419
Committed: https://git.openstack.org/cgit/stackforge/puppet-keystone/commit/?id=9802add6566646dc1e4a2812d19c2ab4724ff594
Submitter: Jenkins
Branch: stable/havana

commit 9802add6566646dc1e4a2812d19c2ab4724ff594
Author: Mark T. Voelker <email address hidden>
Date: Mon Mar 3 23:37:36 2014 -0500

    Don't set keystone endpoint by default

    In the current version of the keystone::init class, the default
    values for keystone endpoint URLs are unversioned, which can lead
    to problems connecting to the endpoint. This patch reverts to
    the old behavior of not setting them at all by default, and includes
    sample values with versioned URL's both in the header and tests
    so that users know to use versioned URL's if they choose to
    override the defaults. It also updates the documentation in the
    header accordingly and tidies it up a bit.

    Conflicts:

     manifests/init.pp

    Change-Id: Ib14edf152c1d208418f101ed48cdc38b18f840a2
    Closes-Bug: #1287520
    (cherry picked from commit Ib14edf152c1d208418f101ed48cdc38b18f840a2)

tags: added: in-stable-havana
Mathieu Gagné (mgagne)
Changed in puppet-keystone:
milestone: none → 4.0.0
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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