)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"c2eb9758a9b0e9a4a8e20c9c1885d43616b19b96","unresolved":true,"context_lines":[{"line_number":7,"context_line":"Remove Block Storage API v2 support"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The Block Storage API v2 was deprecated in Pike by change"},{"line_number":10,"context_line":"I913c44799cddc37c3342729ec0ef34068db5b2d4 and is (finally)"},{"line_number":11,"context_line":"being removed in Xena.  So remove v2 support from devstack."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I856d78648d28ac4cad0fb212bef1ae6ad32fca90"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"47af1228_194710a4","line":11,"range":{"start_line":10,"start_character":46,"end_line":11,"end_character":21},"updated":"2021-05-18 16:56:12.000000000","message":"can you add cinder patch link which removing these or any other ref link of discussion, I still see endpoint support in https://github.com/openstack/cinder/blob/ba27068fc0cc8a70e2a2e2fcfe6ab9896dab1908/etc/cinder/api-paste.ini#L9","commit_id":"7438107d3394fe8951d3d64bf5611111f31e8864"}],"lib/tempest":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"5ff2e6cc8f80ffba9e94c1580acaab957f57a2ac","unresolved":true,"context_lines":[{"line_number":463,"context_line":"    # used to try different Volume API version targets. The tempest"},{"line_number":464,"context_line":"    # default it to \u0027volumev3\u0027(v3 APIs endpoint) , so only set this"},{"line_number":465,"context_line":"    # if you want to change it."},{"line_number":466,"context_line":"    if [[ -n \"$TEMPEST_VOLUME_TYPE\" ]]; then"},{"line_number":467,"context_line":"        iniset $TEMPEST_CONFIG volume catalog_type $TEMPEST_VOLUME_TYPE"},{"line_number":468,"context_line":"    fi"},{"line_number":469,"context_line":"    # Only turn on TEMPEST_VOLUME_MANAGE_SNAPSHOT by default for \"lvm\" backends"},{"line_number":470,"context_line":"    if [[ \"$CINDER_ENABLED_BACKENDS\" \u003d\u003d *\"lvm\"* ]]; then"},{"line_number":471,"context_line":"        TEMPEST_VOLUME_MANAGE_SNAPSHOT\u003d${TEMPEST_VOLUME_MANAGE_SNAPSHOT:-True}"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"cd774849_5baf74f7","line":468,"range":{"start_line":466,"start_character":0,"end_line":468,"end_character":6},"updated":"2021-05-17 23:08:57.000000000","message":"Is there any point in leaving this, or should it be removed?","commit_id":"7438107d3394fe8951d3d64bf5611111f31e8864"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"c2eb9758a9b0e9a4a8e20c9c1885d43616b19b96","unresolved":true,"context_lines":[{"line_number":463,"context_line":"    # used to try different Volume API version targets. The tempest"},{"line_number":464,"context_line":"    # default it to \u0027volumev3\u0027(v3 APIs endpoint) , so only set this"},{"line_number":465,"context_line":"    # if you want to change it."},{"line_number":466,"context_line":"    if [[ -n \"$TEMPEST_VOLUME_TYPE\" ]]; then"},{"line_number":467,"context_line":"        iniset $TEMPEST_CONFIG volume catalog_type $TEMPEST_VOLUME_TYPE"},{"line_number":468,"context_line":"    fi"},{"line_number":469,"context_line":"    # Only turn on TEMPEST_VOLUME_MANAGE_SNAPSHOT by default for \"lvm\" backends"},{"line_number":470,"context_line":"    if [[ \"$CINDER_ENABLED_BACKENDS\" \u003d\u003d *\"lvm\"* ]]; then"},{"line_number":471,"context_line":"        TEMPEST_VOLUME_MANAGE_SNAPSHOT\u003d${TEMPEST_VOLUME_MANAGE_SNAPSHOT:-True}"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"e15c67b9_96a935a5","line":468,"range":{"start_line":466,"start_character":0,"end_line":468,"end_character":6},"in_reply_to":"cd774849_5baf74f7","updated":"2021-05-18 16:56:12.000000000","message":"it is used in job tempest-cinder-v2-api job[1] which I will make it to run on wallaby (last release where v2 is present) and then we will good to remove TEMPEST_VOLUME_TYPE from here.\n\n[1]https://opendev.org/openstack/tempest/src/branch/master/zuul.d/integrated-gate.yaml#L419","commit_id":"7438107d3394fe8951d3d64bf5611111f31e8864"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"3d937c296693a9babaf84563cd915b2ad1dedf51","unresolved":true,"context_lines":[{"line_number":463,"context_line":"    # used to try different Volume API version targets. The tempest"},{"line_number":464,"context_line":"    # default it to \u0027volumev3\u0027(v3 APIs endpoint) , so only set this"},{"line_number":465,"context_line":"    # if you want to change it."},{"line_number":466,"context_line":"    if [[ -n \"$TEMPEST_VOLUME_TYPE\" ]]; then"},{"line_number":467,"context_line":"        iniset $TEMPEST_CONFIG volume catalog_type $TEMPEST_VOLUME_TYPE"},{"line_number":468,"context_line":"    fi"},{"line_number":469,"context_line":"    # Only turn on TEMPEST_VOLUME_MANAGE_SNAPSHOT by default for \"lvm\" backends"},{"line_number":470,"context_line":"    if [[ \"$CINDER_ENABLED_BACKENDS\" \u003d\u003d *\"lvm\"* ]]; then"},{"line_number":471,"context_line":"        TEMPEST_VOLUME_MANAGE_SNAPSHOT\u003d${TEMPEST_VOLUME_MANAGE_SNAPSHOT:-True}"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"055887ff_9b3efa78","line":468,"range":{"start_line":466,"start_character":0,"end_line":468,"end_character":6},"in_reply_to":"e15c67b9_96a935a5","updated":"2021-05-18 20:04:35.000000000","message":"its working fine in https://review.opendev.org/c/openstack/tempest/+/792025\n\nso we are good to remove the TEMPEST_VOLUME_TYPE completely from devstack master.","commit_id":"7438107d3394fe8951d3d64bf5611111f31e8864"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"3d937c296693a9babaf84563cd915b2ad1dedf51","unresolved":true,"context_lines":[{"line_number":489,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled volume_revert $(trueorfalse False TEMPEST_VOLUME_REVERT_TO_SNAPSHOT)"},{"line_number":490,"context_line":"    local tempest_volume_min_microversion\u003d${TEMPEST_VOLUME_MIN_MICROVERSION:-None}"},{"line_number":491,"context_line":"    local tempest_volume_max_microversion\u003d${TEMPEST_VOLUME_MAX_MICROVERSION:-\"latest\"}"},{"line_number":492,"context_line":"    # Reset microversions to None where v2 is running which does not support microversion."},{"line_number":493,"context_line":"    # Both \"None\" means no microversion testing."},{"line_number":494,"context_line":"    if [[ \"$TEMPEST_VOLUME_TYPE\" \u003d\u003d \"volumev2\" ]]; then"},{"line_number":495,"context_line":"        tempest_volume_min_microversion\u003dNone"},{"line_number":496,"context_line":"        tempest_volume_max_microversion\u003dNone"},{"line_number":497,"context_line":"    fi"},{"line_number":498,"context_line":"    if [ \"$tempest_volume_min_microversion\" \u003d\u003d \"None\" ]; then"},{"line_number":499,"context_line":"        inicomment $TEMPEST_CONFIG volume min_microversion"},{"line_number":500,"context_line":"    else"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"6f3ac1bc_19a3540b","line":497,"range":{"start_line":492,"start_character":0,"end_line":497,"end_character":6},"updated":"2021-05-18 20:04:35.000000000","message":"and this too","commit_id":"7438107d3394fe8951d3d64bf5611111f31e8864"}]}
