swift-dispersion-report broken in kilo

Bug #1479006 reported by Darren Birkett
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack-Ansible
Fix Released
Medium
Darren Birkett
Kilo
Fix Released
Medium
Darren Birkett
Trunk
Fix Released
Medium
Darren Birkett

Bug Description

In kilo, the swift dispersion user is set up as part of the swift_service_project_name tenant (by default, 'service'). However, the dispersion.conf file is populated using the keystone_admin_tenant_name (by default 'admin').

This is a remnant from juno, where the swift dispersion user was in fact part of the keystone_admin_tenant.

Changing the dispersion.conf template to use:

auth_user = {{ swift_service_project_name }}:{{ swift_dispersion_user }}

instead of:

auth_user = {{ keystone_admin_tenant_name }}:{{ swift_dispersion_user }}

will allow swift-dispersion-report to work again.

No further change needed

Revision history for this message
Darren Birkett (darren-birkett) wrote :
Changed in openstack-ansible:
status: New → In Progress
assignee: nobody → Darren Birkett (darren-birkett)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to os-ansible-deployment (master)

Reviewed: https://review.openstack.org/206572
Committed: https://git.openstack.org/cgit/stackforge/os-ansible-deployment/commit/?id=23e796c344ba87f0ac6bee436a9b3d3ad4bfb269
Submitter: Jenkins
Branch: master

commit 23e796c344ba87f0ac6bee436a9b3d3ad4bfb269
Author: Darren Birkett <email address hidden>
Date: Tue Jul 28 15:55:20 2015 +0100

    set correct swift dispersion tenant

    In kilo, the swift dispersion user is created as a member of
    the 'service' tenant. However, we are populating the
    swift-dispersion.conf as though it was a member of the admin tenant (as
    it was in juno). This commit fixes that, and makes
    swift-dispersion-report (and any monitoring plugins that rely on it)
    work again

    Change-Id: I0e9b9ccaa487ee0242346ee279821aa09dd5b022
    Closes-Bug: #1479006

Changed in openstack-ansible:
status: In Progress → Fix Committed
Dolph Mathews (dolph)
Changed in openstack-ansible:
importance: Undecided → Medium
Revision history for this message
Darren Birkett (darren-birkett) wrote :
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to os-ansible-deployment (kilo)

Reviewed: https://review.openstack.org/206913
Committed: https://git.openstack.org/cgit/stackforge/os-ansible-deployment/commit/?id=c18a025e5a0d201f21a9a2097730f3030dd28b9a
Submitter: Jenkins
Branch: kilo

commit c18a025e5a0d201f21a9a2097730f3030dd28b9a
Author: Darren Birkett <email address hidden>
Date: Tue Jul 28 15:55:20 2015 +0100

    set correct swift dispersion tenant

    In kilo, the swift dispersion user is created as a member of
    the 'service' tenant. However, we are populating the
    swift-dispersion.conf as though it was a member of the admin tenant (as
    it was in juno). This commit fixes that, and makes
    swift-dispersion-report (and any monitoring plugins that rely on it)
    work again

    Change-Id: I0e9b9ccaa487ee0242346ee279821aa09dd5b022
    Closes-Bug: #1479006
    (cherry picked from commit 23e796c344ba87f0ac6bee436a9b3d3ad4bfb269)

Revision history for this message
Davanum Srinivas (DIMS) (dims-v) wrote : Fix included in openstack/openstack-ansible 11.2.11

This issue was fixed in the openstack/openstack-ansible 11.2.11 release.

Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/openstack-ansible 11.2.12

This issue was fixed in the openstack/openstack-ansible 11.2.12 release.

Revision history for this message
Davanum Srinivas (DIMS) (dims-v) wrote : Fix included in openstack/openstack-ansible 11.2.14

This issue was fixed in the openstack/openstack-ansible 11.2.14 release.

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.