)]}'
{"lib/keystone":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"6da08ed3844911d13b67579023cd893e97927ad9","unresolved":false,"context_lines":[{"line_number":610,"context_line":"    fi"},{"line_number":611,"context_line":""},{"line_number":612,"context_line":"    echo \"Waiting for keystone to start...\""},{"line_number":613,"context_line":"    # Check that the keystone service is running. Even if the tls tunnel"},{"line_number":614,"context_line":"    # should be enabled, make sure the internal port is checked using"},{"line_number":615,"context_line":"    # unencryted traffic at this point."},{"line_number":616,"context_line":"    # If running in Apache, use the path rather than port."}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"3a461143_8a9efcb6","line":613,"updated":"2017-01-31 10:59:11.000000000","message":"The code comment (2nd and 3rd sentences) must be moved inside your wait_for_keystone function.","commit_id":"19ec90a1e2c9fb56e34c95ad87b54b425e27a87c"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"9c55d0dd7aac7fa490bbb2597f58a3a22572ad8a","unresolved":false,"context_lines":[{"line_number":610,"context_line":"    fi"},{"line_number":611,"context_line":""},{"line_number":612,"context_line":"    echo \"Waiting for keystone to start...\""},{"line_number":613,"context_line":"    # Check that the keystone service is running. Even if the tls tunnel"},{"line_number":614,"context_line":"    # should be enabled, make sure the internal port is checked using"},{"line_number":615,"context_line":"    # unencryted traffic at this point."},{"line_number":616,"context_line":"    # If running in Apache, use the path rather than port."}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"3a461143_4a6c149c","line":613,"in_reply_to":"3a461143_8a9efcb6","updated":"2017-01-31 11:02:09.000000000","message":"Done","commit_id":"19ec90a1e2c9fb56e34c95ad87b54b425e27a87c"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"f4aefb195c2cbdef149c8c7633fb9b043ff25fda","unresolved":false,"context_lines":[{"line_number":593,"context_line":"    # should be enabled, make sure the internal port is checked using"},{"line_number":594,"context_line":"    # unencryted traffic at this point."},{"line_number":595,"context_line":"    # If running in Apache, use the path rather than port."},{"line_number":596,"context_line":"    local service_uri\u003d$auth_protocol://$KEYSTONE_SERVICE_HOST:$service_port/v$IDENTITY_API_VERSION/"},{"line_number":597,"context_line":"    if [ \"$KEYSTONE_DEPLOY\" \u003d\u003d \"mod_wsgi\" ]; then"},{"line_number":598,"context_line":"        service_uri\u003d$auth_protocol://$KEYSTONE_SERVICE_HOST/identity/v$IDENTITY_API_VERSION/"},{"line_number":599,"context_line":"    fi"}],"source_content_type":"application/x-shellscript","patch_set":6,"id":"3a461143_553aece7","line":596,"updated":"2017-01-31 14:06:55.000000000","message":"nit: Part of me feels like it would be good to limit variable expansion and place the value in quotation marks, but since it is not done in other places here from what I can see, lgtm.","commit_id":"c277b43bb33b87f5d62f4e1681946f734b54cc7b"}],"lib/nova":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"f4aefb195c2cbdef149c8c7633fb9b043ff25fda","unresolved":false,"context_lines":[{"line_number":813,"context_line":"        while [[ \\$(openstack compute service list --host \\$(hostname) --service nova-compute -f value -c State) !\u003d \u0027up\u0027 ]]; do"},{"line_number":814,"context_line":"            sleep 1"},{"line_number":815,"context_line":"        done"},{"line_number":816,"context_line":"EOF"},{"line_number":817,"context_line":"    time_stop \"wait_for_nova_compute\""},{"line_number":818,"context_line":"    return $rval"},{"line_number":819,"context_line":"}"}],"source_content_type":"application/x-shellscript","patch_set":6,"id":"3a461143_2061a0a9","line":816,"updated":"2017-01-31 14:06:55.000000000","message":"nit: This kind of looks a little weird, but it is readable.","commit_id":"c277b43bb33b87f5d62f4e1681946f734b54cc7b"}]}