)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"80a0fc08d3b242cce867839fa6c307dc94c0c67e","unresolved":false,"context_lines":[{"line_number":10,"context_line":"when booting."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"APIImpact"},{"line_number":13,"context_line":"DocImpact"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"bp: boot-instance-specific-storage-backend"},{"line_number":16,"context_line":"Change-Id: Id1724e69c26478f62673caac3595ff9f5e5be173"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"3f79a3b5_84815950","line":13,"range":{"start_line":13,"start_character":0,"end_line":13,"end_character":9},"updated":"2018-09-13 21:27:00.000000000","message":"You can remove this, it generates a bug which we don\u0027t need.","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"273a7a769c32151f237a4ebc40b3420c41d95384","unresolved":false,"context_lines":[{"line_number":10,"context_line":"when booting."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"APIImpact"},{"line_number":13,"context_line":"DocImpact"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"bp: boot-instance-specific-storage-backend"},{"line_number":16,"context_line":"Change-Id: Id1724e69c26478f62673caac3595ff9f5e5be173"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"3f79a3b5_499fd564","line":13,"range":{"start_line":13,"start_character":0,"end_line":13,"end_character":9},"in_reply_to":"3f79a3b5_84815950","updated":"2018-09-14 06:26:40.000000000","message":"Remove it.","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"}],"specs/stein/approved/boot-instance-specific-storage-backend.rst":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"f52eff3a0732893669daf47ef9f7559460c6a4f8","unresolved":false,"context_lines":[{"line_number":32,"context_line":"they only need to specify a different backend to send the create request"},{"line_number":33,"context_line":"again."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"Proposed change"},{"line_number":36,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":37,"context_line":"Add a new microversion to Servers create API to support specified volume type"},{"line_number":38,"context_line":"when booting instances."}],"source_content_type":"text/x-rst","patch_set":9,"id":"3f79a3b5_502d098d","line":35,"updated":"2018-09-17 14:09:11.000000000","message":"Should probably have some more detail here. This would only apply to BDMs with source_type of blank, image and snapshot and the volume_type will be passed from nova-api through to nova-compute where the volume gets created and then attached to the new server.\n\nAlso, will nova-api validate that the requested volume_type actually exists in cinder so we can fail fast if it doesn\u0027t? Otherwise the user could be waiting a long time just to see the server build fail when nova-compute tries to create the volume and the type doesn\u0027t exist (or the user doesn\u0027t have access to it).","commit_id":"08d11f6193161b20361b8b53f8bdcd22f0ebbb5a"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"089f45d7b539d3c10edcf46daa327229ae93883a","unresolved":false,"context_lines":[{"line_number":32,"context_line":"they only need to specify a different backend to send the create request"},{"line_number":33,"context_line":"again."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"Proposed change"},{"line_number":36,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":37,"context_line":"Add a new microversion to Servers create API to support specified volume type"},{"line_number":38,"context_line":"when booting instances."}],"source_content_type":"text/x-rst","patch_set":9,"id":"3f79a3b5_f0bc556d","line":35,"in_reply_to":"3f79a3b5_502d098d","updated":"2018-09-17 14:43:24.000000000","message":"Done","commit_id":"08d11f6193161b20361b8b53f8bdcd22f0ebbb5a"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"f52eff3a0732893669daf47ef9f7559460c6a4f8","unresolved":false,"context_lines":[{"line_number":107,"context_line":""},{"line_number":108,"context_line":"Developer impact"},{"line_number":109,"context_line":"----------------"},{"line_number":110,"context_line":"Users have a convenient way for specifying the volume backend of booting"},{"line_number":111,"context_line":"an instance in a single API call."},{"line_number":112,"context_line":""},{"line_number":113,"context_line":"Upgrade impact"},{"line_number":114,"context_line":"--------------"}],"source_content_type":"text/x-rst","patch_set":9,"id":"3f79a3b5_703fe5b4","line":111,"range":{"start_line":110,"start_character":0,"end_line":111,"end_character":33},"updated":"2018-09-17 14:09:11.000000000","message":"This section can just be \u0027None\u0027 like the others. I should have mentioned this in the earlier patch set.","commit_id":"08d11f6193161b20361b8b53f8bdcd22f0ebbb5a"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"089f45d7b539d3c10edcf46daa327229ae93883a","unresolved":false,"context_lines":[{"line_number":107,"context_line":""},{"line_number":108,"context_line":"Developer impact"},{"line_number":109,"context_line":"----------------"},{"line_number":110,"context_line":"Users have a convenient way for specifying the volume backend of booting"},{"line_number":111,"context_line":"an instance in a single API call."},{"line_number":112,"context_line":""},{"line_number":113,"context_line":"Upgrade impact"},{"line_number":114,"context_line":"--------------"}],"source_content_type":"text/x-rst","patch_set":9,"id":"3f79a3b5_10c231ea","line":111,"range":{"start_line":110,"start_character":0,"end_line":111,"end_character":33},"in_reply_to":"3f79a3b5_703fe5b4","updated":"2018-09-17 14:43:24.000000000","message":"Done","commit_id":"08d11f6193161b20361b8b53f8bdcd22f0ebbb5a"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"ebe79364f650073a7f0dd0862dae7560e49640fa","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://blueprints.launchpad.net/nova/+spec/boot-instance-specific-storage-backend"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This blueprint proposed to add support specified volume_type for instances"},{"line_number":14,"context_line":"when booting."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Problem description"},{"line_number":17,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":10,"id":"3f79a3b5_56e1822a","line":14,"range":{"start_line":13,"start_character":0,"end_line":14,"end_character":13},"updated":"2018-09-17 14:49:51.000000000","message":"I think this should be \"proposes adding support for specifying volume_type when booting instances.\"","commit_id":"cc5e64e91973f9a790154e2cdc2f9cb319144575"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"596d47adecf96e4cb36394575b3d06912d652e19","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://blueprints.launchpad.net/nova/+spec/boot-instance-specific-storage-backend"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This blueprint proposed to add support specified volume_type for instances"},{"line_number":14,"context_line":"when booting."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Problem description"},{"line_number":17,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":10,"id":"3f79a3b5_b6cb3670","line":14,"range":{"start_line":13,"start_character":0,"end_line":14,"end_character":13},"in_reply_to":"3f79a3b5_56e1822a","updated":"2018-09-17 15:00:10.000000000","message":"Done","commit_id":"cc5e64e91973f9a790154e2cdc2f9cb319144575"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"ebe79364f650073a7f0dd0862dae7560e49640fa","unresolved":false,"context_lines":[{"line_number":16,"context_line":"Problem description"},{"line_number":17,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":18,"context_line":"Currently, when creating a new instance with boot from a volume, the user can"},{"line_number":19,"context_line":"only create a volume by mirroring the specified volume type. User will have to"},{"line_number":20,"context_line":"first create a volume from the image and then boot instance with another API"},{"line_number":21,"context_line":"call. When the user wants to boot the instance on the specified backend, this"},{"line_number":22,"context_line":"is not friendly to the user when there are multiple storage backends in the"}],"source_content_type":"text/x-rst","patch_set":10,"id":"3f79a3b5_d63b5298","line":19,"range":{"start_line":19,"start_character":5,"end_line":19,"end_character":59},"updated":"2018-09-17 14:49:51.000000000","message":"I\u0027m not sure what this means. I\u0027m guessing it should be:\n\n\"Currently, when creating a new boot-from-volume instance, the user can only control the volume_type of the volume by pre-creating it in cinder and handing it to nova during the boot process.\"","commit_id":"cc5e64e91973f9a790154e2cdc2f9cb319144575"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"596d47adecf96e4cb36394575b3d06912d652e19","unresolved":false,"context_lines":[{"line_number":16,"context_line":"Problem description"},{"line_number":17,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":18,"context_line":"Currently, when creating a new instance with boot from a volume, the user can"},{"line_number":19,"context_line":"only create a volume by mirroring the specified volume type. User will have to"},{"line_number":20,"context_line":"first create a volume from the image and then boot instance with another API"},{"line_number":21,"context_line":"call. When the user wants to boot the instance on the specified backend, this"},{"line_number":22,"context_line":"is not friendly to the user when there are multiple storage backends in the"}],"source_content_type":"text/x-rst","patch_set":10,"id":"3f79a3b5_96939a5c","line":19,"range":{"start_line":19,"start_character":5,"end_line":19,"end_character":59},"in_reply_to":"3f79a3b5_d63b5298","updated":"2018-09-17 15:00:10.000000000","message":"Done","commit_id":"cc5e64e91973f9a790154e2cdc2f9cb319144575"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"ebe79364f650073a7f0dd0862dae7560e49640fa","unresolved":false,"context_lines":[{"line_number":34,"context_line":""},{"line_number":35,"context_line":"Proposed change"},{"line_number":36,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":37,"context_line":"Add a new microversion to Servers create API to support specified volume type"},{"line_number":38,"context_line":"when booting instances."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"This would only apply to BDMs with ``source_type`` of blank, image and"}],"source_content_type":"text/x-rst","patch_set":10,"id":"3f79a3b5_366406b9","line":37,"range":{"start_line":37,"start_character":26,"end_line":37,"end_character":33},"updated":"2018-09-17 14:49:51.000000000","message":"servers","commit_id":"cc5e64e91973f9a790154e2cdc2f9cb319144575"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"596d47adecf96e4cb36394575b3d06912d652e19","unresolved":false,"context_lines":[{"line_number":34,"context_line":""},{"line_number":35,"context_line":"Proposed change"},{"line_number":36,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":37,"context_line":"Add a new microversion to Servers create API to support specified volume type"},{"line_number":38,"context_line":"when booting instances."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"This would only apply to BDMs with ``source_type`` of blank, image and"}],"source_content_type":"text/x-rst","patch_set":10,"id":"3f79a3b5_f6b1aee1","line":37,"range":{"start_line":37,"start_character":26,"end_line":37,"end_character":33},"in_reply_to":"3f79a3b5_366406b9","updated":"2018-09-17 15:00:10.000000000","message":"Done","commit_id":"cc5e64e91973f9a790154e2cdc2f9cb319144575"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"ebe79364f650073a7f0dd0862dae7560e49640fa","unresolved":false,"context_lines":[{"line_number":121,"context_line":""},{"line_number":122,"context_line":"Upgrade impact"},{"line_number":123,"context_line":"--------------"},{"line_number":124,"context_line":"Specify the minimum compute service version, the API will have to determine"},{"line_number":125,"context_line":"if the minimum compute service version across the deployment (all cells) is"},{"line_number":126,"context_line":"high enough to support user-specified volume types."},{"line_number":127,"context_line":""}],"source_content_type":"text/x-rst","patch_set":10,"id":"3f79a3b5_b6f496d0","line":124,"range":{"start_line":124,"start_character":0,"end_line":124,"end_character":43},"updated":"2018-09-17 14:49:51.000000000","message":"This doesn\u0027t make sense to me.","commit_id":"cc5e64e91973f9a790154e2cdc2f9cb319144575"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"596d47adecf96e4cb36394575b3d06912d652e19","unresolved":false,"context_lines":[{"line_number":121,"context_line":""},{"line_number":122,"context_line":"Upgrade impact"},{"line_number":123,"context_line":"--------------"},{"line_number":124,"context_line":"Specify the minimum compute service version, the API will have to determine"},{"line_number":125,"context_line":"if the minimum compute service version across the deployment (all cells) is"},{"line_number":126,"context_line":"high enough to support user-specified volume types."},{"line_number":127,"context_line":""}],"source_content_type":"text/x-rst","patch_set":10,"id":"3f79a3b5_56c0a22b","line":124,"range":{"start_line":124,"start_character":0,"end_line":124,"end_character":43},"in_reply_to":"3f79a3b5_b6f496d0","updated":"2018-09-17 15:00:10.000000000","message":"Done","commit_id":"cc5e64e91973f9a790154e2cdc2f9cb319144575"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"7495c843981d8dac4673c5d3e07ce6d12a85ade8","unresolved":false,"context_lines":[{"line_number":37,"context_line":"Add a new microversion to the servers create API to support specifying volume"},{"line_number":38,"context_line":"type when booting instances."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"This would only apply to BDMs with ``source_type`` of blank, image and"},{"line_number":41,"context_line":"snapshot. The ``volume_type`` will be passed from ``nova-api`` through to"},{"line_number":42,"context_line":"``nova-compute`` (via the BlockDeviceMapping object) where the volume gets"},{"line_number":43,"context_line":"created and then attached to the new server."},{"line_number":44,"context_line":""}],"source_content_type":"text/x-rst","patch_set":11,"id":"3f79a3b5_c441bf0c","line":41,"range":{"start_line":40,"start_character":0,"end_line":41,"end_character":9},"updated":"2018-09-25 13:20:57.000000000","message":"also only valid when dest_type is volume?","commit_id":"d4be64cfabae1d0172d1f1e37012e946e62d8004"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"5f3ec599ba08d6f7cb50981e1f1bec1210874666","unresolved":false,"context_lines":[{"line_number":37,"context_line":"Add a new microversion to the servers create API to support specifying volume"},{"line_number":38,"context_line":"type when booting instances."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"This would only apply to BDMs with ``source_type`` of blank, image and"},{"line_number":41,"context_line":"snapshot. The ``volume_type`` will be passed from ``nova-api`` through to"},{"line_number":42,"context_line":"``nova-compute`` (via the BlockDeviceMapping object) where the volume gets"},{"line_number":43,"context_line":"created and then attached to the new server."},{"line_number":44,"context_line":""}],"source_content_type":"text/x-rst","patch_set":11,"id":"3f79a3b5_9e8a2f62","line":41,"range":{"start_line":40,"start_character":0,"end_line":41,"end_character":9},"in_reply_to":"3f79a3b5_c441bf0c","updated":"2018-09-25 16:20:10.000000000","message":"Yeah, true, the spec should be updated.\n\nAnother thing that came up during review of the code change is that the compute API will support the user passing the volume type ID or name, and nova will validate as appropriate and pass through to cinder\u0027s volume create API which also takes volume type ID or name.","commit_id":"d4be64cfabae1d0172d1f1e37012e946e62d8004"}],"specs/stein/approved/support-volume-type-instance-when-boot.rst":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"80a0fc08d3b242cce867839fa6c307dc94c0c67e","unresolved":false,"context_lines":[{"line_number":8,"context_line":"Support add volume_type for instances when booting"},{"line_number":9,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://blueprints.launchpad.net/nova/+spec/boot-instance-specific-storage-backend"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This blueprint proposed to add support specified volume_type for instances"},{"line_number":14,"context_line":"when booting."}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_84e6394a","line":11,"range":{"start_line":11,"start_character":44,"end_line":11,"end_character":82},"updated":"2018-09-13 21:27:00.000000000","message":"The spec filename needs to match the blueprint name, so rename the spec file.","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"65a6a0cd365cf2cf9ba507b12b61aa9d58cb2be4","unresolved":false,"context_lines":[{"line_number":8,"context_line":"Support add volume_type for instances when booting"},{"line_number":9,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://blueprints.launchpad.net/nova/+spec/boot-instance-specific-storage-backend"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This blueprint proposed to add support specified volume_type for instances"},{"line_number":14,"context_line":"when booting."}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_a0147eea","line":11,"range":{"start_line":11,"start_character":44,"end_line":11,"end_character":82},"in_reply_to":"3f79a3b5_84e6394a","updated":"2018-09-14 06:21:25.000000000","message":"Ok, rename to \"boot-instance-specific-storage-backend\".","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"80a0fc08d3b242cce867839fa6c307dc94c0c67e","unresolved":false,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Problem description"},{"line_number":17,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":18,"context_line":"Volume type for servers are supported in microversion 2.64, but currently we"},{"line_number":19,"context_line":"can only create a volume by mirroring the specified volume type, and then"},{"line_number":20,"context_line":"we boot the instances from the volume.User will have to first create a volume"},{"line_number":21,"context_line":"from the image and then boot instance with another API call. When the user"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_c4e3d133","line":18,"range":{"start_line":18,"start_character":0,"end_line":18,"end_character":58},"updated":"2018-09-13 21:27:00.000000000","message":"Huh?\n\nhttps://docs.openstack.org/nova/latest/reference/api-microversion-history.html#id58","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"65a6a0cd365cf2cf9ba507b12b61aa9d58cb2be4","unresolved":false,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Problem description"},{"line_number":17,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":18,"context_line":"Volume type for servers are supported in microversion 2.64, but currently we"},{"line_number":19,"context_line":"can only create a volume by mirroring the specified volume type, and then"},{"line_number":20,"context_line":"we boot the instances from the volume.User will have to first create a volume"},{"line_number":21,"context_line":"from the image and then boot instance with another API call. When the user"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_00b452f1","line":18,"range":{"start_line":18,"start_character":0,"end_line":18,"end_character":58},"in_reply_to":"3f79a3b5_c4e3d133","updated":"2018-09-14 06:21:25.000000000","message":"Sorry, there is a problem described here, it will be updated.","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"80a0fc08d3b242cce867839fa6c307dc94c0c67e","unresolved":false,"context_lines":[{"line_number":17,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":18,"context_line":"Volume type for servers are supported in microversion 2.64, but currently we"},{"line_number":19,"context_line":"can only create a volume by mirroring the specified volume type, and then"},{"line_number":20,"context_line":"we boot the instances from the volume.User will have to first create a volume"},{"line_number":21,"context_line":"from the image and then boot instance with another API call. When the user"},{"line_number":22,"context_line":"wants to boot the instance on the specified backend, this is not friendly to"},{"line_number":23,"context_line":"the user when there are multiple storage backends in the environment."}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_e4de0d69","line":20,"range":{"start_line":20,"start_character":37,"end_line":20,"end_character":38},"updated":"2018-09-13 21:27:00.000000000","message":"need a space between sentences","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"65a6a0cd365cf2cf9ba507b12b61aa9d58cb2be4","unresolved":false,"context_lines":[{"line_number":17,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":18,"context_line":"Volume type for servers are supported in microversion 2.64, but currently we"},{"line_number":19,"context_line":"can only create a volume by mirroring the specified volume type, and then"},{"line_number":20,"context_line":"we boot the instances from the volume.User will have to first create a volume"},{"line_number":21,"context_line":"from the image and then boot instance with another API call. When the user"},{"line_number":22,"context_line":"wants to boot the instance on the specified backend, this is not friendly to"},{"line_number":23,"context_line":"the user when there are multiple storage backends in the environment."}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_e08d9635","line":20,"range":{"start_line":20,"start_character":37,"end_line":20,"end_character":38},"in_reply_to":"3f79a3b5_e4de0d69","updated":"2018-09-14 06:21:25.000000000","message":"Done","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"80a0fc08d3b242cce867839fa6c307dc94c0c67e","unresolved":false,"context_lines":[{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Use Cases"},{"line_number":26,"context_line":"---------"},{"line_number":27,"context_line":"As an user, I would like to specified volume type to my instances when I boot"},{"line_number":28,"context_line":"them, especially when I doing bulk boot, I may want to specify a different"},{"line_number":29,"context_line":"storage backend for the instance created by this call."},{"line_number":30,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_24be4500","line":27,"range":{"start_line":27,"start_character":28,"end_line":27,"end_character":37},"updated":"2018-09-13 21:27:00.000000000","message":"specify","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"65a6a0cd365cf2cf9ba507b12b61aa9d58cb2be4","unresolved":false,"context_lines":[{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Use Cases"},{"line_number":26,"context_line":"---------"},{"line_number":27,"context_line":"As an user, I would like to specified volume type to my instances when I boot"},{"line_number":28,"context_line":"them, especially when I doing bulk boot, I may want to specify a different"},{"line_number":29,"context_line":"storage backend for the instance created by this call."},{"line_number":30,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_a0dd5ede","line":27,"range":{"start_line":27,"start_character":28,"end_line":27,"end_character":37},"in_reply_to":"3f79a3b5_24be4500","updated":"2018-09-14 06:21:25.000000000","message":"Done","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"80a0fc08d3b242cce867839fa6c307dc94c0c67e","unresolved":false,"context_lines":[{"line_number":25,"context_line":"Use Cases"},{"line_number":26,"context_line":"---------"},{"line_number":27,"context_line":"As an user, I would like to specified volume type to my instances when I boot"},{"line_number":28,"context_line":"them, especially when I doing bulk boot, I may want to specify a different"},{"line_number":29,"context_line":"storage backend for the instance created by this call."},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"Proposed change"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_c4bc3108","line":28,"range":{"start_line":28,"start_character":22,"end_line":28,"end_character":23},"updated":"2018-09-13 21:27:00.000000000","message":"I am","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"65a6a0cd365cf2cf9ba507b12b61aa9d58cb2be4","unresolved":false,"context_lines":[{"line_number":25,"context_line":"Use Cases"},{"line_number":26,"context_line":"---------"},{"line_number":27,"context_line":"As an user, I would like to specified volume type to my instances when I boot"},{"line_number":28,"context_line":"them, especially when I doing bulk boot, I may want to specify a different"},{"line_number":29,"context_line":"storage backend for the instance created by this call."},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"Proposed change"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_60f1866a","line":28,"range":{"start_line":28,"start_character":22,"end_line":28,"end_character":23},"in_reply_to":"3f79a3b5_c4bc3108","updated":"2018-09-14 06:21:25.000000000","message":"Done","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"80a0fc08d3b242cce867839fa6c307dc94c0c67e","unresolved":false,"context_lines":[{"line_number":26,"context_line":"---------"},{"line_number":27,"context_line":"As an user, I would like to specified volume type to my instances when I boot"},{"line_number":28,"context_line":"them, especially when I doing bulk boot, I may want to specify a different"},{"line_number":29,"context_line":"storage backend for the instance created by this call."},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"Proposed change"},{"line_number":32,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_845dd9ed","line":29,"updated":"2018-09-13 21:27:00.000000000","message":"Hmm, by bulk boot do you mean the multi-create support where you create multiple servers in a single request using the min_count/max_count parameters? Because I\u0027m pretty sure the block_device_mapping_v2 applies to *all* servers created in that request. Maybe you just mean \u0027bulk boot\u0027 by creating multiple servers in separate requests but at the same time.","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"65a6a0cd365cf2cf9ba507b12b61aa9d58cb2be4","unresolved":false,"context_lines":[{"line_number":26,"context_line":"---------"},{"line_number":27,"context_line":"As an user, I would like to specified volume type to my instances when I boot"},{"line_number":28,"context_line":"them, especially when I doing bulk boot, I may want to specify a different"},{"line_number":29,"context_line":"storage backend for the instance created by this call."},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"Proposed change"},{"line_number":32,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_6032e611","line":29,"in_reply_to":"3f79a3b5_845dd9ed","updated":"2018-09-14 06:21:25.000000000","message":"Yes, I mean \u0027bulk boot\u0027 by creating multiple servers in separate requests but at the same time. However, if the user wants to boot instance on a different backend, they only need to specify a different backend to send the create request again.","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"80a0fc08d3b242cce867839fa6c307dc94c0c67e","unresolved":false,"context_lines":[{"line_number":35,"context_line":""},{"line_number":36,"context_line":"Alternatives"},{"line_number":37,"context_line":"------------"},{"line_number":38,"context_line":"Keep the current implementation."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"Data model impact"},{"line_number":41,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_a46015a6","line":38,"range":{"start_line":38,"start_character":0,"end_line":38,"end_character":32},"updated":"2018-09-13 21:27:00.000000000","message":"The alternative is what you already mentioned - create the volume with the non-default type in cinder and then provide the volume to nova when creating the server.","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"65a6a0cd365cf2cf9ba507b12b61aa9d58cb2be4","unresolved":false,"context_lines":[{"line_number":35,"context_line":""},{"line_number":36,"context_line":"Alternatives"},{"line_number":37,"context_line":"------------"},{"line_number":38,"context_line":"Keep the current implementation."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"Data model impact"},{"line_number":41,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_a3b1e846","line":38,"range":{"start_line":38,"start_character":0,"end_line":38,"end_character":32},"in_reply_to":"3f79a3b5_a46015a6","updated":"2018-09-14 06:21:25.000000000","message":"Yeah, add this description.","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"80a0fc08d3b242cce867839fa6c307dc94c0c67e","unresolved":false,"context_lines":[{"line_number":39,"context_line":""},{"line_number":40,"context_line":"Data model impact"},{"line_number":41,"context_line":"-----------------"},{"line_number":42,"context_line":"None"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"REST API impact"},{"line_number":45,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_645afdf2","line":42,"updated":"2018-09-13 21:27:00.000000000","message":"We\u0027ll have to store the volume_type on the BlockDeviceMapping object (and block_device_mappings table in the DB).","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"65a6a0cd365cf2cf9ba507b12b61aa9d58cb2be4","unresolved":false,"context_lines":[{"line_number":39,"context_line":""},{"line_number":40,"context_line":"Data model impact"},{"line_number":41,"context_line":"-----------------"},{"line_number":42,"context_line":"None"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"REST API impact"},{"line_number":45,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_23cdd8b6","line":42,"in_reply_to":"3f79a3b5_645afdf2","updated":"2018-09-14 06:21:25.000000000","message":"LGTM.","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"80a0fc08d3b242cce867839fa6c307dc94c0c67e","unresolved":false,"context_lines":[{"line_number":57,"context_line":"        ..."},{"line_number":58,"context_line":"    }"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"The volume_type will be also be included in response if setted ::"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"    {"},{"line_number":63,"context_line":"        ..."}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_440ca1eb","line":60,"range":{"start_line":60,"start_character":41,"end_line":60,"end_character":62},"updated":"2018-09-13 21:27:00.000000000","message":"The POST /servers response doesn\u0027t include the bdms, so what response are you referring to here? Showing server details? Showing attached volume details? I don\u0027t know that we really need this in the response - the user can get the attached volume id from the compute API and then lookup the volume details from the block storage API.","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"65a6a0cd365cf2cf9ba507b12b61aa9d58cb2be4","unresolved":false,"context_lines":[{"line_number":57,"context_line":"        ..."},{"line_number":58,"context_line":"    }"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"The volume_type will be also be included in response if setted ::"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"    {"},{"line_number":63,"context_line":"        ..."}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_03b45cf1","line":60,"range":{"start_line":60,"start_character":41,"end_line":60,"end_character":62},"in_reply_to":"3f79a3b5_440ca1eb","updated":"2018-09-14 06:21:25.000000000","message":"Yes, this is what cinder should return, remove this.","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"80a0fc08d3b242cce867839fa6c307dc94c0c67e","unresolved":false,"context_lines":[{"line_number":65,"context_line":"        ..."},{"line_number":66,"context_line":"    }"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"The volume_type will be none if not setted ::"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"    {"},{"line_number":71,"context_line":"        ..."},{"line_number":72,"context_line":"        \"volume_type\": None"},{"line_number":73,"context_line":"        ..."},{"line_number":74,"context_line":"    }"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"Security impact"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_24f165df","line":74,"range":{"start_line":68,"start_character":0,"end_line":74,"end_character":5},"updated":"2018-09-13 21:27:00.000000000","message":"None in what? The response? I don\u0027t see why we\u0027d include it in a compute API response (as noted above).","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"65a6a0cd365cf2cf9ba507b12b61aa9d58cb2be4","unresolved":false,"context_lines":[{"line_number":65,"context_line":"        ..."},{"line_number":66,"context_line":"    }"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"The volume_type will be none if not setted ::"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"    {"},{"line_number":71,"context_line":"        ..."},{"line_number":72,"context_line":"        \"volume_type\": None"},{"line_number":73,"context_line":"        ..."},{"line_number":74,"context_line":"    }"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"Security impact"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_e3c84076","line":74,"range":{"start_line":68,"start_character":0,"end_line":74,"end_character":5},"in_reply_to":"3f79a3b5_24f165df","updated":"2018-09-14 06:21:25.000000000","message":"This would not include in compute API, remove it.","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"80a0fc08d3b242cce867839fa6c307dc94c0c67e","unresolved":false,"context_lines":[{"line_number":80,"context_line":""},{"line_number":81,"context_line":"Notifications impact"},{"line_number":82,"context_line":"--------------------"},{"line_number":83,"context_line":"None"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"Other end user impact"},{"line_number":86,"context_line":"---------------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_241ac517","line":83,"updated":"2018-09-13 21:27:00.000000000","message":"We have BlockDevicePayload in versioned notifications for servers, so we\u0027d want to include the volume_type field.\n\nhttps://github.com/openstack/nova/blob/ffdd8098383a6d5502fbfa7c921633b24512aadf/nova/notifications/objects/instance.py#L434","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"65a6a0cd365cf2cf9ba507b12b61aa9d58cb2be4","unresolved":false,"context_lines":[{"line_number":80,"context_line":""},{"line_number":81,"context_line":"Notifications impact"},{"line_number":82,"context_line":"--------------------"},{"line_number":83,"context_line":"None"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"Other end user impact"},{"line_number":86,"context_line":"---------------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_23d2b8d2","line":83,"in_reply_to":"3f79a3b5_241ac517","updated":"2018-09-14 06:21:25.000000000","message":"Ok, Add ``volume_type`` field to BlockDevicePayload object.","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"80a0fc08d3b242cce867839fa6c307dc94c0c67e","unresolved":false,"context_lines":[{"line_number":84,"context_line":""},{"line_number":85,"context_line":"Other end user impact"},{"line_number":86,"context_line":"---------------------"},{"line_number":87,"context_line":"User will be able to specified volume type when boot instances using"},{"line_number":88,"context_line":"specific microversion. python-novaclient will also make modifications"},{"line_number":89,"context_line":"to support this feature."},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"Performance Impact"},{"line_number":92,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_e4e84de0","line":89,"range":{"start_line":87,"start_character":0,"end_line":89,"end_character":24},"updated":"2018-09-13 21:27:00.000000000","message":"Just say python-novaclient and python-openstackclient will be updated.\n\nWe should also use this section to discuss image-defined BDMs (see the etherpad for details). In summary, when we snapshot a volume-backed server, the block_device_mapping_v2 image metadata will include the volume_type from the BDM record so if the user then creates another server from that snapshot, the volume that nova creates from that snapshot will use the same volume_type. If a user wishes to change that volume type in the image metadata, they can via the image API.","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"f52eff3a0732893669daf47ef9f7559460c6a4f8","unresolved":false,"context_lines":[{"line_number":84,"context_line":""},{"line_number":85,"context_line":"Other end user impact"},{"line_number":86,"context_line":"---------------------"},{"line_number":87,"context_line":"User will be able to specified volume type when boot instances using"},{"line_number":88,"context_line":"specific microversion. python-novaclient will also make modifications"},{"line_number":89,"context_line":"to support this feature."},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"Performance Impact"},{"line_number":92,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_3502bf0d","line":89,"range":{"start_line":87,"start_character":0,"end_line":89,"end_character":24},"in_reply_to":"3f79a3b5_26b726f8","updated":"2018-09-17 14:09:11.000000000","message":"\u003e But I have a question with \"If a user wishes to change that volume type in the image metadata, they can via the image API.\", does it mean creating a new volume using this image to specify the volume type?\n\nYes, see:\n\nhttps://docs.openstack.org/nova/latest/user/block-device-mapping.html\n\nThis tempest test will also explain the workflow for image-defined BDMs:\n\nhttps://github.com/openstack/tempest/blob/3674fb13825a60be9eae6341ec5f7d1fdc7472b4/tempest/scenario/test_volume_boot_pattern.py#L210","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"65a6a0cd365cf2cf9ba507b12b61aa9d58cb2be4","unresolved":false,"context_lines":[{"line_number":84,"context_line":""},{"line_number":85,"context_line":"Other end user impact"},{"line_number":86,"context_line":"---------------------"},{"line_number":87,"context_line":"User will be able to specified volume type when boot instances using"},{"line_number":88,"context_line":"specific microversion. python-novaclient will also make modifications"},{"line_number":89,"context_line":"to support this feature."},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"Performance Impact"},{"line_number":92,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_26b726f8","line":89,"range":{"start_line":87,"start_character":0,"end_line":89,"end_character":24},"in_reply_to":"3f79a3b5_e4e84de0","updated":"2018-09-14 06:21:25.000000000","message":"\" when we snapshot a volume-backed server, the block_device_mapping_v2 image metadata will include the volume_type from the BDM record so if the user then creates another server from that snapshot, the volume that nova creates from that snapshot will use the same volume_type. If a user wishes to change that volume type in the image metadata, they can via the image API.\"\nI think this paragraph is right here.:-)\nThis contains the places where image-defined BDMs need attention.\n---\nBut I have a question with \"If a user wishes to change that volume type in the image metadata, they can via the image API.\", does it mean creating a new volume using this image to specify the volume type?","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"80a0fc08d3b242cce867839fa6c307dc94c0c67e","unresolved":false,"context_lines":[{"line_number":94,"context_line":""},{"line_number":95,"context_line":"Other deployer impact"},{"line_number":96,"context_line":"---------------------"},{"line_number":97,"context_line":"A new microversion will be added."},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"Developer impact"},{"line_number":100,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_84b959ed","line":97,"range":{"start_line":97,"start_character":0,"end_line":97,"end_character":33},"updated":"2018-09-13 21:27:00.000000000","message":"None","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"65a6a0cd365cf2cf9ba507b12b61aa9d58cb2be4","unresolved":false,"context_lines":[{"line_number":94,"context_line":""},{"line_number":95,"context_line":"Other deployer impact"},{"line_number":96,"context_line":"---------------------"},{"line_number":97,"context_line":"A new microversion will be added."},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"Developer impact"},{"line_number":100,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_83e2ac9f","line":97,"range":{"start_line":97,"start_character":0,"end_line":97,"end_character":33},"in_reply_to":"3f79a3b5_84b959ed","updated":"2018-09-14 06:21:25.000000000","message":"Done","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"80a0fc08d3b242cce867839fa6c307dc94c0c67e","unresolved":false,"context_lines":[{"line_number":98,"context_line":""},{"line_number":99,"context_line":"Developer impact"},{"line_number":100,"context_line":"----------------"},{"line_number":101,"context_line":"Users have a convenient way for specified the backend of booting an instance"},{"line_number":102,"context_line":"in a single API call."},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"Upgrade impact"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_04cd4985","line":101,"range":{"start_line":101,"start_character":32,"end_line":101,"end_character":41},"updated":"2018-09-13 21:27:00.000000000","message":"specifying","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"80a0fc08d3b242cce867839fa6c307dc94c0c67e","unresolved":false,"context_lines":[{"line_number":98,"context_line":""},{"line_number":99,"context_line":"Developer impact"},{"line_number":100,"context_line":"----------------"},{"line_number":101,"context_line":"Users have a convenient way for specified the backend of booting an instance"},{"line_number":102,"context_line":"in a single API call."},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"Upgrade impact"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_e4c12d50","line":101,"range":{"start_line":101,"start_character":46,"end_line":101,"end_character":53},"updated":"2018-09-13 21:27:00.000000000","message":"volume backend","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"65a6a0cd365cf2cf9ba507b12b61aa9d58cb2be4","unresolved":false,"context_lines":[{"line_number":98,"context_line":""},{"line_number":99,"context_line":"Developer impact"},{"line_number":100,"context_line":"----------------"},{"line_number":101,"context_line":"Users have a convenient way for specified the backend of booting an instance"},{"line_number":102,"context_line":"in a single API call."},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"Upgrade impact"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_a6701642","line":101,"range":{"start_line":101,"start_character":32,"end_line":101,"end_character":41},"in_reply_to":"3f79a3b5_04cd4985","updated":"2018-09-14 06:21:25.000000000","message":"Done","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"65a6a0cd365cf2cf9ba507b12b61aa9d58cb2be4","unresolved":false,"context_lines":[{"line_number":98,"context_line":""},{"line_number":99,"context_line":"Developer impact"},{"line_number":100,"context_line":"----------------"},{"line_number":101,"context_line":"Users have a convenient way for specified the backend of booting an instance"},{"line_number":102,"context_line":"in a single API call."},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"Upgrade impact"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_468d8228","line":101,"range":{"start_line":101,"start_character":46,"end_line":101,"end_character":53},"in_reply_to":"3f79a3b5_e4c12d50","updated":"2018-09-14 06:21:25.000000000","message":"Done","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"80a0fc08d3b242cce867839fa6c307dc94c0c67e","unresolved":false,"context_lines":[{"line_number":103,"context_line":""},{"line_number":104,"context_line":"Upgrade impact"},{"line_number":105,"context_line":"--------------"},{"line_number":106,"context_line":"None"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"Implementation"},{"line_number":109,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_a48fb520","line":106,"updated":"2018-09-13 21:27:00.000000000","message":"Since nova-compute is what creates the volume and will need to receive the specified type from the API, there is an upgrade impact such that the API will have to determine if the minimum compute service version across the deployment (all cells) is high enough to support user-specified volume types (that is a temporary check for Stein and we can remove in the T release). For example, see how we handle this for boot from volume with multi-attach volumes in the API:\n\nhttps://github.com/openstack/nova/blob/ffdd8098383a6d5502fbfa7c921633b24512aadf/nova/compute/api.py#L3964-L3978","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"65a6a0cd365cf2cf9ba507b12b61aa9d58cb2be4","unresolved":false,"context_lines":[{"line_number":103,"context_line":""},{"line_number":104,"context_line":"Upgrade impact"},{"line_number":105,"context_line":"--------------"},{"line_number":106,"context_line":"None"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"Implementation"},{"line_number":109,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_c9e44511","line":106,"in_reply_to":"3f79a3b5_a48fb520","updated":"2018-09-14 06:21:25.000000000","message":"Specify the minimum compute service version, such as \"MIN_COMPUTE_VOLUME_TYPE \u003d 36\".","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"80a0fc08d3b242cce867839fa6c307dc94c0c67e","unresolved":false,"context_lines":[{"line_number":124,"context_line":""},{"line_number":125,"context_line":"Testing"},{"line_number":126,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":127,"context_line":"* Add related unittest"},{"line_number":128,"context_line":"* Add related functional test"},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"Documentation Impact"},{"line_number":131,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_249ba564","line":128,"range":{"start_line":127,"start_character":0,"end_line":128,"end_character":29},"updated":"2018-09-13 21:27:00.000000000","message":"We would also require Tempest tests for this to ensure the cross-project integration is working (and covered in CI).","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"65a6a0cd365cf2cf9ba507b12b61aa9d58cb2be4","unresolved":false,"context_lines":[{"line_number":124,"context_line":""},{"line_number":125,"context_line":"Testing"},{"line_number":126,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":127,"context_line":"* Add related unittest"},{"line_number":128,"context_line":"* Add related functional test"},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"Documentation Impact"},{"line_number":131,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_e6f28e9b","line":128,"range":{"start_line":127,"start_character":0,"end_line":128,"end_character":29},"in_reply_to":"3f79a3b5_249ba564","updated":"2018-09-14 06:21:25.000000000","message":"Yeah, add Tempest tests.","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"80a0fc08d3b242cce867839fa6c307dc94c0c67e","unresolved":false,"context_lines":[{"line_number":134,"context_line":""},{"line_number":135,"context_line":"References"},{"line_number":136,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":137,"context_line":"None"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"History"},{"line_number":140,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_64b2fde7","line":137,"updated":"2018-09-13 21:27:00.000000000","message":"We have some references here. I\u0027d at least link to http://lists.openstack.org/pipermail/openstack-dev/2018-July/132052.html and https://etherpad.openstack.org/p/nova-ptg-stein for discussions around this feature.","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"},{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"65a6a0cd365cf2cf9ba507b12b61aa9d58cb2be4","unresolved":false,"context_lines":[{"line_number":134,"context_line":""},{"line_number":135,"context_line":"References"},{"line_number":136,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":137,"context_line":"None"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"History"},{"line_number":140,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f79a3b5_c653d26b","line":137,"in_reply_to":"3f79a3b5_64b2fde7","updated":"2018-09-14 06:21:25.000000000","message":"Add these references and simple introduce.","commit_id":"109f7a0cdc2011a16ae5b73e92bfd412774c477c"}]}