)]}'
{"magnumclient/v1/bays_shell.py":[{"author":{"_account_id":7230,"name":"Vijendar Komalla","email":"vijendar.komalla@rackspace.com","username":"vijendar"},"change_message_id":"80134b78fca3882189b468ad2df4a3f8c1631a49","unresolved":false,"context_lines":[{"line_number":107,"context_line":"    opts[\u0027discovery_url\u0027] \u003d args.discovery_url"},{"line_number":108,"context_line":"    opts[\u0027bay_create_timeout\u0027] \u003d args.timeout"},{"line_number":109,"context_line":"    try:"},{"line_number":110,"context_line":"        if args.magnum_api_version \u003c 1.2:"},{"line_number":111,"context_line":"            bay \u003d cs.bays.create(**opts)"},{"line_number":112,"context_line":"            _show_bay(bay)"},{"line_number":113,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":1,"id":"3ac371cc_b3e37bd8","line":110,"updated":"2016-08-17 15:43:16.000000000","message":"Here magnum_api_version seems to be a string and you are trying to compare it with a float. May be you need to do something like below:\nif float(args.magnum_api_version) \u003c 1.2:\nIf magnum_api_version could be empty/none, then do following\nif args.magnum_api_version and float(args.magnum_api_version \u003c 1.2):","commit_id":"9a573d368714d18145773a68a720d76c0a02a71d"},{"author":{"_account_id":21469,"name":"Jaycen Grant","email":"jaycen.v.grant@intel.com","username":"jvgrant"},"change_message_id":"e5142ff9a467ffdda05d01460a6a7d38cfe889f7","unresolved":false,"context_lines":[{"line_number":112,"context_line":"            _show_bay(bay)"},{"line_number":113,"context_line":"        else:"},{"line_number":114,"context_line":"            cs.bays.create(**opts)"},{"line_number":115,"context_line":"            print(\"Request to create bay %s has been accepted.\" % args.name)"},{"line_number":116,"context_line":"    except Exception as e:"},{"line_number":117,"context_line":"        print(\"Create for bay %s failed: %s\" %"},{"line_number":118,"context_line":"              (opts[\u0027name\u0027], e))"}],"source_content_type":"text/x-python","patch_set":1,"id":"3ac371cc_d65f3e3f","line":115,"updated":"2016-08-16 18:27:46.000000000","message":"should we include the UUID here?","commit_id":"9a573d368714d18145773a68a720d76c0a02a71d"},{"author":{"_account_id":7230,"name":"Vijendar Komalla","email":"vijendar.komalla@rackspace.com","username":"vijendar"},"change_message_id":"80134b78fca3882189b468ad2df4a3f8c1631a49","unresolved":false,"context_lines":[{"line_number":170,"context_line":"def do_bay_update(cs, args):"},{"line_number":171,"context_line":"    \"\"\"Update information about the given bay.\"\"\""},{"line_number":172,"context_line":"    patch \u003d magnum_utils.args_array_to_patch(args.op, args.attributes[0])"},{"line_number":173,"context_line":"    if args.magnum_api_version \u003c 1.2:"},{"line_number":174,"context_line":"        bay \u003d cs.bays.update(args.bay, patch)"},{"line_number":175,"context_line":"        _show_bay(bay)"},{"line_number":176,"context_line":"    else:"}],"source_content_type":"text/x-python","patch_set":1,"id":"3ac371cc_f3005368","line":173,"updated":"2016-08-17 15:43:16.000000000","message":"same here.","commit_id":"9a573d368714d18145773a68a720d76c0a02a71d"}]}