swift3 / README.md needs to be updated to match keystone middleware filter options

Bug #1381758 reported by ariday
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Swift3
Fix Released
Undecided
Unassigned

Bug Description

swift3/README.md in https://github.com/stackforge/swift3/blob/master/README.md needs to be updated to match keystone middleware filter options.

--Step 2 If you use keystone, suggest to change to:
[pipeline:main]
    pipeline = catch_errors cache swift3 s3token authtoken keystone proxy-server

  ----But keystone middleware (https://github.com/stackforge/swift3/blob/master/swift3/middleware.py) checks filter order
  check_filter_order(pipeline, ['swift3', 's3token', 'authtoken', 'keystoneauth', 'proxy-server']):

  ----For this reason README.md should change from keystone to keystoneauth:
  [pipeline:main]
      pipeline = catch_errors cache swift3 s3token authtoken keystoneauth proxy-server
  ----The swift proxy-server.conf should also change filter:keystone to filter:keystoneauth.

--Step 3 should use keystoneclient.middleware.s3_token:filter_factory for paste.filter_factory instead of keystone.middleware.s3_token:filter_factory (if is not changed proxy service cannot be restarted)

Changed in swift3:
status: New → Confirmed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to swift3 (master)

Reviewed: https://review.openstack.org/129335
Committed: https://git.openstack.org/cgit/stackforge/swift3/commit/?id=8622b1cdab07b65d3e6267f9506835399262f4c0
Submitter: Jenkins
Branch: master

commit 8622b1cdab07b65d3e6267f9506835399262f4c0
Author: Bill Owen <email address hidden>
Date: Fri Oct 17 10:34:56 2014 -0700

    Update documentation to use keystoneauth name

    Update documentation to use keystoneauth in the proxy server pipeline. Change 5e372016 added
    pipeline checking and explicitly looks for keystoneauth name, so README should match.

    Also update package name for s3_token middleware to be keystoneclient, not keystone.

    Closes-Bug: #1381758
    Change-Id: I0d769e47ffc32725c1df2def4d9d46c25474132e

Changed in swift3:
status: Confirmed → Fix Committed
Changed in swift3:
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.