)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"aa1127b7c8ded27e66cfb00cbe2e4c1ce688cc23","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Michelle (Shelly) Mutu-Grigg \u003cshellymutugrigg@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2020-10-09 22:50:34 +1300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Update .rst files remove -os-baremetal-api-version"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"There are currently parameters within the .rst files that are"},{"line_number":10,"context_line":"referencing old --os-baremetal-api-versions after the \u0027openstack\u0027"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9f560f44_65e371a1","line":7,"range":{"start_line":7,"start_character":18,"end_line":7,"end_character":50},"updated":"2020-10-09 10:19:48.000000000","message":"Since you also fixed some openstack baremetal commands that I requested I would just use Update .rst files in the title.","commit_id":"9d2dd1c3fceacab5e78e50b93e5eaa04387cfa5e"},{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"aa1127b7c8ded27e66cfb00cbe2e4c1ce688cc23","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Update .rst files remove -os-baremetal-api-version"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"There are currently parameters within the .rst files that are"},{"line_number":10,"context_line":"referencing old --os-baremetal-api-versions after the \u0027openstack\u0027"},{"line_number":11,"context_line":"or \u0027openstack baremetal\u0027 commands.  This commit completes"},{"line_number":12,"context_line":"Task 40964 under story 20088194 to remove these instances."},{"line_number":13,"context_line":"Also updates to remove \u0027openstack\u0027 from \u0027openstack baremetal\u0027 instances"},{"line_number":14,"context_line":"Story: 2008194"},{"line_number":15,"context_line":"Task: 40964"},{"line_number":16,"context_line":"Change-id"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9f560f44_c5f35d6c","line":13,"range":{"start_line":9,"start_character":0,"end_line":13,"end_character":71},"updated":"2020-10-09 10:19:48.000000000","message":"I would rephrase for something like:\n\nThis commit removes `--os-baremetal-api-version` from CLI examples and also updates some `openstack baremetal` commands to be `baremetal`.","commit_id":"9d2dd1c3fceacab5e78e50b93e5eaa04387cfa5e"},{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"aa1127b7c8ded27e66cfb00cbe2e4c1ce688cc23","unresolved":false,"context_lines":[{"line_number":13,"context_line":"Also updates to remove \u0027openstack\u0027 from \u0027openstack baremetal\u0027 instances"},{"line_number":14,"context_line":"Story: 2008194"},{"line_number":15,"context_line":"Task: 40964"},{"line_number":16,"context_line":"Change-id"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: I4421e98afdee9ed97506cb7e5d69da966cffd963"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9f560f44_e56f8126","line":16,"range":{"start_line":16,"start_character":0,"end_line":16,"end_character":9},"updated":"2020-10-09 10:19:48.000000000","message":"This you can remove","commit_id":"9d2dd1c3fceacab5e78e50b93e5eaa04387cfa5e"}],"doc/source/admin/boot-from-volume.rst":[{"author":{"_account_id":24828,"name":"Kaifeng Wang","email":"kaifeng.w@gmail.com","username":"wangkf"},"change_message_id":"6709d8467d459cbe1b8e5e952211028caf7f89a7","unresolved":false,"context_lines":[{"line_number":53,"context_line":"storage operations. For example, to set the storage interface to ``cinder``"},{"line_number":54,"context_line":"on an existing node::"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"    baremetal node set \\"},{"line_number":57,"context_line":"             --storage-interface cinder $NODE_UUID"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"A default storage interface can be specified in ironic.conf. See the"}],"source_content_type":"text/x-rst","patch_set":4,"id":"9f560f44_74bb1374","line":56,"updated":"2020-10-10 14:33:12.000000000","message":"could we merge two lines? the line break seems unnecessary here. ditto for other changes where suitable.","commit_id":"d32a04f2d53327448526a12311621f82a99fda76"},{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"cababb2e65bc8be318a23b9c1a01df4ceb0cbee6","unresolved":false,"context_lines":[{"line_number":53,"context_line":"storage operations. For example, to set the storage interface to ``cinder``"},{"line_number":54,"context_line":"on an existing node::"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"    baremetal node set \\"},{"line_number":57,"context_line":"             --storage-interface cinder $NODE_UUID"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"A default storage interface can be specified in ironic.conf. See the"}],"source_content_type":"text/x-rst","patch_set":4,"id":"9f560f44_0da28a3f","line":56,"in_reply_to":"9f560f44_74bb1374","updated":"2020-10-11 11:26:42.000000000","message":"Yeah that makes sense \u003d)","commit_id":"d32a04f2d53327448526a12311621f82a99fda76"}],"doc/source/admin/console.rst":[{"author":{"_account_id":24828,"name":"Kaifeng Wang","email":"kaifeng.w@gmail.com","username":"wangkf"},"change_message_id":"6709d8467d459cbe1b8e5e952211028caf7f89a7","unresolved":false,"context_lines":[{"line_number":119,"context_line":"  If the node uses a hardware type, for example ``ipmi``, set the node\u0027s"},{"line_number":120,"context_line":"  console interface to ``ipmitool-shellinabox``::"},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"    baremetal node set \u003cnode-uuid\u003e \\"},{"line_number":123,"context_line":"       --console-interface ipmitool-shellinabox"},{"line_number":124,"context_line":""},{"line_number":125,"context_line":"  Enable the web console, for example::"}],"source_content_type":"text/x-rst","patch_set":4,"id":"9f560f44_b4b12b94","line":122,"range":{"start_line":122,"start_character":24,"end_line":122,"end_character":33},"updated":"2020-10-10 14:33:12.000000000","message":"Maybe we could take the opportunity to update \u003cnode-uuid\u003e to \u003cnode\u003e in this patch","commit_id":"d32a04f2d53327448526a12311621f82a99fda76"},{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"cababb2e65bc8be318a23b9c1a01df4ceb0cbee6","unresolved":false,"context_lines":[{"line_number":119,"context_line":"  If the node uses a hardware type, for example ``ipmi``, set the node\u0027s"},{"line_number":120,"context_line":"  console interface to ``ipmitool-shellinabox``::"},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"    baremetal node set \u003cnode-uuid\u003e \\"},{"line_number":123,"context_line":"       --console-interface ipmitool-shellinabox"},{"line_number":124,"context_line":""},{"line_number":125,"context_line":"  Enable the web console, for example::"}],"source_content_type":"text/x-rst","patch_set":4,"id":"9f560f44_2da5ce48","line":122,"range":{"start_line":122,"start_character":24,"end_line":122,"end_character":33},"in_reply_to":"9f560f44_b4b12b94","updated":"2020-10-11 11:26:42.000000000","message":"Since this is just an initial task for outreachy contributors maybe can be done in another patch I would say (I\u0027m not against the idea)","commit_id":"d32a04f2d53327448526a12311621f82a99fda76"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"d30fd454aca88e7f591b456e8c74d9283cc2853f","unresolved":false,"context_lines":[{"line_number":205,"context_line":"  If the node uses a hardware type, for example ``ipmi``, set the node\u0027s"},{"line_number":206,"context_line":"  console interface to ``ipmitool-socat``::"},{"line_number":207,"context_line":""},{"line_number":208,"context_line":"    baremetal node set \u003cnode\u003e --console-interface ipmitool-socat"},{"line_number":209,"context_line":""},{"line_number":210,"context_line":"  Enable the serial console, for example::"},{"line_number":211,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f65232a_cf34b74d","line":208,"updated":"2020-10-22 02:20:07.000000000","message":"I wonder why gerrit is showing \u003e is added....","commit_id":"d5ca2ac7f0e2323400b22bc7b8cc2c33a3f1ef5a"}],"doc/source/admin/drivers/idrac.rst":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"d30fd454aca88e7f591b456e8c74d9283cc2853f","unresolved":false,"context_lines":[{"line_number":384,"context_line":".. code-block:: bash"},{"line_number":385,"context_line":""},{"line_number":386,"context_line":"  baremetal node clean --clean-steps \\"},{"line_number":387,"context_line":"    \u0027[{\"interface\": \"raid\", \"step\": \"create_configuration\"}]\u0027 ${node_uuid}"},{"line_number":388,"context_line":""},{"line_number":389,"context_line":""},{"line_number":390,"context_line":"When the physical disk names or controller names are not known, the"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f65232a_2ff0cb90","line":387,"range":{"start_line":387,"start_character":62,"end_line":387,"end_character":74},"updated":"2020-10-22 02:20:07.000000000","message":"It would be good to change this to \u003cnode\u003e","commit_id":"d5ca2ac7f0e2323400b22bc7b8cc2c33a3f1ef5a"}],"doc/source/admin/drivers/ilo.rst":[{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"226b60ea6b5c92725b7dbef46731010b4a069ec5","unresolved":false,"context_lines":[{"line_number":1642,"context_line":""},{"line_number":1643,"context_line":"    \u003cnode\u003e[_\u003cinstance-uuid\u003e]_update_firmware_sum_\u003ctimestamp yyyy-mm-dd-hh-mm-ss\u003e.tar.gz"},{"line_number":1644,"context_line":"    or"},{"line_number":1645,"context_line":"    \u003cnode\u003e[_\u003cinstance-uuid\u003e]_flash_firmware_sum_\u003ctimestamp yyyy-mm-dd-hh-mm-ss\u003e.tar.gz"},{"line_number":1646,"context_line":""},{"line_number":1647,"context_line":"Refer to :ref:`retrieve_deploy_ramdisk_logs` for more information on enabling and"},{"line_number":1648,"context_line":"viewing the logs returned from the ramdisk."}],"source_content_type":"text/x-rst","patch_set":5,"id":"7f6b1bfe_cbc64b68","line":1645,"updated":"2020-10-15 20:09:37.000000000","message":"In this case, \u003cnode\u003e will always be the uuid, never the name. I don\u0027t really think it is a problem to change these docs to \u003cnode\u003e though.","commit_id":"d5ca2ac7f0e2323400b22bc7b8cc2c33a3f1ef5a"}],"doc/source/admin/node-deployment.rst":[{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"226b60ea6b5c92725b7dbef46731010b4a069ec5","unresolved":false,"context_lines":[{"line_number":103,"context_line":"failed, run the following command; it will return the value in the node\u0027s"},{"line_number":104,"context_line":"``driver_internal_info`` field::"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"    baremetal node show $node_ident -f value -c driver_internal_info"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"The ``deploy_steps`` field will contain a list of all remaining steps with"},{"line_number":109,"context_line":"their priorities, and the first one listed is the step currently in progress or"}],"source_content_type":"text/x-rst","patch_set":5,"id":"7f6b1bfe_cb748b38","line":106,"range":{"start_line":106,"start_character":24,"end_line":106,"end_character":35},"updated":"2020-10-15 20:09:37.000000000","message":"huh, this file has a different convention, $node_ident instead of \u003cnode\u003e everywhere else. If there is a another revision to this change it would be good to make this consistent","commit_id":"d5ca2ac7f0e2323400b22bc7b8cc2c33a3f1ef5a"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"d30fd454aca88e7f591b456e8c74d9283cc2853f","unresolved":false,"context_lines":[{"line_number":103,"context_line":"failed, run the following command; it will return the value in the node\u0027s"},{"line_number":104,"context_line":"``driver_internal_info`` field::"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"    baremetal node show $node_ident -f value -c driver_internal_info"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"The ``deploy_steps`` field will contain a list of all remaining steps with"},{"line_number":109,"context_line":"their priorities, and the first one listed is the step currently in progress or"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f65232a_0fd06ff7","line":106,"range":{"start_line":106,"start_character":24,"end_line":106,"end_character":35},"in_reply_to":"7f6b1bfe_cb748b38","updated":"2020-10-22 02:20:07.000000000","message":"Agreed.","commit_id":"d5ca2ac7f0e2323400b22bc7b8cc2c33a3f1ef5a"}],"doc/source/admin/portgroups.rst":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"6d27df865535666106598fc64a5f573332cade95","unresolved":false,"context_lines":[{"line_number":48,"context_line":"   along with, optionally, its name, address, mode, properties, and if it"},{"line_number":49,"context_line":"   supports fallback to standalone ports::"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"   baremetal port group create \\"},{"line_number":52,"context_line":"    --node $NODE_UUID --name test --address fa:ab:25:48:fd:ba --mode 802.3ad \\"},{"line_number":53,"context_line":"    --property miimon\u003d100 --property xmit_hash_policy\u003d\"layer2+3\" \\"},{"line_number":54,"context_line":"    --support-standalone-ports"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9f560f44_259219fd","line":51,"updated":"2020-10-09 10:07:41.000000000","message":"docs: Literal block expected; none found.","commit_id":"9d2dd1c3fceacab5e78e50b93e5eaa04387cfa5e"},{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"aa1127b7c8ded27e66cfb00cbe2e4c1ce688cc23","unresolved":false,"context_lines":[{"line_number":48,"context_line":"   along with, optionally, its name, address, mode, properties, and if it"},{"line_number":49,"context_line":"   supports fallback to standalone ports::"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"   baremetal port group create \\"},{"line_number":52,"context_line":"    --node $NODE_UUID --name test --address fa:ab:25:48:fd:ba --mode 802.3ad \\"},{"line_number":53,"context_line":"    --property miimon\u003d100 --property xmit_hash_policy\u003d\"layer2+3\" \\"},{"line_number":54,"context_line":"    --support-standalone-ports"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9f560f44_c0386bd5","line":51,"in_reply_to":"9f560f44_259219fd","updated":"2020-10-09 10:19:48.000000000","message":"I think you need to add an empty space so \"baremetal\" will be alligned with the other lines.\n\nLocally you can run some tests to make sure your change is ok, see https://docs.openstack.org/ironic/latest/contributor/dev-quickstart.html#running-unit-tests-locally\n\nYou should run pep8, docs (since you are only editing rst files)","commit_id":"9d2dd1c3fceacab5e78e50b93e5eaa04387cfa5e"},{"author":{"_account_id":24828,"name":"Kaifeng Wang","email":"kaifeng.w@gmail.com","username":"wangkf"},"change_message_id":"6709d8467d459cbe1b8e5e952211028caf7f89a7","unresolved":false,"context_lines":[{"line_number":48,"context_line":"   along with, optionally, its name, address, mode, properties, and if it"},{"line_number":49,"context_line":"   supports fallback to standalone ports::"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"    baremetal port group create \\"},{"line_number":52,"context_line":"    --node $NODE_UUID --name test --address fa:ab:25:48:fd:ba --mode 802.3ad \\"},{"line_number":53,"context_line":"    --property miimon\u003d100 --property xmit_hash_policy\u003d\"layer2+3\" \\"},{"line_number":54,"context_line":"    --support-standalone-ports"}],"source_content_type":"text/x-rst","patch_set":4,"id":"9f560f44_34a51b49","line":51,"updated":"2020-10-10 14:33:12.000000000","message":"I don\u0027t understand why some commands are switched to baremetal, while others are not, I think either we keep documenting \"openstack baremetal\" or just \"baremetal\", by doing this, the doc looks consistent to users. If we don\u0027t reach feature parity between \"openstack baremetal\" and \"baremetal\", I would prefer to keep the \"openstack baremetal\" command throughout the doc and have a page for the \"baremetal\" command, if they are actually identical we just need a note that they are interchangeable and use one of them throughout the document, but this is just my opinion.","commit_id":"d32a04f2d53327448526a12311621f82a99fda76"},{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"cababb2e65bc8be318a23b9c1a01df4ceb0cbee6","unresolved":false,"context_lines":[{"line_number":48,"context_line":"   along with, optionally, its name, address, mode, properties, and if it"},{"line_number":49,"context_line":"   supports fallback to standalone ports::"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"    baremetal port group create \\"},{"line_number":52,"context_line":"    --node $NODE_UUID --name test --address fa:ab:25:48:fd:ba --mode 802.3ad \\"},{"line_number":53,"context_line":"    --property miimon\u003d100 --property xmit_hash_policy\u003d\"layer2+3\" \\"},{"line_number":54,"context_line":"    --support-standalone-ports"}],"source_content_type":"text/x-rst","patch_set":4,"id":"9f560f44_6d60660e","line":51,"in_reply_to":"9f560f44_34a51b49","updated":"2020-10-11 11:26:42.000000000","message":"The initial idea is to try to make our doc less openstack specific I would say. Let\u0027s get some feedback from the community during the upstream meeting to see how we should move forward.","commit_id":"d32a04f2d53327448526a12311621f82a99fda76"}],"doc/source/admin/upgrade-to-hardware-types.rst":[{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"226b60ea6b5c92725b7dbef46731010b4a069ec5","unresolved":false,"context_lines":[{"line_number":93,"context_line":"    export OS_BAREMETAL_API_VERSION\u003d1.31"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"    for uuid in $(baremetal node list --driver pxe_ipmitool -f value -c UUID); do"},{"line_number":96,"context_line":"        baremetal node set $uuid --driver ipmi --deploy-interface iscsi"},{"line_number":97,"context_line":"    done"},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"    for uuid in $(baremetal node list --driver agent_ipmitool -f value -c UUID); do"}],"source_content_type":"text/x-rst","patch_set":5,"id":"7f6b1bfe_ebace78d","line":96,"range":{"start_line":96,"start_character":27,"end_line":96,"end_character":32},"updated":"2020-10-15 20:09:37.000000000","message":"this should also be \u003cnode\u003e","commit_id":"d5ca2ac7f0e2323400b22bc7b8cc2c33a3f1ef5a"}],"doc/source/install/configure-nova-flavors.rst":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"d30fd454aca88e7f591b456e8c74d9283cc2853f","unresolved":false,"context_lines":[{"line_number":66,"context_line":""},{"line_number":67,"context_line":".. code-block:: console"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"      $ baremetal node set $NODE_UUID --resource-class baremetal.with-GPU"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":".. warning::"},{"line_number":72,"context_line":"    It is possible to **add** a resource class to ``active`` nodes, but it is"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f65232a_af04db7c","line":69,"range":{"start_line":69,"start_character":28,"end_line":69,"end_character":37},"updated":"2020-10-22 02:20:07.000000000","message":"It would be excellent to revise the formatting to use \u003cnode\u003e instead of $NODE_UUID","commit_id":"d5ca2ac7f0e2323400b22bc7b8cc2c33a3f1ef5a"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"d30fd454aca88e7f591b456e8c74d9283cc2853f","unresolved":false,"context_lines":[{"line_number":118,"context_line":""},{"line_number":119,"context_line":".. code-block:: console"},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"      $ baremetal node add trait $NODE_UUID CUSTOM_TRAIT1 HW_CPU_X86_VMX"},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"Then, update the flavor to require these traits:"},{"line_number":124,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f65232a_eff53385","line":121,"range":{"start_line":121,"start_character":33,"end_line":121,"end_character":44},"updated":"2020-10-22 02:20:07.000000000","message":"Same comment as above.","commit_id":"d5ca2ac7f0e2323400b22bc7b8cc2c33a3f1ef5a"}]}
