)]}'
{"recipes/openrc.rb":[{"author":{"_account_id":7128,"name":"Mark Vanderwiel","email":"vanderwl@us.ibm.com","username":"kramvan"},"change_message_id":"d6e8b8aa6b85fdd9ea5735613953e9f9129ec60f","unresolved":false,"context_lines":[{"line_number":42,"context_line":"ksadmin_pass \u003d get_password \u0027user\u0027, ksadmin_user"},{"line_number":43,"context_line":"identity_endpoint \u003d endpoint \u0027identity-api\u0027"},{"line_number":44,"context_line":"volume_api_path \u003d node[\u0027openstack\u0027][\u0027endpoints\u0027][\u0027block-storage-api\u0027][\u0027path\u0027]"},{"line_number":45,"context_line":"volume_api_version \u003d volume_api_path[/\\d/].to_i unless volume_api_path.to_s.empty?"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"directory node[\u0027openstack\u0027][\u0027openrc\u0027][\u0027path\u0027] do"},{"line_number":48,"context_line":"  owner node[\u0027openstack\u0027][\u0027openrc\u0027][\u0027user\u0027]"}],"source_content_type":"text/x-ruby","patch_set":3,"id":"5a890539_7ba22bfe","line":45,"updated":"2014-11-14 16:45:33.000000000","message":"Missing the initial set to default of volume_api_version \u003d 2, as this line is conditional with the unless part.","commit_id":"73b1117a24b00ce7c195f44d7ea34b3b109eeece"},{"author":{"_account_id":7128,"name":"Mark Vanderwiel","email":"vanderwl@us.ibm.com","username":"kramvan"},"change_message_id":"d6e8b8aa6b85fdd9ea5735613953e9f9129ec60f","unresolved":false,"context_lines":[{"line_number":66,"context_line":"  )"},{"line_number":67,"context_line":"end"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"volume_api_path \u003d node[\u0027openstack\u0027][\u0027endpoints\u0027][\u0027block-storage-api\u0027][\u0027path\u0027]"},{"line_number":70,"context_line":"volume_api_version \u003d volume_api_path[/v\\d/].to_i unless volume_api_path.to_s.empty?"}],"source_content_type":"text/x-ruby","patch_set":3,"id":"5a890539_1b3207f2","line":69,"updated":"2014-11-14 16:45:33.000000000","message":"Oops, these 2 lines are cut/paste error?","commit_id":"73b1117a24b00ce7c195f44d7ea34b3b109eeece"}],"templates/default/openrc.erb":[{"author":{"_account_id":7128,"name":"Mark Vanderwiel","email":"vanderwl@us.ibm.com","username":"kramvan"},"change_message_id":"d7efb9610600a5909d5e2eb231762c8eb5606e3b","unresolved":false,"context_lines":[{"line_number":6,"context_line":"export OS_TENANT_NAME\u003d\u003c%\u003d @tenant %\u003e"},{"line_number":7,"context_line":"export OS_AUTH_URL\u003d\u003c%\u003d @identity_endpoint %\u003e"},{"line_number":8,"context_line":"export OS_REGION_NAME\u003d\u003c%\u003d node[\u0027openstack\u0027][\u0027region\u0027] %\u003e"},{"line_number":9,"context_line":"export OS_VOLUME_API_VERSION\u003d2"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"\u003c% if node[\u0027openstack\u0027][\u0027misc_openrc\u0027] %\u003e"},{"line_number":12,"context_line":"# Misc options"}],"source_content_type":"application/x-erb","patch_set":2,"id":"5a890539_8424dc60","line":9,"updated":"2014-11-14 14:32:38.000000000","message":"I would rather see this value generated from the endpoint definition.\ndefault[\u0027openstack\u0027][\u0027endpoints\u0027][\u0027block-storage-api\u0027][\u0027path\u0027] \u003d \u0027/v2/%(tenant_id)s\u0027\n\nSomething like this in the recipe:\n\nvolume_api_version \u003d 2\n\nvolume_api_path \u003d node[\u0027openstack\u0027][\u0027endpoints\u0027][\u0027block-storage-api\u0027][\u0027path\u0027]\n\nvolume_api_version \u003d volume_api_path[/v\\d/].to_i unless \nvolume_api_path.to_s.empty?\n\npass volume_api_version into template as variable.","commit_id":"12d447ba7ffff196497f70c11b718c43f9c5d3b3"},{"author":{"_account_id":7148,"name":"Zhu Zhu","email":"bjzzu.zz@gmail.com","username":"zhuzhubj"},"change_message_id":"5b1c6d7b56feeceb093eb5a11633f0c8a51a07fd","unresolved":false,"context_lines":[{"line_number":6,"context_line":"export OS_TENANT_NAME\u003d\u003c%\u003d @tenant %\u003e"},{"line_number":7,"context_line":"export OS_AUTH_URL\u003d\u003c%\u003d @identity_endpoint %\u003e"},{"line_number":8,"context_line":"export OS_REGION_NAME\u003d\u003c%\u003d node[\u0027openstack\u0027][\u0027region\u0027] %\u003e"},{"line_number":9,"context_line":"export OS_VOLUME_API_VERSION\u003d2"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"\u003c% if node[\u0027openstack\u0027][\u0027misc_openrc\u0027] %\u003e"},{"line_number":12,"context_line":"# Misc options"}],"source_content_type":"application/x-erb","patch_set":2,"id":"5a890539_90b41683","line":9,"in_reply_to":"5a890539_8424dc60","updated":"2014-11-14 16:12:59.000000000","message":"good suggestion. Looks volume_api_path[/\\d/] could work. Thanks.","commit_id":"12d447ba7ffff196497f70c11b718c43f9c5d3b3"}]}