)]}'
{"novaclient/exceptions.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"86ea34aa268d124e7962ae62cdd070433bf32c2c","unresolved":false,"context_lines":[{"line_number":46,"context_line":"    pass"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"class BadCommand(Exception):"},{"line_number":50,"context_line":"    pass"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"7af24918_08ebc8b1","line":49,"range":{"start_line":49,"start_character":6,"end_line":49,"end_character":16},"updated":"2016-03-04 20:08:18.000000000","message":"Why is this added? You\u0027re still going to get the ERROR: prefix on the failure:\n\nhttps://github.com/openstack/python-novaclient/blob/master/novaclient/shell.py#L1083","commit_id":"b2081c87145a69b9b13df63044efbf2a001d6495"}],"novaclient/v2/shell.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"426a2a94179383f11f83c6e0c09bdb686e9b4bee","unresolved":false,"context_lines":[{"line_number":882,"context_line":"    elif args.flavor:"},{"line_number":883,"context_line":"        flavor \u003d _find_flavor(cs, args.flavor)"},{"line_number":884,"context_line":"        if flavor.is_public:"},{"line_number":885,"context_line":"            print(_(\"Flavor is public.\"))"},{"line_number":886,"context_line":"            return"},{"line_number":887,"context_line":"        kwargs \u003d {\u0027flavor\u0027: flavor}"},{"line_number":888,"context_line":"    elif args.tenant:"}],"source_content_type":"text/x-python","patch_set":1,"id":"7af24918_cd29f5f4","line":885,"range":{"start_line":885,"start_character":21,"end_line":885,"end_character":37},"updated":"2016-03-04 18:59:03.000000000","message":"Use what the nova REST API returns:\n\nhttps://github.com/openstack/nova/blob/master/nova/api/openstack/compute/flavor_access.py#L59","commit_id":"c4e2ae16ad7da36814d0b769bed1cdc7672bb7e4"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"504683e92c99bae7c568df44efe42f924f2d464d","unresolved":false,"context_lines":[{"line_number":883,"context_line":"        flavor \u003d _find_flavor(cs, args.flavor)"},{"line_number":884,"context_line":"        if flavor.is_public:"},{"line_number":885,"context_line":"            print(_(\"Flavor is public.\"))"},{"line_number":886,"context_line":"            return"},{"line_number":887,"context_line":"        kwargs \u003d {\u0027flavor\u0027: flavor}"},{"line_number":888,"context_line":"    elif args.tenant:"},{"line_number":889,"context_line":"        kwargs \u003d {\u0027tenant\u0027: args.tenant}"}],"source_content_type":"text/x-python","patch_set":1,"id":"7af24918_adf1096d","line":886,"range":{"start_line":886,"start_character":12,"end_line":886,"end_character":18},"updated":"2016-03-04 19:02:34.000000000","message":"Also, the REST API returns a 404 in this case, so by you changing this from raising a CommandError, you\u0027re changing the API so it doesn\u0027t return a non-0 error code. We probably shouldn\u0027t do that.","commit_id":"c4e2ae16ad7da36814d0b769bed1cdc7672bb7e4"}]}
