)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":31981,"name":"Tarun Mathur","email":"tarun.mathur@dell.com"},"change_message_id":"6fa573e6b634a07fee6071143fb4d68ccc865d9c","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Helen Walsh \u003chelen.walsh@emc.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2020-05-13 14:46:20 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"PowerMax Driver - Usurri Documentation"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This change addresses documentation updates required"},{"line_number":10,"context_line":"for the Usurri release."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"ff570b3c_4ad5c3e4","line":7,"range":{"start_line":7,"start_character":18,"end_line":7,"end_character":24},"updated":"2020-05-15 12:13:02.000000000","message":"Ussuri","commit_id":"95b15eb3cb21d5be30f6762c7c93866fa426c3b4"},{"author":{"_account_id":31981,"name":"Tarun Mathur","email":"tarun.mathur@dell.com"},"change_message_id":"6fa573e6b634a07fee6071143fb4d68ccc865d9c","unresolved":false,"context_lines":[{"line_number":7,"context_line":"PowerMax Driver - Usurri Documentation"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This change addresses documentation updates required"},{"line_number":10,"context_line":"for the Usurri release."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I6602fbac143eb59c654010dd0e96af5ec066a112"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"ff570b3c_aa073f82","line":10,"range":{"start_line":10,"start_character":8,"end_line":10,"end_character":14},"updated":"2020-05-15 12:13:02.000000000","message":"Ussuri","commit_id":"95b15eb3cb21d5be30f6762c7c93866fa426c3b4"}],"doc/source/configuration/block-storage/drivers/dell-emc-powermax-driver.rst":[{"author":{"_account_id":10379,"name":"Rajini Karthik","email":"rrajini@gmail.com","username":"rajinir"},"change_message_id":"142b514bfe5d753359a019137b80ef33c9e239d6","unresolved":false,"context_lines":[{"line_number":35,"context_line":"Download Solutions Enabler and Unisphere from the Dell EMC\u0027s support web site"},{"line_number":36,"context_line":"(login is required). See the ``Dell EMC Solutions Enabler 9.1.x Installation"},{"line_number":37,"context_line":"and Configuration Guide`` and ``Dell EMC Unisphere for PowerMax Installation"},{"line_number":38,"context_line":"Guide`` at the `Dell Support`_ site."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":".. note::"},{"line_number":41,"context_line":""}],"source_content_type":"text/x-rst","patch_set":7,"id":"ff570b3c_38d637c7","line":38,"updated":"2020-05-12 14:44:20.000000000","message":"\u0027Dell EMC Support\u0027","commit_id":"cc00cb72c6abc35f62c491d90a6b88aa5197d809"},{"author":{"_account_id":10379,"name":"Rajini Karthik","email":"rrajini@gmail.com","username":"rajinir"},"change_message_id":"142b514bfe5d753359a019137b80ef33c9e239d6","unresolved":false,"context_lines":[{"line_number":109,"context_line":"------------------"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"To activate your entitlements and obtain your PowerMax license files, visit the"},{"line_number":112,"context_line":"Service Center on `Dell Support`_, as directed on your License Authorization"},{"line_number":113,"context_line":"Code (LAC) letter emailed to you."},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"-  For help with missing or incorrect entitlements after activation"}],"source_content_type":"text/x-rst","patch_set":7,"id":"ff570b3c_98dae3e6","line":112,"updated":"2020-05-12 14:44:20.000000000","message":"Ditto","commit_id":"cc00cb72c6abc35f62c491d90a6b88aa5197d809"},{"author":{"_account_id":10379,"name":"Rajini Karthik","email":"rrajini@gmail.com","username":"rajinir"},"change_message_id":"142b514bfe5d753359a019137b80ef33c9e239d6","unresolved":false,"context_lines":[{"line_number":133,"context_line":"----------------------------------------------"},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"If you require help or assistance with PowerMax and Cinder please open a"},{"line_number":136,"context_line":"Service Request (SR) through standard support channels at `Dell Support`_."},{"line_number":137,"context_line":"When opening a SR please include the following information:"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"- Array Model \u0026 uCode level"}],"source_content_type":"text/x-rst","patch_set":7,"id":"ff570b3c_b8e1a79d","line":136,"updated":"2020-05-12 14:44:20.000000000","message":"Ditto everywhere on Dell EMC Support","commit_id":"cc00cb72c6abc35f62c491d90a6b88aa5197d809"},{"author":{"_account_id":31981,"name":"Tarun Mathur","email":"tarun.mathur@dell.com"},"change_message_id":"6fa573e6b634a07fee6071143fb4d68ccc865d9c","unresolved":false,"context_lines":[{"line_number":1304,"context_line":"Live migration configuration"},{"line_number":1305,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~~~"},{"line_number":1306,"context_line":""},{"line_number":1307,"context_line":"Please refer to the official Openstack documentation on"},{"line_number":1308,"context_line":"`configuring migrations`_ and `live migration usage`_ for more information."},{"line_number":1309,"context_line":""},{"line_number":1310,"context_line":".. note::"}],"source_content_type":"text/x-rst","patch_set":8,"id":"ff570b3c_782fc4a0","line":1307,"range":{"start_line":1307,"start_character":29,"end_line":1307,"end_character":38},"updated":"2020-05-15 12:13:02.000000000","message":"Should the S in Openstack be capitalized?","commit_id":"95b15eb3cb21d5be30f6762c7c93866fa426c3b4"},{"author":{"_account_id":31981,"name":"Tarun Mathur","email":"tarun.mathur@dell.com"},"change_message_id":"6fa573e6b634a07fee6071143fb4d68ccc865d9c","unresolved":false,"context_lines":[{"line_number":1373,"context_line":"~~~~~~~~"},{"line_number":1374,"context_line":""},{"line_number":1375,"context_line":"For our use case shown below, we have three hosts with host names ``HostA``,"},{"line_number":1376,"context_line":"``HostB`` and ``HostC``. ``HostA`` is the compute node while ``HostB`` and"},{"line_number":1377,"context_line":"``HostC`` are the compute nodes. The following were also used in live"},{"line_number":1378,"context_line":"migration."},{"line_number":1379,"context_line":""},{"line_number":1380,"context_line":"- 2GB bootable volume using the CirrOS image."}],"source_content_type":"text/x-rst","patch_set":8,"id":"ff570b3c_d817f050","line":1377,"range":{"start_line":1376,"start_character":25,"end_line":1377,"end_character":31},"updated":"2020-05-15 12:13:02.000000000","message":"All hosts are documented as compute nodes","commit_id":"95b15eb3cb21d5be30f6762c7c93866fa426c3b4"},{"author":{"_account_id":31981,"name":"Tarun Mathur","email":"tarun.mathur@dell.com"},"change_message_id":"6fa573e6b634a07fee6071143fb4d68ccc865d9c","unresolved":false,"context_lines":[{"line_number":1643,"context_line":""},{"line_number":1644,"context_line":".. note::"},{"line_number":1645,"context_line":""},{"line_number":1646,"context_line":"   To add more than one Unisphere fai-lover target create additional"},{"line_number":1647,"context_line":"   ``u4p_failover_target`` details for the Unisphere instance. These will be"},{"line_number":1648,"context_line":"   cycled through in a first-in, first-out (FIFO) basis, the first fail-over"},{"line_number":1649,"context_line":"   target in ``cinder.conf`` will be the first backup instance of Unisphere"}],"source_content_type":"text/x-rst","patch_set":8,"id":"ff570b3c_aa29ff8a","line":1646,"range":{"start_line":1646,"start_character":34,"end_line":1646,"end_character":43},"updated":"2020-05-15 12:13:02.000000000","message":"The hyphen is between i and l instead of being between l and o","commit_id":"95b15eb3cb21d5be30f6762c7c93866fa426c3b4"},{"author":{"_account_id":31981,"name":"Tarun Mathur","email":"tarun.mathur@dell.com"},"change_message_id":"6fa573e6b634a07fee6071143fb4d68ccc865d9c","unresolved":false,"context_lines":[{"line_number":1718,"context_line":"- Tags can be modified on the backend stanza, but none will ever be removed,"},{"line_number":1719,"context_line":"  only added."},{"line_number":1720,"context_line":"- There is no restriction on creating or deleting tags of OpenStack storage"},{"line_number":1721,"context_line":"  groups or array outside of OpenStack e.g. Unisphere for PowerMax UI.  The"},{"line_number":1722,"context_line":"  max number of 8 tags will apply however, as this is a Unisphere for"},{"line_number":1723,"context_line":"  PowerMax limit."},{"line_number":1724,"context_line":""}],"source_content_type":"text/x-rst","patch_set":8,"id":"ff570b3c_ca80b349","line":1721,"range":{"start_line":1721,"start_character":12,"end_line":1721,"end_character":17},"updated":"2020-05-15 12:13:02.000000000","message":"Should this be changed to plural - arrays?","commit_id":"95b15eb3cb21d5be30f6762c7c93866fa426c3b4"},{"author":{"_account_id":31981,"name":"Tarun Mathur","email":"tarun.mathur@dell.com"},"change_message_id":"6fa573e6b634a07fee6071143fb4d68ccc865d9c","unresolved":false,"context_lines":[{"line_number":1772,"context_line":"- Only the short host name and port group name components can be renamed"},{"line_number":1773,"context_line":"  within the storage group, initiator group and masking view names."},{"line_number":1774,"context_line":"- If the ``powermax_short_host_name_template`` and"},{"line_number":1775,"context_line":"  ``powermax_port_group_name_template``, do not adhere to the rules, then"},{"line_number":1776,"context_line":"  the operation will fail early and gracefully with a clear description as"},{"line_number":1777,"context_line":"  to the problem."},{"line_number":1778,"context_line":"- The templates cannot be changed once volumes have been attached using the"}],"source_content_type":"text/x-rst","patch_set":8,"id":"ff570b3c_4a4f8333","line":1775,"range":{"start_line":1775,"start_character":39,"end_line":1775,"end_character":40},"updated":"2020-05-15 12:13:02.000000000","message":"Seems redundant","commit_id":"95b15eb3cb21d5be30f6762c7c93866fa426c3b4"},{"author":{"_account_id":31981,"name":"Tarun Mathur","email":"tarun.mathur@dell.com"},"change_message_id":"6fa573e6b634a07fee6071143fb4d68ccc865d9c","unresolved":false,"context_lines":[{"line_number":1993,"context_line":""},{"line_number":1994,"context_line":"Setting multiple replication devices in cinder.conf allows the use of all the"},{"line_number":1995,"context_line":"supported replication modes simultaneously. Up to three replication devices"},{"line_number":1996,"context_line":"can be set, one for each of the replication modes available. An additional"},{"line_number":1997,"context_line":"volume type extra specification is then used to determine which replication"},{"line_number":1998,"context_line":"device should be utilized when attempting to perform an operation on a"},{"line_number":1999,"context_line":"volume which is replication enabled. All details, guidelines and"},{"line_number":2000,"context_line":"recommendations set out in the `Configure a single replication target`_"}],"source_content_type":"text/x-rst","patch_set":8,"id":"ff570b3c_0a20eb47","line":1997,"range":{"start_line":1996,"start_character":61,"end_line":1997,"end_character":31},"updated":"2020-05-15 12:13:02.000000000","message":"Is the \u0027extra\u0027 redundant?","commit_id":"95b15eb3cb21d5be30f6762c7c93866fa426c3b4"},{"author":{"_account_id":31981,"name":"Tarun Mathur","email":"tarun.mathur@dell.com"},"change_message_id":"6fa573e6b634a07fee6071143fb4d68ccc865d9c","unresolved":false,"context_lines":[{"line_number":2111,"context_line":""},{"line_number":2112,"context_line":".. note::"},{"line_number":2113,"context_line":""},{"line_number":2114,"context_line":"    In a legacy system where volume types are present that were replication"},{"line_number":2115,"context_line":"    enabled before adding multiple replication devices. The"},{"line_number":2116,"context_line":"    ``storagetype:replication_device_backend_id`` should be omitted from any"},{"line_number":2117,"context_line":"    volume type that does/will use the legacy ``replication_device`` i.e."},{"line_number":2118,"context_line":"    when ``storagetype:replication_device_backend_id`` is omitted the"},{"line_number":2119,"context_line":"    replication_device with a ``backend_id`` of ``backend_id_legacy_rep``"},{"line_number":2120,"context_line":"    will be used."}],"source_content_type":"text/x-rst","patch_set":8,"id":"ff570b3c_8ac23b58","line":2117,"range":{"start_line":2114,"start_character":4,"end_line":2117,"end_character":68},"updated":"2020-05-15 12:13:02.000000000","message":"Should this be a single sentence?","commit_id":"95b15eb3cb21d5be30f6762c7c93866fa426c3b4"},{"author":{"_account_id":31981,"name":"Tarun Mathur","email":"tarun.mathur@dell.com"},"change_message_id":"6fa573e6b634a07fee6071143fb4d68ccc865d9c","unresolved":false,"context_lines":[{"line_number":2127,"context_line":"* Replication Group operations are available for volumes in Synchronous mode"},{"line_number":2128,"context_line":"  only."},{"line_number":2129,"context_line":""},{"line_number":2130,"context_line":"* The Usurri release of OpenStack supports retyping in-use volumes to and from"},{"line_number":2131,"context_line":"  replication enabled volume types with limited exception of volumes with"},{"line_number":2132,"context_line":"  Metro replication enabled. To retype to a volume-type that is Metro enabled"},{"line_number":2133,"context_line":"  the volume **must** first be detached then retyped. The reason for this is"}],"source_content_type":"text/x-rst","patch_set":8,"id":"ff570b3c_caad539c","line":2130,"range":{"start_line":2130,"start_character":6,"end_line":2130,"end_character":12},"updated":"2020-05-15 12:13:02.000000000","message":"Need two s\u0027s instead of two r\u0027s","commit_id":"95b15eb3cb21d5be30f6762c7c93866fa426c3b4"},{"author":{"_account_id":31981,"name":"Tarun Mathur","email":"tarun.mathur@dell.com"},"change_message_id":"6fa573e6b634a07fee6071143fb4d68ccc865d9c","unresolved":false,"context_lines":[{"line_number":2250,"context_line":""},{"line_number":2251,"context_line":".. note::"},{"line_number":2252,"context_line":""},{"line_number":2253,"context_line":"   The Usurri release of OpenStack supports retyping in-use volumes to and from"},{"line_number":2254,"context_line":"   replication enabled volume types with limited exception of volumes with"},{"line_number":2255,"context_line":"   Metro replication enabled. To retype to a volume-type that is Metro enabled"},{"line_number":2256,"context_line":"   the volume **must** first be detached then retyped. The reason for this is"}],"source_content_type":"text/x-rst","patch_set":8,"id":"ff570b3c_8a951b36","line":2253,"range":{"start_line":2253,"start_character":7,"end_line":2253,"end_character":13},"updated":"2020-05-15 12:13:02.000000000","message":"Ussuri","commit_id":"95b15eb3cb21d5be30f6762c7c93866fa426c3b4"},{"author":{"_account_id":31981,"name":"Tarun Mathur","email":"tarun.mathur@dell.com"},"change_message_id":"6fa573e6b634a07fee6071143fb4d68ccc865d9c","unresolved":false,"context_lines":[{"line_number":2714,"context_line":"Cinder backup support"},{"line_number":2715,"context_line":"---------------------"},{"line_number":2716,"context_line":""},{"line_number":2717,"context_line":"PowerMax Cinder drivers support Cinder backup functionality. For further"},{"line_number":2718,"context_line":"information on setup, configuration, and usage please see the official"},{"line_number":2719,"context_line":"OpenStack `volume backup`_ documentation and related `volume backup CLI`_"},{"line_number":2720,"context_line":"guide."}],"source_content_type":"text/x-rst","patch_set":8,"id":"ff570b3c_8ae35bd9","line":2717,"range":{"start_line":2717,"start_character":9,"end_line":2717,"end_character":23},"updated":"2020-05-15 12:13:02.000000000","message":"Shouldn\u0027t this be singular?","commit_id":"95b15eb3cb21d5be30f6762c7c93866fa426c3b4"},{"author":{"_account_id":31981,"name":"Tarun Mathur","email":"tarun.mathur@dell.com"},"change_message_id":"6fa573e6b634a07fee6071143fb4d68ccc865d9c","unresolved":false,"context_lines":[{"line_number":2715,"context_line":"---------------------"},{"line_number":2716,"context_line":""},{"line_number":2717,"context_line":"PowerMax Cinder drivers support Cinder backup functionality. For further"},{"line_number":2718,"context_line":"information on setup, configuration, and usage please see the official"},{"line_number":2719,"context_line":"OpenStack `volume backup`_ documentation and related `volume backup CLI`_"},{"line_number":2720,"context_line":"guide."},{"line_number":2721,"context_line":""}],"source_content_type":"text/x-rst","patch_set":8,"id":"ff570b3c_eae497e2","line":2718,"range":{"start_line":2718,"start_character":35,"end_line":2718,"end_character":36},"updated":"2020-05-15 12:13:02.000000000","message":"redundant comma","commit_id":"95b15eb3cb21d5be30f6762c7c93866fa426c3b4"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d4e55b8444c1faf9da93fd97c8f239d45cb5f88f","unresolved":false,"context_lines":[{"line_number":47,"context_line":"   +-----------+------------------------+-------------+"},{"line_number":48,"context_line":"   | OpenStack | Unisphere for PowerMax | PowerMax OS |"},{"line_number":49,"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+"},{"line_number":50,"context_line":"   | Usurri    | 9.1.x                  | 5978.479    |"},{"line_number":51,"context_line":"   +-----------+------------------------+-------------+"},{"line_number":52,"context_line":"   | Train     | 9.1.x                  | 5978.444    |"},{"line_number":53,"context_line":"   +-----------+------------------------+-------------+"}],"source_content_type":"text/x-rst","patch_set":10,"id":"bf51134e_23a9b5de","line":50,"range":{"start_line":50,"start_character":5,"end_line":50,"end_character":11},"updated":"2020-06-19 03:11:00.000000000","message":"Ussuri","commit_id":"3cc08543a3df3bf32d9ade173ea856f807276b2d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d4e55b8444c1faf9da93fd97c8f239d45cb5f88f","unresolved":false,"context_lines":[{"line_number":278,"context_line":"is then added. Storage groups contain volumes created from a pool, attached"},{"line_number":279,"context_line":"to a single host, over a single connection type (iSCSI or FC). ``[protocol]``"},{"line_number":280,"context_line":"is either ``I`` for volumes attached over iSCSI or ``F`` for volumes attached"},{"line_number":281,"context_line":"over Fiber Channel. PowerMax Cinder driver utilizes cascaded storage groups -"},{"line_number":282,"context_line":"a ``parent`` storage group which is associated with the masking view, which"},{"line_number":283,"context_line":"contains ``child`` storage groups for each configured"},{"line_number":284,"context_line":"SRP/slo/workload/compression-enabled or disabled/replication-enabled or"}],"source_content_type":"text/x-rst","patch_set":10,"id":"bf51134e_0385915a","line":281,"range":{"start_line":281,"start_character":5,"end_line":281,"end_character":10},"updated":"2020-06-19 03:11:00.000000000","message":"not your change, but I believe this should be \u0027Fibre\u0027","commit_id":"3cc08543a3df3bf32d9ade173ea856f807276b2d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d4e55b8444c1faf9da93fd97c8f239d45cb5f88f","unresolved":false,"context_lines":[{"line_number":330,"context_line":"   .. note::"},{"line_number":331,"context_line":""},{"line_number":332,"context_line":"      You must discover storage arrays before you can use the PowerMax drivers."},{"line_number":333,"context_line":"      Follow instructions in ``Dell EMC Solutions Enabler 9.1.x Installation"},{"line_number":334,"context_line":"      and Configuration Guide`` on `Dell EMC Support`_ for more details."},{"line_number":335,"context_line":""},{"line_number":336,"context_line":"#. Download Unisphere from `Dell EMC Support`_ and install it."}],"source_content_type":"text/x-rst","patch_set":10,"id":"bf51134e_e3ecdd2a","line":333,"range":{"start_line":333,"start_character":29,"end_line":333,"end_character":31},"updated":"2020-06-19 03:11:00.000000000","message":"This is going to render in monospace font.  You probably want italics here (i.e., use * instead of ``).  You can do that on a follow-up if you want, looks like `` is used a lot for documentation section titles in this document.","commit_id":"3cc08543a3df3bf32d9ade173ea856f807276b2d"},{"author":{"_account_id":12670,"name":"Helen Walsh","email":"helen.walsh@emc.com","username":"walshh2"},"change_message_id":"2b0748c0f7c7528cad60100465447746569603ed","unresolved":false,"context_lines":[{"line_number":330,"context_line":"   .. note::"},{"line_number":331,"context_line":""},{"line_number":332,"context_line":"      You must discover storage arrays before you can use the PowerMax drivers."},{"line_number":333,"context_line":"      Follow instructions in ``Dell EMC Solutions Enabler 9.1.x Installation"},{"line_number":334,"context_line":"      and Configuration Guide`` on `Dell EMC Support`_ for more details."},{"line_number":335,"context_line":""},{"line_number":336,"context_line":"#. Download Unisphere from `Dell EMC Support`_ and install it."}],"source_content_type":"text/x-rst","patch_set":10,"id":"bf51134e_685b8453","line":333,"range":{"start_line":333,"start_character":29,"end_line":333,"end_character":31},"in_reply_to":"bf51134e_e3ecdd2a","updated":"2020-06-19 12:16:51.000000000","message":"Might follow up with this one, we do reference docs a lot","commit_id":"3cc08543a3df3bf32d9ade173ea856f807276b2d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d4e55b8444c1faf9da93fd97c8f239d45cb5f88f","unresolved":false,"context_lines":[{"line_number":403,"context_line":""},{"line_number":404,"context_line":"   Service Level can be added to ``cinder.conf`` when the backend is the"},{"line_number":405,"context_line":"   default case and there is no associated volume type. This not a recommended"},{"line_number":406,"context_line":"   configuration as it is too restrictive. Workload is ``NONE`` for PowerMax"},{"line_number":407,"context_line":"   and any All Flash with PowerMax OS (5978) or greater."},{"line_number":408,"context_line":""},{"line_number":409,"context_line":"   +--------------------+----------------------------+----------+----------+"}],"source_content_type":"text/x-rst","patch_set":10,"id":"bf51134e_83d08121","line":406,"range":{"start_line":406,"start_character":57,"end_line":406,"end_character":61},"updated":"2020-06-19 03:11:00.000000000","message":"You should be consistent about this.  It\u0027s NONE here, but None at line 412.  (Happens several places in this file.)","commit_id":"3cc08543a3df3bf32d9ade173ea856f807276b2d"},{"author":{"_account_id":12670,"name":"Helen Walsh","email":"helen.walsh@emc.com","username":"walshh2"},"change_message_id":"9aa140a20607b7c342847859bc77f4c748471bf2","unresolved":false,"context_lines":[{"line_number":403,"context_line":""},{"line_number":404,"context_line":"   Service Level can be added to ``cinder.conf`` when the backend is the"},{"line_number":405,"context_line":"   default case and there is no associated volume type. This not a recommended"},{"line_number":406,"context_line":"   configuration as it is too restrictive. Workload is ``NONE`` for PowerMax"},{"line_number":407,"context_line":"   and any All Flash with PowerMax OS (5978) or greater."},{"line_number":408,"context_line":""},{"line_number":409,"context_line":"   +--------------------+----------------------------+----------+----------+"}],"source_content_type":"text/x-rst","patch_set":10,"id":"bf51134e_6846e423","line":406,"range":{"start_line":406,"start_character":57,"end_line":406,"end_character":61},"in_reply_to":"bf51134e_83d08121","updated":"2020-06-19 12:13:32.000000000","message":"Workloads are NONE in unisphere, as opposed to None everywhere else.  It is a little confusing alright.","commit_id":"3cc08543a3df3bf32d9ade173ea856f807276b2d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d4e55b8444c1faf9da93fd97c8f239d45cb5f88f","unresolved":false,"context_lines":[{"line_number":413,"context_line":"   +--------------------+----------------------------+----------+----------+"},{"line_number":414,"context_line":""},{"line_number":415,"context_line":""},{"line_number":416,"context_line":"To configure PowerMax block storage Add the following entries to"},{"line_number":417,"context_line":"``/etc/cinder/cinder.conf``:"},{"line_number":418,"context_line":""},{"line_number":419,"context_line":".. code-block:: ini"}],"source_content_type":"text/x-rst","patch_set":10,"id":"bf51134e_63cbad71","line":416,"range":{"start_line":416,"start_character":35,"end_line":416,"end_character":38},"updated":"2020-06-19 03:11:00.000000000","message":"s/ A/, a/","commit_id":"3cc08543a3df3bf32d9ade173ea856f807276b2d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d4e55b8444c1faf9da93fd97c8f239d45cb5f88f","unresolved":false,"context_lines":[{"line_number":725,"context_line":"tiered level of service based on cost. The Nova/Cinder QoS offer similar"},{"line_number":726,"context_line":"functionality based on volume type setting limits on host storage bandwidth"},{"line_number":727,"context_line":"per service offering. Each volume type is tied to specific QoS attributes"},{"line_number":728,"context_line":"some of which are unique to each storage vendor. In the hyper-visor, the QoS"},{"line_number":729,"context_line":"limits the following:"},{"line_number":730,"context_line":""},{"line_number":731,"context_line":"- Limit by throughput - Total bytes/sec, read bytes/sec, write bytes/sec"}],"source_content_type":"text/x-rst","patch_set":10,"id":"bf51134e_03b911cb","line":728,"range":{"start_line":728,"start_character":56,"end_line":728,"end_character":67},"updated":"2020-06-19 03:11:00.000000000","message":"I think the usual way to spell this is without the hyphen, i.e., \u0027hypervisor\u0027 (analogous with \u0027supervisor\u0027).","commit_id":"3cc08543a3df3bf32d9ade173ea856f807276b2d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d4e55b8444c1faf9da93fd97c8f239d45cb5f88f","unresolved":false,"context_lines":[{"line_number":742,"context_line":"end/back end agnostic"},{"line_number":743,"context_line":""},{"line_number":744,"context_line":"- ``total_iops_sec`` - Maximum IOPs (in I/Os per second). Valid values range"},{"line_number":745,"context_line":"  from 100 IO/Sec to 100,000 IO/sec."},{"line_number":746,"context_line":"- ``total_bytes_sec`` - Maximum bandwidth (throughput) in bytes per second."},{"line_number":747,"context_line":"  Valid values range from 1048576 bytes (1MB) to 104857600000 bytes (100,000MB)"},{"line_number":748,"context_line":""}],"source_content_type":"text/x-rst","patch_set":10,"id":"bf51134e_23b6d5da","line":745,"range":{"start_line":745,"start_character":24,"end_line":745,"end_character":25},"updated":"2020-06-19 03:11:00.000000000","message":"do you really allow a comma in the value?","commit_id":"3cc08543a3df3bf32d9ade173ea856f807276b2d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d4e55b8444c1faf9da93fd97c8f239d45cb5f88f","unresolved":false,"context_lines":[{"line_number":838,"context_line":" +-----------------------+-----------------------+"},{"line_number":839,"context_line":""},{"line_number":840,"context_line":"#. Create QoS specifications with the prerequisite values above. The consumer"},{"line_number":841,"context_line":"   in this case use case is both for front-end and back-end:"},{"line_number":842,"context_line":""},{"line_number":843,"context_line":"   .. code-block:: console"},{"line_number":844,"context_line":""}],"source_content_type":"text/x-rst","patch_set":10,"id":"bf51134e_63cc4d61","line":841,"range":{"start_line":841,"start_character":3,"end_line":841,"end_character":24},"updated":"2020-06-19 03:11:00.000000000","message":"do you mean \"in this case\" or \"in this use case\" ?","commit_id":"3cc08543a3df3bf32d9ade173ea856f807276b2d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d4e55b8444c1faf9da93fd97c8f239d45cb5f88f","unresolved":false,"context_lines":[{"line_number":882,"context_line":"``Libvirt`` includes an extra ``xml`` flag within the ``\u003cdisk\u003e`` section called"},{"line_number":883,"context_line":"``iotune`` that is responsible for rate limitation. To confirm that, first get"},{"line_number":884,"context_line":"the ``OS-EXT-SRV-ATTR:instance_name`` value of the server instance"},{"line_number":885,"context_line":"i.e. ``instance-00000003``."},{"line_number":886,"context_line":""},{"line_number":887,"context_line":".. code-block:: console"},{"line_number":888,"context_line":""}],"source_content_type":"text/x-rst","patch_set":10,"id":"bf51134e_4301496b","line":885,"range":{"start_line":885,"start_character":0,"end_line":885,"end_character":4},"updated":"2020-06-19 03:11:00.000000000","message":"OpenStack documentation style is to use \", that is,\" instead of \"i.e.\" (though I suspect you actually mean \", for example,\" here).","commit_id":"3cc08543a3df3bf32d9ade173ea856f807276b2d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d4e55b8444c1faf9da93fd97c8f239d45cb5f88f","unresolved":false,"context_lines":[{"line_number":929,"context_line":""},{"line_number":930,"context_line":"   $ virsh dumpxml instance-00000003 | grep -1 \"total_bytes_sec\\|total_iops_sec\""},{"line_number":931,"context_line":""},{"line_number":932,"context_line":"The output of the command contains the ``xml`` below. It is found between the"},{"line_number":933,"context_line":"``\u003cdisk\u003e`` start and end tag."},{"line_number":934,"context_line":""},{"line_number":935,"context_line":".. code-block:: xml"}],"source_content_type":"text/x-rst","patch_set":10,"id":"bf51134e_63f56d83","line":932,"range":{"start_line":932,"start_character":39,"end_line":932,"end_character":46},"updated":"2020-06-19 03:11:00.000000000","message":"Since you\u0027re talking about the markup language, this should probably be simply XML (all caps, no formatting).","commit_id":"3cc08543a3df3bf32d9ade173ea856f807276b2d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d4e55b8444c1faf9da93fd97c8f239d45cb5f88f","unresolved":false,"context_lines":[{"line_number":1076,"context_line":"   # yum install sysfsutils sg3-utils    #file system utilities"},{"line_number":1077,"context_line":""},{"line_number":1078,"context_line":""},{"line_number":1079,"context_line":"Multi-path configuration file"},{"line_number":1080,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"},{"line_number":1081,"context_line":""},{"line_number":1082,"context_line":"The multi-path configuration file may be edited for better management and"}],"source_content_type":"text/x-rst","patch_set":10,"id":"bf51134e_6323ad06","line":1079,"range":{"start_line":1079,"start_character":0,"end_line":1079,"end_character":10},"updated":"2020-06-19 03:11:00.000000000","message":"I think this is usually \u0027multipath\u0027 with no hyphen.  I suggest dropping the hyphen since the name of the config file doesn\u0027t have a hyphen, and the config options mentioned below don\u0027t either.","commit_id":"3cc08543a3df3bf32d9ade173ea856f807276b2d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d4e55b8444c1faf9da93fd97c8f239d45cb5f88f","unresolved":false,"context_lines":[{"line_number":1242,"context_line":"   ignored because compression is not an available feature on a VMAX3 Hybrid."},{"line_number":1243,"context_line":""},{"line_number":1244,"context_line":""},{"line_number":1245,"context_line":"12. Over-subscription support"},{"line_number":1246,"context_line":"-----------------------------"},{"line_number":1247,"context_line":""},{"line_number":1248,"context_line":"Please refer to the official OpenStack `over-subscription documentation`_ for"}],"source_content_type":"text/x-rst","patch_set":10,"id":"bf51134e_0332b110","line":1245,"range":{"start_line":1245,"start_character":4,"end_line":1245,"end_character":21},"updated":"2020-06-19 03:11:00.000000000","message":"What\u0027s the deal with all the hyphens?  This is usually spelled \u0027oversubscription\u0027.  (The doc filename you reverence below has a hyphen, but the text in that doc doesn\u0027t use a hyphen.)","commit_id":"3cc08543a3df3bf32d9ade173ea856f807276b2d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d4e55b8444c1faf9da93fd97c8f239d45cb5f88f","unresolved":false,"context_lines":[{"line_number":1352,"context_line":""},{"line_number":1353,"context_line":"#. Export ``NOVA-INST-DIR/instances`` from ``HostA``, and ensure it is readable"},{"line_number":1354,"context_line":"   and writable by the Compute user on ``HostB`` and ``HostC``. Please refer to"},{"line_number":1355,"context_line":"   the relevant OS documentation for further details, e.g."},{"line_number":1356,"context_line":"   `Ubuntu NFS Documentation`_"},{"line_number":1357,"context_line":""},{"line_number":1358,"context_line":"#. On all compute nodes, enable the ``execute/search`` bit on your shared"}],"source_content_type":"text/x-rst","patch_set":10,"id":"bf51134e_433ca9e1","line":1355,"range":{"start_line":1355,"start_character":54,"end_line":1355,"end_character":58},"updated":"2020-06-19 03:11:00.000000000","message":"OpenStack style is to use \"for example\" here.","commit_id":"3cc08543a3df3bf32d9ade173ea856f807276b2d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d4e55b8444c1faf9da93fd97c8f239d45cb5f88f","unresolved":false,"context_lines":[{"line_number":1427,"context_line":"                                 server_lm_1"},{"line_number":1428,"context_line":""},{"line_number":1429,"context_line":"#. Run the command on step 3 above when the instance is back in available"},{"line_number":1430,"context_line":"   status. The hyper-visor should be on Host A."},{"line_number":1431,"context_line":""},{"line_number":1432,"context_line":"#. Run the command on Step 4 on Host A to confirm that the instance is"},{"line_number":1433,"context_line":"   created through ``virsh``."}],"source_content_type":"text/x-rst","patch_set":10,"id":"bf51134e_e31dfd73","line":1430,"range":{"start_line":1430,"start_character":15,"end_line":1430,"end_character":26},"updated":"2020-06-19 03:11:00.000000000","message":"Stop! You\u0027re killing me!","commit_id":"3cc08543a3df3bf32d9ade173ea856f807276b2d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d4e55b8444c1faf9da93fd97c8f239d45cb5f88f","unresolved":false,"context_lines":[{"line_number":1433,"context_line":"   created through ``virsh``."},{"line_number":1434,"context_line":""},{"line_number":1435,"context_line":""},{"line_number":1436,"context_line":"14. Multi-attach support"},{"line_number":1437,"context_line":"------------------------"},{"line_number":1438,"context_line":""},{"line_number":1439,"context_line":"PowerMax cinder driver supports the ability to attach a volume to multiple"}],"source_content_type":"text/x-rst","patch_set":10,"id":"bf51134e_66657be1","line":1436,"range":{"start_line":1436,"start_character":4,"end_line":1436,"end_character":16},"updated":"2020-06-19 03:11:00.000000000","message":"OK, this one is OK, I guess.  The cinder doc [0] seems to use it both ways.\n\n[0] https://docs.openstack.org/cinder/latest/admin/blockstorage-volume-multiattach.html","commit_id":"3cc08543a3df3bf32d9ade173ea856f807276b2d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d4e55b8444c1faf9da93fd97c8f239d45cb5f88f","unresolved":false,"context_lines":[{"line_number":1588,"context_line":"- All required instances of Unisphere for PowerMax are set up and configured"},{"line_number":1589,"context_line":"  for the array(s)"},{"line_number":1590,"context_line":"- Array(s) are locally registered with the instance of Unisphere that will be"},{"line_number":1591,"context_line":"  used as a fail-over instance. There are two fail-over types, local and"},{"line_number":1592,"context_line":"  remote:"},{"line_number":1593,"context_line":""},{"line_number":1594,"context_line":"  - `Local failover` - Primary Unisphere is unreachable, fail-over to"}],"source_content_type":"text/x-rst","patch_set":10,"id":"bf51134e_86dd2f72","line":1591,"range":{"start_line":1591,"start_character":12,"end_line":1591,"end_character":21},"updated":"2020-06-19 03:11:00.000000000","message":"You know what I\u0027m going to say ... it\u0027s less confusing to spell it without the hyphen, especially since the config options don\u0027t have hyphens.","commit_id":"3cc08543a3df3bf32d9ade173ea856f807276b2d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d4e55b8444c1faf9da93fd97c8f239d45cb5f88f","unresolved":false,"context_lines":[{"line_number":1704,"context_line":""},{"line_number":1705,"context_line":"Assumptions, restrictions and prerequisites"},{"line_number":1706,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"},{"line_number":1707,"context_line":""},{"line_number":1708,"context_line":"- The storage group tag(s) is associated with a volume type extra spec key"},{"line_number":1709,"context_line":"  storagetype:storagegrouptags."},{"line_number":1710,"context_line":"- The array tag is associated with the backend stanza using key"}],"source_content_type":"text/x-rst","patch_set":10,"id":"bf51134e_666f3b92","line":1707,"updated":"2020-06-19 03:11:00.000000000","message":"I think you might want to use `` `` around some of the below to make the config names stand out.","commit_id":"3cc08543a3df3bf32d9ade173ea856f807276b2d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d4e55b8444c1faf9da93fd97c8f239d45cb5f88f","unresolved":false,"context_lines":[{"line_number":1745,"context_line":"   powermax_array_tag_list \u003d [openstack1, openstack2]"},{"line_number":1746,"context_line":""},{"line_number":1747,"context_line":""},{"line_number":1748,"context_line":"21. PowerMax short host name and port group name override"},{"line_number":1749,"context_line":"---------------------------------------------------------"},{"line_number":1750,"context_line":""},{"line_number":1751,"context_line":"This functionality allows the user to customize the short host name and port"}],"source_content_type":"text/x-rst","patch_set":10,"id":"bf51134e_66f45bd0","line":1748,"range":{"start_line":1748,"start_character":49,"end_line":1748,"end_character":57},"updated":"2020-06-19 03:11:00.000000000","message":"\\o/","commit_id":"3cc08543a3df3bf32d9ade173ea856f807276b2d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d4e55b8444c1faf9da93fd97c8f239d45cb5f88f","unresolved":false,"context_lines":[{"line_number":1759,"context_line":"offers a little bit more flexibility to determine how these truncated"},{"line_number":1760,"context_line":"components should look."},{"line_number":1761,"context_line":""},{"line_number":1762,"context_line":"Assumptions, restrictions, and pre-requisites"},{"line_number":1763,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"},{"line_number":1764,"context_line":""},{"line_number":1765,"context_line":"- Backward compatibility with old format is preserved."}],"source_content_type":"text/x-rst","patch_set":10,"id":"bf51134e_661d7b0e","line":1762,"range":{"start_line":1762,"start_character":31,"end_line":1762,"end_character":45},"updated":"2020-06-19 03:11:00.000000000","message":"You usually don\u0027t see this with a hyphen.","commit_id":"3cc08543a3df3bf32d9ade173ea856f807276b2d"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"106daac8f26a015304f2c54120f2ade0b810e81d","unresolved":false,"context_lines":[{"line_number":2019,"context_line":"       value of these must ``not`` be ``backend_id_legacy_rep``."},{"line_number":2020,"context_line":""},{"line_number":2021,"context_line":"Example existing backend stanza pre-multiple replication:"},{"line_number":2022,"context_line":"    .. code-block:: console"},{"line_number":2023,"context_line":""},{"line_number":2024,"context_line":"      enabled_backends \u003d POWERMAX_FC_REPLICATION"},{"line_number":2025,"context_line":"      [POWERMAX_FC_REPLICATION]"}],"source_content_type":"text/x-rst","patch_set":10,"id":"ff570b3c_7c7fd865","line":2022,"range":{"start_line":2022,"start_character":4,"end_line":2022,"end_character":27},"updated":"2020-06-12 15:08:03.000000000","message":"This and the next one probably shouldn\u0027t be indented. This is making the sentences into a definition list.\n\nIt doesn\u0027t make the formatting look bad though.","commit_id":"3cc08543a3df3bf32d9ade173ea856f807276b2d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d4e55b8444c1faf9da93fd97c8f239d45cb5f88f","unresolved":false,"context_lines":[{"line_number":2128,"context_line":"* Replication Group operations are available for volumes in Synchronous mode"},{"line_number":2129,"context_line":"  only."},{"line_number":2130,"context_line":""},{"line_number":2131,"context_line":"* The Ussuri release of OpenStack supports retyping in-use volumes to and from"},{"line_number":2132,"context_line":"  replication enabled volume types with limited exception of volumes with"},{"line_number":2133,"context_line":"  Metro replication enabled. To retype to a volume-type that is Metro enabled"},{"line_number":2134,"context_line":"  the volume **must** first be detached then retyped. The reason for this is"}],"source_content_type":"text/x-rst","patch_set":10,"id":"bf51134e_c6ca876b","line":2131,"range":{"start_line":2131,"start_character":52,"end_line":2131,"end_character":58},"updated":"2020-06-19 03:11:00.000000000","message":"This one is perfectly OK.","commit_id":"3cc08543a3df3bf32d9ade173ea856f807276b2d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d4e55b8444c1faf9da93fd97c8f239d45cb5f88f","unresolved":false,"context_lines":[{"line_number":2155,"context_line":""},{"line_number":2156,"context_line":".. note::"},{"line_number":2157,"context_line":""},{"line_number":2158,"context_line":"   Fail-over and Fail-back operations are not applicable in Metro"},{"line_number":2159,"context_line":"   configurations."},{"line_number":2160,"context_line":""},{"line_number":2161,"context_line":"In the event of a disaster, or where there is required downtime, upgrade"}],"source_content_type":"text/x-rst","patch_set":10,"id":"bf51134e_e6922b4b","line":2158,"range":{"start_line":2158,"start_character":3,"end_line":2158,"end_character":26},"updated":"2020-06-19 03:11:00.000000000","message":"Usually not hyphenated.","commit_id":"3cc08543a3df3bf32d9ade173ea856f807276b2d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d4e55b8444c1faf9da93fd97c8f239d45cb5f88f","unresolved":false,"context_lines":[{"line_number":2271,"context_line":"   Retyping an in-use volume to a metro enabled volume type is not currently"},{"line_number":2272,"context_line":"   supported via storage-assisted migration. This retype can still be"},{"line_number":2273,"context_line":"   performed using host-assisted migration by setting the migration-policy"},{"line_number":2274,"context_line":"   to on-demand."},{"line_number":2275,"context_line":"   cinder retype --migration-policy on-demand \u003cvolume\u003e \u003cvolume-type\u003e"},{"line_number":2276,"context_line":""},{"line_number":2277,"context_line":"#. For migrating a volume from one Service Level or Workload combination to"}],"source_content_type":"text/x-rst","patch_set":10,"id":"bf51134e_86952f43","line":2274,"range":{"start_line":2274,"start_character":6,"end_line":2274,"end_character":15},"updated":"2020-06-19 03:11:00.000000000","message":"probably want `` `` around this","commit_id":"3cc08543a3df3bf32d9ade173ea856f807276b2d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d4e55b8444c1faf9da93fd97c8f239d45cb5f88f","unresolved":false,"context_lines":[{"line_number":2272,"context_line":"   supported via storage-assisted migration. This retype can still be"},{"line_number":2273,"context_line":"   performed using host-assisted migration by setting the migration-policy"},{"line_number":2274,"context_line":"   to on-demand."},{"line_number":2275,"context_line":"   cinder retype --migration-policy on-demand \u003cvolume\u003e \u003cvolume-type\u003e"},{"line_number":2276,"context_line":""},{"line_number":2277,"context_line":"#. For migrating a volume from one Service Level or Workload combination to"},{"line_number":2278,"context_line":"   another, use volume retype with the migration-policy to on-demand. The"}],"source_content_type":"text/x-rst","patch_set":10,"id":"bf51134e_6679fb2d","line":2275,"range":{"start_line":2275,"start_character":3,"end_line":2275,"end_character":68},"updated":"2020-06-19 03:11:00.000000000","message":"this line is getting mashed into the previous paragraph when rendered in HTML","commit_id":"3cc08543a3df3bf32d9ade173ea856f807276b2d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d4e55b8444c1faf9da93fd97c8f239d45cb5f88f","unresolved":false,"context_lines":[{"line_number":2364,"context_line":"Generic volume group operations no longer require the user to specify the"},{"line_number":2365,"context_line":"Cinder CLI version, however, performing generic volume group replication"},{"line_number":2366,"context_line":"operations still require this setting. When running generic volume group"},{"line_number":2367,"context_line":"commands set the value --os-volume-api-version to 3.38. These commands are not"},{"line_number":2368,"context_line":"listed in the latest Cinder CLI documentation so will remain here until added"},{"line_number":2369,"context_line":"to the latest Cinder CLI version or deprecated from Cinder."},{"line_number":2370,"context_line":""}],"source_content_type":"text/x-rst","patch_set":10,"id":"bf51134e_c6ad6789","line":2367,"range":{"start_line":2367,"start_character":23,"end_line":2367,"end_character":54},"updated":"2020-06-19 03:11:00.000000000","message":"I suggest\n\n  ``--os-volume-api-version`` to ``3.38``","commit_id":"3cc08543a3df3bf32d9ade173ea856f807276b2d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"323db541706717fda6d078159aaad8625b69b969","unresolved":false,"context_lines":[{"line_number":1761,"context_line":"components should look."},{"line_number":1762,"context_line":""},{"line_number":1763,"context_line":"Assumptions, restrictions, and prerequisites"},{"line_number":1764,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"},{"line_number":1765,"context_line":""},{"line_number":1766,"context_line":"- Backward compatibility with old format is preserved."},{"line_number":1767,"context_line":"- ``cinder.conf`` will have 2 new configuration options,"}],"source_content_type":"text/x-rst","patch_set":12,"id":"bf51134e_5f690689","line":1764,"range":{"start_line":1764,"start_character":44,"end_line":1764,"end_character":45},"updated":"2020-06-20 02:20:40.000000000","message":"I think you have an extra ~ in the line, but the pep8 job doesn\u0027t seem to care.","commit_id":"e444ecf81033bd21dfa726394250b01ce176fa26"}]}