)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6802,"name":"Joel Coffman","email":"jmc7tp@gmail.com","username":"joel-coffman"},"change_message_id":"85841ca99efc34c643fc37d9f6cf7a3695a69a4a","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Set fixed-key key manager"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"With the key manager refactoring, the key manager will need to be"},{"line_number":10,"context_line":"explicitly set."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I733279864ee1a4aaffc9c8eed81b5e12f8d8821b"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9aed3d3a_3738fa3c","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":32},"updated":"2016-02-29 20:37:03.000000000","message":"Is this change dependent on another one?","commit_id":"a1a71cc853c5c5fb7132317327c846ec52f03b70"}],"lib/cinder":[{"author":{"_account_id":6802,"name":"Joel Coffman","email":"jmc7tp@gmail.com","username":"joel-coffman"},"change_message_id":"85841ca99efc34c643fc37d9f6cf7a3695a69a4a","unresolved":false,"context_lines":[{"line_number":273,"context_line":""},{"line_number":274,"context_line":"    iniset $CINDER_CONF DEFAULT os_region_name \"$REGION_NAME\""},{"line_number":275,"context_line":""},{"line_number":276,"context_line":"    iniset $CINDER_CONF key_manager api_class cinder.keymgr.conf_key_mgr.ConfKeyManager"},{"line_number":277,"context_line":""},{"line_number":278,"context_line":"    if is_service_enabled c-vol \u0026\u0026 [[ -n \"$CINDER_ENABLED_BACKENDS\" ]]; then"},{"line_number":279,"context_line":"        local enabled_backends\u003d\"\""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"9aed3d3a_f77fb2dd","line":276,"updated":"2016-02-29 20:37:03.000000000","message":"I believe that the ConfKeyManager is trying to be removed. Should the default be set to Barbican or another backend that isn\u0027t in danger of deprecation?","commit_id":"a1a71cc853c5c5fb7132317327c846ec52f03b70"},{"author":{"_account_id":8623,"name":"Kaitlin Farr","email":"kaitlin.farr@jhuapl.edu","username":"kaitlin.farr"},"change_message_id":"f6619a687408701291d9103ed9964796262c0749","unresolved":false,"context_lines":[{"line_number":273,"context_line":""},{"line_number":274,"context_line":"    iniset $CINDER_CONF DEFAULT os_region_name \"$REGION_NAME\""},{"line_number":275,"context_line":""},{"line_number":276,"context_line":"    iniset $CINDER_CONF key_manager api_class cinder.keymgr.conf_key_mgr.ConfKeyManager"},{"line_number":277,"context_line":""},{"line_number":278,"context_line":"    if is_service_enabled c-vol \u0026\u0026 [[ -n \"$CINDER_ENABLED_BACKENDS\" ]]; then"},{"line_number":279,"context_line":"        local enabled_backends\u003d\"\""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"bab6814e_bddc1ae5","line":276,"in_reply_to":"9aed3d3a_f77fb2dd","updated":"2016-05-19 19:59:16.000000000","message":"While ConfKeyManager is not ideal for integration testing because it should not be used in actual deployments, it was the default previously used. Making Barbican the default means we\u0027d have to change to default devstack to also configure Barbican.","commit_id":"a1a71cc853c5c5fb7132317327c846ec52f03b70"}]}