)]}'
{"lib/tempest":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"ea2eebabb8867cf312d191f97e88e31c2ad0280e","unresolved":false,"context_lines":[{"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)"},{"line_number":442,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled manage_volume $(trueorfalse True TEMPEST_VOLUME_MANAGE_VOLUME)"},{"line_number":443,"context_line":"    # TODO(ameade): Remove the api_v3 flag when Mitaka and Liberty are end of life."},{"line_number":444,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled api_v3 True"},{"line_number":445,"context_line":"    local tempest_volume_min_microversion\u003d${TEMPEST_VOLUME_MIN_MICROVERSION:-None}"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"9a30ddce_8e1cd614","line":442,"range":{"start_line":442,"start_character":78,"end_line":442,"end_character":82},"updated":"2017-03-08 06:29:40.000000000","message":"I am little bit worried about default value. it makes tests run for all backend and I am sure it will break many CI.\n\nWe faced same issue on manage_snapshot also L441\n\nI suggest to default it to false and enable it for backend support this feature like done in L438-439","commit_id":"2e0d265690181c4bb710a1d13ed447208affc9a1"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"f2c4f4478f96a1f4bf0b397c88feba9d8475bdbd","unresolved":false,"context_lines":[{"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)"},{"line_number":442,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled manage_volume $(trueorfalse True TEMPEST_VOLUME_MANAGE_VOLUME)"},{"line_number":443,"context_line":"    # TODO(ameade): Remove the api_v3 flag when Mitaka and Liberty are end of life."},{"line_number":444,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled api_v3 True"},{"line_number":445,"context_line":"    local tempest_volume_min_microversion\u003d${TEMPEST_VOLUME_MIN_MICROVERSION:-None}"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"da36d5c6_666f30fc","line":442,"updated":"2017-02-25 13:56:05.000000000","message":"now that we have project-config support for local.conf, I\u0027d rather do all of that there.","commit_id":"2e0d265690181c4bb710a1d13ed447208affc9a1"},{"author":{"_account_id":23081,"name":"Jeremy Zhang","email":"zhangjun_inspur@163.com","username":"jeremy.zhang"},"change_message_id":"989bf81555a353d4229c4d424fab4a2506769de9","unresolved":false,"context_lines":[{"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)"},{"line_number":442,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled manage_volume $(trueorfalse True TEMPEST_VOLUME_MANAGE_VOLUME)"},{"line_number":443,"context_line":"    # TODO(ameade): Remove the api_v3 flag when Mitaka and Liberty are end of life."},{"line_number":444,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled api_v3 True"},{"line_number":445,"context_line":"    local tempest_volume_min_microversion\u003d${TEMPEST_VOLUME_MIN_MICROVERSION:-None}"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"9a30ddce_f12613b9","line":442,"range":{"start_line":442,"start_character":78,"end_line":442,"end_character":82},"in_reply_to":"9a30ddce_8e1cd614","updated":"2017-03-08 07:02:05.000000000","message":"@Ghanshyam Mann, I agree with your opinion on the issue.\nIn my local env, only Ceph \u0026 LVM backends have been tested. So for stability, I\u0027ll take your suggestion, thank you!","commit_id":"2e0d265690181c4bb710a1d13ed447208affc9a1"},{"author":{"_account_id":23081,"name":"Jeremy Zhang","email":"zhangjun_inspur@163.com","username":"jeremy.zhang"},"change_message_id":"c0fe9221b374f3f2d9f24124380faec8b2042408","unresolved":false,"context_lines":[{"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)"},{"line_number":442,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled manage_volume $(trueorfalse True TEMPEST_VOLUME_MANAGE_VOLUME)"},{"line_number":443,"context_line":"    # TODO(ameade): Remove the api_v3 flag when Mitaka and Liberty are end of life."},{"line_number":444,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled api_v3 True"},{"line_number":445,"context_line":"    local tempest_volume_min_microversion\u003d${TEMPEST_VOLUME_MIN_MICROVERSION:-None}"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"da36d5c6_3a8c5bd9","line":442,"in_reply_to":"da36d5c6_666f30fc","updated":"2017-02-27 07:22:10.000000000","message":"Hi, Sean Dague.\nA patch is proposed to openstack-infra/project-config, https://review.openstack.org/#/c/438347/, looking forward your advice, thank you!","commit_id":"2e0d265690181c4bb710a1d13ed447208affc9a1"}]}
