auth_token configuration does not need the service catalog

Bug #1257738 reported by Chmouel Boudjnah
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
puppet-swift
Fix Released
Medium
Emilien Macchi
Havana
Fix Released
Medium
Emilien Macchi

Bug Description

in swift's auth_token configuration we need

include_servce_catalog=False

since swift does need it and it would be way better performances.

this setting is only effective on trunk python-keystoneclient

Changed in puppet-swift:
assignee: nobody → Emilien Macchi (emilienm)
Revision history for this message
Chmouel Boudjnah (chmouel) wrote :

review on swit documenting a bit more :

https://review.openstack.org/#/c/59371/

Changed in puppet-swift:
status: New → In Progress
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to puppet-swift (master)

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

Revision history for this message
Emilien Macchi (emilienm) wrote :

By default we could read in python-keystoneclient code that include_service_catalog is set to True.
I have to respect this policy in puppet modules, so if you consider that's not the best choice, we will need to change the default value in keystone client, and then in puppet module.

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

Reviewed: https://review.openstack.org/60521
Committed: http://github.com/stackforge/puppet-swift/commit/2fc0354936bd89889f2e37f92e2a4acb501ca785
Submitter: Jenkins
Branch: master

commit 2fc0354936bd89889f2e37f92e2a4acb501ca785
Author: Emilien Macchi <email address hidden>
Date: Fri Dec 6 15:19:16 2013 +0100

    Set include_service_catalog to False

    include_service_catalog is a parameter in swift authtoken configuration.
    Set to False improves performances.

    Closes-Bug #1257738
    Signed-off-by: Emilien Macchi <email address hidden>

    Change-Id: Ie5dd2de45fce79e4da7ccb5b37d7c69ee1cb5198

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

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

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

Reviewed: https://review.openstack.org/65321
Committed: https://git.openstack.org/cgit/stackforge/puppet-swift/commit/?id=0e33d2dd66b2c0174b17e95eab1fb57edab8f097
Submitter: Jenkins
Branch: stable/havana

commit 0e33d2dd66b2c0174b17e95eab1fb57edab8f097
Author: Emilien Macchi <email address hidden>
Date: Fri Dec 6 15:19:16 2013 +0100

    Set include_service_catalog to False

    include_service_catalog is a parameter in swift authtoken configuration.
    Set to False improves performances.

    Closes-Bug #1257738
    Signed-off-by: Emilien Macchi <email address hidden>

    Change-Id: Ie5dd2de45fce79e4da7ccb5b37d7c69ee1cb5198
    (cherry picked from commit 2fc0354936bd89889f2e37f92e2a4acb501ca785)

tags: added: in-stable-havana
Mathieu Gagné (mgagne)
Changed in puppet-swift:
milestone: none → 3.0.0
Mathieu Gagné (mgagne)
Changed in puppet-swift:
milestone: 3.0.0 → none
Mathieu Gagné (mgagne)
Changed in puppet-swift:
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.