whitespace from config file is not trimmed

Bug #1087018 reported by Michael Basnight
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
oslo-incubator
Fix Released
Medium
Michael Basnight
Grizzly
Fix Released
Medium
Michael Basnight

Bug Description

A config file with a differing set of values from the default do not trim whitespace properly. I see that Bug 1079299 was committed, but this does not fix my case. I have added unit tests to validate it as well.

Revision history for this message
Davanum Srinivas (DIMS) (dims-v) wrote :

Michael,

Can you please provide examples of what does not get trimmed.

-- dims

Changed in oslo:
status: New → Incomplete
status: Incomplete → Confirmed
Revision history for this message
Davanum Srinivas (DIMS) (dims-v) wrote :

Looks like Michael has submitted a patch already - https://review.openstack.org/#/c/17559/

Revision history for this message
Michael Basnight (hubcap) wrote :

Dims, check out the tests. Try to add those w/o the changes to cfg. It'll fail miserably.

Revision history for this message
Davanum Srinivas (DIMS) (dims-v) wrote :

hey @hubcap! yes, found the patch after i saw the bug.

Revision history for this message
Michael Basnight (hubcap) wrote :

Oh duh. U wrote that 2nd comment. Sry!

Mark McLoughlin (markmc)
Changed in oslo:
status: Confirmed → In Progress
assignee: nobody → Michael Basnight (hubcap)
importance: Undecided → Medium
Mark McLoughlin (markmc)
Changed in oslo:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in oslo:
milestone: none → grizzly-2
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.