)]}'
{"cinder/volume/drivers/dell_emc/powerstore/adapter.py":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"54d34bef41101b3dd015c97ee3a1614f1a83966f","unresolved":false,"context_lines":[{"line_number":264,"context_line":"            \"provider_id\": cloned_provider_id,"},{"line_number":265,"context_line":"        }"},{"line_number":266,"context_line":""},{"line_number":267,"context_line":"    def create_volume_from_snapshot(self, volume, snapshot):"},{"line_number":268,"context_line":"        LOG.debug(\"Create PowerStore volume %(volume_name)s of size \""},{"line_number":269,"context_line":"                  \"%(volume_size)s GiB with id %(volume_id)s from snapshot \""},{"line_number":270,"context_line":"                  \"%(snapshot_name)s with id %(snapshot_id)s. PowerStore \""}],"source_content_type":"text/x-python","patch_set":1,"id":"bf51134e_d23e4889","line":267,"range":{"start_line":267,"start_character":8,"end_line":267,"end_character":35},"updated":"2020-07-15 13:38:30.000000000","message":"create_cloned and create from snapshot both need to handle the new volume being requested for a larger size than the original.","commit_id":"2fc6eae95c17d3978e7f9a8645ea9af8b374229b"},{"author":{"_account_id":31016,"name":"Ivan Pchelintsev","email":"i.pchelintsev@yadro.com","username":"pcheli"},"change_message_id":"c58cf8dd73e14fc1d3732b5e61587eff5da2b125","unresolved":false,"context_lines":[{"line_number":264,"context_line":"            \"provider_id\": cloned_provider_id,"},{"line_number":265,"context_line":"        }"},{"line_number":266,"context_line":""},{"line_number":267,"context_line":"    def create_volume_from_snapshot(self, volume, snapshot):"},{"line_number":268,"context_line":"        LOG.debug(\"Create PowerStore volume %(volume_name)s of size \""},{"line_number":269,"context_line":"                  \"%(volume_size)s GiB with id %(volume_id)s from snapshot \""},{"line_number":270,"context_line":"                  \"%(snapshot_name)s with id %(snapshot_id)s. PowerStore \""}],"source_content_type":"text/x-python","patch_set":1,"id":"bf51134e_21dc90e2","line":267,"range":{"start_line":267,"start_character":8,"end_line":267,"end_character":35},"in_reply_to":"bf51134e_d23e4889","updated":"2020-07-15 17:23:15.000000000","message":"create_cloned and create from snap both use _create_volume_from_source where new volume is created and after that is extended if needed.","commit_id":"2fc6eae95c17d3978e7f9a8645ea9af8b374229b"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"c46fd21345fa7cce89bf65e37e3b8f1f99728385","unresolved":false,"context_lines":[{"line_number":381,"context_line":"            source.provider_id,"},{"line_number":382,"context_line":"            entity"},{"line_number":383,"context_line":"        )"},{"line_number":384,"context_line":"        if volume.size \u003e source_size:"},{"line_number":385,"context_line":"            size_in_bytes \u003d utils.gib_to_bytes(volume.size)"},{"line_number":386,"context_line":"            self.client.extend_volume(volume_provider_id, size_in_bytes)"},{"line_number":387,"context_line":"        return volume_provider_id"}],"source_content_type":"text/x-python","patch_set":1,"id":"bf51134e_3348635a","line":384,"range":{"start_line":384,"start_character":8,"end_line":384,"end_character":37},"updated":"2020-07-15 17:31:52.000000000","message":"☑️","commit_id":"2fc6eae95c17d3978e7f9a8645ea9af8b374229b"}],"cinder/volume/drivers/dell_emc/powerstore/client.py":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"54d34bef41101b3dd015c97ee3a1614f1a83966f","unresolved":false,"context_lines":[{"line_number":244,"context_line":"                \"description\": description,"},{"line_number":245,"context_line":"                \"name\": name,"},{"line_number":246,"context_line":"            }"},{"line_number":247,"context_line":"        )"},{"line_number":248,"context_line":"        if r.status_code !\u003d requests.codes.ok:"},{"line_number":249,"context_line":"            msg \u003d (_(\"Failed to create clone %(clone_name)s for \""},{"line_number":250,"context_line":"                     \"PowerStore %(entity)s with id %(entity_id)s.\")"}],"source_content_type":"text/x-python","patch_set":1,"id":"bf51134e_72d65c51","line":247,"updated":"2020-07-15 13:38:30.000000000","message":"Doesn\u0027t look like this handles the case where the new volume could be requested to be larger than the original volume.","commit_id":"2fc6eae95c17d3978e7f9a8645ea9af8b374229b"},{"author":{"_account_id":31016,"name":"Ivan Pchelintsev","email":"i.pchelintsev@yadro.com","username":"pcheli"},"change_message_id":"c58cf8dd73e14fc1d3732b5e61587eff5da2b125","unresolved":false,"context_lines":[{"line_number":244,"context_line":"                \"description\": description,"},{"line_number":245,"context_line":"                \"name\": name,"},{"line_number":246,"context_line":"            }"},{"line_number":247,"context_line":"        )"},{"line_number":248,"context_line":"        if r.status_code !\u003d requests.codes.ok:"},{"line_number":249,"context_line":"            msg \u003d (_(\"Failed to create clone %(clone_name)s for \""},{"line_number":250,"context_line":"                     \"PowerStore %(entity)s with id %(entity_id)s.\")"}],"source_content_type":"text/x-python","patch_set":1,"id":"bf51134e_01850cad","line":247,"in_reply_to":"bf51134e_72d65c51","updated":"2020-07-15 17:23:15.000000000","message":"This request only clones volume without extension.","commit_id":"2fc6eae95c17d3978e7f9a8645ea9af8b374229b"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"fac1539ff28069ac825836cea924bb37f8f4efaf","unresolved":false,"context_lines":[{"line_number":13,"context_line":"# License for the specific language governing permissions and limitations"},{"line_number":14,"context_line":"# under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"\"\"\""},{"line_number":17,"context_line":"REST client for Dell EMC PowerStore."},{"line_number":18,"context_line":"\"\"\""},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"import functools"},{"line_number":21,"context_line":"import json"}],"source_content_type":"text/x-python","patch_set":9,"id":"bf51134e_f0332f12","line":18,"range":{"start_line":16,"start_character":0,"end_line":18,"end_character":3},"updated":"2020-07-21 15:52:53.000000000","message":"Extremely trivial nit, but if you need to do another update, singe line docstrings should be all on one line:\n\n\"\"\"REST client for Dell EMC PowerStore.\"\"\"","commit_id":"0e9130846a3fbe2ddbf86b2b6ac8543e9fa1fb47"},{"author":{"_account_id":31016,"name":"Ivan Pchelintsev","email":"i.pchelintsev@yadro.com","username":"pcheli"},"change_message_id":"31290b12cb844c0a637febe2c6eb6e5f6dd061b4","unresolved":false,"context_lines":[{"line_number":13,"context_line":"# License for the specific language governing permissions and limitations"},{"line_number":14,"context_line":"# under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"\"\"\""},{"line_number":17,"context_line":"REST client for Dell EMC PowerStore."},{"line_number":18,"context_line":"\"\"\""},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"import functools"},{"line_number":21,"context_line":"import json"}],"source_content_type":"text/x-python","patch_set":9,"id":"bf51134e_8a8b941c","line":18,"range":{"start_line":16,"start_character":0,"end_line":18,"end_character":3},"in_reply_to":"bf51134e_f0332f12","updated":"2020-07-22 11:29:12.000000000","message":"Done","commit_id":"0e9130846a3fbe2ddbf86b2b6ac8543e9fa1fb47"}],"cinder/volume/drivers/dell_emc/powerstore/options.py":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"19226c65b945aa8aaa3c2dff88ccbc3b73a5487d","unresolved":false,"context_lines":[{"line_number":31,"context_line":"                default\u003d[],"},{"line_number":32,"context_line":"                help\u003d\"Allowed ports. Comma separated list of PowerStore \""},{"line_number":33,"context_line":"                     \"iSCSI IPs or FC WWNs (ex. 58:cc:f0:98:49:22:07:02) \""},{"line_number":34,"context_line":"                     \"to be used. If option is not set all ports are allowed.\")"},{"line_number":35,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":9,"id":"bf51134e_b0c0972c","line":34,"range":{"start_line":34,"start_character":34,"end_line":34,"end_character":76},"updated":"2020-07-21 15:37:05.000000000","message":"+1","commit_id":"0e9130846a3fbe2ddbf86b2b6ac8543e9fa1fb47"}],"doc/source/configuration/block-storage/drivers/dell-emc-powerstore-driver.rst":[{"author":{"_account_id":10379,"name":"Rajini Karthik","email":"rrajini@gmail.com","username":"rajinir"},"change_message_id":"2811d26ded68dded1e9fc64ea46b1c203f447162","unresolved":false,"context_lines":[{"line_number":41,"context_line":"      # Backend name"},{"line_number":42,"context_line":"      volume_backend_name \u003d \u003cBackend name\u003e"},{"line_number":43,"context_line":"      # PowerStore appliances"},{"line_number":44,"context_line":"      powerstore_appliances \u003d \u003cAppliances names\u003e # Ex. Appliance-1,Appliance-2"},{"line_number":45,"context_line":"      # PowerStore allowed ports"},{"line_number":46,"context_line":"      powerstore_ports \u003d \u003cAllowed ports\u003e # Ex. 58:cc:f0:98:49:22:07:02,58:cc:f0:98:49:23:07:02"},{"line_number":47,"context_line":""}],"source_content_type":"text/x-rst","patch_set":8,"id":"bf51134e_14e185fd","line":44,"updated":"2020-07-20 21:41:45.000000000","message":"Is this optional? Does it support a wildcard to include all like \u0027*\u0027","commit_id":"23f51e7dc3f8fb3ff6cc56baee953bf70ad35eac"},{"author":{"_account_id":31016,"name":"Ivan Pchelintsev","email":"i.pchelintsev@yadro.com","username":"pcheli"},"change_message_id":"87145d0d8541539cd74996e289d754e7881d9491","unresolved":false,"context_lines":[{"line_number":41,"context_line":"      # Backend name"},{"line_number":42,"context_line":"      volume_backend_name \u003d \u003cBackend name\u003e"},{"line_number":43,"context_line":"      # PowerStore appliances"},{"line_number":44,"context_line":"      powerstore_appliances \u003d \u003cAppliances names\u003e # Ex. Appliance-1,Appliance-2"},{"line_number":45,"context_line":"      # PowerStore allowed ports"},{"line_number":46,"context_line":"      powerstore_ports \u003d \u003cAllowed ports\u003e # Ex. 58:cc:f0:98:49:22:07:02,58:cc:f0:98:49:23:07:02"},{"line_number":47,"context_line":""}],"source_content_type":"text/x-rst","patch_set":8,"id":"bf51134e_00b01fdd","line":44,"in_reply_to":"bf51134e_14e185fd","updated":"2020-07-21 06:34:46.000000000","message":"It is required and does not support wildcards or any kind of regular expressions yet. There is also a corresponding error if variable is not set which is raised during driver initialisation.","commit_id":"23f51e7dc3f8fb3ff6cc56baee953bf70ad35eac"},{"author":{"_account_id":10379,"name":"Rajini Karthik","email":"rrajini@gmail.com","username":"rajinir"},"change_message_id":"2811d26ded68dded1e9fc64ea46b1c203f447162","unresolved":false,"context_lines":[{"line_number":43,"context_line":"      # PowerStore appliances"},{"line_number":44,"context_line":"      powerstore_appliances \u003d \u003cAppliances names\u003e # Ex. Appliance-1,Appliance-2"},{"line_number":45,"context_line":"      # PowerStore allowed ports"},{"line_number":46,"context_line":"      powerstore_ports \u003d \u003cAllowed ports\u003e # Ex. 58:cc:f0:98:49:22:07:02,58:cc:f0:98:49:23:07:02"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"Driver options"},{"line_number":49,"context_line":"~~~~~~~~~~~~~~"}],"source_content_type":"text/x-rst","patch_set":8,"id":"bf51134e_74ee01ee","line":46,"updated":"2020-07-20 21:41:45.000000000","message":"Is this optional? Does it support a wildcard to include all like \u0027*\u0027","commit_id":"23f51e7dc3f8fb3ff6cc56baee953bf70ad35eac"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"6eaac6c1cdac447d7790f3795e355fe84dea791f","unresolved":false,"context_lines":[{"line_number":43,"context_line":"      # PowerStore appliances"},{"line_number":44,"context_line":"      powerstore_appliances \u003d \u003cAppliances names\u003e # Ex. Appliance-1,Appliance-2"},{"line_number":45,"context_line":"      # PowerStore allowed ports"},{"line_number":46,"context_line":"      powerstore_ports \u003d \u003cAllowed ports\u003e # Ex. 58:cc:f0:98:49:22:07:02,58:cc:f0:98:49:23:07:02"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"Driver options"},{"line_number":49,"context_line":"~~~~~~~~~~~~~~"}],"source_content_type":"text/x-rst","patch_set":8,"id":"bf51134e_10b6a3dd","line":46,"in_reply_to":"bf51134e_708f1f2e","updated":"2020-07-21 15:36:37.000000000","message":"It would be good to at least note it in the help text.","commit_id":"23f51e7dc3f8fb3ff6cc56baee953bf70ad35eac"},{"author":{"_account_id":31016,"name":"Ivan Pchelintsev","email":"i.pchelintsev@yadro.com","username":"pcheli"},"change_message_id":"87145d0d8541539cd74996e289d754e7881d9491","unresolved":false,"context_lines":[{"line_number":43,"context_line":"      # PowerStore appliances"},{"line_number":44,"context_line":"      powerstore_appliances \u003d \u003cAppliances names\u003e # Ex. Appliance-1,Appliance-2"},{"line_number":45,"context_line":"      # PowerStore allowed ports"},{"line_number":46,"context_line":"      powerstore_ports \u003d \u003cAllowed ports\u003e # Ex. 58:cc:f0:98:49:22:07:02,58:cc:f0:98:49:23:07:02"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"Driver options"},{"line_number":49,"context_line":"~~~~~~~~~~~~~~"}],"source_content_type":"text/x-rst","patch_set":8,"id":"bf51134e_c0daa71b","line":46,"in_reply_to":"bf51134e_74ee01ee","updated":"2020-07-21 06:34:46.000000000","message":"Ports option is optional. If it is not set then all ports are allowed.\nRegular expressions might be supported later.","commit_id":"23f51e7dc3f8fb3ff6cc56baee953bf70ad35eac"},{"author":{"_account_id":10379,"name":"Rajini Karthik","email":"rrajini@gmail.com","username":"rajinir"},"change_message_id":"5bee7c1d02e6ad2d7930b6a9828b1b3800b2fbd5","unresolved":false,"context_lines":[{"line_number":43,"context_line":"      # PowerStore appliances"},{"line_number":44,"context_line":"      powerstore_appliances \u003d \u003cAppliances names\u003e # Ex. Appliance-1,Appliance-2"},{"line_number":45,"context_line":"      # PowerStore allowed ports"},{"line_number":46,"context_line":"      powerstore_ports \u003d \u003cAllowed ports\u003e # Ex. 58:cc:f0:98:49:22:07:02,58:cc:f0:98:49:23:07:02"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"Driver options"},{"line_number":49,"context_line":"~~~~~~~~~~~~~~"}],"source_content_type":"text/x-rst","patch_set":8,"id":"bf51134e_708f1f2e","line":46,"in_reply_to":"bf51134e_c0daa71b","updated":"2020-07-21 15:35:01.000000000","message":"Is there a way to differentiate between optional and required. I thought there was","commit_id":"23f51e7dc3f8fb3ff6cc56baee953bf70ad35eac"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"fac1539ff28069ac825836cea924bb37f8f4efaf","unresolved":false,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":"   .. code-block:: ini"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"      [DEFAULT]"},{"line_number":29,"context_line":"      enabled_backends \u003d powerstore"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"      [powerstore]"}],"source_content_type":"text/x-rst","patch_set":9,"id":"bf51134e_8339cfd9","line":28,"updated":"2020-07-21 15:52:53.000000000","message":"This code-block has extra indentation causing it to also be blockquoted in the generated HTML output. Need to shift the whole block left.","commit_id":"0e9130846a3fbe2ddbf86b2b6ac8543e9fa1fb47"},{"author":{"_account_id":31016,"name":"Ivan Pchelintsev","email":"i.pchelintsev@yadro.com","username":"pcheli"},"change_message_id":"31290b12cb844c0a637febe2c6eb6e5f6dd061b4","unresolved":false,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":"   .. code-block:: ini"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"      [DEFAULT]"},{"line_number":29,"context_line":"      enabled_backends \u003d powerstore"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"      [powerstore]"}],"source_content_type":"text/x-rst","patch_set":9,"id":"bf51134e_ea821000","line":28,"in_reply_to":"bf51134e_8339cfd9","updated":"2020-07-22 11:29:12.000000000","message":"Done","commit_id":"0e9130846a3fbe2ddbf86b2b6ac8543e9fa1fb47"}],"doc/source/reference/support-matrix.ini":[{"author":{"_account_id":18058,"name":"Lucio Seki","email":"lseki@redhat.com","username":"lseki"},"change_message_id":"e08017a27db2c7030eebd06f41f7e0592b890ce8","unresolved":false,"context_lines":[{"line_number":265,"context_line":"  an instance, but not all drivers are able to do this."},{"line_number":266,"context_line":"driver.datera\u003dcomplete"},{"line_number":267,"context_line":"driver.dell_emc_powermax\u003dcomplete"},{"line_number":268,"context_line":"driver.dell_emc_powerstore\u003dcomplete"},{"line_number":269,"context_line":"driver.dell_emc_sc\u003dcomplete"},{"line_number":270,"context_line":"driver.dell_emc_unity\u003dcomplete"},{"line_number":271,"context_line":"driver.dell_emc_vmax_af\u003dcomplete"}],"source_content_type":"text/x-properties","patch_set":11,"id":"bf51134e_553594a7","line":268,"updated":"2020-07-22 14:24:48.000000000","message":"The online extend tests are being skipped [0][1]. Please either:\n\n- set this feature as missing\n- edit tempest.conf to set `[volume-feature-enabled] extend_attached_volume \u003d True` and run the CI again.\n\n\n[0] http://publiclogs.emc.com/58/741158/11/check/DellEMC_PowerStore_iSCSI/f220676/DellEMC_PowerStore_iSCSI/22/console.html\n[1] http://publiclogs.emc.com/58/741158/11/check/DellEMC_PowerStore_FC/81d3428/DellEMC_PowerStore_FC/23/console.html","commit_id":"4e61c6beecc922280df6378f5d21591ddcef1e7d"}]}