)]}'
{"lib/cinder":[{"author":{"_account_id":6984,"name":"Sébastien Han","email":"seb@redhat.com","username":"leseb"},"change_message_id":"a12858db6b6c0e51f4d50a99718ab1b291831872","unresolved":false,"context_lines":[{"line_number":159,"context_line":""},{"line_number":160,"context_line":"    if is_service_enabled cinder; then"},{"line_number":161,"context_line":"        # maybe check for LVM enabled for this?"},{"line_number":162,"context_line":"        sudo rm -rf $CINDER_STATE_PATH/volumes/*"},{"line_number":163,"context_line":"    fi"},{"line_number":164,"context_line":""},{"line_number":165,"context_line":"}"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"1ae5cdf2_45bcf4db","line":162,"updated":"2014-06-17 15:37:11.000000000","message":"Why don\u0027t we let _clean_lvm_backing_file do this (called from cleanup_cinder_backend_${BE_TYPE})","commit_id":"9421e623785e1a87da6135f80604decbfb061a4e"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"dbc9b74fca3f0d106502453cb7d5c4c03fa029d0","unresolved":false,"context_lines":[{"line_number":159,"context_line":""},{"line_number":160,"context_line":"    if is_service_enabled cinder; then"},{"line_number":161,"context_line":"        # maybe check for LVM enabled for this?"},{"line_number":162,"context_line":"        sudo rm -rf $CINDER_STATE_PATH/volumes/*"},{"line_number":163,"context_line":"    fi"},{"line_number":164,"context_line":""},{"line_number":165,"context_line":"}"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"1ae5cdf2_829c606c","line":162,"in_reply_to":"1ae5cdf2_45bcf4db","updated":"2014-06-23 20:58:56.000000000","message":"_clean_lvm_backing_file() already removed the active backing files, I\u0027m thinking this is leftover from the DevStack stone age.  Removed.","commit_id":"9421e623785e1a87da6135f80604decbfb061a4e"},{"author":{"_account_id":6984,"name":"Sébastien Han","email":"seb@redhat.com","username":"leseb"},"change_message_id":"a12858db6b6c0e51f4d50a99718ab1b291831872","unresolved":false,"context_lines":[{"line_number":250,"context_line":"            fi"},{"line_number":251,"context_line":"            enabled_backends+\u003d$BE_NAME,"},{"line_number":252,"context_line":"        done"},{"line_number":253,"context_line":"        iniset $CINDER_CONF DEFAULT enabled_backends ${enabled_backends%,*}"},{"line_number":254,"context_line":"    fi"},{"line_number":255,"context_line":""},{"line_number":256,"context_line":"    if is_service_enabled swift; then"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"1ae5cdf2_45b874c7","line":253,"updated":"2014-06-17 15:37:11.000000000","message":"shouldn\u0027t we also configure a default volume type with default_volume_type? this handles the case where you create a volume without specifying a type.","commit_id":"9421e623785e1a87da6135f80604decbfb061a4e"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"dbc9b74fca3f0d106502453cb7d5c4c03fa029d0","unresolved":false,"context_lines":[{"line_number":250,"context_line":"            fi"},{"line_number":251,"context_line":"            enabled_backends+\u003d$BE_NAME,"},{"line_number":252,"context_line":"        done"},{"line_number":253,"context_line":"        iniset $CINDER_CONF DEFAULT enabled_backends ${enabled_backends%,*}"},{"line_number":254,"context_line":"    fi"},{"line_number":255,"context_line":""},{"line_number":256,"context_line":"    if is_service_enabled swift; then"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"1ae5cdf2_9fe1172d","line":253,"in_reply_to":"1ae5cdf2_45b874c7","updated":"2014-06-23 20:58:56.000000000","message":"Yes we should.  Deciding which might be fun so at first I\u0027ll suggest just using the first enabled backend.\n\nThat said, that\u0027s new functionality, I\u0027d like to add it in a later patch, this is meant to be a re-work of what DevStack supports today.","commit_id":"9421e623785e1a87da6135f80604decbfb061a4e"}],"lib/cinder_backends/lvm":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"4afe1af69ec5c27635953585f9cc7a9327f56354","unresolved":false,"context_lines":[{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    if is_fedora || is_suse; then"},{"line_number":97,"context_line":"        # service is not started by default"},{"line_number":98,"context_line":"        start_service tgtd"},{"line_number":99,"context_line":"    fi"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"    # Remove iscsi targets"}],"source_content_type":"application/octet-stream","patch_set":11,"id":"baada198_1a55abf9","line":98,"updated":"2014-07-11 14:37:28.000000000","message":"I think this is ok to start but please note that this will need to be extended with another option to support platforms that run only LIO iSCSI for the LVM driver and do not have tgtd.  (Currently this is el7/CentOS7.)","commit_id":"21ff84c9cf94e21e59b19f4ba3a93c05f1df6754"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"5651e7986600181b34c4a87b2fa642bf7b1a642f","unresolved":false,"context_lines":[{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    if is_fedora || is_suse; then"},{"line_number":97,"context_line":"        # service is not started by default"},{"line_number":98,"context_line":"        start_service tgtd"},{"line_number":99,"context_line":"    fi"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"    # Remove iscsi targets"}],"source_content_type":"application/octet-stream","patch_set":11,"id":"baada198_dabb33e3","line":98,"in_reply_to":"baada198_1a55abf9","updated":"2014-07-11 14:45:50.000000000","message":"Sure, that will need to be done separately with or without this change.  RHEL7 is already listed as supported, is it currently broken?","commit_id":"21ff84c9cf94e21e59b19f4ba3a93c05f1df6754"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"937bec3e0b10d0b39ff1eb40ae79802885b91715","unresolved":false,"context_lines":[{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    if is_fedora || is_suse; then"},{"line_number":97,"context_line":"        # service is not started by default"},{"line_number":98,"context_line":"        start_service tgtd"},{"line_number":99,"context_line":"    fi"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"    # Remove iscsi targets"}],"source_content_type":"application/octet-stream","patch_set":11,"id":"baada198_fd955133","line":98,"in_reply_to":"baada198_dabb33e3","updated":"2014-07-11 15:08:56.000000000","message":"Well I haven\u0027t tried devstack on it... but the Cinder LVM driver won\u0027t work currently unless you set iscsi_helper \u003d \"lioadm\".  I don\u0027t see anything in devstack doing that.","commit_id":"21ff84c9cf94e21e59b19f4ba3a93c05f1df6754"}]}