)]}'
{"lib/tempest":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"4c69b99bbc1d5cb9113f3c92b3a154cefdcd3272","unresolved":false,"context_lines":[{"line_number":434,"context_line":"    iniset $TEMPEST_CONFIG validation network_for_ssh $TEMPEST_SSH_NETWORK_NAME"},{"line_number":435,"context_line":""},{"line_number":436,"context_line":"    # Volume"},{"line_number":437,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled manage_snapshot $(trueorfalse False TEMPEST_VOLUME_MANAGE_SNAPSHOT)"},{"line_number":438,"context_line":"    # TODO(ynesenenko): Remove the volume_services flag when Liberty and Kilo will correct work with host info."},{"line_number":439,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled volume_services True"},{"line_number":440,"context_line":"    # TODO(ameade): Remove the api_v3 flag when Mitaka and Liberty are end of life."}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"7a3c09a3_f43e02d9","line":437,"updated":"2017-01-15 19:24:04.000000000","message":"I think this should stay like this. Devstack is here to support the testing of the \"reference\" drivers/backends which is LVM. It\u0027s trivially easy for other backends to opt-out.","commit_id":"4492044d6022984232b50d51dcea54bacb97adb5"},{"author":{"_account_id":12924,"name":"Patrick East","email":"east.patrick@gmail.com","username":"patrick.east"},"change_message_id":"4a269234a76e0808db9d72a981c45a719eda9feb","unresolved":false,"context_lines":[{"line_number":434,"context_line":"    iniset $TEMPEST_CONFIG validation network_for_ssh $TEMPEST_SSH_NETWORK_NAME"},{"line_number":435,"context_line":""},{"line_number":436,"context_line":"    # Volume"},{"line_number":437,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled manage_snapshot $(trueorfalse False TEMPEST_VOLUME_MANAGE_SNAPSHOT)"},{"line_number":438,"context_line":"    # TODO(ynesenenko): Remove the volume_services flag when Liberty and Kilo will correct work with host info."},{"line_number":439,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled volume_services True"},{"line_number":440,"context_line":"    # TODO(ameade): Remove the api_v3 flag when Mitaka and Liberty are end of life."}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"7a3c09a3_042c0d01","line":437,"in_reply_to":"7a3c09a3_f43e02d9","updated":"2017-01-16 18:39:08.000000000","message":"I would argue that It\u0027s also trivially easy to make it work for all of them without additional config?","commit_id":"4492044d6022984232b50d51dcea54bacb97adb5"},{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"d201f92b35641084686753b0455ccffae7afc062","unresolved":false,"context_lines":[{"line_number":435,"context_line":""},{"line_number":436,"context_line":"    # Volume"},{"line_number":437,"context_line":"    # Only turn on TEMPEST_VOLUME_MANAGE_SNAPSHOT by default for \"lvm\" backends"},{"line_number":438,"context_line":"    if [[ \"$CINDER_ENABLED_BACKNEDS\" \u003d\u003d *\"lvm\"* ]]; then"},{"line_number":439,"context_line":"        TEMPEST_VOLUME_MANAGE_SNAPSHOT\u003d${TEMPEST_VOLUME_MANAGE_SNAPSHOT:-True}"},{"line_number":440,"context_line":"    fi"},{"line_number":441,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled manage_snapshot $(trueorfalse False TEMPEST_VOLUME_MANAGE_SNAPSHOT)"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"7a3c09a3_5085b704","line":438,"range":{"start_line":438,"start_character":11,"end_line":438,"end_character":35},"updated":"2017-01-17 18:17:28.000000000","message":"$CINDER_ENABLED_BACKENDS","commit_id":"c6bd801a5a05db307b3b72453404e396b36096cc"},{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"d201f92b35641084686753b0455ccffae7afc062","unresolved":false,"context_lines":[{"line_number":435,"context_line":""},{"line_number":436,"context_line":"    # Volume"},{"line_number":437,"context_line":"    # Only turn on TEMPEST_VOLUME_MANAGE_SNAPSHOT by default for \"lvm\" backends"},{"line_number":438,"context_line":"    if [[ \"$CINDER_ENABLED_BACKNEDS\" \u003d\u003d *\"lvm\"* ]]; then"},{"line_number":439,"context_line":"        TEMPEST_VOLUME_MANAGE_SNAPSHOT\u003d${TEMPEST_VOLUME_MANAGE_SNAPSHOT:-True}"},{"line_number":440,"context_line":"    fi"},{"line_number":441,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled manage_snapshot $(trueorfalse False TEMPEST_VOLUME_MANAGE_SNAPSHOT)"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"7a3c09a3_50b3f7e8","line":438,"range":{"start_line":438,"start_character":40,"end_line":438,"end_character":47},"updated":"2017-01-17 18:17:28.000000000","message":"What if CINDER_ENABLED_BACKENDS\u003d\u0027nfs,lvm\u0027? NFS does not support the manage and the test would be run.","commit_id":"c6bd801a5a05db307b3b72453404e396b36096cc"},{"author":{"_account_id":12924,"name":"Patrick East","email":"east.patrick@gmail.com","username":"patrick.east"},"change_message_id":"dd51bc1da8554885ef1cd440e5877961d32c265c","unresolved":false,"context_lines":[{"line_number":435,"context_line":""},{"line_number":436,"context_line":"    # Volume"},{"line_number":437,"context_line":"    # Only turn on TEMPEST_VOLUME_MANAGE_SNAPSHOT by default for \"lvm\" backends"},{"line_number":438,"context_line":"    if [[ \"$CINDER_ENABLED_BACKNEDS\" \u003d\u003d *\"lvm\"* ]]; then"},{"line_number":439,"context_line":"        TEMPEST_VOLUME_MANAGE_SNAPSHOT\u003d${TEMPEST_VOLUME_MANAGE_SNAPSHOT:-True}"},{"line_number":440,"context_line":"    fi"},{"line_number":441,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled manage_snapshot $(trueorfalse False TEMPEST_VOLUME_MANAGE_SNAPSHOT)"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"7a3c09a3_3946534f","line":438,"range":{"start_line":438,"start_character":11,"end_line":438,"end_character":35},"in_reply_to":"7a3c09a3_5085b704","updated":"2017-01-17 19:57:29.000000000","message":"Done","commit_id":"c6bd801a5a05db307b3b72453404e396b36096cc"},{"author":{"_account_id":12924,"name":"Patrick East","email":"east.patrick@gmail.com","username":"patrick.east"},"change_message_id":"dd51bc1da8554885ef1cd440e5877961d32c265c","unresolved":false,"context_lines":[{"line_number":435,"context_line":""},{"line_number":436,"context_line":"    # Volume"},{"line_number":437,"context_line":"    # Only turn on TEMPEST_VOLUME_MANAGE_SNAPSHOT by default for \"lvm\" backends"},{"line_number":438,"context_line":"    if [[ \"$CINDER_ENABLED_BACKNEDS\" \u003d\u003d *\"lvm\"* ]]; then"},{"line_number":439,"context_line":"        TEMPEST_VOLUME_MANAGE_SNAPSHOT\u003d${TEMPEST_VOLUME_MANAGE_SNAPSHOT:-True}"},{"line_number":440,"context_line":"    fi"},{"line_number":441,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled manage_snapshot $(trueorfalse False TEMPEST_VOLUME_MANAGE_SNAPSHOT)"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"7a3c09a3_b9596337","line":438,"range":{"start_line":438,"start_character":40,"end_line":438,"end_character":47},"in_reply_to":"7a3c09a3_50b3f7e8","updated":"2017-01-17 19:57:29.000000000","message":"Then the option needs to be overridden by the env variable. We could force it off if there are any backends other than LVM.","commit_id":"c6bd801a5a05db307b3b72453404e396b36096cc"}]}