)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"49cb5ce3eacedb6ae7dc726634b9186cf7de9e16","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b63bf6a4_27ca86b8","updated":"2021-11-05 17:35:25.000000000","message":"recheck","commit_id":"121917a42ee8a20c75f99cb5c2b0c821436a4101"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"8a6c22c4794bcff5c9ad09d8f8cb0af88c5da835","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"c303c716_611a9f81","updated":"2021-11-07 05:04:40.000000000","message":"I think it\u0027s ok (or better) to leave the pure_iscsi_cidr_list parameter now, and implement the parameter separately because the parameter was implemented during X cycle, and can\u0027t be backported to W.\n\nThe implementation looks almost good but I\u0027d leave some suggestions ","commit_id":"c04cca5f75e43be707f366bde947a681d9a8db1a"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"0ba80675841e0fc3371dc607f40e2cc5c45feac0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"3e8da1b7_27e42138","in_reply_to":"8b1a8509_7cecc095","updated":"2021-11-07 14:07:06.000000000","message":"Looks good to me. Thank you.","commit_id":"c04cca5f75e43be707f366bde947a681d9a8db1a"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"8213760fc04208d6e2733cbd666677a883fd5ea4","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"8b1a8509_7cecc095","in_reply_to":"c303c716_611a9f81","updated":"2021-11-07 05:07:05.000000000","message":"I\u0027ve fixed these items by my side because these are minor, but please feel free to revert the change I made, if you find anything wrong.","commit_id":"c04cca5f75e43be707f366bde947a681d9a8db1a"}],"manifests/backend/pure.pp":[{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"494b75be21c430efb726ac30e2de82c959f08023","unresolved":true,"context_lines":[{"line_number":54,"context_line":"#   (Optional) Determines how the Purity system treates deleted volumes."},{"line_number":55,"context_line":"#   Whether to immeadiately eradicate on delete or leave for auto-eradication"},{"line_number":56,"context_line":"#   in 24 hours"},{"line_number":57,"context_line":"#   Defaults to False"},{"line_number":58,"context_line":"#"},{"line_number":59,"context_line":"# [*pure_iscsi_cidr*]"},{"line_number":60,"context_line":"#   (Optional) Identifies which iSCSI network CIDR should be used for"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"fc0de015_1b1b260c","line":57,"updated":"2021-11-05 13:52:38.000000000","message":"I realize cinder itself default this to False, but unless there\u0027s a reason why puppet-cinder needs to specify a value then it\u0027s generally better to use $::os_service_default","commit_id":"74c1ab53c7c53806ec5a15f0baa32459026701d0"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"a8dc3df4d815c17a7ee924eb3ef693ec2e5d316f","unresolved":false,"context_lines":[{"line_number":54,"context_line":"#   (Optional) Determines how the Purity system treates deleted volumes."},{"line_number":55,"context_line":"#   Whether to immeadiately eradicate on delete or leave for auto-eradication"},{"line_number":56,"context_line":"#   in 24 hours"},{"line_number":57,"context_line":"#   Defaults to False"},{"line_number":58,"context_line":"#"},{"line_number":59,"context_line":"# [*pure_iscsi_cidr*]"},{"line_number":60,"context_line":"#   (Optional) Identifies which iSCSI network CIDR should be used for"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"5114d4d3_458052d7","line":57,"in_reply_to":"fc0de015_1b1b260c","updated":"2021-11-05 15:38:15.000000000","message":"Done","commit_id":"74c1ab53c7c53806ec5a15f0baa32459026701d0"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"494b75be21c430efb726ac30e2de82c959f08023","unresolved":true,"context_lines":[{"line_number":60,"context_line":"#   (Optional) Identifies which iSCSI network CIDR should be used for"},{"line_number":61,"context_line":"#   iscsi connections to the FlashArray is the array is configured with"},{"line_number":62,"context_line":"#   multiple iSCSI VLANs."},{"line_number":63,"context_line":"#   Defaults to \u00270.0.0.0/0\u0027"},{"line_number":64,"context_line":"#"},{"line_number":65,"context_line":"# [*extra_options*]"},{"line_number":66,"context_line":"#   (optional) Hash of extra options to pass to the backend stanza."}],"source_content_type":"text/x-puppet","patch_set":1,"id":"bd3d599f_2d689e95","line":63,"updated":"2021-11-05 13:52:38.000000000","message":"Two things:\n\n1. According to L80 the default is \u0027\u0027, not what you list here.\n2. As I mention above, why not default this to $::os_service_default","commit_id":"74c1ab53c7c53806ec5a15f0baa32459026701d0"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"a8dc3df4d815c17a7ee924eb3ef693ec2e5d316f","unresolved":false,"context_lines":[{"line_number":60,"context_line":"#   (Optional) Identifies which iSCSI network CIDR should be used for"},{"line_number":61,"context_line":"#   iscsi connections to the FlashArray is the array is configured with"},{"line_number":62,"context_line":"#   multiple iSCSI VLANs."},{"line_number":63,"context_line":"#   Defaults to \u00270.0.0.0/0\u0027"},{"line_number":64,"context_line":"#"},{"line_number":65,"context_line":"# [*extra_options*]"},{"line_number":66,"context_line":"#   (optional) Hash of extra options to pass to the backend stanza."}],"source_content_type":"text/x-puppet","patch_set":1,"id":"68cdfbe5_5f71d2f0","line":63,"in_reply_to":"bd3d599f_2d689e95","updated":"2021-11-05 15:38:15.000000000","message":"Done","commit_id":"74c1ab53c7c53806ec5a15f0baa32459026701d0"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"c558572abf3109af07db9c91a395cbb6bd75c5ec","unresolved":true,"context_lines":[{"line_number":58,"context_line":"#"},{"line_number":59,"context_line":"# [*pure_iscsi_cidr*]"},{"line_number":60,"context_line":"#   (Optional) Identifies which iSCSI network CIDR should be used for"},{"line_number":61,"context_line":"#   iscsi connections to the FlashArray is the array is configured with"},{"line_number":62,"context_line":"#   multiple iSCSI VLANs."},{"line_number":63,"context_line":"#   Defaults to $::os_service_default"},{"line_number":64,"context_line":"#"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"855539ab_97fb5f84","line":61,"range":{"start_line":61,"start_character":40,"end_line":61,"end_character":42},"updated":"2021-11-05 21:08:01.000000000","message":"nit: \"if\" (not \"is\")\n\nHowever, when I look at [1] it seems cinder\u0027s pure_iscsi_cidr is superseded by a new pure_iscsi_cidr_list option.\n\nAre you sure you want to add support for the old option, and not the new one?\n\n[1] https://opendev.org/openstack/cinder/src/branch/master/cinder/volume/drivers/pure.py#L93","commit_id":"121917a42ee8a20c75f99cb5c2b0c821436a4101"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"13b73a331a99ab47c47d9f014e9e1d9d0d1c5a71","unresolved":false,"context_lines":[{"line_number":58,"context_line":"#"},{"line_number":59,"context_line":"# [*pure_iscsi_cidr*]"},{"line_number":60,"context_line":"#   (Optional) Identifies which iSCSI network CIDR should be used for"},{"line_number":61,"context_line":"#   iscsi connections to the FlashArray is the array is configured with"},{"line_number":62,"context_line":"#   multiple iSCSI VLANs."},{"line_number":63,"context_line":"#   Defaults to $::os_service_default"},{"line_number":64,"context_line":"#"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"fb1cbd50_ea7dbfd9","line":61,"range":{"start_line":61,"start_character":40,"end_line":61,"end_character":42},"in_reply_to":"4cab6e5a_ef05a520","updated":"2021-11-07 19:27:22.000000000","message":"That makes sense, thanks.","commit_id":"121917a42ee8a20c75f99cb5c2b0c821436a4101"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"b2f17f1681b404316be230f3b9a07d5d97c4099c","unresolved":true,"context_lines":[{"line_number":58,"context_line":"#"},{"line_number":59,"context_line":"# [*pure_iscsi_cidr*]"},{"line_number":60,"context_line":"#   (Optional) Identifies which iSCSI network CIDR should be used for"},{"line_number":61,"context_line":"#   iscsi connections to the FlashArray is the array is configured with"},{"line_number":62,"context_line":"#   multiple iSCSI VLANs."},{"line_number":63,"context_line":"#   Defaults to $::os_service_default"},{"line_number":64,"context_line":"#"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"4cab6e5a_ef05a520","line":61,"range":{"start_line":61,"start_character":40,"end_line":61,"end_character":42},"in_reply_to":"855539ab_97fb5f84","updated":"2021-11-05 21:47:12.000000000","message":"I need to get this one supported first so that I can backport it to Wallaby ready for RHOSP17.\nWhen that is done I plan to do another patch to add the _list param","commit_id":"121917a42ee8a20c75f99cb5c2b0c821436a4101"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"c558572abf3109af07db9c91a395cbb6bd75c5ec","unresolved":true,"context_lines":[{"line_number":76,"context_line":"  $pure_storage_protocol        \u003d \u0027iSCSI\u0027,"},{"line_number":77,"context_line":"  $use_chap_auth                \u003d false,"},{"line_number":78,"context_line":"  $use_multipath_for_image_xfer \u003d true,"},{"line_number":79,"context_line":"  $pure_eradicate_on_delete     \u003d $::os_service_default,"},{"line_number":80,"context_line":"  $pure_iscsi_cidr              \u003d $::os_service_default,"},{"line_number":81,"context_line":"  $manage_volume_type           \u003d false,"},{"line_number":82,"context_line":"  $image_volume_cache_enabled   \u003d true,"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"fc91e3af_74459e8a","line":79,"updated":"2021-11-05 21:08:01.000000000","message":"not sure if this is a nit (other core reviewers may have a stronger opinion), but it would look cleaner if the \"pure_\" parameters were grouped together, and that the order in the documentation header matched the call order.","commit_id":"121917a42ee8a20c75f99cb5c2b0c821436a4101"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"b2f17f1681b404316be230f3b9a07d5d97c4099c","unresolved":false,"context_lines":[{"line_number":76,"context_line":"  $pure_storage_protocol        \u003d \u0027iSCSI\u0027,"},{"line_number":77,"context_line":"  $use_chap_auth                \u003d false,"},{"line_number":78,"context_line":"  $use_multipath_for_image_xfer \u003d true,"},{"line_number":79,"context_line":"  $pure_eradicate_on_delete     \u003d $::os_service_default,"},{"line_number":80,"context_line":"  $pure_iscsi_cidr              \u003d $::os_service_default,"},{"line_number":81,"context_line":"  $manage_volume_type           \u003d false,"},{"line_number":82,"context_line":"  $image_volume_cache_enabled   \u003d true,"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"a8ca2b14_a994a180","line":79,"in_reply_to":"fc91e3af_74459e8a","updated":"2021-11-05 21:47:12.000000000","message":"Put the params in the same order as the documentation","commit_id":"121917a42ee8a20c75f99cb5c2b0c821436a4101"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"8a6c22c4794bcff5c9ad09d8f8cb0af88c5da835","unresolved":true,"context_lines":[{"line_number":58,"context_line":"#"},{"line_number":59,"context_line":"# [*pure_iscsi_cidr*]"},{"line_number":60,"context_line":"#   (Optional) Identifies which iSCSI network CIDR should be used for"},{"line_number":61,"context_line":"#   iscsi connections to the FlashArray is the array is configured with"},{"line_number":62,"context_line":"#   multiple iSCSI VLANs."},{"line_number":63,"context_line":"#   Defaults to $::os_service_default"},{"line_number":64,"context_line":"#"}],"source_content_type":"text/x-puppet","patch_set":4,"id":"54aba457_1d8ba158","line":61,"range":{"start_line":61,"start_character":40,"end_line":61,"end_character":42},"updated":"2021-11-07 05:04:40.000000000","message":"if ?","commit_id":"c04cca5f75e43be707f366bde947a681d9a8db1a"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"8213760fc04208d6e2733cbd666677a883fd5ea4","unresolved":false,"context_lines":[{"line_number":58,"context_line":"#"},{"line_number":59,"context_line":"# [*pure_iscsi_cidr*]"},{"line_number":60,"context_line":"#   (Optional) Identifies which iSCSI network CIDR should be used for"},{"line_number":61,"context_line":"#   iscsi connections to the FlashArray is the array is configured with"},{"line_number":62,"context_line":"#   multiple iSCSI VLANs."},{"line_number":63,"context_line":"#   Defaults to $::os_service_default"},{"line_number":64,"context_line":"#"}],"source_content_type":"text/x-puppet","patch_set":4,"id":"cf9a3d87_f2b5c0b2","line":61,"range":{"start_line":61,"start_character":40,"end_line":61,"end_character":42},"in_reply_to":"54aba457_1d8ba158","updated":"2021-11-07 05:07:05.000000000","message":"Done","commit_id":"c04cca5f75e43be707f366bde947a681d9a8db1a"}],"releasenotes/notes/purestorage_updated_params-b0f474d9ab08c37e.yaml":[{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"494b75be21c430efb726ac30e2de82c959f08023","unresolved":true,"context_lines":[{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Add new parameter cinder::backend::pure::pure_eradicate_on_delete"},{"line_number":5,"context_line":"    Allows setting of the pure_eradicate_on_delete option on the managed"},{"line_number":6,"context_line":"    Cinder backend for PureStorage. The option defaults to False if unset."},{"line_number":7,"context_line":"  - |"},{"line_number":8,"context_line":"    Add new parameter cinder::backend::pure::pure_iscsi_cidr"},{"line_number":9,"context_line":"    Allows setting of the pure_iscsi_cidr option on the managed"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"80b22df4_7d619926","line":6,"range":{"start_line":6,"start_character":36,"end_line":6,"end_character":74},"updated":"2021-11-05 13:52:38.000000000","message":"You can probably skip listing the default values, especially if you let the cinder driver choose its own default by using $::os_service_default","commit_id":"74c1ab53c7c53806ec5a15f0baa32459026701d0"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"a8dc3df4d815c17a7ee924eb3ef693ec2e5d316f","unresolved":false,"context_lines":[{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Add new parameter cinder::backend::pure::pure_eradicate_on_delete"},{"line_number":5,"context_line":"    Allows setting of the pure_eradicate_on_delete option on the managed"},{"line_number":6,"context_line":"    Cinder backend for PureStorage. The option defaults to False if unset."},{"line_number":7,"context_line":"  - |"},{"line_number":8,"context_line":"    Add new parameter cinder::backend::pure::pure_iscsi_cidr"},{"line_number":9,"context_line":"    Allows setting of the pure_iscsi_cidr option on the managed"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"4c83d03c_0e53b4de","line":6,"range":{"start_line":6,"start_character":36,"end_line":6,"end_character":74},"in_reply_to":"80b22df4_7d619926","updated":"2021-11-05 15:38:15.000000000","message":"Done","commit_id":"74c1ab53c7c53806ec5a15f0baa32459026701d0"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"8a6c22c4794bcff5c9ad09d8f8cb0af88c5da835","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Add new parameter cinder::backend::pure::pure_eradicate_on_delete"},{"line_number":5,"context_line":"    Allows setting of the pure_eradicate_on_delete option on the managed"},{"line_number":6,"context_line":"    Cinder backend for PureStorage."},{"line_number":7,"context_line":"  - |"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"86ac065f_01cff5e8","line":4,"range":{"start_line":4,"start_character":22,"end_line":4,"end_character":69},"updated":"2021-11-07 05:04:40.000000000","message":"would be better to quote this by ``...`` like ``cinder::backend::pure::pure_eradicate_on_delete``","commit_id":"c04cca5f75e43be707f366bde947a681d9a8db1a"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"8213760fc04208d6e2733cbd666677a883fd5ea4","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Add new parameter cinder::backend::pure::pure_eradicate_on_delete"},{"line_number":5,"context_line":"    Allows setting of the pure_eradicate_on_delete option on the managed"},{"line_number":6,"context_line":"    Cinder backend for PureStorage."},{"line_number":7,"context_line":"  - |"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"ccc84ca4_ef1ee337","line":4,"range":{"start_line":4,"start_character":22,"end_line":4,"end_character":69},"in_reply_to":"86ac065f_01cff5e8","updated":"2021-11-07 05:07:05.000000000","message":"Done","commit_id":"c04cca5f75e43be707f366bde947a681d9a8db1a"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"8a6c22c4794bcff5c9ad09d8f8cb0af88c5da835","unresolved":true,"context_lines":[{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Add new parameter cinder::backend::pure::pure_eradicate_on_delete"},{"line_number":5,"context_line":"    Allows setting of the pure_eradicate_on_delete option on the managed"},{"line_number":6,"context_line":"    Cinder backend for PureStorage."},{"line_number":7,"context_line":"  - |"},{"line_number":8,"context_line":"    Add new parameter cinder::backend::pure::pure_iscsi_cidr"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"19880c76_51442262","line":5,"range":{"start_line":5,"start_character":4,"end_line":5,"end_character":10},"updated":"2021-11-07 05:04:40.000000000","message":"allows\n\nThis line is squashed into the above line when being rendered to a relrease note doc.","commit_id":"c04cca5f75e43be707f366bde947a681d9a8db1a"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"8213760fc04208d6e2733cbd666677a883fd5ea4","unresolved":false,"context_lines":[{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Add new parameter cinder::backend::pure::pure_eradicate_on_delete"},{"line_number":5,"context_line":"    Allows setting of the pure_eradicate_on_delete option on the managed"},{"line_number":6,"context_line":"    Cinder backend for PureStorage."},{"line_number":7,"context_line":"  - |"},{"line_number":8,"context_line":"    Add new parameter cinder::backend::pure::pure_iscsi_cidr"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"fe8966cb_22bf7ca7","line":5,"range":{"start_line":5,"start_character":4,"end_line":5,"end_character":10},"in_reply_to":"19880c76_51442262","updated":"2021-11-07 05:07:05.000000000","message":"Done","commit_id":"c04cca5f75e43be707f366bde947a681d9a8db1a"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"8a6c22c4794bcff5c9ad09d8f8cb0af88c5da835","unresolved":true,"context_lines":[{"line_number":5,"context_line":"    Allows setting of the pure_eradicate_on_delete option on the managed"},{"line_number":6,"context_line":"    Cinder backend for PureStorage."},{"line_number":7,"context_line":"  - |"},{"line_number":8,"context_line":"    Add new parameter cinder::backend::pure::pure_iscsi_cidr"},{"line_number":9,"context_line":"    Allows setting of the pure_iscsi_cidr option on the managed"},{"line_number":10,"context_line":"    Cinder backend for PureStorage."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"0470a717_f021b4da","line":8,"range":{"start_line":8,"start_character":22,"end_line":8,"end_character":60},"updated":"2021-11-07 05:04:40.000000000","message":"ditto","commit_id":"c04cca5f75e43be707f366bde947a681d9a8db1a"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"8213760fc04208d6e2733cbd666677a883fd5ea4","unresolved":false,"context_lines":[{"line_number":5,"context_line":"    Allows setting of the pure_eradicate_on_delete option on the managed"},{"line_number":6,"context_line":"    Cinder backend for PureStorage."},{"line_number":7,"context_line":"  - |"},{"line_number":8,"context_line":"    Add new parameter cinder::backend::pure::pure_iscsi_cidr"},{"line_number":9,"context_line":"    Allows setting of the pure_iscsi_cidr option on the managed"},{"line_number":10,"context_line":"    Cinder backend for PureStorage."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"0e97cd48_eb11e4ca","line":8,"range":{"start_line":8,"start_character":22,"end_line":8,"end_character":60},"in_reply_to":"0470a717_f021b4da","updated":"2021-11-07 05:07:05.000000000","message":"Done","commit_id":"c04cca5f75e43be707f366bde947a681d9a8db1a"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"8a6c22c4794bcff5c9ad09d8f8cb0af88c5da835","unresolved":true,"context_lines":[{"line_number":6,"context_line":"    Cinder backend for PureStorage."},{"line_number":7,"context_line":"  - |"},{"line_number":8,"context_line":"    Add new parameter cinder::backend::pure::pure_iscsi_cidr"},{"line_number":9,"context_line":"    Allows setting of the pure_iscsi_cidr option on the managed"},{"line_number":10,"context_line":"    Cinder backend for PureStorage."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"74907291_35d3ca16","line":9,"range":{"start_line":9,"start_character":4,"end_line":9,"end_character":10},"updated":"2021-11-07 05:04:40.000000000","message":"ditto","commit_id":"c04cca5f75e43be707f366bde947a681d9a8db1a"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"8213760fc04208d6e2733cbd666677a883fd5ea4","unresolved":false,"context_lines":[{"line_number":6,"context_line":"    Cinder backend for PureStorage."},{"line_number":7,"context_line":"  - |"},{"line_number":8,"context_line":"    Add new parameter cinder::backend::pure::pure_iscsi_cidr"},{"line_number":9,"context_line":"    Allows setting of the pure_iscsi_cidr option on the managed"},{"line_number":10,"context_line":"    Cinder backend for PureStorage."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3693c1b3_eae93499","line":9,"range":{"start_line":9,"start_character":4,"end_line":9,"end_character":10},"in_reply_to":"74907291_35d3ca16","updated":"2021-11-07 05:07:05.000000000","message":"Done","commit_id":"c04cca5f75e43be707f366bde947a681d9a8db1a"}]}