)]}'
{"lib/cinder":[{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"35486a331e5065e0121d5be6933b7653cb946f83","unresolved":false,"context_lines":[{"line_number":246,"context_line":"                configure_cinder_backend_${BE_TYPE} ${BE_NAME}"},{"line_number":247,"context_line":"            fi"},{"line_number":248,"context_line":"            if [[ -z \"$default_type\" ]]; then"},{"line_number":249,"context_line":"                default_type\u003d$BE_NAME"},{"line_number":250,"context_line":"            fi"},{"line_number":251,"context_line":"            enabled_backends+\u003d$BE_NAME,"},{"line_number":252,"context_line":"        done"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"baada198_71678125","line":249,"updated":"2014-07-25 21:21:52.000000000","message":"Would you do me a favor and change the \u0027default_type\u0027 var to \u0027default_name\u0027?  Even though that doesn\u0027t match cinder\u0027s config attribute it is consistent with everything else.","commit_id":"8beb0ed697f93457ce7ff67931d9320cf3e64e12"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"00db6248e2843f07d46a614463808bb235f9f607","unresolved":false,"context_lines":[{"line_number":246,"context_line":"                configure_cinder_backend_${BE_TYPE} ${BE_NAME}"},{"line_number":247,"context_line":"            fi"},{"line_number":248,"context_line":"            if [[ -z \"$default_type\" ]]; then"},{"line_number":249,"context_line":"                default_type\u003d$BE_NAME"},{"line_number":250,"context_line":"            fi"},{"line_number":251,"context_line":"            enabled_backends+\u003d$BE_NAME,"},{"line_number":252,"context_line":"        done"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"baada198_ac5aa27f","line":249,"in_reply_to":"baada198_0cc38e58","updated":"2014-07-25 21:41:36.000000000","message":"There is plenty of confusion, yeah, and it may be mine.  But everything in DevStack\u0027s new cinder backend stuff is named like this:\n\n* Cinder API\u0027s volume type name is what I\u0027m calling BE_NAME\n* cinder.conf\u0027s volume_backend_name is what I\u0027m calling BE_TYPE\n* cinder.conf wants the API\u0027s \u0027name\u0027 value in default_volume_type.\n\nI\u0027d prefer to have the point of friction in DevStack to be where default_volume_type is set with what is called a \u0027name\u0027 everywhere else.","commit_id":"8beb0ed697f93457ce7ff67931d9320cf3e64e12"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"ec1e10eceed02ac75fd0f56589de2bc387217349","unresolved":false,"context_lines":[{"line_number":246,"context_line":"                configure_cinder_backend_${BE_TYPE} ${BE_NAME}"},{"line_number":247,"context_line":"            fi"},{"line_number":248,"context_line":"            if [[ -z \"$default_type\" ]]; then"},{"line_number":249,"context_line":"                default_type\u003d$BE_NAME"},{"line_number":250,"context_line":"            fi"},{"line_number":251,"context_line":"            enabled_backends+\u003d$BE_NAME,"},{"line_number":252,"context_line":"        done"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"baada198_ecf49ad0","line":249,"in_reply_to":"baada198_71678125","updated":"2014-07-25 21:31:08.000000000","message":"Not sure I\u0027m following, the var is used for the cinder default_volume_type option, so default_type as the var name seems to make sense, it\u0027s just the BE_NAME var that\u0027s setting the value.","commit_id":"8beb0ed697f93457ce7ff67931d9320cf3e64e12"},{"author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"change_message_id":"9f49bedcd3e6eb990d7192c004814df84f0cdd49","unresolved":false,"context_lines":[{"line_number":246,"context_line":"                configure_cinder_backend_${BE_TYPE} ${BE_NAME}"},{"line_number":247,"context_line":"            fi"},{"line_number":248,"context_line":"            if [[ -z \"$default_type\" ]]; then"},{"line_number":249,"context_line":"                default_type\u003d$BE_NAME"},{"line_number":250,"context_line":"            fi"},{"line_number":251,"context_line":"            enabled_backends+\u003d$BE_NAME,"},{"line_number":252,"context_line":"        done"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"baada198_0cc38e58","line":249,"in_reply_to":"baada198_71678125","updated":"2014-07-25 21:33:31.000000000","message":"so I think that might make things more confusing.  One of the reasons is I for example customize this to use \"lvm\" like we had before which includes multiple backend-names.  For limited devstack I guess this isn\u0027t a big deal but it does add confusion IMO.","commit_id":"8beb0ed697f93457ce7ff67931d9320cf3e64e12"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"56aea4593080b80af33c38e0089661fb7bc57c74","unresolved":false,"context_lines":[{"line_number":246,"context_line":"                configure_cinder_backend_${BE_TYPE} ${BE_NAME}"},{"line_number":247,"context_line":"            fi"},{"line_number":248,"context_line":"            if [[ -z \"$default_type\" ]]; then"},{"line_number":249,"context_line":"                default_type\u003d$BE_NAME"},{"line_number":250,"context_line":"            fi"},{"line_number":251,"context_line":"            enabled_backends+\u003d$BE_NAME,"},{"line_number":252,"context_line":"        done"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"baada198_0c538e3a","line":249,"in_reply_to":"baada198_ac5aa27f","updated":"2014-07-25 21:47:45.000000000","message":"Done","commit_id":"8beb0ed697f93457ce7ff67931d9320cf3e64e12"},{"author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"change_message_id":"af89f303fa9def935c01d6e394d9b778522e703a","unresolved":false,"context_lines":[{"line_number":246,"context_line":"                configure_cinder_backend_${BE_TYPE} ${BE_NAME}"},{"line_number":247,"context_line":"            fi"},{"line_number":248,"context_line":"            if [[ -z \"$default_type\" ]]; then"},{"line_number":249,"context_line":"                default_type\u003d$BE_NAME"},{"line_number":250,"context_line":"            fi"},{"line_number":251,"context_line":"            enabled_backends+\u003d$BE_NAME,"},{"line_number":252,"context_line":"        done"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"baada198_82f92584","line":249,"in_reply_to":"baada198_ac5aa27f","updated":"2014-07-25 22:31:34.000000000","message":"fair enough, but just to be clear; the point I was trying to make was that cinder doesn\u0027t actually want or care if the name is used for default_volume_type, but the way the new lib for multi-backend works this is the only way to keep things \"working\" due to some other assumptions.\n\nAnyway... it\u0027s not that big of a deal to me; and you make a really good point.  At some point it may be worth revisiting how the new cinder backend code is setup but not that big of a deal.","commit_id":"8beb0ed697f93457ce7ff67931d9320cf3e64e12"}]}