)]}'
{"lib/tempest":[{"author":{"_account_id":17716,"name":"Luz Cazares","email":"luz.cazares@intel.com","username":"luzcazares"},"change_message_id":"1593328bb78caea2dfea657cd9e183f40efb4b76","unresolved":false,"context_lines":[{"line_number":275,"context_line":"        iniset $TEMPEST_CONFIG identity-feature-enabled api_v2 True"},{"line_number":276,"context_line":"    else"},{"line_number":277,"context_line":"        # Skip Identity API v2 tests by default"},{"line_number":278,"context_line":"        iniset $TEMPEST_CONFIG identity-feature-enabled api_v2 False"},{"line_number":279,"context_line":"    fi"},{"line_number":280,"context_line":"    iniset $TEMPEST_CONFIG identity auth_version ${TEMPEST_AUTH_VERSION:-v3}"},{"line_number":281,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"9a30ddce_6edeab12","line":278,"range":{"start_line":278,"start_character":8,"end_line":278,"end_character":68},"updated":"2017-03-08 17:30:25.000000000","message":"nit: should this be moved to LN273 (have it as the default too) and remove else statement?","commit_id":"dc6e55021861afceae2a7778b299df94ceab0a6e"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"0fb320ca0f2b9d371f43e78503590e77058da376","unresolved":false,"context_lines":[{"line_number":275,"context_line":"        iniset $TEMPEST_CONFIG identity-feature-enabled api_v2 True"},{"line_number":276,"context_line":"    else"},{"line_number":277,"context_line":"        # Skip Identity API v2 tests by default"},{"line_number":278,"context_line":"        iniset $TEMPEST_CONFIG identity-feature-enabled api_v2 False"},{"line_number":279,"context_line":"    fi"},{"line_number":280,"context_line":"    iniset $TEMPEST_CONFIG identity auth_version ${TEMPEST_AUTH_VERSION:-v3}"},{"line_number":281,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"9a30ddce_74d75523","line":278,"range":{"start_line":278,"start_character":8,"end_line":278,"end_character":68},"in_reply_to":"9a30ddce_6edeab12","updated":"2017-03-08 20:40:50.000000000","message":"Yeah, we could. Actually I did it before posting this patch.\nHowever, the change seems a little complex and I guess it could be a little hard to be reviewed.\nCurrent change is very simple and easy to understand the purpose I think.","commit_id":"dc6e55021861afceae2a7778b299df94ceab0a6e"},{"author":{"_account_id":17716,"name":"Luz Cazares","email":"luz.cazares@intel.com","username":"luzcazares"},"change_message_id":"1593328bb78caea2dfea657cd9e183f40efb4b76","unresolved":false,"context_lines":[{"line_number":277,"context_line":"        # Skip Identity API v2 tests by default"},{"line_number":278,"context_line":"        iniset $TEMPEST_CONFIG identity-feature-enabled api_v2 False"},{"line_number":279,"context_line":"    fi"},{"line_number":280,"context_line":"    iniset $TEMPEST_CONFIG identity auth_version ${TEMPEST_AUTH_VERSION:-v3}"},{"line_number":281,"context_line":""},{"line_number":282,"context_line":"    if is_ssl_enabled_service \"key\" || is_service_enabled tls-proxy; then"},{"line_number":283,"context_line":"        iniset $TEMPEST_CONFIG identity ca_certificates_file $SSL_BUNDLE_FILE"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"9a30ddce_ced5d7b9","line":280,"range":{"start_line":280,"start_character":4,"end_line":280,"end_character":76},"updated":"2017-03-08 17:30:25.000000000","message":"nit: should line be placed at LN273?","commit_id":"dc6e55021861afceae2a7778b299df94ceab0a6e"}]}