)]}'
{"cinderclient/tests/unit/v3/test_shell.py":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"62f83f0af566a46f16675213b01cd482fca90fe1","unresolved":true,"context_lines":[{"line_number":1652,"context_line":"        self.assert_called(\u0027POST\u0027, \u0027/backups/1234/restore\u0027,"},{"line_number":1653,"context_line":"                           body\u003dexpected)"},{"line_number":1654,"context_line":""},{"line_number":1655,"context_line":"    def test_restore_with_name_error(self):"},{"line_number":1656,"context_line":"        self.assertRaises(exceptions.CommandError, self.run_command,"},{"line_number":1657,"context_line":"                          \u0027backup-restore 1234 --volume fake_vol --name \u0027"},{"line_number":1658,"context_line":"                          \u0027restore_vol\u0027)"}],"source_content_type":"text/x-python","patch_set":8,"id":"3ca973d6_6525095b","line":1655,"range":{"start_line":1655,"start_character":8,"end_line":1655,"end_character":36},"updated":"2021-02-16 16:47:28.000000000","message":"I\u0027m not sure what we\u0027re actually testing here as this raises because the volume \u0027fake_vol\u0027 doesn\u0027t exist and is not related to the name priority of volume over user provided name.","commit_id":"7e3566ed04c8f664f6e1df0614499989f6b3560a"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"9ba329a41c89ad8a06617cdcc19990390fc75754","unresolved":true,"context_lines":[{"line_number":1652,"context_line":"        self.assert_called(\u0027POST\u0027, \u0027/backups/1234/restore\u0027,"},{"line_number":1653,"context_line":"                           body\u003dexpected)"},{"line_number":1654,"context_line":""},{"line_number":1655,"context_line":"    def test_restore_with_name_error(self):"},{"line_number":1656,"context_line":"        self.assertRaises(exceptions.CommandError, self.run_command,"},{"line_number":1657,"context_line":"                          \u0027backup-restore 1234 --volume fake_vol --name \u0027"},{"line_number":1658,"context_line":"                          \u0027restore_vol\u0027)"}],"source_content_type":"text/x-python","patch_set":8,"id":"f2b2fb62_292b79bd","line":1655,"range":{"start_line":1655,"start_character":8,"end_line":1655,"end_character":36},"in_reply_to":"3ca973d6_6525095b","updated":"2021-02-19 18:10:54.000000000","message":"Yeah, I copied this test from the v2 code and didn\u0027t study it in depth. It really should be named \"test_restore_with_volume_error\" because it\u0027s checking what happens if you specify a \"--volume\" that doesn\u0027t exist. The \"--name\" argument doesn\u0027t play a role in the test, because the CommandError exception is thrown before the name argument becomes relevant.","commit_id":"7e3566ed04c8f664f6e1df0614499989f6b3560a"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"62f83f0af566a46f16675213b01cd482fca90fe1","unresolved":true,"context_lines":[{"line_number":1789,"context_line":"        # of a specific volume type, or in a different AZ (the fake \u00271234\u0027"},{"line_number":1790,"context_line":"        # backup is in az1)."},{"line_number":1791,"context_line":"        if volume_type or availability_zone \u003d\u003d \u0027different-az\u0027:"},{"line_number":1792,"context_line":"            mock_stub_restore.assert_not_called()"},{"line_number":1793,"context_line":"        else:"},{"line_number":1794,"context_line":"            mock_stub_restore.assert_called_once()"},{"line_number":1795,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"e9cfe5c3_8ad314c7","line":1792,"range":{"start_line":1792,"start_character":12,"end_line":1792,"end_character":49},"updated":"2021-02-16 16:47:28.000000000","message":"Along with restore not called, it would be better to assert that create volume is called by mocking _stub_volume (not totally sure), that would strengthen the test for any regression for restoring into volume case","commit_id":"7e3566ed04c8f664f6e1df0614499989f6b3560a"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"9ba329a41c89ad8a06617cdcc19990390fc75754","unresolved":true,"context_lines":[{"line_number":1789,"context_line":"        # of a specific volume type, or in a different AZ (the fake \u00271234\u0027"},{"line_number":1790,"context_line":"        # backup is in az1)."},{"line_number":1791,"context_line":"        if volume_type or availability_zone \u003d\u003d \u0027different-az\u0027:"},{"line_number":1792,"context_line":"            mock_stub_restore.assert_not_called()"},{"line_number":1793,"context_line":"        else:"},{"line_number":1794,"context_line":"            mock_stub_restore.assert_called_once()"},{"line_number":1795,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"9e1216a9_005d9185","line":1792,"range":{"start_line":1792,"start_character":12,"end_line":1792,"end_character":49},"in_reply_to":"e9cfe5c3_8ad314c7","updated":"2021-02-19 18:10:54.000000000","message":"Conceptually, yes, but you can\u0027t just mock \u0027_stub_volume\u0027 because it plays a broader role in the v2/fakes code. I spent some time trying to add a mock wrapper, but gave up after I couldn\u0027t get it to work in a reasonable amount of time. In the end, I gave up trying to make it work, but I agree it would be nice.","commit_id":"7e3566ed04c8f664f6e1df0614499989f6b3560a"}],"cinderclient/v3/shell.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"8337146d27e97ad7b92383ecd460ccb9479e2281","unresolved":true,"context_lines":[{"line_number":251,"context_line":"        if args.name:"},{"line_number":252,"context_line":"            args.name \u003d None"},{"line_number":253,"context_line":"            print(\u0027Mutually exclusive options are specified simultaneously: \u0027"},{"line_number":254,"context_line":"                  \u0027\"--volume and --name\". The --volume option takes priority.\u0027)"},{"line_number":255,"context_line":"    else:"},{"line_number":256,"context_line":"        volume_id \u003d None"},{"line_number":257,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"fcf8054b_3ae6cfa2","line":254,"range":{"start_line":254,"start_character":19,"end_line":254,"end_character":40},"updated":"2020-11-24 00:11:39.000000000","message":"I wonder whether this should be\n\n  \"volume\" and \"name\"\n\n(it would be consistent with lines 239 and 246 and 261-262)","commit_id":"fd885d1291de5c49a52ca640cf8e1a8c88804efc"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"3c673aee75ea5d331506ed916d95792ba4e6c71d","unresolved":false,"context_lines":[{"line_number":251,"context_line":"        if args.name:"},{"line_number":252,"context_line":"            args.name \u003d None"},{"line_number":253,"context_line":"            print(\u0027Mutually exclusive options are specified simultaneously: \u0027"},{"line_number":254,"context_line":"                  \u0027\"--volume and --name\". The --volume option takes priority.\u0027)"},{"line_number":255,"context_line":"    else:"},{"line_number":256,"context_line":"        volume_id \u003d None"},{"line_number":257,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"25e6be47_6a855712","line":254,"range":{"start_line":254,"start_character":19,"end_line":254,"end_character":40},"in_reply_to":"4bf60c3a_52c002b8","updated":"2020-11-24 20:46:36.000000000","message":"Done","commit_id":"fd885d1291de5c49a52ca640cf8e1a8c88804efc"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"560d527447df34617648d46326e7846dd419df02","unresolved":true,"context_lines":[{"line_number":251,"context_line":"        if args.name:"},{"line_number":252,"context_line":"            args.name \u003d None"},{"line_number":253,"context_line":"            print(\u0027Mutually exclusive options are specified simultaneously: \u0027"},{"line_number":254,"context_line":"                  \u0027\"--volume and --name\". The --volume option takes priority.\u0027)"},{"line_number":255,"context_line":"    else:"},{"line_number":256,"context_line":"        volume_id \u003d None"},{"line_number":257,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"4bf60c3a_52c002b8","line":254,"range":{"start_line":254,"start_character":19,"end_line":254,"end_character":40},"in_reply_to":"fcf8054b_3ae6cfa2","updated":"2020-11-24 18:25:15.000000000","message":"I\u0027ll be happy to make that change. My patch originally, er, \"honored\" the equivalent v2 command\u0027s message [1].\n\n[1] https://opendev.org/openstack/python-cinderclient/src/branch/master/cinderclient/v2/shell.py#L1315","commit_id":"fd885d1291de5c49a52ca640cf8e1a8c88804efc"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"8337146d27e97ad7b92383ecd460ccb9479e2281","unresolved":true,"context_lines":[{"line_number":257,"context_line":""},{"line_number":258,"context_line":"    volume_type \u003d getattr(args, \u0027volume_type\u0027, None)"},{"line_number":259,"context_line":"    az \u003d getattr(args, \u0027availability_zone\u0027, None)"},{"line_number":260,"context_line":"    if (volume_type or az) and not args.name:"},{"line_number":261,"context_line":"        msg \u003d (\u0027The \"volume-type\" and \"availability-zone\" options are valid \u0027"},{"line_number":262,"context_line":"               \u0027only when used with the \"name\" option.\u0027)"},{"line_number":263,"context_line":"        raise exceptions.ClientException(code\u003d1, message\u003dmsg)"}],"source_content_type":"text/x-python","patch_set":2,"id":"33eb784b_23c3bb91","line":260,"range":{"start_line":260,"start_character":31,"end_line":260,"end_character":44},"updated":"2020-11-24 00:11:39.000000000","message":"I wonder whether instead we should be checking for not args.volume here.  The behavior of the backup-restore API (at least according to the api-ref) is that \"If no existing volume UUID is provided, the backup will be restored to a new volume matching the size and name of the originally backed up volume\".  So I think we should allow name to be unspecified and change the error to \"VT and AZ are valid only when not using the --volume option\".  What do you think?","commit_id":"fd885d1291de5c49a52ca640cf8e1a8c88804efc"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"560d527447df34617648d46326e7846dd419df02","unresolved":true,"context_lines":[{"line_number":257,"context_line":""},{"line_number":258,"context_line":"    volume_type \u003d getattr(args, \u0027volume_type\u0027, None)"},{"line_number":259,"context_line":"    az \u003d getattr(args, \u0027availability_zone\u0027, None)"},{"line_number":260,"context_line":"    if (volume_type or az) and not args.name:"},{"line_number":261,"context_line":"        msg \u003d (\u0027The \"volume-type\" and \"availability-zone\" options are valid \u0027"},{"line_number":262,"context_line":"               \u0027only when used with the \"name\" option.\u0027)"},{"line_number":263,"context_line":"        raise exceptions.ClientException(code\u003d1, message\u003dmsg)"}],"source_content_type":"text/x-python","patch_set":2,"id":"c9d19d3f_5e337664","line":260,"range":{"start_line":260,"start_character":31,"end_line":260,"end_character":44},"in_reply_to":"33eb784b_23c3bb91","updated":"2020-11-24 18:25:15.000000000","message":"Ooh, good catch. I neglected the fact that the command doesn\u0027t require you specify a name, in which case it restores to a new volume named restore_backup_\u003cbackup.volume_id\u003e","commit_id":"fd885d1291de5c49a52ca640cf8e1a8c88804efc"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"3c673aee75ea5d331506ed916d95792ba4e6c71d","unresolved":false,"context_lines":[{"line_number":257,"context_line":""},{"line_number":258,"context_line":"    volume_type \u003d getattr(args, \u0027volume_type\u0027, None)"},{"line_number":259,"context_line":"    az \u003d getattr(args, \u0027availability_zone\u0027, None)"},{"line_number":260,"context_line":"    if (volume_type or az) and not args.name:"},{"line_number":261,"context_line":"        msg \u003d (\u0027The \"volume-type\" and \"availability-zone\" options are valid \u0027"},{"line_number":262,"context_line":"               \u0027only when used with the \"name\" option.\u0027)"},{"line_number":263,"context_line":"        raise exceptions.ClientException(code\u003d1, message\u003dmsg)"}],"source_content_type":"text/x-python","patch_set":2,"id":"0380504f_263fdf8f","line":260,"range":{"start_line":260,"start_character":31,"end_line":260,"end_character":44},"in_reply_to":"c9d19d3f_5e337664","updated":"2020-11-24 20:46:36.000000000","message":"Done","commit_id":"fd885d1291de5c49a52ca640cf8e1a8c88804efc"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"2336f7d3731978fea0aed61a1717c04f735c90e2","unresolved":true,"context_lines":[{"line_number":228,"context_line":"@utils.arg(\u0027--name\u0027, metavar\u003d\u0027\u003cname\u003e\u0027,"},{"line_number":229,"context_line":"           default\u003dNone,"},{"line_number":230,"context_line":"           help\u003d\u0027Use the name for new volume creation to restore. \u0027"},{"line_number":231,"context_line":"           \u0027This is mutually exclusive with --volume  and --volume \u0027"},{"line_number":232,"context_line":"           \u0027takes priority. \u0027"},{"line_number":233,"context_line":"           \u0027Default\u003dNone.\u0027)"},{"line_number":234,"context_line":"@utils.arg(\u0027--volume-type\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"2e0880c5_17be76cd","line":231,"range":{"start_line":231,"start_character":53,"end_line":231,"end_character":54},"updated":"2020-12-07 17:43:36.000000000","message":"extra space","commit_id":"0bed65d908f89a8339aa5acd802f52cb6039e01e"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"7959380e888d88eccc1a5121ca4e668158a1ed2b","unresolved":false,"context_lines":[{"line_number":228,"context_line":"@utils.arg(\u0027--name\u0027, metavar\u003d\u0027\u003cname\u003e\u0027,"},{"line_number":229,"context_line":"           default\u003dNone,"},{"line_number":230,"context_line":"           help\u003d\u0027Use the name for new volume creation to restore. \u0027"},{"line_number":231,"context_line":"           \u0027This is mutually exclusive with --volume  and --volume \u0027"},{"line_number":232,"context_line":"           \u0027takes priority. \u0027"},{"line_number":233,"context_line":"           \u0027Default\u003dNone.\u0027)"},{"line_number":234,"context_line":"@utils.arg(\u0027--volume-type\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"becc9550_c78c1a77","line":231,"range":{"start_line":231,"start_character":53,"end_line":231,"end_character":54},"in_reply_to":"2e0880c5_17be76cd","updated":"2020-12-07 17:59:48.000000000","message":"Dangit! Fixed.","commit_id":"0bed65d908f89a8339aa5acd802f52cb6039e01e"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"1378c6141416b892032860c8916f2c32ed4fc042","unresolved":true,"context_lines":[{"line_number":269,"context_line":"        # Implement restoring a backup to a newly created volume of a"},{"line_number":270,"context_line":"        # specific volume type or in a different AZ by using the"},{"line_number":271,"context_line":"        # volume-create API."},{"line_number":272,"context_line":"        volume_name \u003d args.name or \u0027restore_backup_%s\u0027 % backup.volume_id"},{"line_number":273,"context_line":"        volume \u003d cs.volumes.create(size\u003dbackup.size,"},{"line_number":274,"context_line":"                                   name\u003dvolume_name,"},{"line_number":275,"context_line":"                                   volume_type\u003dvolume_type,"}],"source_content_type":"text/x-python","patch_set":3,"id":"74269ea4_04cb2bba","line":272,"updated":"2020-12-07 17:25:21.000000000","message":"Right, you need to set this because you are using the volume-create API, not the backup-restore API.  Just so no one else has to look this up, the backup-restore API [0] does (eventually) use the naming scheme that Alan is using here.\n\n[0] https://opendev.org/openstack/cinder/src/commit/95d00bd7a6658e2cd3066e5ae77d3a9a63bc0c8f/cinder/backup/api.py#L354","commit_id":"0bed65d908f89a8339aa5acd802f52cb6039e01e"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"aa2faba546f0c1a7252b224ce1719156602d002c","unresolved":true,"context_lines":[{"line_number":269,"context_line":"        # Implement restoring a backup to a newly created volume of a"},{"line_number":270,"context_line":"        # specific volume type or in a different AZ by using the"},{"line_number":271,"context_line":"        # volume-create API."},{"line_number":272,"context_line":"        volume_name \u003d args.name or \u0027restore_backup_%s\u0027 % backup.volume_id"},{"line_number":273,"context_line":"        volume \u003d cs.volumes.create(size\u003dbackup.size,"},{"line_number":274,"context_line":"                                   name\u003dvolume_name,"},{"line_number":275,"context_line":"                                   volume_type\u003dvolume_type,"}],"source_content_type":"text/x-python","patch_set":4,"id":"747ccb26_e18bf14d","line":272,"range":{"start_line":272,"start_character":57,"end_line":272,"end_character":73},"updated":"2021-01-05 23:11:12.000000000","message":"I just looked at this again.  I think the backup-restore API uses the id from the backup (not the id of the volume it\u0027s a backup of) here, so maybe this should be backup.id?  Definitely double-check me on this, though.  Here\u0027s the cinder code to look at:\nhttps://opendev.org/openstack/cinder/src/commit/95d00bd7a6658e2cd3066e5ae77d3a9a63bc0c8f/cinder/backup/api.py#L354","commit_id":"51e589cdb01caf885bddf802ce738f6e9afc794f"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"623f300fb65d2a4145dd9d68b6e2574900cff55b","unresolved":false,"context_lines":[{"line_number":269,"context_line":"        # Implement restoring a backup to a newly created volume of a"},{"line_number":270,"context_line":"        # specific volume type or in a different AZ by using the"},{"line_number":271,"context_line":"        # volume-create API."},{"line_number":272,"context_line":"        volume_name \u003d args.name or \u0027restore_backup_%s\u0027 % backup.volume_id"},{"line_number":273,"context_line":"        volume \u003d cs.volumes.create(size\u003dbackup.size,"},{"line_number":274,"context_line":"                                   name\u003dvolume_name,"},{"line_number":275,"context_line":"                                   volume_type\u003dvolume_type,"}],"source_content_type":"text/x-python","patch_set":4,"id":"bb018b5b_706ac280","line":272,"range":{"start_line":272,"start_character":57,"end_line":272,"end_character":73},"in_reply_to":"747ccb26_e18bf14d","updated":"2021-01-07 00:00:31.000000000","message":"You\u0027re right! *facepalm* I think my brain simply assumed the volume_id was of interest, and I neglected to carefully read the code. I\u0027ll post an update.","commit_id":"51e589cdb01caf885bddf802ce738f6e9afc794f"}],"releasenotes/notes/enhance-backup-restore-shell-command-0cf55df6ca4b4c55.yaml":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"8337146d27e97ad7b92383ecd460ccb9479e2281","unresolved":true,"context_lines":[{"line_number":4,"context_line":"    Enhance the \u0027backup-restore\u0027 shell command to support restoring to a new"},{"line_number":5,"context_line":"    volume created with a specific volume type and/or in a different AZ. New"},{"line_number":6,"context_line":"    \u0027--volume-type\u0027 and \u0027--availability-zone\u0027 arguments are compatible with"},{"line_number":7,"context_line":"    cinder API microversion v3.47 onward."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7f0a03d2_e8ffbd7c","line":7,"updated":"2020-11-24 00:11:39.000000000","message":"Please use two backticks (``) instead of a single quote (\u0027).  It will render the content in monospace, which is very important here, because in the \"regular\" non-fixed-width font, the \u0027--\u0027 in the command line options is usually rendered as a single \"em\" dash.","commit_id":"fd885d1291de5c49a52ca640cf8e1a8c88804efc"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"560d527447df34617648d46326e7846dd419df02","unresolved":true,"context_lines":[{"line_number":4,"context_line":"    Enhance the \u0027backup-restore\u0027 shell command to support restoring to a new"},{"line_number":5,"context_line":"    volume created with a specific volume type and/or in a different AZ. New"},{"line_number":6,"context_line":"    \u0027--volume-type\u0027 and \u0027--availability-zone\u0027 arguments are compatible with"},{"line_number":7,"context_line":"    cinder API microversion v3.47 onward."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"fc931a9f_9c56839f","line":7,"in_reply_to":"7f0a03d2_e8ffbd7c","updated":"2020-11-24 18:25:15.000000000","message":"Ack, should I also use double backticks around \u0027backup-restore\u0027 in L4?","commit_id":"fd885d1291de5c49a52ca640cf8e1a8c88804efc"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"3c673aee75ea5d331506ed916d95792ba4e6c71d","unresolved":false,"context_lines":[{"line_number":4,"context_line":"    Enhance the \u0027backup-restore\u0027 shell command to support restoring to a new"},{"line_number":5,"context_line":"    volume created with a specific volume type and/or in a different AZ. New"},{"line_number":6,"context_line":"    \u0027--volume-type\u0027 and \u0027--availability-zone\u0027 arguments are compatible with"},{"line_number":7,"context_line":"    cinder API microversion v3.47 onward."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"91e86f5e_266d73df","line":7,"in_reply_to":"cd3dcae7_602670e5","updated":"2020-11-24 20:46:36.000000000","message":"Done","commit_id":"fd885d1291de5c49a52ca640cf8e1a8c88804efc"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"319cc23c0327240e335f5d644d72eb78fb019a84","unresolved":true,"context_lines":[{"line_number":4,"context_line":"    Enhance the \u0027backup-restore\u0027 shell command to support restoring to a new"},{"line_number":5,"context_line":"    volume created with a specific volume type and/or in a different AZ. New"},{"line_number":6,"context_line":"    \u0027--volume-type\u0027 and \u0027--availability-zone\u0027 arguments are compatible with"},{"line_number":7,"context_line":"    cinder API microversion v3.47 onward."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"cd3dcae7_602670e5","line":7,"in_reply_to":"fc931a9f_9c56839f","updated":"2020-11-24 20:09:33.000000000","message":"Yes, I think so, because that\u0027s literally the command.","commit_id":"fd885d1291de5c49a52ca640cf8e1a8c88804efc"}]}