)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"9b5378d1a03adc20b4709be224efecfb3e113186","unresolved":false,"context_lines":[{"line_number":13,"context_line":"Change-Id: I55fb5e3f23e763ca93648bdfbebba6447281717a"},{"line_number":14,"context_line":"Closes-Bug: 1611155"},{"line_number":15,"context_line":"Depends-On: I940904fbbd44b595759883d944a54dd37fac5c35"},{"line_number":16,"context_line":"Signed-off-by: Ricardo Araújo Santos \u003cricardo@lsd.ufcg.edu.br\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3ac371cc_ba02ce07","line":16,"updated":"2016-08-18 20:59:20.000000000","message":"OpenStack does not follow the Signed-off-by convention. Please remove this from the commit message.","commit_id":"b3526fa2f677e1c3d31770fff26d6893d8ea2dac"}],"doc/source/drivers/oneview.rst":[{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"9b5378d1a03adc20b4709be224efecfb3e113186","unresolved":false,"context_lines":[{"line_number":298,"context_line":"The migration of a node from an ironic deployment using ``pre-allocation``"},{"line_number":299,"context_line":"model to the new ``dynamic allocation`` model can be done by using"},{"line_number":300,"context_line":"``ironic-oneview-cli`` facilities to migrate nodes (further details on [8]_)."},{"line_number":301,"context_line":"However, the same results can be achieved using the Ironic CLI for each node as"},{"line_number":302,"context_line":"explained below."},{"line_number":303,"context_line":""},{"line_number":304,"context_line":"Firstly, it is recommended to migrate nodes which have ``provision_state`` set"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3ac371cc_7a197679","line":301,"updated":"2016-08-18 20:59:20.000000000","message":"s/Ironic/ironic/","commit_id":"b3526fa2f677e1c3d31770fff26d6893d8ea2dac"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"9b5378d1a03adc20b4709be224efecfb3e113186","unresolved":false,"context_lines":[{"line_number":303,"context_line":""},{"line_number":304,"context_line":"Firstly, it is recommended to migrate nodes which have ``provision_state`` set"},{"line_number":305,"context_line":"to one of the following: `enroll`, `manageable`, `available`, `active`,"},{"line_number":306,"context_line":"`inspect failed`, `clean failed` or `error`. You can verify the state the nodes"},{"line_number":307,"context_line":"doing::"},{"line_number":308,"context_line":""},{"line_number":309,"context_line":"  $ ironic node-list"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3ac371cc_5ae63a17","line":306,"updated":"2016-08-18 20:59:20.000000000","message":"A few things:\n\n1) What states can you /not/ migrate nodes in? That might be a shorter list. An explanation why they can\u0027t be migrated in this state would be useful too.\n\n2) A grammar fix: \"You can verify the node\u0027s state by doing::\n\n3) You switch rapidly here from working on a list of nodes to talking about a single node. You might want to be more clear, for instance:\n\n\nStep 1: Find nodes to migrate\n-----------------------------\n(how to find nodes that need migration -- including *all* the criteria needed to determine if the node needs migration, such as states, and if it\u0027s already been migrated). You may even be able to use the filter argument to the ironic CLI to make this simpler for the user.\n\n\nStep 2: Which migration process?\n----------------------------------\n(use state and instance_uuid to determine if it needs a migration for a node w/an instance, or a migration for a node w/o an instance)\n\n\nand so on. But this part is a suggestion, take however much of it you wish.","commit_id":"b3526fa2f677e1c3d31770fff26d6893d8ea2dac"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"9b5378d1a03adc20b4709be224efecfb3e113186","unresolved":false,"context_lines":[{"line_number":310,"context_line":""},{"line_number":311,"context_line":"Secondly, put the node in maintenance mode doing::"},{"line_number":312,"context_line":""},{"line_number":313,"context_line":"  $ ironic node-set-maintenance \u003cnode_uuid\u003e on"},{"line_number":314,"context_line":""},{"line_number":315,"context_line":"Once assured the node is in maintenance mode, we can perform the migration"},{"line_number":316,"context_line":"itself. We check if the node has an instance associated with it by looking the"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3ac371cc_ba500e0c","line":313,"updated":"2016-08-18 20:59:20.000000000","message":"Shouldn\u0027t you set a reason here, and it\u0027s \"true\", not on.\n\nironic node-set-maintenance --reason \"Migrating node to dynamic allocation\" UUID true","commit_id":"b3526fa2f677e1c3d31770fff26d6893d8ea2dac"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"9b5378d1a03adc20b4709be224efecfb3e113186","unresolved":false,"context_lines":[{"line_number":312,"context_line":""},{"line_number":313,"context_line":"  $ ironic node-set-maintenance \u003cnode_uuid\u003e on"},{"line_number":314,"context_line":""},{"line_number":315,"context_line":"Once assured the node is in maintenance mode, we can perform the migration"},{"line_number":316,"context_line":"itself. We check if the node has an instance associated with it by looking the"},{"line_number":317,"context_line":"value of ``instance_uuid`` property. If it is set to `None`, the node is idle"},{"line_number":318,"context_line":"and we simply remove the Server Profile from the Server Hardware in OneView."}],"source_content_type":"text/x-rst","patch_set":1,"id":"3ac371cc_fa5a8629","line":315,"updated":"2016-08-18 20:59:20.000000000","message":"s/assured//","commit_id":"b3526fa2f677e1c3d31770fff26d6893d8ea2dac"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"9b5378d1a03adc20b4709be224efecfb3e113186","unresolved":false,"context_lines":[{"line_number":313,"context_line":"  $ ironic node-set-maintenance \u003cnode_uuid\u003e on"},{"line_number":314,"context_line":""},{"line_number":315,"context_line":"Once assured the node is in maintenance mode, we can perform the migration"},{"line_number":316,"context_line":"itself. We check if the node has an instance associated with it by looking the"},{"line_number":317,"context_line":"value of ``instance_uuid`` property. If it is set to `None`, the node is idle"},{"line_number":318,"context_line":"and we simply remove the Server Profile from the Server Hardware in OneView."},{"line_number":319,"context_line":"On the other case, the node has an active instance associated and we need to set"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3ac371cc_3af53ecb","line":316,"updated":"2016-08-18 20:59:20.000000000","message":"Inconsistent person here. Being more directive will solve this, like \"Check if the node has an instance associated by looking at the value of ``instance_uuid``\".\n\nAlso, I think you can simplify this a lot -- above when you\u0027re talking about states, you should be able to collapse down this state -- in general usage of ironic, you shouldn\u0027t have a node in deploying or active unless it has an instance_uuid. There\u0027s basically two migration processes and you can separate them earlier on to make the process more clear.","commit_id":"b3526fa2f677e1c3d31770fff26d6893d8ea2dac"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"9b5378d1a03adc20b4709be224efecfb3e113186","unresolved":false,"context_lines":[{"line_number":328,"context_line":""},{"line_number":329,"context_line":"And finally remove the node from maintenance mode doing::"},{"line_number":330,"context_line":""},{"line_number":331,"context_line":"  $ ironic node-set-maintenance \u003cnode_uuid\u003e off"},{"line_number":332,"context_line":""},{"line_number":333,"context_line":"3rd Party Tools"},{"line_number":334,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3ac371cc_9af0b2cb","line":331,"updated":"2016-08-18 20:59:20.000000000","message":"s/off/false/","commit_id":"b3526fa2f677e1c3d31770fff26d6893d8ea2dac"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"8dbf5264f74746096500072428882c2d3970cc8d","unresolved":false,"context_lines":[{"line_number":310,"context_line":"values [9_] are: `enroll`, `manageable`, `available`, `active` and `error`."},{"line_number":311,"context_line":""},{"line_number":312,"context_line":"One can migrate nodes in `clean failed` and `inspect failed` without any major"},{"line_number":313,"context_line":"problems as they can only be removed from such states using::"},{"line_number":314,"context_line":""},{"line_number":315,"context_line":"  $ ironic node-set-provision-state manage"},{"line_number":316,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"1ac06dbe_70f33d0b","line":313,"updated":"2016-08-19 21:02:00.000000000","message":"I\u0027m not sure what you mean by this paragraph? If you just mean clean failed and inspect failed nodes can be migrated, I\u0027d just add them to the above list.","commit_id":"32a18e2fb0da21160e910eca81cee9a1ecf315ef"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"8dbf5264f74746096500072428882c2d3970cc8d","unresolved":false,"context_lines":[{"line_number":316,"context_line":""},{"line_number":317,"context_line":"Dynamic allocation mode changes the way a Server Profile is associated with a"},{"line_number":318,"context_line":"node. In ``pre-allocation`` mode, when a node is registered in ironic, there"},{"line_number":319,"context_line":"must be a Server Profile applied to the Server Hardware represented by the given"},{"line_number":320,"context_line":"node what means, from the OneView point of view, the hardware is in use. In the"},{"line_number":321,"context_line":"``dynamic allocation`` mode a Server Hardware is associated only when the node"},{"line_number":322,"context_line":"is in use by the Compute service or the OneView itself. As a result, there are"}],"source_content_type":"text/x-rst","patch_set":2,"id":"1ac06dbe_906fd991","line":319,"updated":"2016-08-19 21:02:00.000000000","message":"If \"Server Profile\" and \"Server Hardware\" are oneview specific terms, they should probably be enclosed in quotation marks to indicate that -- otherwise they should be lower cased.","commit_id":"32a18e2fb0da21160e910eca81cee9a1ecf315ef"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"8dbf5264f74746096500072428882c2d3970cc8d","unresolved":false,"context_lines":[{"line_number":320,"context_line":"node what means, from the OneView point of view, the hardware is in use. In the"},{"line_number":321,"context_line":"``dynamic allocation`` mode a Server Hardware is associated only when the node"},{"line_number":322,"context_line":"is in use by the Compute service or the OneView itself. As a result, there are"},{"line_number":323,"context_line":"different steps to perform if the node is in use by the Compute service, in"},{"line_number":324,"context_line":"other words, when the `provisioning_state` is set to `active`."},{"line_number":325,"context_line":""},{"line_number":326,"context_line":".. note::"}],"source_content_type":"text/x-rst","patch_set":2,"id":"1ac06dbe_308f0525","line":323,"updated":"2016-08-19 21:02:00.000000000","message":"Lets just say \"As a result, there are different steps to perform if the node has an instance provisioned, in other words, when the `provisioning_state` is set to `active`.","commit_id":"32a18e2fb0da21160e910eca81cee9a1ecf315ef"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"8dbf5264f74746096500072428882c2d3970cc8d","unresolved":false,"context_lines":[{"line_number":328,"context_line":"   a `dynamic_allocation` field set to ``True`` in the `driver_info` namespace"},{"line_number":329,"context_line":"   doing::"},{"line_number":330,"context_line":""},{"line_number":331,"context_line":"     @ironic node-show  --fields driver_info"},{"line_number":332,"context_line":""},{"line_number":333,"context_line":"Migrating nodes in `active` state"},{"line_number":334,"context_line":"^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^"}],"source_content_type":"text/x-rst","patch_set":2,"id":"1ac06dbe_d09cb16b","line":331,"updated":"2016-08-19 21:02:00.000000000","message":"I think you meant $ instead of @? Plus spacing","commit_id":"32a18e2fb0da21160e910eca81cee9a1ecf315ef"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"8dbf5264f74746096500072428882c2d3970cc8d","unresolved":false,"context_lines":[{"line_number":362,"context_line":""},{"line_number":363,"context_line":"List nodes that are a given `provision_state` state doing::"},{"line_number":364,"context_line":""},{"line_number":365,"context_line":"  $ ironic node-list --provision-state \u003cprovision_state\u003e --fields uuid driver_info"},{"line_number":366,"context_line":""},{"line_number":367,"context_line":"Then, for each node in the list execute the following steps:"},{"line_number":368,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"1ac06dbe_109ac914","line":365,"updated":"2016-08-19 21:02:00.000000000","message":"You can remove/change this if you want since (like your comment) it\u0027s not possible to filter by all of them :)","commit_id":"32a18e2fb0da21160e910eca81cee9a1ecf315ef"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"8dbf5264f74746096500072428882c2d3970cc8d","unresolved":false,"context_lines":[{"line_number":366,"context_line":""},{"line_number":367,"context_line":"Then, for each node in the list execute the following steps:"},{"line_number":368,"context_line":""},{"line_number":369,"context_line":"1. Remove the node from the resource pool to prevent Bare Metal service from"},{"line_number":370,"context_line":"executing periodic tasks associated with the node and also prevent Compute"},{"line_number":371,"context_line":"service to place an instance on the node. In order to do this, put the node in"},{"line_number":372,"context_line":"maintenance mode using its ``uuid`` doing::"}],"source_content_type":"text/x-rst","patch_set":2,"id":"1ac06dbe_d055d11d","line":369,"updated":"2016-08-19 21:02:00.000000000","message":"You can just say:\n\n1. Place the node in maintenance mode to prevent ironic from working on the node during the migration. In order ...","commit_id":"32a18e2fb0da21160e910eca81cee9a1ecf315ef"}]}