)]}'
{"rally/cli/commands/verify.py":[{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"b2b126bf7bc21e69934421fdee12b7ed0fe9fc3b","unresolved":false,"context_lines":[{"line_number":154,"context_line":"    @envutils.with_default_verifier_id(cli_arg_name\u003d\"verifier-id\")"},{"line_number":155,"context_line":"    @plugins.ensure_plugins_are_loaded"},{"line_number":156,"context_line":"    def show_verifier(self, api, verifier_id\u003dNone):"},{"line_number":157,"context_line":"        \"\"\"Show information about verifier.\"\"\""},{"line_number":158,"context_line":"        verifier \u003d api.verifier.get(verifier_id)"},{"line_number":159,"context_line":"        fields \u003d [\"UUID\", \"Name\", \"Description\", \"Type\", \"Namespace\","},{"line_number":160,"context_line":"                  \"Created at\", \"Updated at\", \"Status\", \"Source\", \"Version\","}],"source_content_type":"text/x-python","patch_set":2,"id":"5a3905b3_d77634d6","line":157,"range":{"start_line":157,"start_character":11,"end_line":157,"end_character":42},"updated":"2017-01-21 14:11:49.000000000","message":"Let\u0027s keep consistency with other commands.\n\n\n\"\"\"Show a verifier.\"\"\"","commit_id":"c12e0503069d4fd4f340bde9ca281eeb4b0d5789"},{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"21e9cbbabeb846ccb78d610296ac39eb9322cd6f","unresolved":false,"context_lines":[{"line_number":154,"context_line":"    @envutils.with_default_verifier_id(cli_arg_name\u003d\"verifier-id\")"},{"line_number":155,"context_line":"    @plugins.ensure_plugins_are_loaded"},{"line_number":156,"context_line":"    def show_verifier(self, api, verifier_id\u003dNone):"},{"line_number":157,"context_line":"        \"\"\"Show information about verifier.\"\"\""},{"line_number":158,"context_line":"        verifier \u003d api.verifier.get(verifier_id)"},{"line_number":159,"context_line":"        fields \u003d [\"UUID\", \"Name\", \"Description\", \"Type\", \"Namespace\","},{"line_number":160,"context_line":"                  \"Created at\", \"Updated at\", \"Status\", \"Source\", \"Version\","}],"source_content_type":"text/x-python","patch_set":2,"id":"5a3905b3_a3ea250d","line":157,"range":{"start_line":157,"start_character":11,"end_line":157,"end_character":42},"in_reply_to":"5a3905b3_d77634d6","updated":"2017-01-23 10:28:05.000000000","message":"\"\"\"Show a verifier.\"\"\" is more unified, but less informative.","commit_id":"c12e0503069d4fd4f340bde9ca281eeb4b0d5789"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"b2b126bf7bc21e69934421fdee12b7ed0fe9fc3b","unresolved":false,"context_lines":[{"line_number":158,"context_line":"        verifier \u003d api.verifier.get(verifier_id)"},{"line_number":159,"context_line":"        fields \u003d [\"UUID\", \"Name\", \"Description\", \"Type\", \"Namespace\","},{"line_number":160,"context_line":"                  \"Created at\", \"Updated at\", \"Status\", \"Source\", \"Version\","},{"line_number":161,"context_line":"                  \"System-wide\", \"Active\", \"Extra settings\"]"},{"line_number":162,"context_line":"        used_verifier \u003d envutils.get_global(envutils.ENV_VERIFIER)"},{"line_number":163,"context_line":"        formatters \u003d {"},{"line_number":164,"context_line":"            \"Created at\": lambda v: v.created_at.replace(microsecond\u003d0),"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a3905b3_7742e069","line":161,"updated":"2017-01-21 14:11:49.000000000","message":"Let\u0027s add \u0027Location\u0027 field at the end of the list.","commit_id":"c12e0503069d4fd4f340bde9ca281eeb4b0d5789"},{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"21e9cbbabeb846ccb78d610296ac39eb9322cd6f","unresolved":false,"context_lines":[{"line_number":158,"context_line":"        verifier \u003d api.verifier.get(verifier_id)"},{"line_number":159,"context_line":"        fields \u003d [\"UUID\", \"Name\", \"Description\", \"Type\", \"Namespace\","},{"line_number":160,"context_line":"                  \"Created at\", \"Updated at\", \"Status\", \"Source\", \"Version\","},{"line_number":161,"context_line":"                  \"System-wide\", \"Active\", \"Extra settings\"]"},{"line_number":162,"context_line":"        used_verifier \u003d envutils.get_global(envutils.ENV_VERIFIER)"},{"line_number":163,"context_line":"        formatters \u003d {"},{"line_number":164,"context_line":"            \"Created at\": lambda v: v.created_at.replace(microsecond\u003d0),"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a3905b3_8318c976","line":161,"in_reply_to":"5a3905b3_7742e069","updated":"2017-01-23 10:28:05.000000000","message":"Done","commit_id":"c12e0503069d4fd4f340bde9ca281eeb4b0d5789"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"b2b126bf7bc21e69934421fdee12b7ed0fe9fc3b","unresolved":false,"context_lines":[{"line_number":165,"context_line":"            \"Updated at\": lambda v: v.updated_at.replace(microsecond\u003d0),"},{"line_number":166,"context_line":"            \"Active\": lambda v: u\"\\u2714\" if v.uuid \u003d\u003d used_verifier else \"\","},{"line_number":167,"context_line":"            \"Extra settings\": lambda v: (json.dumps(v.extra_settings, indent\u003d4)"},{"line_number":168,"context_line":"                                         if v.extra_settings else \"\")"},{"line_number":169,"context_line":"        }"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":"        cliutils.print_dict(verifier, fields\u003dfields, formatters\u003dformatters,"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a3905b3_17543c1f","line":168,"updated":"2017-01-21 14:11:49.000000000","message":"\"Location\": lambda v: v.manager.repo_dir","commit_id":"c12e0503069d4fd4f340bde9ca281eeb4b0d5789"},{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"21e9cbbabeb846ccb78d610296ac39eb9322cd6f","unresolved":false,"context_lines":[{"line_number":165,"context_line":"            \"Updated at\": lambda v: v.updated_at.replace(microsecond\u003d0),"},{"line_number":166,"context_line":"            \"Active\": lambda v: u\"\\u2714\" if v.uuid \u003d\u003d used_verifier else \"\","},{"line_number":167,"context_line":"            \"Extra settings\": lambda v: (json.dumps(v.extra_settings, indent\u003d4)"},{"line_number":168,"context_line":"                                         if v.extra_settings else \"\")"},{"line_number":169,"context_line":"        }"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":"        cliutils.print_dict(verifier, fields\u003dfields, formatters\u003dformatters,"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a3905b3_638e7d7a","line":168,"in_reply_to":"5a3905b3_17543c1f","updated":"2017-01-23 10:28:05.000000000","message":"not sure that it is the right location. home_dir looks better, but it is not critical right now.","commit_id":"c12e0503069d4fd4f340bde9ca281eeb4b0d5789"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"bd4dbeb3dc73ebea8ce7f6195199724a7a0d65f5","unresolved":false,"context_lines":[{"line_number":165,"context_line":"            \"Updated at\": lambda v: v.updated_at.replace(microsecond\u003d0),"},{"line_number":166,"context_line":"            \"Active\": lambda v: u\"\\u2714\" if v.uuid \u003d\u003d used_verifier else \"\","},{"line_number":167,"context_line":"            \"Extra settings\": lambda v: (json.dumps(v.extra_settings, indent\u003d4)"},{"line_number":168,"context_line":"                                         if v.extra_settings else \"\")"},{"line_number":169,"context_line":"        }"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":"        cliutils.print_dict(verifier, fields\u003dfields, formatters\u003dformatters,"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a3905b3_43a2619a","line":168,"in_reply_to":"5a3905b3_638e7d7a","updated":"2017-01-23 10:36:23.000000000","message":"Home dir is deployment-specific dir.","commit_id":"c12e0503069d4fd4f340bde9ca281eeb4b0d5789"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"92aa329ae0d96a98d2509ea86833edbb81e796b4","unresolved":false,"context_lines":[{"line_number":149,"context_line":"                    \"command `rally verify create-verifier`.\"))"},{"line_number":150,"context_line":""},{"line_number":151,"context_line":"    @cliutils.help_group(\"verifier\")"},{"line_number":152,"context_line":"    @cliutils.args(\"--id\", dest\u003d\"verifier_id\", type\u003dstr, required\u003dTrue,"},{"line_number":153,"context_line":"                   help\u003d\"Verifier name or UUID. \" + LIST_VERIFIERS_HINT)"},{"line_number":154,"context_line":"    @envutils.with_default_verifier_id(cli_arg_name\u003d\"verifier-id\")"},{"line_number":155,"context_line":"    @plugins.ensure_plugins_are_loaded"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a3905b3_548859c5","line":152,"range":{"start_line":152,"start_character":57,"end_line":152,"end_character":71},"updated":"2017-01-23 11:39:49.000000000","message":"It should be removed for proper work. Please, look at other commands.","commit_id":"25c2a0f329dfddf2381418fcb5c5e3cbd53e268e"},{"author":{"_account_id":19011,"name":"Anton Studenov","email":"tohin1332@gmail.com","username":"tohin"},"change_message_id":"25aaf88c645338f24aa2d3a02a57e3381746a1b1","unresolved":false,"context_lines":[{"line_number":151,"context_line":"    @cliutils.help_group(\"verifier\")"},{"line_number":152,"context_line":"    @cliutils.args(\"--id\", dest\u003d\"verifier_id\", type\u003dstr, required\u003dTrue,"},{"line_number":153,"context_line":"                   help\u003d\"Verifier name or UUID. \" + LIST_VERIFIERS_HINT)"},{"line_number":154,"context_line":"    @envutils.with_default_verifier_id(cli_arg_name\u003d\"verifier-id\")"},{"line_number":155,"context_line":"    @plugins.ensure_plugins_are_loaded"},{"line_number":156,"context_line":"    def show_verifier(self, api, verifier_id\u003dNone):"},{"line_number":157,"context_line":"        \"\"\"Show detailed information about verifier.\"\"\""}],"source_content_type":"text/x-python","patch_set":3,"id":"5a3905b3_9e6740b6","line":154,"range":{"start_line":154,"start_character":53,"end_line":154,"end_character":64},"updated":"2017-01-23 10:51:56.000000000","message":"Shouldn\u0027t it be \"id\" in case of --id argument ?","commit_id":"25c2a0f329dfddf2381418fcb5c5e3cbd53e268e"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"1a07cd2d4a4f3f15bebf59678a673b97ac75ceff","unresolved":false,"context_lines":[{"line_number":151,"context_line":"    @cliutils.help_group(\"verifier\")"},{"line_number":152,"context_line":"    @cliutils.args(\"--id\", dest\u003d\"verifier_id\", type\u003dstr, required\u003dTrue,"},{"line_number":153,"context_line":"                   help\u003d\"Verifier name or UUID. \" + LIST_VERIFIERS_HINT)"},{"line_number":154,"context_line":"    @envutils.with_default_verifier_id(cli_arg_name\u003d\"verifier-id\")"},{"line_number":155,"context_line":"    @plugins.ensure_plugins_are_loaded"},{"line_number":156,"context_line":"    def show_verifier(self, api, verifier_id\u003dNone):"},{"line_number":157,"context_line":"        \"\"\"Show detailed information about verifier.\"\"\""}],"source_content_type":"text/x-python","patch_set":3,"id":"5a3905b3_f9c77e95","line":154,"range":{"start_line":154,"start_character":53,"end_line":154,"end_character":64},"in_reply_to":"5a3905b3_9e6740b6","updated":"2017-01-23 11:03:49.000000000","message":"Yep, actually it should be\n\n\n@envutils.with_default_verifier_id()\n\n\nYou need to rebase this on https://review.openstack.org/#/c/422089/ .","commit_id":"25c2a0f329dfddf2381418fcb5c5e3cbd53e268e"},{"author":{"_account_id":19011,"name":"Anton Studenov","email":"tohin1332@gmail.com","username":"tohin"},"change_message_id":"25aaf88c645338f24aa2d3a02a57e3381746a1b1","unresolved":false,"context_lines":[{"line_number":153,"context_line":"                   help\u003d\"Verifier name or UUID. \" + LIST_VERIFIERS_HINT)"},{"line_number":154,"context_line":"    @envutils.with_default_verifier_id(cli_arg_name\u003d\"verifier-id\")"},{"line_number":155,"context_line":"    @plugins.ensure_plugins_are_loaded"},{"line_number":156,"context_line":"    def show_verifier(self, api, verifier_id\u003dNone):"},{"line_number":157,"context_line":"        \"\"\"Show detailed information about verifier.\"\"\""},{"line_number":158,"context_line":"        verifier \u003d api.verifier.get(verifier_id)"},{"line_number":159,"context_line":"        fields \u003d [\"UUID\", \"Name\", \"Description\", \"Type\", \"Namespace\","}],"source_content_type":"text/x-python","patch_set":3,"id":"5a3905b3_bed4fc1f","line":156,"range":{"start_line":156,"start_character":44,"end_line":156,"end_character":49},"updated":"2017-01-23 10:51:56.000000000","message":"I think it is better to remove the default.","commit_id":"25c2a0f329dfddf2381418fcb5c5e3cbd53e268e"},{"author":{"_account_id":19011,"name":"Anton Studenov","email":"tohin1332@gmail.com","username":"tohin"},"change_message_id":"96d0c761c4f966e20741fc172d5f031b32919b20","unresolved":false,"context_lines":[{"line_number":153,"context_line":"                   help\u003d\"Verifier name or UUID. \" + LIST_VERIFIERS_HINT)"},{"line_number":154,"context_line":"    @envutils.with_default_verifier_id(cli_arg_name\u003d\"verifier-id\")"},{"line_number":155,"context_line":"    @plugins.ensure_plugins_are_loaded"},{"line_number":156,"context_line":"    def show_verifier(self, api, verifier_id\u003dNone):"},{"line_number":157,"context_line":"        \"\"\"Show detailed information about verifier.\"\"\""},{"line_number":158,"context_line":"        verifier \u003d api.verifier.get(verifier_id)"},{"line_number":159,"context_line":"        fields \u003d [\"UUID\", \"Name\", \"Description\", \"Type\", \"Namespace\","}],"source_content_type":"text/x-python","patch_set":3,"id":"5a3905b3_99159ae6","line":156,"range":{"start_line":156,"start_character":44,"end_line":156,"end_character":49},"in_reply_to":"5a3905b3_59aed2e6","updated":"2017-01-23 11:06:44.000000000","message":"So it is ok to get verifier with api.verifier.get(None) ?","commit_id":"25c2a0f329dfddf2381418fcb5c5e3cbd53e268e"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"92aa329ae0d96a98d2509ea86833edbb81e796b4","unresolved":false,"context_lines":[{"line_number":153,"context_line":"                   help\u003d\"Verifier name or UUID. \" + LIST_VERIFIERS_HINT)"},{"line_number":154,"context_line":"    @envutils.with_default_verifier_id(cli_arg_name\u003d\"verifier-id\")"},{"line_number":155,"context_line":"    @plugins.ensure_plugins_are_loaded"},{"line_number":156,"context_line":"    def show_verifier(self, api, verifier_id\u003dNone):"},{"line_number":157,"context_line":"        \"\"\"Show detailed information about verifier.\"\"\""},{"line_number":158,"context_line":"        verifier \u003d api.verifier.get(verifier_id)"},{"line_number":159,"context_line":"        fields \u003d [\"UUID\", \"Name\", \"Description\", \"Type\", \"Namespace\","}],"source_content_type":"text/x-python","patch_set":3,"id":"5a3905b3_348bcdc4","line":156,"range":{"start_line":156,"start_character":44,"end_line":156,"end_character":49},"in_reply_to":"5a3905b3_99159ae6","updated":"2017-01-23 11:39:49.000000000","message":"Without verifier_id\u003dNone @envutils.with_default_verifier_id will not work correctly.","commit_id":"25c2a0f329dfddf2381418fcb5c5e3cbd53e268e"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"1a07cd2d4a4f3f15bebf59678a673b97ac75ceff","unresolved":false,"context_lines":[{"line_number":153,"context_line":"                   help\u003d\"Verifier name or UUID. \" + LIST_VERIFIERS_HINT)"},{"line_number":154,"context_line":"    @envutils.with_default_verifier_id(cli_arg_name\u003d\"verifier-id\")"},{"line_number":155,"context_line":"    @plugins.ensure_plugins_are_loaded"},{"line_number":156,"context_line":"    def show_verifier(self, api, verifier_id\u003dNone):"},{"line_number":157,"context_line":"        \"\"\"Show detailed information about verifier.\"\"\""},{"line_number":158,"context_line":"        verifier \u003d api.verifier.get(verifier_id)"},{"line_number":159,"context_line":"        fields \u003d [\"UUID\", \"Name\", \"Description\", \"Type\", \"Namespace\","}],"source_content_type":"text/x-python","patch_set":3,"id":"5a3905b3_59aed2e6","line":156,"range":{"start_line":156,"start_character":44,"end_line":156,"end_character":49},"in_reply_to":"5a3905b3_bed4fc1f","updated":"2017-01-23 11:03:49.000000000","message":"No, the default value is needed.","commit_id":"25c2a0f329dfddf2381418fcb5c5e3cbd53e268e"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"79be80da2b3197ecece558e100bbaccb83b8c6e5","unresolved":false,"context_lines":[{"line_number":164,"context_line":"        formatters \u003d {"},{"line_number":165,"context_line":"            \"Created at\": lambda v: v.created_at.replace(microsecond\u003d0),"},{"line_number":166,"context_line":"            \"Updated at\": lambda v: v.updated_at.replace(microsecond\u003d0),"},{"line_number":167,"context_line":"            \"Active\": lambda v: u\"\\u2714\" if v.uuid \u003d\u003d used_verifier else \"\","},{"line_number":168,"context_line":"            \"Extra settings\": lambda v: (json.dumps(v.extra_settings, indent\u003d4)"},{"line_number":169,"context_line":"                                         if v.extra_settings else \"\"),"},{"line_number":170,"context_line":"            \"Location\": lambda v: v.manager.repo_dir"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a3905b3_4f67e0e8","line":167,"range":{"start_line":167,"start_character":74,"end_line":167,"end_character":76},"updated":"2017-01-23 11:45:06.000000000","message":"Let\u0027s use \"-\" instead.","commit_id":"25c2a0f329dfddf2381418fcb5c5e3cbd53e268e"},{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"36998a5ef29ead9b32971e2f8bcab362f28e0c7b","unresolved":false,"context_lines":[{"line_number":160,"context_line":"        fields \u003d [\"UUID\", \"Status\", \"Name\", \"Description\", \"Type\", \"Namespace\","},{"line_number":161,"context_line":"                  \"Created at\", \"Updated at\", \"Source\", \"Version\","},{"line_number":162,"context_line":"                  \"System-wide\", \"Active\", \"Extra settings\", \"Location\","},{"line_number":163,"context_line":"                  \"Venv location\"]"},{"line_number":164,"context_line":"        used_verifier \u003d envutils.get_global(envutils.ENV_VERIFIER)"},{"line_number":165,"context_line":"        formatters \u003d {"},{"line_number":166,"context_line":"            \"Created at\": lambda v: v.created_at.replace(microsecond\u003d0),"}],"source_content_type":"text/x-python","patch_set":5,"id":"5a3905b3_edc22152","line":163,"range":{"start_line":163,"start_character":18,"end_line":163,"end_character":33},"updated":"2017-01-24 09:39:04.000000000","message":"It is better to add extra check:\n\n    if not verifier.system_side:\n         fields.append(\"Venv location\")\n\nBtw, not sure that we need \"Location\" and \"Venv location\" in normal mode. Maybe it is better to put it in debug?","commit_id":"4d6ab98114ec7e0943d694f23f6913f45d6fac17"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"7178bb1788ce558b21ecb047c30c89c82cf7bfd7","unresolved":false,"context_lines":[{"line_number":160,"context_line":"        fields \u003d [\"UUID\", \"Status\", \"Name\", \"Description\", \"Type\", \"Namespace\","},{"line_number":161,"context_line":"                  \"Created at\", \"Updated at\", \"Source\", \"Version\","},{"line_number":162,"context_line":"                  \"System-wide\", \"Active\", \"Extra settings\", \"Location\","},{"line_number":163,"context_line":"                  \"Venv location\"]"},{"line_number":164,"context_line":"        used_verifier \u003d envutils.get_global(envutils.ENV_VERIFIER)"},{"line_number":165,"context_line":"        formatters \u003d {"},{"line_number":166,"context_line":"            \"Created at\": lambda v: v.created_at.replace(microsecond\u003d0),"}],"source_content_type":"text/x-python","patch_set":5,"id":"5a3905b3_73ad349a","line":163,"range":{"start_line":163,"start_character":18,"end_line":163,"end_character":33},"in_reply_to":"5a3905b3_edc22152","updated":"2017-01-24 10:18:56.000000000","message":"I think we should print it in the normal mode. But let\u0027s add some warning message that users can do any changes in the verifier repo and the verifier venv, but at your own risk.","commit_id":"4d6ab98114ec7e0943d694f23f6913f45d6fac17"}]}
