)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"6a9af1bab5c223a906c27fc62423c176d03b7452","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Since Pike release, Cinder is able to extend an attached volume. This"},{"line_number":10,"context_line":"feature is enabled by default on Cinder and therefore should be tested"},{"line_number":11,"context_line":"on Tempest."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I020e281558eda7aee7e00b669037a28e780e2188"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"3f79a3b5_0ed2785c","line":12,"range":{"start_line":12,"start_character":0,"end_line":12,"end_character":0},"updated":"2018-10-07 12:46:49.000000000","message":"You can make this patch depends-on on tempest patch and test here.\nWith that we can tests the other backend also specially ceph (with experimental check as ceph job is experimental queue).\n\nDepends-On: https://review.openstack.org/#/c/578463/","commit_id":"62c812a49c99c3e00e259b2e25f5a100fd79bec5"}],"lib/tempest":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"81178547de33c0e768952ecbee2919bb9b80f2f1","unresolved":false,"context_lines":[{"line_number":450,"context_line":"        TEMPEST_VOLUME_MANAGE_VOLUME\u003d${TEMPEST_VOLUME_MANAGE_VOLUME:-True}"},{"line_number":451,"context_line":"    fi"},{"line_number":452,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled manage_volume $(trueorfalse False TEMPEST_VOLUME_MANAGE_VOLUME)"},{"line_number":453,"context_line":"    # Only turn on TEMPEST_EXTEND_ATTACHED_VOLUME by default for \"lvm\" backends"},{"line_number":454,"context_line":"    # in Cinder and the libvirt driver in Nova."},{"line_number":455,"context_line":"    if [[ \"$CINDER_ENABLED_BACKENDS\" \u003d\u003d *\"lvm\"* ]] \u0026\u0026 [ \"$VIRT_DRIVER\" \u003d \"libvirt\" ]; then"},{"line_number":456,"context_line":"        TEMPEST_EXTEND_ATTACHED_VOLUME\u003d${TEMPEST_EXTEND_ATTACHED_VOLUME:-True}"},{"line_number":457,"context_line":"    fi"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"5f7c97a3_ae062285","line":454,"range":{"start_line":453,"start_character":4,"end_line":454,"end_character":47},"updated":"2018-06-13 22:08:22.000000000","message":"This is clearly invalid now, and the lvm conditions in here. Also, the assertion about the libvirt virt driver is still true, there are no other virt drivers in nova besides libvirt that support the volume extend operation on in-use volumes, so your default enable logic is going to have to at least consider that, otherwise this impacts not only cinder 3rd party CI but also other nova virt driver CI, like xenserver.","commit_id":"faf5acc966153c979d6ac8b614d6dc9c151edde6"},{"author":{"_account_id":18058,"name":"Lucio Seki","email":"lseki@redhat.com","username":"lseki"},"change_message_id":"6d0ce151b8e45688aef6eb229faef6ff123d254c","unresolved":false,"context_lines":[{"line_number":450,"context_line":"        TEMPEST_VOLUME_MANAGE_VOLUME\u003d${TEMPEST_VOLUME_MANAGE_VOLUME:-True}"},{"line_number":451,"context_line":"    fi"},{"line_number":452,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled manage_volume $(trueorfalse False TEMPEST_VOLUME_MANAGE_VOLUME)"},{"line_number":453,"context_line":"    # Only turn on TEMPEST_EXTEND_ATTACHED_VOLUME by default for \"lvm\" backends"},{"line_number":454,"context_line":"    # in Cinder and the libvirt driver in Nova."},{"line_number":455,"context_line":"    if [[ \"$CINDER_ENABLED_BACKENDS\" \u003d\u003d *\"lvm\"* ]] \u0026\u0026 [ \"$VIRT_DRIVER\" \u003d \"libvirt\" ]; then"},{"line_number":456,"context_line":"        TEMPEST_EXTEND_ATTACHED_VOLUME\u003d${TEMPEST_EXTEND_ATTACHED_VOLUME:-True}"},{"line_number":457,"context_line":"    fi"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"5f7c97a3_777c51ea","line":454,"range":{"start_line":453,"start_character":4,"end_line":454,"end_character":47},"in_reply_to":"5f7c97a3_ae062285","updated":"2018-06-14 14:17:14.000000000","message":"Done","commit_id":"faf5acc966153c979d6ac8b614d6dc9c151edde6"},{"author":{"_account_id":18058,"name":"Lucio Seki","email":"lseki@redhat.com","username":"lseki"},"change_message_id":"54069382406070607181787dfe8ad5a79b719114","unresolved":false,"context_lines":[{"line_number":454,"context_line":"    if [ \"$VIRT_DRIVER\" \u003d \"libvirt\" ]; then"},{"line_number":455,"context_line":"        TEMPEST_EXTEND_ATTACHED_VOLUME\u003d${TEMPEST_EXTEND_ATTACHED_VOLUME:-True}"},{"line_number":456,"context_line":"    fi"},{"line_number":457,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled extend_attached_volume $(trueorfalse True TEMPEST_EXTEND_ATTACHED_VOLUME)"},{"line_number":458,"context_line":"    # TODO(ameade): Remove the api_v3 flag when Mitaka and Liberty are end of life."},{"line_number":459,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled api_v3 True"},{"line_number":460,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled api_v1 $(trueorfalse False TEMPEST_VOLUME_API_V1)"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"5f7c97a3_d70d253e","line":457,"range":{"start_line":457,"start_character":87,"end_line":457,"end_character":91},"updated":"2018-06-14 14:27:56.000000000","message":"oops I need to set the \"else\" case to False","commit_id":"7d00e80c7a03575e429c30a3139e2bdfb5499e9f"},{"author":{"_account_id":18058,"name":"Lucio Seki","email":"lseki@redhat.com","username":"lseki"},"change_message_id":"f42365e1a8f1ba134deff35956e1775338946525","unresolved":false,"context_lines":[{"line_number":454,"context_line":"    if [ \"$VIRT_DRIVER\" \u003d \"libvirt\" ]; then"},{"line_number":455,"context_line":"        TEMPEST_EXTEND_ATTACHED_VOLUME\u003d${TEMPEST_EXTEND_ATTACHED_VOLUME:-True}"},{"line_number":456,"context_line":"    fi"},{"line_number":457,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled extend_attached_volume $(trueorfalse True TEMPEST_EXTEND_ATTACHED_VOLUME)"},{"line_number":458,"context_line":"    # TODO(ameade): Remove the api_v3 flag when Mitaka and Liberty are end of life."},{"line_number":459,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled api_v3 True"},{"line_number":460,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled api_v1 $(trueorfalse False TEMPEST_VOLUME_API_V1)"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"5f7c97a3_d297738f","line":457,"range":{"start_line":457,"start_character":87,"end_line":457,"end_character":91},"in_reply_to":"5f7c97a3_d70d253e","updated":"2018-06-14 14:33:10.000000000","message":"Done","commit_id":"7d00e80c7a03575e429c30a3139e2bdfb5499e9f"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"78995875c90db74b180feca84f9cb2cd029c7fdc","unresolved":false,"context_lines":[{"line_number":451,"context_line":"    fi"},{"line_number":452,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled manage_volume $(trueorfalse False TEMPEST_VOLUME_MANAGE_VOLUME)"},{"line_number":453,"context_line":"    # Turn on TEMPEST_EXTEND_ATTACHED_VOLUME by default for every backend"},{"line_number":454,"context_line":"    # in Cinder and the libvirt driver in Nova"},{"line_number":455,"context_line":"    if [ \"$VIRT_DRIVER\" \u003d \"libvirt\" ]; then"},{"line_number":456,"context_line":"        TEMPEST_EXTEND_ATTACHED_VOLUME\u003d${TEMPEST_EXTEND_ATTACHED_VOLUME:-True}"},{"line_number":457,"context_line":"    fi"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"5f7c97a3_e6ac9189","line":454,"updated":"2018-06-22 03:49:26.000000000","message":"I think this would read better as \n\n\"TEMPEST_EXTEND_ATTACHED_VOLUME should be enabled for all Cinder backends when using the libvirt driver in Nova\"","commit_id":"deab82b6343d787a669ea44c9a41dadb3275a537"},{"author":{"_account_id":18058,"name":"Lucio Seki","email":"lseki@redhat.com","username":"lseki"},"change_message_id":"5319930c96de69748012482229c42935bb417fc0","unresolved":false,"context_lines":[{"line_number":451,"context_line":"    fi"},{"line_number":452,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled manage_volume $(trueorfalse False TEMPEST_VOLUME_MANAGE_VOLUME)"},{"line_number":453,"context_line":"    # Turn on TEMPEST_EXTEND_ATTACHED_VOLUME by default for every backend"},{"line_number":454,"context_line":"    # in Cinder and the libvirt driver in Nova"},{"line_number":455,"context_line":"    if [ \"$VIRT_DRIVER\" \u003d \"libvirt\" ]; then"},{"line_number":456,"context_line":"        TEMPEST_EXTEND_ATTACHED_VOLUME\u003d${TEMPEST_EXTEND_ATTACHED_VOLUME:-True}"},{"line_number":457,"context_line":"    fi"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"5f7c97a3_a0a3af9a","line":454,"in_reply_to":"5f7c97a3_e6ac9189","updated":"2018-06-25 18:59:23.000000000","message":"Done","commit_id":"deab82b6343d787a669ea44c9a41dadb3275a537"}]}