)]}'
{"lib/nova":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"eb4a6ddefd5ae4768192ab1eeebb11eb14ab6533","unresolved":false,"context_lines":[{"line_number":49,"context_line":"NOVA_API_PASTE_INI\u003d${NOVA_API_PASTE_INI:-$NOVA_CONF_DIR/api-paste.ini}"},{"line_number":50,"context_line":"# NOVA_API_VERSION valid options"},{"line_number":51,"context_line":"#   - v2default - setup API end points as nova does out of the box"},{"line_number":52,"context_line":"#   - v21default - make v21 the default on /v2"},{"line_number":53,"context_line":"# NOTE(sdague): this is for transitional testing of the Nova v21 API."},{"line_number":54,"context_line":"# Expect to remove in L or M."},{"line_number":55,"context_line":"NOVA_API_VERSION\u003d${NOVA_API_VERSION-v21default}"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"9a80dd14_05a0cef2","line":52,"updated":"2015-03-20 01:30:20.000000000","message":"How about changing here like:\n\n - default - Don\u0027t change API endpoint and use default endpoint\n - v2default - make v2 the default on /v2\n - v21default - make v21 the default on /v2\n\nand this default value is still \"default\".\non project-config side, we specify \"v2default\" for v2 API job and \"v21default\" for v2.1 API job.\nThe other jobs(like postgres, neutron, etc) are depends on what is default values of Nova side. That would fit to development status.","commit_id":"89ffe4b26db8e8642a82731e9566936e37b37c5f"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"eb4a6ddefd5ae4768192ab1eeebb11eb14ab6533","unresolved":false,"context_lines":[{"line_number":52,"context_line":"#   - v21default - make v21 the default on /v2"},{"line_number":53,"context_line":"# NOTE(sdague): this is for transitional testing of the Nova v21 API."},{"line_number":54,"context_line":"# Expect to remove in L or M."},{"line_number":55,"context_line":"NOVA_API_VERSION\u003d${NOVA_API_VERSION-v21default}"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"if is_ssl_enabled_service \"nova\" || is_service_enabled tls-proxy; then"},{"line_number":58,"context_line":"    NOVA_SERVICE_PROTOCOL\u003d\"https\""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"9a80dd14_05d52e42","line":55,"updated":"2015-03-20 01:30:20.000000000","message":"As the above suggestion, it is find to keep \"default\" here. but maybe we need to fix here like\n\n \tNOVA_API_VERSION\u003d${NOVA_API_VERSION:-default}\n\nby adding \":\"","commit_id":"89ffe4b26db8e8642a82731e9566936e37b37c5f"}],"lib/tempest":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"b88156807746366e1c4877cd1f60f8ccd361a3ec","unresolved":false,"context_lines":[{"line_number":85,"context_line":"# Nova catalog type. Default it to computev21 as v2.1 APIs are current API for Nova."},{"line_number":86,"context_line":"# compute - Nova v2 API"},{"line_number":87,"context_line":"# computev21 - Nova v2.1 API"},{"line_number":88,"context_line":"COMPUTE_CATALOG_TYPE\u003d$(COMPUTE_CATALOG_TYPE:-computev21)"},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"# Functions"},{"line_number":91,"context_line":"# ---------"}],"source_content_type":"application/x-shellscript","patch_set":5,"id":"3a50d1a3_307342e9","line":88,"updated":"2015-07-27 14:29:15.000000000","message":"You need quotes around computev21","commit_id":"8eceacbd09625681543b63c4d5bd89f52b423ad5"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"651ebe87846d11110ab2eeaa86e19629914da0a1","unresolved":false,"context_lines":[{"line_number":85,"context_line":"# Nova catalog type. Default it to computev21 as v2.1 APIs are current API for Nova."},{"line_number":86,"context_line":"# compute - Nova v2 API"},{"line_number":87,"context_line":"# computev21 - Nova v2.1 API"},{"line_number":88,"context_line":"COMPUTE_CATALOG_TYPE\u003d$(COMPUTE_CATALOG_TYPE:-computev21)"},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"# Functions"},{"line_number":91,"context_line":"# ---------"}],"source_content_type":"application/x-shellscript","patch_set":5,"id":"3a50d1a3_67ad799a","line":88,"in_reply_to":"3a50d1a3_307342e9","updated":"2015-07-28 00:12:59.000000000","message":"ahh, and i used wrong braces also.","commit_id":"8eceacbd09625681543b63c4d5bd89f52b423ad5"}]}
