)]}'
{"novaclient/v1_1/shell.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"32bf74745599224d8628fba956a06e80a1aca634","unresolved":false,"context_lines":[{"line_number":49,"context_line":"        raise exceptions.CommandError(\"you need to specify a Flavor ID \")"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"    if args.num_instances:"},{"line_number":52,"context_line":"        if args.num_instances \u003c\u003d 1:"},{"line_number":53,"context_line":"            raise exceptions.CommandError(\"num_instances should be \u003e 1\")"},{"line_number":54,"context_line":"        max_count \u003d int(args.num_instances)"},{"line_number":55,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2F7LU%3D","line":52,"updated":"2012-12-20 16:05:30.000000000","message":"Hrm.  Since you haven\u0027t specified the type of the num_instances option, it will be a string here; a test shows that Python considers the string \"0\" to be greater than the integer 1, which means this raise can never be triggered.","commit_id":"b2250b595fbc7abcc1ac7898796289679d58a709"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"32bf74745599224d8628fba956a06e80a1aca634","unresolved":false,"context_lines":[{"line_number":154,"context_line":"     default\u003dNone,"},{"line_number":155,"context_line":"     metavar\u003d\u0027\u003cimage\u003e\u0027,"},{"line_number":156,"context_line":"     help\u003d\"Image ID (see \u0027nova image-list\u0027). \")"},{"line_number":157,"context_line":"@utils.arg(\u0027--num_instances\u0027,"},{"line_number":158,"context_line":"     default\u003dNone,"},{"line_number":159,"context_line":"     metavar\u003d\u0027\u003cnumber\u003e\u0027,"},{"line_number":160,"context_line":"     help\u003d\"boot multi instances at a time\")"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2F7LM%3D","line":157,"updated":"2012-12-20 16:05:30.000000000","message":"We are trying to avoid underscores in option names; the option should be named \"--num-instances\".  Also, as noted above, you need to specify a type for this option.","commit_id":"b2250b595fbc7abcc1ac7898796289679d58a709"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"19e9943d54db43cacf475c2e7f92d85d3f4aa867","unresolved":false,"context_lines":[{"line_number":160,"context_line":"     metavar\u003d\u0027\u003cnumber\u003e\u0027,"},{"line_number":161,"context_line":"     help\u003d\"boot multi instances at a time\")"},{"line_number":162,"context_line":"@utils.arg(\u0027--num_instances\u0027,"},{"line_number":163,"context_line":"    help\u003dargparse.SUPPRESS)"},{"line_number":164,"context_line":"@utils.arg(\u0027--meta\u0027,"},{"line_number":165,"context_line":"     metavar\u003d\"\u003ckey\u003dvalue\u003e\","},{"line_number":166,"context_line":"     action\u003d\u0027append\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2F6KY%3D","line":163,"updated":"2012-12-21 16:02:20.000000000","message":"The \u0027_\u0027 options that are left in here are for backwards compatibility.  Since you\u0027re just adding a new option, I\u0027d recommend dropping the --num_instances entirely.","commit_id":"e2dd2de8033c8b6cb075ede2cfd2811efd2a3e46"}]}