reuse default value for signing_dir setting from middleware which is sane

Bug #1313780 reported by Ihar Hrachyshka
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Low
Ihar Hrachyshka

Bug Description

In neutron.conf, we specify our custom value for signing_dir. There is no real need to do it, instead it's safe to use the default value which is a securely created temporary directory below TMPDIR (usually /tmp). Most other services (f.e. nova) use the default behaviour provided by keystone middleware, so we're better reduce the maintenance effort on our side and stick to the common ground.

It's always a good thing to reduce the number of explicit settings in default configuration.

Kyle Mestery (mestery)
Changed in neutron:
importance: Undecided → Low
Kyle Mestery (mestery)
Changed in neutron:
status: New → Triaged
Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote : Fix merged to neutron (master)

Reviewed: https://review.openstack.org/90723
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=e423126965fb79d35f5f2281974cee6a216af8d4
Submitter: Jenkins
Branch: master

commit e423126965fb79d35f5f2281974cee6a216af8d4
Author: Ihar Hrachyshka <email address hidden>
Date: Mon Apr 28 15:05:30 2014 +0200

    Removed signing_dir from neutron.conf

    The default value from keystone middleware ($TMPDIR) is good enough,
    so stick to it.

    Closes-Bug: 1313780
    Change-Id: Ideb393f9010f4f1706392ccdfdd5fa7307bda070

Changed in neutron:
status: Triaged → Fix Committed
Kyle Mestery (mestery)
Changed in neutron:
assignee: nobody → Ihar Hrachyshka (ihar-hrachyshka)
milestone: none → juno-1
Thierry Carrez (ttx)
Changed in neutron:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in neutron:
milestone: juno-1 → 2014.2
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.