)]}'
{"templates/default/heat.conf.erb":[{"author":{"_account_id":7128,"name":"Mark Vanderwiel","email":"vanderwl@us.ibm.com","username":"kramvan"},"change_message_id":"6730309f8f19aa9ee2672d660511e4ec9597a42c","unresolved":false,"context_lines":[{"line_number":37,"context_line":"# (string value)"},{"line_number":38,"context_line":"\u003c% if node[\u0027openstack\u0027][\u0027orchestration\u0027][\u0027deferred_auth_method\u0027] -%\u003e"},{"line_number":39,"context_line":"deferred_auth_method\u003d\u003c%\u003d node[\u0027openstack\u0027][\u0027orchestration\u0027][\u0027deferred_auth_method\u0027] %\u003e"},{"line_number":40,"context_line":"\u003c% else -%\u003e"},{"line_number":41,"context_line":"#deferred_auth_method\u003dpassword"},{"line_number":42,"context_line":"\u003c% end -%\u003e"},{"line_number":43,"context_line":""}],"source_content_type":"application/x-erb","patch_set":1,"id":"9a80dd14_bc94708b","line":40,"updated":"2015-03-09 14:30:56.000000000","message":"I don\u0027t think this else is needed if just for a comment line now that the attribute is handling this.","commit_id":"c5c882b4cb7abe8453adc4c19c82d93a6982e32c"},{"author":{"_account_id":7128,"name":"Mark Vanderwiel","email":"vanderwl@us.ibm.com","username":"kramvan"},"change_message_id":"c062436ce8bbdbf0b6cd8639731fcaa25687d5c4","unresolved":false,"context_lines":[{"line_number":37,"context_line":"# (string value)"},{"line_number":38,"context_line":"\u003c% if node[\u0027openstack\u0027][\u0027orchestration\u0027][\u0027deferred_auth_method\u0027] -%\u003e"},{"line_number":39,"context_line":"deferred_auth_method\u003d\u003c%\u003d node[\u0027openstack\u0027][\u0027orchestration\u0027][\u0027deferred_auth_method\u0027] %\u003e"},{"line_number":40,"context_line":"\u003c% else -%\u003e"},{"line_number":41,"context_line":"#deferred_auth_method\u003dpassword"},{"line_number":42,"context_line":"\u003c% end -%\u003e"},{"line_number":43,"context_line":""}],"source_content_type":"application/x-erb","patch_set":1,"id":"9a80dd14_a7d6eac1","line":40,"in_reply_to":"9a80dd14_38330cf7","updated":"2015-03-10 14:12:09.000000000","message":"In my opinion, this is one of those cleanup items that should be done, removing this unnecessary logic clutter sounds like the right way to go here.    The other issue is that the defaults for all these commented keys need to be maintained, which just doubles the work we already have in the attributes default files.","commit_id":"c5c882b4cb7abe8453adc4c19c82d93a6982e32c"},{"author":{"_account_id":7404,"name":"Ethan Lynn","email":"xuanlangjian@gmail.com","username":"GaryLynn"},"change_message_id":"21f1b00a363e6c275cdc6e6955f5f5761be89716","unresolved":false,"context_lines":[{"line_number":37,"context_line":"# (string value)"},{"line_number":38,"context_line":"\u003c% if node[\u0027openstack\u0027][\u0027orchestration\u0027][\u0027deferred_auth_method\u0027] -%\u003e"},{"line_number":39,"context_line":"deferred_auth_method\u003d\u003c%\u003d node[\u0027openstack\u0027][\u0027orchestration\u0027][\u0027deferred_auth_method\u0027] %\u003e"},{"line_number":40,"context_line":"\u003c% else -%\u003e"},{"line_number":41,"context_line":"#deferred_auth_method\u003dpassword"},{"line_number":42,"context_line":"\u003c% end -%\u003e"},{"line_number":43,"context_line":""}],"source_content_type":"application/x-erb","patch_set":1,"id":"9a80dd14_38330cf7","line":40,"in_reply_to":"9a80dd14_bc94708b","updated":"2015-03-10 06:38:02.000000000","message":"Will remove this in next patchset. And I notice that L126-L128 and other place are also using \u003c% else -%\u003e logic like me. \n\nIs that codes needed? If not, I can upload another patch to remove them.","commit_id":"c5c882b4cb7abe8453adc4c19c82d93a6982e32c"}]}