)]}'
{"openstackclient/compute/v2/flavor.py":[{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"056fa67e749470b6cb40e0bb27cdd269800cd16e","unresolved":false,"context_lines":[{"line_number":42,"context_line":"            metavar\u003d\"\u003cid\u003e\","},{"line_number":43,"context_line":"            help\u003d\"Unique ID (integer or UUID) for the new flavor.\""},{"line_number":44,"context_line":"                 \" If specifying \u0027auto\u0027, a UUID will be generated as id\")"},{"line_number":45,"context_line":"        parser.add_argument("},{"line_number":46,"context_line":"            \"ram\","},{"line_number":47,"context_line":"            metavar\u003d\"\u003cram\u003e\","},{"line_number":48,"context_line":"            help\u003d\"Memory size in MB\")"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2Bx78%3D","line":45,"updated":"2013-02-19 13:23:40.000000000","message":"There are several numerical options here. If you set the type to int, argparse will do the type conversion for you, including reporting errors. http://docs.python.org/2/library/argparse.html#the-add-argument-method","commit_id":"4209e5fef442237502d96078af287721ac5ca711"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"056fa67e749470b6cb40e0bb27cdd269800cd16e","unresolved":false,"context_lines":[{"line_number":69,"context_line":"            metavar\u003d\"\u003cfactor\u003e\","},{"line_number":70,"context_line":"            help\u003d\"RX/TX factor (default 1)\","},{"line_number":71,"context_line":"            default\u003d1)"},{"line_number":72,"context_line":"        parser.add_argument("},{"line_number":73,"context_line":"            \"--is-public\","},{"line_number":74,"context_line":"            metavar\u003d\"\u003cis-public\u003e\","},{"line_number":75,"context_line":"            help\u003d\"Make flavor accessible to the public (default true)\","}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2Bx8I%3D","line":72,"updated":"2013-02-19 13:23:40.000000000","message":"This should be set up as a boolean option, with separate switches using store_true and store_false actions.","commit_id":"4209e5fef442237502d96078af287721ac5ca711"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"bd5bf402ccc5cd20bde97214605f894178f1ef14","unresolved":false,"context_lines":[{"line_number":76,"context_line":"            help\u003d\"RX/TX factor (default 1)\","},{"line_number":77,"context_line":"            default\u003d1)"},{"line_number":78,"context_line":"        parser.add_argument("},{"line_number":79,"context_line":"            \"--is-public\","},{"line_number":80,"context_line":"            metavar\u003d\"\u003cis-public\u003e\","},{"line_number":81,"context_line":"            help\u003d\"Make flavor accessible to the public (default false)\","},{"line_number":82,"context_line":"            action\u003d\"store_true\")"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2BqA8%3D","line":79,"updated":"2013-02-22 16:59:54.000000000","message":"I want to be consistent with using two complementary flags here, like --public / --private similar to the --enable / --disable used elsewhere","commit_id":"6b927b3579648f07840612b88da7eaf9c6999554"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"bd5bf402ccc5cd20bde97214605f894178f1ef14","unresolved":false,"context_lines":[{"line_number":77,"context_line":"            default\u003d1)"},{"line_number":78,"context_line":"        parser.add_argument("},{"line_number":79,"context_line":"            \"--is-public\","},{"line_number":80,"context_line":"            metavar\u003d\"\u003cis-public\u003e\","},{"line_number":81,"context_line":"            help\u003d\"Make flavor accessible to the public (default false)\","},{"line_number":82,"context_line":"            action\u003d\"store_true\")"},{"line_number":83,"context_line":"        return parser"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2BqAU%3D","line":80,"updated":"2013-02-22 16:59:54.000000000","message":"This causes \u0027help create flavor\u0027 to fail","commit_id":"6b927b3579648f07840612b88da7eaf9c6999554"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"1a70c42572a911972070f82e4d3eb18c6e29188a","unresolved":false,"context_lines":[{"line_number":76,"context_line":"            help\u003d\"RX/TX factor (default 1)\","},{"line_number":77,"context_line":"            default\u003d1)"},{"line_number":78,"context_line":"        parser.add_argument("},{"line_number":79,"context_line":"            \"--private\","},{"line_number":80,"context_line":"            metavar\u003d\"\u003cprivate\u003e\","},{"line_number":81,"context_line":"            help\u003d\"Make flavor inaccessible to the public (default false)\","},{"line_number":82,"context_line":"            action\u003d\"store_true\")"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2BoZA%3D","line":79,"updated":"2013-02-25 05:22:54.000000000","message":"I would still prefer to use a --public/--private pair of options.  But the variable name here is \u0027public\u0027 so you\u0027ll need a dest\u003d argument.","commit_id":"577fc49d596eab3a0d20755d0de3d1c7d9866113"},{"author":{"_account_id":209,"name":"Josh Kearney","email":"josh@jk0.org","username":"jk0"},"change_message_id":"97120694efae8b40bb517807fee7af4689c3e165","unresolved":false,"context_lines":[{"line_number":76,"context_line":"            help\u003d\"RX/TX factor (default 1)\","},{"line_number":77,"context_line":"            default\u003d1)"},{"line_number":78,"context_line":"        parser.add_argument("},{"line_number":79,"context_line":"            \"--private\","},{"line_number":80,"context_line":"            metavar\u003d\"\u003cprivate\u003e\","},{"line_number":81,"context_line":"            help\u003d\"Make flavor inaccessible to the public (default false)\","},{"line_number":82,"context_line":"            action\u003d\"store_true\")"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2BnuA%3D","line":79,"in_reply_to":"AAAAM3%2F%2BoZA%3D","updated":"2013-02-25 16:36:33.000000000","message":"Oops, it should be passing the \u0027public\u0027 variable down in take_action().","commit_id":"577fc49d596eab3a0d20755d0de3d1c7d9866113"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"1a70c42572a911972070f82e4d3eb18c6e29188a","unresolved":false,"context_lines":[{"line_number":77,"context_line":"            default\u003d1)"},{"line_number":78,"context_line":"        parser.add_argument("},{"line_number":79,"context_line":"            \"--private\","},{"line_number":80,"context_line":"            metavar\u003d\"\u003cprivate\u003e\","},{"line_number":81,"context_line":"            help\u003d\"Make flavor inaccessible to the public (default false)\","},{"line_number":82,"context_line":"            action\u003d\"store_true\")"},{"line_number":83,"context_line":"        return parser"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2BoZE%3D","line":80,"updated":"2013-02-25 05:22:54.000000000","message":"This metavar is not allowed for an option without an argument.","commit_id":"577fc49d596eab3a0d20755d0de3d1c7d9866113"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"1a70c42572a911972070f82e4d3eb18c6e29188a","unresolved":false,"context_lines":[{"line_number":141,"context_line":"        columns \u003d ("},{"line_number":142,"context_line":"            \"ID\","},{"line_number":143,"context_line":"            \"Name\","},{"line_number":144,"context_line":"            \"Memory_MB\","},{"line_number":145,"context_line":"            \"Disk\","},{"line_number":146,"context_line":"            \"Ephemeral\","},{"line_number":147,"context_line":"            \"Swap\","}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2BoY8%3D","line":144,"updated":"2013-02-25 05:22:54.000000000","message":"The column name her is \u0027Ram\u0027","commit_id":"577fc49d596eab3a0d20755d0de3d1c7d9866113"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"84baa9d65f156b7ffc55cbd244c38df81505f8f7","unresolved":false,"context_lines":[{"line_number":141,"context_line":"        columns \u003d ("},{"line_number":142,"context_line":"            \"ID\","},{"line_number":143,"context_line":"            \"Name\","},{"line_number":144,"context_line":"            \"Memory_MB\","},{"line_number":145,"context_line":"            \"Disk\","},{"line_number":146,"context_line":"            \"Ephemeral\","},{"line_number":147,"context_line":"            \"Swap\","}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2Bnq4%3D","line":144,"in_reply_to":"AAAAM3%2F%2Bnt0%3D","updated":"2013-02-25 17:04:12.000000000","message":"Yes, but in this case I changed \u0027Memory_MB\u0027 (blank column) to \u0027Ram\u0027 and got data.  Look at the show flavor output, it\u0027s \u0027ram\u0027 there too.","commit_id":"577fc49d596eab3a0d20755d0de3d1c7d9866113"},{"author":{"_account_id":209,"name":"Josh Kearney","email":"josh@jk0.org","username":"jk0"},"change_message_id":"97120694efae8b40bb517807fee7af4689c3e165","unresolved":false,"context_lines":[{"line_number":141,"context_line":"        columns \u003d ("},{"line_number":142,"context_line":"            \"ID\","},{"line_number":143,"context_line":"            \"Name\","},{"line_number":144,"context_line":"            \"Memory_MB\","},{"line_number":145,"context_line":"            \"Disk\","},{"line_number":146,"context_line":"            \"Ephemeral\","},{"line_number":147,"context_line":"            \"Swap\","}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2Bnt0%3D","line":144,"in_reply_to":"AAAAM3%2F%2BoY8%3D","updated":"2013-02-25 16:36:33.000000000","message":"Don\u0027t the columns have to be matched with the response, otherwise it will result in a blank column? That\u0027s been my experience.","commit_id":"577fc49d596eab3a0d20755d0de3d1c7d9866113"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"1a70c42572a911972070f82e4d3eb18c6e29188a","unresolved":false,"context_lines":[{"line_number":146,"context_line":"            \"Ephemeral\","},{"line_number":147,"context_line":"            \"Swap\","},{"line_number":148,"context_line":"            \"VCPUs\","},{"line_number":149,"context_line":"            \"RXTX_Factor\","},{"line_number":150,"context_line":"            \"Is_Public\","},{"line_number":151,"context_line":"            \"Extra_Specs\""},{"line_number":152,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2BoY4%3D","line":149,"updated":"2013-02-25 05:22:54.000000000","message":"You can use spaces instead of underscores here so the column headings look better.","commit_id":"577fc49d596eab3a0d20755d0de3d1c7d9866113"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"84baa9d65f156b7ffc55cbd244c38df81505f8f7","unresolved":false,"context_lines":[{"line_number":146,"context_line":"            \"Ephemeral\","},{"line_number":147,"context_line":"            \"Swap\","},{"line_number":148,"context_line":"            \"VCPUs\","},{"line_number":149,"context_line":"            \"RXTX_Factor\","},{"line_number":150,"context_line":"            \"Is_Public\","},{"line_number":151,"context_line":"            \"Extra_Specs\""},{"line_number":152,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2BnrU%3D","line":149,"in_reply_to":"AAAAM3%2F%2Bntw%3D","updated":"2013-02-25 17:04:12.000000000","message":"utils.get_item_properties() handles the mapping","commit_id":"577fc49d596eab3a0d20755d0de3d1c7d9866113"},{"author":{"_account_id":209,"name":"Josh Kearney","email":"josh@jk0.org","username":"jk0"},"change_message_id":"97120694efae8b40bb517807fee7af4689c3e165","unresolved":false,"context_lines":[{"line_number":146,"context_line":"            \"Ephemeral\","},{"line_number":147,"context_line":"            \"Swap\","},{"line_number":148,"context_line":"            \"VCPUs\","},{"line_number":149,"context_line":"            \"RXTX_Factor\","},{"line_number":150,"context_line":"            \"Is_Public\","},{"line_number":151,"context_line":"            \"Extra_Specs\""},{"line_number":152,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2Bntw%3D","line":149,"in_reply_to":"AAAAM3%2F%2BoY4%3D","updated":"2013-02-25 16:36:33.000000000","message":"Same as above. ^","commit_id":"577fc49d596eab3a0d20755d0de3d1c7d9866113"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"d504f6965a62e95aa3415eae7dff41ba41fc8d2b","unresolved":false,"context_lines":[{"line_number":75,"context_line":"            metavar\u003d\"\u003cfactor\u003e\","},{"line_number":76,"context_line":"            help\u003d\"RX/TX factor (default 1)\","},{"line_number":77,"context_line":"            default\u003d1)"},{"line_number":78,"context_line":"        parser.add_argument("},{"line_number":79,"context_line":"            \"--public\","},{"line_number":80,"context_line":"            help\u003d\"Make flavor inaccessible to the public\","},{"line_number":81,"context_line":"            action\u003d\"store_true\")"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAANH%2F%2F%2B4Q%3D","line":78,"updated":"2013-02-28 17:36:02.000000000","message":"These should be an exclusive pair, example in identity/v2_0/tenant.py:SetTenant()","commit_id":"a5cfde72b7713cd480620b1be7458dfed87da7e8"}]}