)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":22724,"name":"Galyna Zholtkevych","email":"galynazholtkevych1991@gmail.com","username":"gzholtkevych"},"change_message_id":"48196b346d3dc3820d58e214819def99dd804cda","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Correct typos in doc_files"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Fix."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I47fc7ce7c6c8a83ab9b0ffae666f49cc90da9c0f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"da36d5c6_8e5cfbee","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":4},"updated":"2017-02-28 08:32:39.000000000","message":"+1. The commit message in rare cases can be without description\nBut here this sentence does not make sence, either erase or complete the description","commit_id":"d28306ba128872e5360dd17a6e5a123de7eeae64"},{"author":{"_account_id":23883,"name":"Joanna Taryma","email":"joanna.taryma@intel.com","username":"jtaryma"},"change_message_id":"9aac013cff01b4bbf62ae7810af6dd0f7aff31e3","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Correct typos in doc_files"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Fix."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I47fc7ce7c6c8a83ab9b0ffae666f49cc90da9c0f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"da36d5c6_4cae99e8","line":9,"updated":"2017-02-27 22:57:07.000000000","message":"I don\u0027t get this sentence :)","commit_id":"d28306ba128872e5360dd17a6e5a123de7eeae64"},{"author":{"_account_id":22724,"name":"Galyna Zholtkevych","email":"galynazholtkevych1991@gmail.com","username":"gzholtkevych"},"change_message_id":"f55df00a5d566d1461abe30a826e0d55eb370be5","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     yuhui \u003cyuhui@inspur.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-02-28 09:00:04 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Correct typos in doc_files"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I47fc7ce7c6c8a83ab9b0ffae666f49cc90da9c0f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"da36d5c6_d11cd36c","line":7,"range":{"start_line":7,"start_character":17,"end_line":7,"end_character":26},"updated":"2017-02-28 10:37:06.000000000","message":"s/doc_files/doc files","commit_id":"0ba1b253902983d086632ceec9423539a8a7c330"},{"author":{"_account_id":22724,"name":"Galyna Zholtkevych","email":"galynazholtkevych1991@gmail.com","username":"gzholtkevych"},"change_message_id":"f55df00a5d566d1461abe30a826e0d55eb370be5","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2017-02-28 09:00:04 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Correct typos in doc_files"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I47fc7ce7c6c8a83ab9b0ffae666f49cc90da9c0f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"da36d5c6_d133f3f4","line":8,"updated":"2017-02-28 10:37:06.000000000","message":"I would prefer to specify which doc files you are correcting, because there are plenty of docs in the ironic tree, but you change only three ones","commit_id":"0ba1b253902983d086632ceec9423539a8a7c330"}],"doc/source/drivers/ilo.rst":[{"author":{"_account_id":23883,"name":"Joanna Taryma","email":"joanna.taryma@intel.com","username":"jtaryma"},"change_message_id":"9aac013cff01b4bbf62ae7810af6dd0f7aff31e3","unresolved":false,"context_lines":[{"line_number":40,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"* `proliantutils \u003chttps://pypi.python.org/pypi/proliantutils\u003e`_ is a python package"},{"line_number":43,"context_line":"  which contains set of modules for managing HPE ProLiant hardware."},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"  Install ``proliantutils`` module on the ironic conductor node. Minimum"},{"line_number":46,"context_line":"  version required is 2.2.0::"}],"source_content_type":"text/x-rst","patch_set":1,"id":"da36d5c6_ececad2b","line":43,"updated":"2017-02-27 22:57:07.000000000","message":"s/set/a set/","commit_id":"d28306ba128872e5360dd17a6e5a123de7eeae64"},{"author":{"_account_id":23883,"name":"Joanna Taryma","email":"joanna.taryma@intel.com","username":"jtaryma"},"change_message_id":"9aac013cff01b4bbf62ae7810af6dd0f7aff31e3","unresolved":false,"context_lines":[{"line_number":48,"context_line":"   $ pip install \"proliantutils\u003e\u003d2.2.0\""},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"* ``ipmitool`` command must be present on the service node(s) where"},{"line_number":51,"context_line":"  ``ironic-conductor`` is running. On most distros, this is provided as part"},{"line_number":52,"context_line":"  of the ``ipmitool`` package. Refer to `Hardware Inspection Support`_ for more"},{"line_number":53,"context_line":"  information on recommended version."},{"line_number":54,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"da36d5c6_8ce6d142","line":51,"updated":"2017-02-27 22:57:07.000000000","message":"s/this/it/ ?","commit_id":"d28306ba128872e5360dd17a6e5a123de7eeae64"},{"author":{"_account_id":23883,"name":"Joanna Taryma","email":"joanna.taryma@intel.com","username":"jtaryma"},"change_message_id":"9aac013cff01b4bbf62ae7810af6dd0f7aff31e3","unresolved":false,"context_lines":[{"line_number":62,"context_line":"   \u003chttp://docs.openstack.org/developer/glance/configuring.html#configuring-the-swift-storage-backend\u003e`_."},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"2. Set a temp-url key for Glance user in Swift. For example, if you have"},{"line_number":65,"context_line":"   configured Glance with user ``glance-swift`` and tenant as ``service``,"},{"line_number":66,"context_line":"   then run the below command::"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"    swift --os-username\u003dservice:glance-swift post -m temp-url-key:mysecretkeyforglance"}],"source_content_type":"text/x-rst","patch_set":1,"id":"da36d5c6_ac68f588","line":65,"updated":"2017-02-27 22:57:07.000000000","message":"shouldn\u0027t the openstack project names be lowercase?\nhttps://governance.openstack.org/tc/reference/service-project-naming.html","commit_id":"d28306ba128872e5360dd17a6e5a123de7eeae64"},{"author":{"_account_id":23883,"name":"Joanna Taryma","email":"joanna.taryma@intel.com","username":"jtaryma"},"change_message_id":"9aac013cff01b4bbf62ae7810af6dd0f7aff31e3","unresolved":false,"context_lines":[{"line_number":67,"context_line":""},{"line_number":68,"context_line":"    swift --os-username\u003dservice:glance-swift post -m temp-url-key:mysecretkeyforglance"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"3. Fill the required parameters in the ``[glance]`` section   in"},{"line_number":71,"context_line":"   ``/etc/ironic/ironic.conf``. Normally you would be required to fill in the"},{"line_number":72,"context_line":"   following details::"},{"line_number":73,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"da36d5c6_2c45e516","line":70,"updated":"2017-02-27 22:57:07.000000000","message":"s/   / /","commit_id":"d28306ba128872e5360dd17a6e5a123de7eeae64"},{"author":{"_account_id":23883,"name":"Joanna Taryma","email":"joanna.taryma@intel.com","username":"jtaryma"},"change_message_id":"9aac013cff01b4bbf62ae7810af6dd0f7aff31e3","unresolved":false,"context_lines":[{"line_number":201,"context_line":"Target Users"},{"line_number":202,"context_line":"~~~~~~~~~~~~"},{"line_number":203,"context_line":""},{"line_number":204,"context_line":"* Users who do not want to use PXE/TFTP protocol on their data centres."},{"line_number":205,"context_line":""},{"line_number":206,"context_line":"* Users who have concerns with PXE protocol\u0027s security issues and want to have a"},{"line_number":207,"context_line":"  security enhanced PXE-less deployment mechanism."}],"source_content_type":"text/x-rst","patch_set":1,"id":"da36d5c6_ec2f4d3c","line":204,"updated":"2017-02-27 22:57:07.000000000","message":"on/in ?","commit_id":"d28306ba128872e5360dd17a6e5a123de7eeae64"},{"author":{"_account_id":23883,"name":"Joanna Taryma","email":"joanna.taryma@intel.com","username":"jtaryma"},"change_message_id":"9aac013cff01b4bbf62ae7810af6dd0f7aff31e3","unresolved":false,"context_lines":[{"line_number":254,"context_line":"* Remote Console"},{"line_number":255,"context_line":"* HW Sensors"},{"line_number":256,"context_line":"* Works well for machines with resource constraints (lesser amount of memory)."},{"line_number":257,"context_line":"* Supports for out-of-band hardware inspection."},{"line_number":258,"context_line":"* Swiftless deploy for intermediate images"},{"line_number":259,"context_line":"* HTTP(S) Based Deploy."},{"line_number":260,"context_line":"* iLO drivers with standalone ironic."}],"source_content_type":"text/x-rst","patch_set":1,"id":"da36d5c6_6c1c3d14","line":257,"updated":"2017-02-27 22:57:07.000000000","message":"this should stay as it was (this is a list of nouns)","commit_id":"d28306ba128872e5360dd17a6e5a123de7eeae64"},{"author":{"_account_id":22226,"name":"Varun Gadiraju","email":"varun.gadiraju@intel.com","username":"gadirajv"},"change_message_id":"22be42b467d78ffdc54acf55f1cd1fc6f53ecabf","unresolved":false,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"The ``iscsi_ilo`` and ``agent_ilo`` drivers provide security enhanced"},{"line_number":24,"context_line":"PXE-less deployment by using iLO virtual media to boot up the bare metal node."},{"line_number":25,"context_line":"These drivers send management info through management channel and separate"},{"line_number":26,"context_line":"it from data channel which is used for deployment."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"``iscsi_ilo`` and ``agent_ilo`` drivers use deployment ramdisk"}],"source_content_type":"text/x-rst","patch_set":4,"id":"ba2be162_cf001169","line":25,"range":{"start_line":25,"start_character":35,"end_line":25,"end_character":61},"updated":"2017-03-03 20:50:40.000000000","message":"s/through management channel/ through the management channel","commit_id":"2caf7ac73b858a0a155fce861216a450137c558c"},{"author":{"_account_id":22226,"name":"Varun Gadiraju","email":"varun.gadiraju@intel.com","username":"gadirajv"},"change_message_id":"22be42b467d78ffdc54acf55f1cd1fc6f53ecabf","unresolved":false,"context_lines":[{"line_number":23,"context_line":"The ``iscsi_ilo`` and ``agent_ilo`` drivers provide security enhanced"},{"line_number":24,"context_line":"PXE-less deployment by using iLO virtual media to boot up the bare metal node."},{"line_number":25,"context_line":"These drivers send management info through management channel and separate"},{"line_number":26,"context_line":"it from data channel which is used for deployment."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"``iscsi_ilo`` and ``agent_ilo`` drivers use deployment ramdisk"},{"line_number":29,"context_line":"built from ``diskimage-builder``. The ``iscsi_ilo`` driver deploys from"}],"source_content_type":"text/x-rst","patch_set":4,"id":"ba2be162_8f31b95c","line":26,"range":{"start_line":26,"start_character":3,"end_line":26,"end_character":20},"updated":"2017-03-03 20:50:40.000000000","message":"s/from data channel/\"from the data channel\"","commit_id":"2caf7ac73b858a0a155fce861216a450137c558c"},{"author":{"_account_id":22226,"name":"Varun Gadiraju","email":"varun.gadiraju@intel.com","username":"gadirajv"},"change_message_id":"22be42b467d78ffdc54acf55f1cd1fc6f53ecabf","unresolved":false,"context_lines":[{"line_number":201,"context_line":"Target Users"},{"line_number":202,"context_line":"~~~~~~~~~~~~"},{"line_number":203,"context_line":""},{"line_number":204,"context_line":"* Users who do not want to use PXE/TFTP protocol in their data centres."},{"line_number":205,"context_line":""},{"line_number":206,"context_line":"* Users who have concerns with PXE protocol\u0027s security issues and want to have a"},{"line_number":207,"context_line":"  security enhanced PXE-less deployment mechanism."}],"source_content_type":"text/x-rst","patch_set":4,"id":"ba2be162_4f4421b6","line":204,"range":{"start_line":204,"start_character":63,"end_line":204,"end_character":70},"updated":"2017-03-03 20:50:40.000000000","message":"s/centres/centers","commit_id":"2caf7ac73b858a0a155fce861216a450137c558c"},{"author":{"_account_id":13295,"name":"Mario Villaplana","email":"mario.villaplana@gmail.com","username":"mariojv"},"change_message_id":"5dd998161873415d20f2944739ceb9639225f944","unresolved":false,"context_lines":[{"line_number":201,"context_line":"Target Users"},{"line_number":202,"context_line":"~~~~~~~~~~~~"},{"line_number":203,"context_line":""},{"line_number":204,"context_line":"* Users who do not want to use PXE/TFTP protocol in their data centers."},{"line_number":205,"context_line":""},{"line_number":206,"context_line":"* Users who have concerns with PXE protocol\u0027s security issues and want to have a"},{"line_number":207,"context_line":"  security enhanced PXE-less deployment mechanism."}],"source_content_type":"text/x-rst","patch_set":6,"id":"9a30ddce_133979f2","line":204,"range":{"start_line":204,"start_character":63,"end_line":204,"end_character":70},"updated":"2017-03-10 15:48:39.000000000","message":"nit: centres or centers are both fine :)","commit_id":"8bc960b31781043be677e1be1bd0aed3d65f9282"}],"doc/source/drivers/oneview.rst":[{"author":{"_account_id":22724,"name":"Galyna Zholtkevych","email":"galynazholtkevych1991@gmail.com","username":"gzholtkevych"},"change_message_id":"48196b346d3dc3820d58e214819def99dd804cda","unresolved":false,"context_lines":[{"line_number":52,"context_line":"actual resource sharing among ironic and OneView users."},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"Since OneView can claim nodes in ``available`` state at any time, a set of"},{"line_number":55,"context_line":"tasks run periodically to detect nodes in use by OneView. A node in use by"},{"line_number":56,"context_line":"OneView is placed in ``manageable`` state and has maintenance mode set. Once"},{"line_number":57,"context_line":"the node is no longer in use, these tasks will make place them back in"},{"line_number":58,"context_line":"``available`` state and clear maintenance mode."}],"source_content_type":"text/x-rst","patch_set":1,"id":"da36d5c6_6efd076e","line":55,"range":{"start_line":55,"start_character":6,"end_line":55,"end_character":10},"updated":"2017-02-28 08:32:39.000000000","message":"IMHO, here it is correct change. \nif the verb `runs` would relate to tasks, the `running` instead of `runs` would be used.","commit_id":"d28306ba128872e5360dd17a6e5a123de7eeae64"},{"author":{"_account_id":23883,"name":"Joanna Taryma","email":"joanna.taryma@intel.com","username":"jtaryma"},"change_message_id":"9aac013cff01b4bbf62ae7810af6dd0f7aff31e3","unresolved":false,"context_lines":[{"line_number":52,"context_line":"actual resource sharing among ironic and OneView users."},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"Since OneView can claim nodes in ``available`` state at any time, a set of"},{"line_number":55,"context_line":"tasks run periodically to detect nodes in use by OneView. A node in use by"},{"line_number":56,"context_line":"OneView is placed in ``manageable`` state and has maintenance mode set. Once"},{"line_number":57,"context_line":"the node is no longer in use, these tasks will make place them back in"},{"line_number":58,"context_line":"``available`` state and clear maintenance mode."}],"source_content_type":"text/x-rst","patch_set":1,"id":"da36d5c6_0cb3c10b","line":55,"updated":"2017-02-27 22:57:07.000000000","message":"here i am not sure, if set is a subject here, it should should stay like it was.","commit_id":"d28306ba128872e5360dd17a6e5a123de7eeae64"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"d67394f8627939965a6da9c2ad39892e627d54a6","unresolved":false,"context_lines":[{"line_number":52,"context_line":"actual resource sharing among ironic and OneView users."},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"Since OneView can claim nodes in ``available`` state at any time, a set of"},{"line_number":55,"context_line":"tasks running periodically to detect nodes in use by OneView. A node in use by"},{"line_number":56,"context_line":"OneView is placed in ``manageable`` state and has maintenance mode set. Once"},{"line_number":57,"context_line":"the node is no longer in use, these tasks will make place them back in"},{"line_number":58,"context_line":"``available`` state and clear maintenance mode."}],"source_content_type":"text/x-rst","patch_set":3,"id":"ba2be162_9a418e3c","line":55,"updated":"2017-03-01 10:34:04.000000000","message":"I think this was correct: \"a set ... runs\"","commit_id":"682047dfa281d51c0d7eaee13a7adfd607894212"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"d67394f8627939965a6da9c2ad39892e627d54a6","unresolved":false,"context_lines":[{"line_number":315,"context_line":"   Ironic manages OneView machines either when they have"},{"line_number":316,"context_line":"   a Server Profile applied by the driver or when they don\u0027t have any Server"},{"line_number":317,"context_line":"   Profile. Trying to change the power state of the machine in OneView without"},{"line_number":318,"context_line":"   first assigning a Server Profile, will lead to allowing Ironic to revert the"},{"line_number":319,"context_line":"   power state change. Ironic will NOT change the power state of machines"},{"line_number":320,"context_line":"   which the Server Profile was applied by another OneView user."},{"line_number":321,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"ba2be162_ba3e52b6","line":318,"updated":"2017-03-01 10:34:04.000000000","message":"while we\u0027re here, I\u0027d change \"lead to allowing\" to just \"allow\" or something like \"cause\"","commit_id":"682047dfa281d51c0d7eaee13a7adfd607894212"},{"author":{"_account_id":22226,"name":"Varun Gadiraju","email":"varun.gadiraju@intel.com","username":"gadirajv"},"change_message_id":"22be42b467d78ffdc54acf55f1cd1fc6f53ecabf","unresolved":false,"context_lines":[{"line_number":315,"context_line":"   Ironic manages OneView machines either when they have"},{"line_number":316,"context_line":"   a Server Profile applied by the driver or when they don\u0027t have any Server"},{"line_number":317,"context_line":"   Profile. Trying to change the power state of the machine in OneView without"},{"line_number":318,"context_line":"   first assigning a Server Profile, will lead to allowing Ironic to revert the"},{"line_number":319,"context_line":"   power state change. Ironic will NOT change the power state of machines"},{"line_number":320,"context_line":"   which the Server Profile was applied by another OneView user."},{"line_number":321,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"ba2be162_6f7c85ec","line":318,"range":{"start_line":318,"start_character":35,"end_line":318,"end_character":36},"updated":"2017-03-03 20:50:40.000000000","message":"this comma is grammatically incorrect. This change is not necessary :)","commit_id":"2caf7ac73b858a0a155fce861216a450137c558c"},{"author":{"_account_id":22226,"name":"Varun Gadiraju","email":"varun.gadiraju@intel.com","username":"gadirajv"},"change_message_id":"22be42b467d78ffdc54acf55f1cd1fc6f53ecabf","unresolved":false,"context_lines":[{"line_number":347,"context_line":"provisioned, in other words, when the `provisioning_state` is set to `active`."},{"line_number":348,"context_line":""},{"line_number":349,"context_line":".. note::"},{"line_number":350,"context_line":"   Verify if the node has not already been migrated, checking if there is"},{"line_number":351,"context_line":"   a `dynamic_allocation` field set to ``True`` in the `driver_info` namespace"},{"line_number":352,"context_line":"   doing::"},{"line_number":353,"context_line":""},{"line_number":354,"context_line":"     $ ironic node-show  --fields driver_info"},{"line_number":355,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"ba2be162_af591d90","line":352,"range":{"start_line":350,"start_character":0,"end_line":352,"end_character":10},"updated":"2017-03-03 20:50:40.000000000","message":"This whole sentence could be better written.\n\nI would change to \"Verify if the node has not already been migrated by checking if there is a `dynamic_allocation` field set to ``True`` in the `driver_info` namespace::\"","commit_id":"2caf7ac73b858a0a155fce861216a450137c558c"},{"author":{"_account_id":11878,"name":"Rushil Chugh","email":"rushil.chugh@gmail.com","username":"rushil"},"change_message_id":"30cbb3c598b3a2c11c9f73ece59a9881176dd456","unresolved":false,"context_lines":[{"line_number":349,"context_line":".. note::"},{"line_number":350,"context_line":"   Verify if the node has not already been migrated by checking if there is"},{"line_number":351,"context_line":"   a `dynamic_allocation` field set to ``True`` in the `driver_info`"},{"line_number":352,"context_line":"   namespace::"},{"line_number":353,"context_line":""},{"line_number":354,"context_line":"     $ ironic node-show  --fields driver_info"},{"line_number":355,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"ba2be162_c2c7ea61","line":352,"range":{"start_line":352,"start_character":3,"end_line":352,"end_character":12},"updated":"2017-03-06 01:03:14.000000000","message":"s/namespace/namespace by doing","commit_id":"d722aef1aa20e292f743c34017e092ff6a9b389a"}]}