)]}'
{"zunclient/v1/containers_shell.py":[{"author":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"change_message_id":"c369e9533e3f78f2d0383dfb2cfc8af728061322","unresolved":false,"context_lines":[{"line_number":176,"context_line":"@utils.arg(\u0027--all-tenants\u0027,"},{"line_number":177,"context_line":"           action\u003d\"store_true\","},{"line_number":178,"context_line":"           default\u003dFalse,"},{"line_number":179,"context_line":"           help\u003d\u0027Delete container(s) in all tenant by name.\u0027)"},{"line_number":180,"context_line":"def do_delete(cs, args):"},{"line_number":181,"context_line":"    \"\"\"Delete specified containers.\"\"\""},{"line_number":182,"context_line":"    delete_containers \u003d []"}],"source_content_type":"text/x-python","patch_set":1,"id":"7ffa3b31_0cb5ca31","line":179,"range":{"start_line":179,"start_character":17,"end_line":179,"end_character":58},"updated":"2017-04-14 22:51:43.000000000","message":"Suggest to change it to:\n\n\u0027Delete contaner(s) in another tenant (Admin only).\u0027","commit_id":"e0cd22e9e9ca9d9d9653dbe0cf28705ebcf87767"},{"author":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"change_message_id":"c369e9533e3f78f2d0383dfb2cfc8af728061322","unresolved":false,"context_lines":[{"line_number":176,"context_line":"@utils.arg(\u0027--all-tenants\u0027,"},{"line_number":177,"context_line":"           action\u003d\"store_true\","},{"line_number":178,"context_line":"           default\u003dFalse,"},{"line_number":179,"context_line":"           help\u003d\u0027Delete container(s) in all tenant by name.\u0027)"},{"line_number":180,"context_line":"def do_delete(cs, args):"},{"line_number":181,"context_line":"    \"\"\"Delete specified containers.\"\"\""},{"line_number":182,"context_line":"    delete_containers \u003d []"}],"source_content_type":"text/x-python","patch_set":1,"id":"7ffa3b31_accb76a3","line":179,"range":{"start_line":179,"start_character":50,"end_line":179,"end_character":58},"updated":"2017-04-14 22:51:43.000000000","message":"Why it is by name? How about by uuid?","commit_id":"e0cd22e9e9ca9d9d9653dbe0cf28705ebcf87767"},{"author":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"change_message_id":"c369e9533e3f78f2d0383dfb2cfc8af728061322","unresolved":false,"context_lines":[{"line_number":181,"context_line":"    \"\"\"Delete specified containers.\"\"\""},{"line_number":182,"context_line":"    delete_containers \u003d []"},{"line_number":183,"context_line":"    for container in args.containers:"},{"line_number":184,"context_line":"        if args.all_tenants and not uuidutils.is_uuid_like(container):"},{"line_number":185,"context_line":"            opts \u003d {}"},{"line_number":186,"context_line":"            opts[\u0027all_tenants\u0027] \u003d True"},{"line_number":187,"context_line":"            all_containers \u003d cs.containers.list(**opts)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7ffa3b31_ac401609","line":184,"range":{"start_line":184,"start_character":46,"end_line":184,"end_character":58},"updated":"2017-04-14 22:51:43.000000000","message":"I imagine \u0027all_tenants\u0027 can work with uuid as well? If it can, most of the code in line #184 - #194 is unnecessary.","commit_id":"e0cd22e9e9ca9d9d9653dbe0cf28705ebcf87767"}]}