)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13609,"name":"Pavel Boldin","email":"pboldin@mirantis.com","username":"pboldin"},"change_message_id":"3ded06bbf3601cddb161ba067698e5d7d734a81d","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Boris Pavlovic \u003cboris@pavlovic.me\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-04-16 00:50:43 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix rally show commands when we have existing users in deployment"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch contains 2 fixes:"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"dac4157f_2ae8c6b3","line":7,"updated":"2015-04-21 11:22:06.000000000","message":"\"when we have\" -\u003e \"for\"","commit_id":"71873a48e8e02a5e2c36955a4fcb81a72999ea4a"},{"author":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"change_message_id":"2ccb0cfeb14d2ffe0e379d6ad48dfc71fd832f10","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Boris Pavlovic \u003cboris@pavlovic.me\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-04-16 00:50:43 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix rally show commands when we have existing users in deployment"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch contains 2 fixes:"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"bab921fb_941a756d","line":7,"in_reply_to":"dac4157f_2ae8c6b3","updated":"2015-04-23 18:46:42.000000000","message":"Done","commit_id":"71873a48e8e02a5e2c36955a4fcb81a72999ea4a"},{"author":{"_account_id":13609,"name":"Pavel Boldin","email":"pboldin@mirantis.com","username":"pboldin"},"change_message_id":"3ded06bbf3601cddb161ba067698e5d7d734a81d","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix rally show commands when we have existing users in deployment"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch contains 2 fixes:"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"  1) Show commands catch properly exceptions from keystone"},{"line_number":12,"context_line":"     and display human readble output for Rally users."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"dac4157f_cacdca5e","line":9,"updated":"2015-04-21 11:22:06.000000000","message":"well, this seems to be three of them below","commit_id":"71873a48e8e02a5e2c36955a4fcb81a72999ea4a"},{"author":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"change_message_id":"2ccb0cfeb14d2ffe0e379d6ad48dfc71fd832f10","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix rally show commands when we have existing users in deployment"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch contains 2 fixes:"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"  1) Show commands catch properly exceptions from keystone"},{"line_number":12,"context_line":"     and display human readble output for Rally users."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"bab921fb_7426c1bb","line":9,"in_reply_to":"dac4157f_cacdca5e","updated":"2015-04-23 18:46:42.000000000","message":"Done","commit_id":"71873a48e8e02a5e2c36955a4fcb81a72999ea4a"},{"author":{"_account_id":13609,"name":"Pavel Boldin","email":"pboldin@mirantis.com","username":"pboldin"},"change_message_id":"3ded06bbf3601cddb161ba067698e5d7d734a81d","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch contains 2 fixes:"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"  1) Show commands catch properly exceptions from keystone"},{"line_number":12,"context_line":"     and display human readble output for Rally users."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"  2) Each user should print own table with resources."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"dac4157f_2affa668","line":11,"updated":"2015-04-21 11:22:06.000000000","message":"Rally `show\u0027 commands","commit_id":"71873a48e8e02a5e2c36955a4fcb81a72999ea4a"},{"author":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"change_message_id":"2ccb0cfeb14d2ffe0e379d6ad48dfc71fd832f10","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch contains 2 fixes:"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"  1) Show commands catch properly exceptions from keystone"},{"line_number":12,"context_line":"     and display human readble output for Rally users."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"  2) Each user should print own table with resources."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"bab921fb_744da171","line":11,"in_reply_to":"dac4157f_2affa668","updated":"2015-04-23 18:46:42.000000000","message":"Done","commit_id":"71873a48e8e02a5e2c36955a4fcb81a72999ea4a"},{"author":{"_account_id":13609,"name":"Pavel Boldin","email":"pboldin@mirantis.com","username":"pboldin"},"change_message_id":"3ded06bbf3601cddb161ba067698e5d7d734a81d","unresolved":false,"context_lines":[{"line_number":9,"context_line":"This patch contains 2 fixes:"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"  1) Show commands catch properly exceptions from keystone"},{"line_number":12,"context_line":"     and display human readble output for Rally users."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"  2) Each user should print own table with resources."},{"line_number":15,"context_line":"     Before this patch table rows were added (without flush)"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"dac4157f_0a02a26f","line":12,"updated":"2015-04-21 11:22:06.000000000","message":"readable errors","commit_id":"71873a48e8e02a5e2c36955a4fcb81a72999ea4a"},{"author":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"change_message_id":"2ccb0cfeb14d2ffe0e379d6ad48dfc71fd832f10","unresolved":false,"context_lines":[{"line_number":9,"context_line":"This patch contains 2 fixes:"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"  1) Show commands catch properly exceptions from keystone"},{"line_number":12,"context_line":"     and display human readble output for Rally users."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"  2) Each user should print own table with resources."},{"line_number":15,"context_line":"     Before this patch table rows were added (without flush)"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"bab921fb_b44b3959","line":12,"in_reply_to":"dac4157f_0a02a26f","updated":"2015-04-23 18:46:42.000000000","message":"Done","commit_id":"71873a48e8e02a5e2c36955a4fcb81a72999ea4a"},{"author":{"_account_id":13609,"name":"Pavel Boldin","email":"pboldin@mirantis.com","username":"pboldin"},"change_message_id":"3ded06bbf3601cddb161ba067698e5d7d734a81d","unresolved":false,"context_lines":[{"line_number":11,"context_line":"  1) Show commands catch properly exceptions from keystone"},{"line_number":12,"context_line":"     and display human readble output for Rally users."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"  2) Each user should print own table with resources."},{"line_number":15,"context_line":"     Before this patch table rows were added (without flush)"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"  3) Print headers with information about which user\u0027s resource"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"dac4157f_0a398241","line":14,"updated":"2015-04-21 11:22:06.000000000","message":"For each user his own table with resources should be printed. (User is not printing anything, rally does).","commit_id":"71873a48e8e02a5e2c36955a4fcb81a72999ea4a"},{"author":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"change_message_id":"2ccb0cfeb14d2ffe0e379d6ad48dfc71fd832f10","unresolved":false,"context_lines":[{"line_number":11,"context_line":"  1) Show commands catch properly exceptions from keystone"},{"line_number":12,"context_line":"     and display human readble output for Rally users."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"  2) Each user should print own table with resources."},{"line_number":15,"context_line":"     Before this patch table rows were added (without flush)"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"  3) Print headers with information about which user\u0027s resource"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"bab921fb_749b61ca","line":14,"in_reply_to":"dac4157f_0a398241","updated":"2015-04-23 18:46:42.000000000","message":"Done","commit_id":"71873a48e8e02a5e2c36955a4fcb81a72999ea4a"},{"author":{"_account_id":13609,"name":"Pavel Boldin","email":"pboldin@mirantis.com","username":"pboldin"},"change_message_id":"3ded06bbf3601cddb161ba067698e5d7d734a81d","unresolved":false,"context_lines":[{"line_number":12,"context_line":"     and display human readble output for Rally users."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"  2) Each user should print own table with resources."},{"line_number":15,"context_line":"     Before this patch table rows were added (without flush)"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"  3) Print headers with information about which user\u0027s resource"},{"line_number":18,"context_line":"     are in table"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"dac4157f_ea408ea7","line":15,"updated":"2015-04-21 11:22:06.000000000","message":"not \u0027added\u0027 but \u0027summarized\u0027.","commit_id":"71873a48e8e02a5e2c36955a4fcb81a72999ea4a"},{"author":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"change_message_id":"2ccb0cfeb14d2ffe0e379d6ad48dfc71fd832f10","unresolved":false,"context_lines":[{"line_number":12,"context_line":"     and display human readble output for Rally users."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"  2) Each user should print own table with resources."},{"line_number":15,"context_line":"     Before this patch table rows were added (without flush)"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"  3) Print headers with information about which user\u0027s resource"},{"line_number":18,"context_line":"     are in table"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"bab921fb_d4ac6def","line":15,"in_reply_to":"dac4157f_ea408ea7","updated":"2015-04-23 18:46:42.000000000","message":"I am not sure that this makes any sense, but ok","commit_id":"71873a48e8e02a5e2c36955a4fcb81a72999ea4a"},{"author":{"_account_id":13609,"name":"Pavel Boldin","email":"pboldin@mirantis.com","username":"pboldin"},"change_message_id":"3ded06bbf3601cddb161ba067698e5d7d734a81d","unresolved":false,"context_lines":[{"line_number":14,"context_line":"  2) Each user should print own table with resources."},{"line_number":15,"context_line":"     Before this patch table rows were added (without flush)"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"  3) Print headers with information about which user\u0027s resource"},{"line_number":18,"context_line":"     are in table"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Change-Id: Ibcc2051797bf61b3a4db3a9c781ecf750ec8f3ea"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"dac4157f_6a135eb9","line":17,"updated":"2015-04-21 11:22:06.000000000","message":"either \"resources are\" or \"resource is\"","commit_id":"71873a48e8e02a5e2c36955a4fcb81a72999ea4a"},{"author":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"change_message_id":"2ccb0cfeb14d2ffe0e379d6ad48dfc71fd832f10","unresolved":false,"context_lines":[{"line_number":14,"context_line":"  2) Each user should print own table with resources."},{"line_number":15,"context_line":"     Before this patch table rows were added (without flush)"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"  3) Print headers with information about which user\u0027s resource"},{"line_number":18,"context_line":"     are in table"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Change-Id: Ibcc2051797bf61b3a4db3a9c781ecf750ec8f3ea"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"bab921fb_1481c54a","line":17,"in_reply_to":"dac4157f_6a135eb9","updated":"2015-04-23 18:46:42.000000000","message":"Done","commit_id":"71873a48e8e02a5e2c36955a4fcb81a72999ea4a"}],"rally/cmd/cliutils.py":[{"author":{"_account_id":8507,"name":"Mikhail Dubov","email":"mdubov@mirantis.com","username":"msdubov"},"change_message_id":"94d61a574f09bf0ed5b4f5f36b4410fa82e749d1","unresolved":false,"context_lines":[{"line_number":152,"context_line":""},{"line_number":153,"context_line":"@decorator.decorator"},{"line_number":154,"context_line":"def process_keystone_exc(f, *args, **kwargs):"},{"line_number":155,"context_line":"    from keystoneclient import exceptions"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"    try:"},{"line_number":158,"context_line":"        return f(*args, **kwargs)"}],"source_content_type":"text/x-python","patch_set":3,"id":"da9b358b_82559d1e","line":155,"updated":"2015-03-30 09:42:03.000000000","message":"Let\u0027s avoid name conflict with Rally exceptions module:\n\n    from keystoneclient import exceptions as keystone_exc:\n        ...","commit_id":"82ecf5e170e385e1eca0cf44472016ec7339dd1d"},{"author":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"change_message_id":"4f24d8d2daad2bf0f6aa7550439bdca05628457e","unresolved":false,"context_lines":[{"line_number":152,"context_line":""},{"line_number":153,"context_line":"@decorator.decorator"},{"line_number":154,"context_line":"def process_keystone_exc(f, *args, **kwargs):"},{"line_number":155,"context_line":"    from keystoneclient import exceptions"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"    try:"},{"line_number":158,"context_line":"        return f(*args, **kwargs)"}],"source_content_type":"text/x-python","patch_set":3,"id":"5aa745d1_17268cba","line":155,"in_reply_to":"7aaa499b_e23462df","updated":"2015-04-15 22:57:28.000000000","message":"Done","commit_id":"82ecf5e170e385e1eca0cf44472016ec7339dd1d"},{"author":{"_account_id":8576,"name":"Yair Fried","email":"yfried@redhat.com","username":"yfried"},"change_message_id":"e5a0a68bd128c246993a1789d0ffa5bb76d0c31e","unresolved":false,"context_lines":[{"line_number":152,"context_line":""},{"line_number":153,"context_line":"@decorator.decorator"},{"line_number":154,"context_line":"def process_keystone_exc(f, *args, **kwargs):"},{"line_number":155,"context_line":"    from keystoneclient import exceptions"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"    try:"},{"line_number":158,"context_line":"        return f(*args, **kwargs)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7aaa499b_e23462df","line":155,"in_reply_to":"da9b358b_82559d1e","updated":"2015-04-13 07:54:53.000000000","message":"+1","commit_id":"82ecf5e170e385e1eca0cf44472016ec7339dd1d"},{"author":{"_account_id":8507,"name":"Mikhail Dubov","email":"mdubov@mirantis.com","username":"msdubov"},"change_message_id":"94d61a574f09bf0ed5b4f5f36b4410fa82e749d1","unresolved":false,"context_lines":[{"line_number":159,"context_line":"    except exceptions.Unauthorized as e:"},{"line_number":160,"context_line":"        print(_(\"User credentials are wrong! \\n%s\") % e)"},{"line_number":161,"context_line":"        return 1"},{"line_number":162,"context_line":"    except exceptions.AuthorizationFailure as e:"},{"line_number":163,"context_line":"        print(_(\"Not able to connect to Keystone! \\n%s\") % e)"},{"line_number":164,"context_line":"        return 1"},{"line_number":165,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"da9b358b_3d303456","line":162,"updated":"2015-03-30 09:42:03.000000000","message":"Shouldn\u0027t we catch \u0027ConnectionRefused\u0027 as well? This is an exception that occurs when Rally can\u0027t reach the deployment","commit_id":"82ecf5e170e385e1eca0cf44472016ec7339dd1d"},{"author":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"change_message_id":"4f24d8d2daad2bf0f6aa7550439bdca05628457e","unresolved":false,"context_lines":[{"line_number":159,"context_line":"    except exceptions.Unauthorized as e:"},{"line_number":160,"context_line":"        print(_(\"User credentials are wrong! \\n%s\") % e)"},{"line_number":161,"context_line":"        return 1"},{"line_number":162,"context_line":"    except exceptions.AuthorizationFailure as e:"},{"line_number":163,"context_line":"        print(_(\"Not able to connect to Keystone! \\n%s\") % e)"},{"line_number":164,"context_line":"        return 1"},{"line_number":165,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"5aa745d1_d76dc4cd","line":162,"in_reply_to":"7aaa499b_425c3625","updated":"2015-04-15 22:57:28.000000000","message":"Done","commit_id":"82ecf5e170e385e1eca0cf44472016ec7339dd1d"},{"author":{"_account_id":8576,"name":"Yair Fried","email":"yfried@redhat.com","username":"yfried"},"change_message_id":"e5a0a68bd128c246993a1789d0ffa5bb76d0c31e","unresolved":false,"context_lines":[{"line_number":159,"context_line":"    except exceptions.Unauthorized as e:"},{"line_number":160,"context_line":"        print(_(\"User credentials are wrong! \\n%s\") % e)"},{"line_number":161,"context_line":"        return 1"},{"line_number":162,"context_line":"    except exceptions.AuthorizationFailure as e:"},{"line_number":163,"context_line":"        print(_(\"Not able to connect to Keystone! \\n%s\") % e)"},{"line_number":164,"context_line":"        return 1"},{"line_number":165,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"7aaa499b_425c3625","line":162,"in_reply_to":"da9b358b_3d303456","updated":"2015-04-13 07:54:53.000000000","message":"+1","commit_id":"82ecf5e170e385e1eca0cf44472016ec7339dd1d"},{"author":{"_account_id":8507,"name":"Mikhail Dubov","email":"mdubov@mirantis.com","username":"msdubov"},"change_message_id":"94d61a574f09bf0ed5b4f5f36b4410fa82e749d1","unresolved":false,"context_lines":[{"line_number":160,"context_line":"        print(_(\"User credentials are wrong! \\n%s\") % e)"},{"line_number":161,"context_line":"        return 1"},{"line_number":162,"context_line":"    except exceptions.AuthorizationFailure as e:"},{"line_number":163,"context_line":"        print(_(\"Not able to connect to Keystone! \\n%s\") % e)"},{"line_number":164,"context_line":"        return 1"},{"line_number":165,"context_line":""},{"line_number":166,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"da9b358b_22449172","line":163,"updated":"2015-03-30 09:42:03.000000000","message":"s/\"Not able\"/\"Unable\"","commit_id":"82ecf5e170e385e1eca0cf44472016ec7339dd1d"},{"author":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"change_message_id":"4f24d8d2daad2bf0f6aa7550439bdca05628457e","unresolved":false,"context_lines":[{"line_number":160,"context_line":"        print(_(\"User credentials are wrong! \\n%s\") % e)"},{"line_number":161,"context_line":"        return 1"},{"line_number":162,"context_line":"    except exceptions.AuthorizationFailure as e:"},{"line_number":163,"context_line":"        print(_(\"Not able to connect to Keystone! \\n%s\") % e)"},{"line_number":164,"context_line":"        return 1"},{"line_number":165,"context_line":""},{"line_number":166,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"5aa745d1_b77238ac","line":163,"in_reply_to":"da9b358b_22449172","updated":"2015-04-15 22:57:28.000000000","message":"Done","commit_id":"82ecf5e170e385e1eca0cf44472016ec7339dd1d"}],"rally/cmd/commands/show.py":[{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"389ac89afd1ddd8e4d5ca0a474a9b058f6309aa2","unresolved":false,"context_lines":[{"line_number":29,"context_line":""},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"@decorator.decorator"},{"line_number":32,"context_line":"def process_keystone_exc(f, *args, **kwargs):"},{"line_number":33,"context_line":"    from keystoneclient import exceptions"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    try:"}],"source_content_type":"text/x-python","patch_set":2,"id":"da9b358b_b8745fcf","line":32,"updated":"2015-03-29 17:41:22.000000000","message":"imo, module `rally.cmd.cliutils` is a better place for this decorator","commit_id":"24cb76291670025a49b5ca3f0cf9c88966725d3a"},{"author":{"_account_id":8507,"name":"Mikhail Dubov","email":"mdubov@mirantis.com","username":"msdubov"},"change_message_id":"94d61a574f09bf0ed5b4f5f36b4410fa82e749d1","unresolved":false,"context_lines":[{"line_number":33,"context_line":"    \"\"\""},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    def _print_header(self, resource_name, credentials):"},{"line_number":36,"context_line":"        print(\"\\n%(resource)s for `%(user)s` user in `%(tenant)s` tenant:\""},{"line_number":37,"context_line":"              % {\"resource\": resource_name,"},{"line_number":38,"context_line":"                 \"user\": credentials[\"username\"],"},{"line_number":39,"context_line":"                 \"tenant\": credentials[\"tenant_name\"]})"}],"source_content_type":"text/x-python","patch_set":3,"id":"da9b358b_bd48c42e","line":36,"updated":"2015-03-30 09:42:03.000000000","message":"Perhaps it will be a little bit more clear if you put user/tenant names as a second word:\n\n    print(\"\\n%(resource)s for user `%(user)s` in tenant `%(tenant)s`:\"","commit_id":"82ecf5e170e385e1eca0cf44472016ec7339dd1d"},{"author":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"change_message_id":"4f24d8d2daad2bf0f6aa7550439bdca05628457e","unresolved":false,"context_lines":[{"line_number":33,"context_line":"    \"\"\""},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    def _print_header(self, resource_name, credentials):"},{"line_number":36,"context_line":"        print(\"\\n%(resource)s for `%(user)s` user in `%(tenant)s` tenant:\""},{"line_number":37,"context_line":"              % {\"resource\": resource_name,"},{"line_number":38,"context_line":"                 \"user\": credentials[\"username\"],"},{"line_number":39,"context_line":"                 \"tenant\": credentials[\"tenant_name\"]})"}],"source_content_type":"text/x-python","patch_set":3,"id":"5aa745d1_375f4810","line":36,"in_reply_to":"da9b358b_bd48c42e","updated":"2015-04-15 22:57:28.000000000","message":"Done","commit_id":"82ecf5e170e385e1eca0cf44472016ec7339dd1d"},{"author":{"_account_id":12395,"name":"Roman Vasylets","email":"pomeo92@gmail.com","username":"rvasilets"},"change_message_id":"01facbadc33cd0252e206ed97cf791c8b3a311ad","unresolved":false,"context_lines":[{"line_number":33,"context_line":"    \"\"\""},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    def _print_header(self, resource_name, credentials):"},{"line_number":36,"context_line":"        print(\"\\n%(resource)s for user `%(user)s` in tenant `%(tenant)s`:\""},{"line_number":37,"context_line":"              % {\"resource\": resource_name,"},{"line_number":38,"context_line":"                 \"user\": credentials[\"username\"],"},{"line_number":39,"context_line":"                 \"tenant\": credentials[\"tenant_name\"]})"}],"source_content_type":"text/x-python","patch_set":4,"id":"dac4157f_6a2c7e02","line":36,"updated":"2015-04-22 14:21:00.000000000","message":"Why you don\u0027t use here _?","commit_id":"71873a48e8e02a5e2c36955a4fcb81a72999ea4a"},{"author":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"change_message_id":"2ccb0cfeb14d2ffe0e379d6ad48dfc71fd832f10","unresolved":false,"context_lines":[{"line_number":33,"context_line":"    \"\"\""},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    def _print_header(self, resource_name, credentials):"},{"line_number":36,"context_line":"        print(\"\\n%(resource)s for user `%(user)s` in tenant `%(tenant)s`:\""},{"line_number":37,"context_line":"              % {\"resource\": resource_name,"},{"line_number":38,"context_line":"                 \"user\": credentials[\"username\"],"},{"line_number":39,"context_line":"                 \"tenant\": credentials[\"tenant_name\"]})"}],"source_content_type":"text/x-python","patch_set":4,"id":"bab921fb_6fbb8ec5","line":36,"in_reply_to":"dac4157f_6a2c7e02","updated":"2015-04-23 18:46:42.000000000","message":"Done","commit_id":"71873a48e8e02a5e2c36955a4fcb81a72999ea4a"},{"author":{"_account_id":10475,"name":"Alexander Maretskiy","email":"amaretskiy@mirantis.com","username":"maretskiy"},"change_message_id":"cadc5b84e65b3265575caef2d43836d2a233dff7","unresolved":false,"context_lines":[{"line_number":66,"context_line":"                               for col in float_cols]))"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"        for endpoint_dict in self._get_endpoints(deployment):"},{"line_number":69,"context_line":"            self._print_header(\"Images\", endpoint_dict)"},{"line_number":70,"context_line":"            table_rows \u003d []"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"            clients \u003d osclients.Clients(objects.Endpoint(**endpoint_dict))"}],"source_content_type":"text/x-python","patch_set":4,"id":"bab921fb_c665e045","line":69,"updated":"2015-04-23 16:39:02.000000000","message":"This should be moved to L71, immediately before cliutils.print_list() - so header will not be printed in case of error\n\nSame for similar cases below.","commit_id":"71873a48e8e02a5e2c36955a4fcb81a72999ea4a"},{"author":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"change_message_id":"2ccb0cfeb14d2ffe0e379d6ad48dfc71fd832f10","unresolved":false,"context_lines":[{"line_number":66,"context_line":"                               for col in float_cols]))"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"        for endpoint_dict in self._get_endpoints(deployment):"},{"line_number":69,"context_line":"            self._print_header(\"Images\", endpoint_dict)"},{"line_number":70,"context_line":"            table_rows \u003d []"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"            clients \u003d osclients.Clients(objects.Endpoint(**endpoint_dict))"}],"source_content_type":"text/x-python","patch_set":4,"id":"bab921fb_cd31780d","line":69,"in_reply_to":"bab921fb_c665e045","updated":"2015-04-23 18:46:42.000000000","message":"It is better to keep this as is. \n\nFor example, \n\n Images for user X1, tenant Y1\n\n  Table\n\n  Images for user X2, tenant Y2\n\n  \u003cTraceBack\u003e \n\n\nSo it will be simpler for Rally users to debug.\nLike they will now that Images for user X2 and Y2 for some reason are not available.","commit_id":"71873a48e8e02a5e2c36955a4fcb81a72999ea4a"},{"author":{"_account_id":8507,"name":"Mikhail Dubov","email":"mdubov@mirantis.com","username":"msdubov"},"change_message_id":"9b9c9daf82f71459268b8a5482bfccfad01fefb4","unresolved":false,"context_lines":[{"line_number":67,"context_line":""},{"line_number":68,"context_line":"        for endpoint_dict in self._get_endpoints(deployment):"},{"line_number":69,"context_line":"            self._print_header(\"Images\", endpoint_dict)"},{"line_number":70,"context_line":"            table_rows \u003d []"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"            clients \u003d osclients.Clients(objects.Endpoint(**endpoint_dict))"},{"line_number":73,"context_line":"            glance_client \u003d clients.glance()"}],"source_content_type":"text/x-python","patch_set":4,"id":"dac4157f_c0912b54","line":70,"updated":"2015-04-21 12:08:47.000000000","message":"It\u0027s bad that this bug with no flush was not caught by unit tests. I believe UTs should be modified a bit to check what happens when there is more than one endpoint:\n\nhttps://github.com/openstack/rally/blob/master/tests/unit/cmd/commands/test_show.py#L48","commit_id":"71873a48e8e02a5e2c36955a4fcb81a72999ea4a"},{"author":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"change_message_id":"2ccb0cfeb14d2ffe0e379d6ad48dfc71fd832f10","unresolved":false,"context_lines":[{"line_number":67,"context_line":""},{"line_number":68,"context_line":"        for endpoint_dict in self._get_endpoints(deployment):"},{"line_number":69,"context_line":"            self._print_header(\"Images\", endpoint_dict)"},{"line_number":70,"context_line":"            table_rows \u003d []"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"            clients \u003d osclients.Clients(objects.Endpoint(**endpoint_dict))"},{"line_number":73,"context_line":"            glance_client \u003d clients.glance()"}],"source_content_type":"text/x-python","patch_set":4,"id":"bab921fb_cd605838","line":70,"in_reply_to":"dac4157f_c0912b54","updated":"2015-04-23 18:46:42.000000000","message":"Done","commit_id":"71873a48e8e02a5e2c36955a4fcb81a72999ea4a"}]}