)]}'
{"files/keystone_data.sh":[{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"af696abf42043ea5ef37acabc0019fdcee3baa14","unresolved":false,"context_lines":[{"line_number":124,"context_line":"    # Ceilometer needs ResellerAdmin role to access swift account stats."},{"line_number":125,"context_line":"    keystone user-role-add --tenant $SERVICE_TENANT_NAME \\"},{"line_number":126,"context_line":"        --user ceilometer \\"},{"line_number":127,"context_line":"        --role ResellerAdmin"},{"line_number":128,"context_line":"    if [[ \"$KEYSTONE_CATALOG_BACKEND\" \u003d \u0027sql\u0027 ]]; then"},{"line_number":129,"context_line":"        keystone service-create \\"},{"line_number":130,"context_line":"            --name\u003dceilometer \\"}],"source_content_type":"text/x-sh","patch_set":2,"id":"AAAAUH%2F%2BsfM%3D","side":"PARENT","line":127,"updated":"2014-01-23 03:23:01.000000000","message":"-1: This should not be removed","commit_id":"6d323f43b499482765438a964ac11874d4af3aec"},{"author":{"_account_id":7126,"name":"Vadim Rovachev","email":"vrovachev@mirantis.com","username":"vrovachev"},"change_message_id":"7f61de72b31d54de8fb1e97253c60d44cd84ce89","unresolved":false,"context_lines":[{"line_number":124,"context_line":"    # Ceilometer needs ResellerAdmin role to access swift account stats."},{"line_number":125,"context_line":"    keystone user-role-add --tenant $SERVICE_TENANT_NAME \\"},{"line_number":126,"context_line":"        --user ceilometer \\"},{"line_number":127,"context_line":"        --role ResellerAdmin"},{"line_number":128,"context_line":"    if [[ \"$KEYSTONE_CATALOG_BACKEND\" \u003d \u0027sql\u0027 ]]; then"},{"line_number":129,"context_line":"        keystone service-create \\"},{"line_number":130,"context_line":"            --name\u003dceilometer \\"}],"source_content_type":"text/x-sh","patch_set":2,"id":"AAAAUH%2F%2BrH4%3D","side":"PARENT","line":127,"in_reply_to":"AAAAUH%2F%2BrSs%3D","updated":"2014-01-23 10:23:53.000000000","message":"Apropos both methods of ceilometer service registration are caused under identical conditions. But the code in lib/ceilometer is implemented better. I suggest to trace registration Swift service. Registration is implemented in lib/swift.\nIn this regard, I am sure that method implemented in this file is unnecessary.","commit_id":"6d323f43b499482765438a964ac11874d4af3aec"},{"author":{"_account_id":7126,"name":"Vadim Rovachev","email":"vrovachev@mirantis.com","username":"vrovachev"},"change_message_id":"cab8aeab48198e796dc6cc5c899cd2b5b3907652","unresolved":false,"context_lines":[{"line_number":124,"context_line":"    # Ceilometer needs ResellerAdmin role to access swift account stats."},{"line_number":125,"context_line":"    keystone user-role-add --tenant $SERVICE_TENANT_NAME \\"},{"line_number":126,"context_line":"        --user ceilometer \\"},{"line_number":127,"context_line":"        --role ResellerAdmin"},{"line_number":128,"context_line":"    if [[ \"$KEYSTONE_CATALOG_BACKEND\" \u003d \u0027sql\u0027 ]]; then"},{"line_number":129,"context_line":"        keystone service-create \\"},{"line_number":130,"context_line":"            --name\u003dceilometer \\"}],"source_content_type":"text/x-sh","patch_set":2,"id":"AAAAUH%2F%2BrSs%3D","side":"PARENT","line":127,"in_reply_to":"AAAAUH%2F%2BsfM%3D","updated":"2014-01-23 10:00:46.000000000","message":"Then we have to remove this code from lib/ceilometer.\n2 ceilometer service in keystone is bad.","commit_id":"6d323f43b499482765438a964ac11874d4af3aec"}]}