)]}'
{"/PATCHSET_LEVEL":[{"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":"d62a7b76499e99a15f00e677be32ef69b346bbce","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"006c6bc3_06c4dac0","updated":"2023-09-08 19:31:13.000000000","message":"I think I cleaned up what Dmitry was concerned about.","commit_id":"76abc6930abe1147276cdcd6ac66831f36fc8321"},{"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":"7befc850eb85a68d80037ee40937a27a5fd78176","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"0cc7e544_ba156a94","updated":"2023-09-07 21:17:18.000000000","message":"Ironic cores + releases team; should Ironic mention the Nova driver changes from here? https://review.opendev.org/c/openstack/releases/+/894213 seems like there\u0027s value in a pointer at a minimum.","commit_id":"76abc6930abe1147276cdcd6ac66831f36fc8321"},{"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":"d303c292d6b6a87a1e5c4ddb4c6459986711b390","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"5adc25b7_27bb4e72","updated":"2023-09-07 21:15:24.000000000","message":"Please permit a small window for Ironic cores to review these before landing. Thanks!","commit_id":"76abc6930abe1147276cdcd6ac66831f36fc8321"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"0b9415c307921b392b80972c2ad08f7155bba4ad","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4aeb5614_04786b4b","in_reply_to":"006c6bc3_06c4dac0","updated":"2023-09-13 14:45:58.000000000","message":"One typo that Dmitry mentioned is still in the text.","commit_id":"76abc6930abe1147276cdcd6ac66831f36fc8321"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"0b9415c307921b392b80972c2ad08f7155bba4ad","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"47a819ad_22e38fc3","updated":"2023-09-13 14:45:58.000000000","message":"There are some nits (giving -1 to highlight), otherwise so far this looks like this: https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_f00/894228/3/check/openstack-tox-docs/f00fa5a/docs/bobcat/highlights.html#ironic","commit_id":"7fa287f59098813075d42b80d8d3bfeccd82e728"},{"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":"7823225fcecdf0e4f2dac7f5dbd0b079b6ba376c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"c6038bfc_36d665ce","updated":"2023-09-13 20:26:59.000000000","message":"Thanks for highlighting the typo I missed, and and and and that I missed ;)","commit_id":"bd15bc6dd99b54fc0577e72cec129e3ba6a540c2"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"e4654cb8aee265c5b6093bcf36a258ee3639befd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"0ff5d192_0e24b6c4","in_reply_to":"c6038bfc_36d665ce","updated":"2023-09-14 08:23:51.000000000","message":"Thanks, too, for the update :) Let\u0027s get this merged then! \\o/","commit_id":"bd15bc6dd99b54fc0577e72cec129e3ba6a540c2"}],"deliverables/bobcat/ironic.yaml":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"9f930a85fe85ec715d46bef9f73e0b44967c1a53","unresolved":true,"context_lines":[{"line_number":28,"context_line":"    a step-based action, those steps can include actions to be run on the child"},{"line_number":29,"context_line":"    node. See `parent node documentation \u003chttps://docs.openstack.org/ironic/latest/admin/cleaning.html#parent-nodes\u003e`_"},{"line_number":30,"context_line":"    for more information."},{"line_number":31,"context_line":"  - Ironic now has basic support for serviving nodes in ``ACTIVE`` state. "},{"line_number":32,"context_line":"    Service Steps allows operators to use steps to perform service on"},{"line_number":33,"context_line":"    deployed nodes similarly to how they are used during deployment and"},{"line_number":34,"context_line":"    cleaning. Previously, Ironic would not perform operations on active nodes."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"f8f76dd2_01b858da","line":31,"range":{"start_line":31,"start_character":37,"end_line":31,"end_character":46},"updated":"2023-09-08 07:50:00.000000000","message":"servicing. and maybe better introduce it as \"servicing\" (to make it clear we\u0027re talking about a new term)","commit_id":"76abc6930abe1147276cdcd6ac66831f36fc8321"},{"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":"764797f6afd1d54e270a9ea5f79c2b1a36e43b88","unresolved":false,"context_lines":[{"line_number":28,"context_line":"    a step-based action, those steps can include actions to be run on the child"},{"line_number":29,"context_line":"    node. See `parent node documentation \u003chttps://docs.openstack.org/ironic/latest/admin/cleaning.html#parent-nodes\u003e`_"},{"line_number":30,"context_line":"    for more information."},{"line_number":31,"context_line":"  - Ironic now has basic support for serviving nodes in ``ACTIVE`` state. "},{"line_number":32,"context_line":"    Service Steps allows operators to use steps to perform service on"},{"line_number":33,"context_line":"    deployed nodes similarly to how they are used during deployment and"},{"line_number":34,"context_line":"    cleaning. Previously, Ironic would not perform operations on active nodes."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3e324c8d_6729d2f0","line":31,"range":{"start_line":31,"start_character":37,"end_line":31,"end_character":46},"in_reply_to":"f8f76dd2_01b858da","updated":"2023-09-13 20:25:36.000000000","message":"Done","commit_id":"76abc6930abe1147276cdcd6ac66831f36fc8321"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"9f930a85fe85ec715d46bef9f73e0b44967c1a53","unresolved":true,"context_lines":[{"line_number":37,"context_line":"  - Ironic has promoted support for firmware upgrading and information to a"},{"line_number":38,"context_line":"    new interface and API named ``FirmwareInterface``. This includes new API"},{"line_number":39,"context_line":"    support, at ``/v1/nodes/{node_ident}/firmware`` for getting firmware"},{"line_number":40,"context_line":"    information. In addition, operators can use the steps provided by the new"},{"line_number":41,"context_line":"    ``FirmwareInterface`` their firmware upgrade process for step-based"},{"line_number":42,"context_line":"    automations. Initial support in this release is limited to redfish hardware."},{"line_number":43,"context_line":"  - Additional steps have been added to enable operators to control the flow of"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"64b03cf9_6d987386","line":40,"updated":"2023-09-08 07:50:00.000000000","message":"nit: \"clean/deploy steps\"? (do we support deploy steps even?)","commit_id":"76abc6930abe1147276cdcd6ac66831f36fc8321"},{"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":"80c2b09ee5ca388358e1cde9a7b7d6fe6bca3051","unresolved":true,"context_lines":[{"line_number":37,"context_line":"  - Ironic has promoted support for firmware upgrading and information to a"},{"line_number":38,"context_line":"    new interface and API named ``FirmwareInterface``. This includes new API"},{"line_number":39,"context_line":"    support, at ``/v1/nodes/{node_ident}/firmware`` for getting firmware"},{"line_number":40,"context_line":"    information. In addition, operators can use the steps provided by the new"},{"line_number":41,"context_line":"    ``FirmwareInterface`` their firmware upgrade process for step-based"},{"line_number":42,"context_line":"    automations. Initial support in this release is limited to redfish hardware."},{"line_number":43,"context_line":"  - Additional steps have been added to enable operators to control the flow of"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"9ef340a8_a7d6eecd","line":40,"in_reply_to":"64b03cf9_6d987386","updated":"2023-09-08 19:02:36.000000000","message":"We need to come up with a name for these steps to make them reasonable to talk about.\n\nAnd yeah, we have deploy templates which can contain steps. I\u0027d like to work next cycle to make them even more parallel (see: https://review.opendev.org/c/openstack/ironic-specs/+/890164 which I\u0027ll be working on more after release)","commit_id":"76abc6930abe1147276cdcd6ac66831f36fc8321"},{"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":"764797f6afd1d54e270a9ea5f79c2b1a36e43b88","unresolved":true,"context_lines":[{"line_number":37,"context_line":"  - Ironic has promoted support for firmware upgrading and information to a"},{"line_number":38,"context_line":"    new interface and API named ``FirmwareInterface``. This includes new API"},{"line_number":39,"context_line":"    support, at ``/v1/nodes/{node_ident}/firmware`` for getting firmware"},{"line_number":40,"context_line":"    information. In addition, operators can use the steps provided by the new"},{"line_number":41,"context_line":"    ``FirmwareInterface`` their firmware upgrade process for step-based"},{"line_number":42,"context_line":"    automations. Initial support in this release is limited to redfish hardware."},{"line_number":43,"context_line":"  - Additional steps have been added to enable operators to control the flow of"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"5c63044f_e6b56e21","line":40,"in_reply_to":"9ef340a8_a7d6eecd","updated":"2023-09-13 20:25:36.000000000","message":"I added some language to help clarify this.","commit_id":"76abc6930abe1147276cdcd6ac66831f36fc8321"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"9f930a85fe85ec715d46bef9f73e0b44967c1a53","unresolved":true,"context_lines":[{"line_number":39,"context_line":"    support, at ``/v1/nodes/{node_ident}/firmware`` for getting firmware"},{"line_number":40,"context_line":"    information. In addition, operators can use the steps provided by the new"},{"line_number":41,"context_line":"    ``FirmwareInterface`` their firmware upgrade process for step-based"},{"line_number":42,"context_line":"    automations. Initial support in this release is limited to redfish hardware."},{"line_number":43,"context_line":"  - Additional steps have been added to enable operators to control the flow of"},{"line_number":44,"context_line":"    the automated process. The first new step, ``wait``, which waits for a"},{"line_number":45,"context_line":"    specified amount of time or until the next heartbeat. The second new step,"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"529b5053_09de281a","line":42,"range":{"start_line":42,"start_character":17,"end_line":42,"end_character":79},"updated":"2023-09-08 07:50:00.000000000","message":"Not yet merged..","commit_id":"76abc6930abe1147276cdcd6ac66831f36fc8321"},{"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":"80c2b09ee5ca388358e1cde9a7b7d6fe6bca3051","unresolved":true,"context_lines":[{"line_number":39,"context_line":"    support, at ``/v1/nodes/{node_ident}/firmware`` for getting firmware"},{"line_number":40,"context_line":"    information. In addition, operators can use the steps provided by the new"},{"line_number":41,"context_line":"    ``FirmwareInterface`` their firmware upgrade process for step-based"},{"line_number":42,"context_line":"    automations. Initial support in this release is limited to redfish hardware."},{"line_number":43,"context_line":"  - Additional steps have been added to enable operators to control the flow of"},{"line_number":44,"context_line":"    the automated process. The first new step, ``wait``, which waits for a"},{"line_number":45,"context_line":"    specified amount of time or until the next heartbeat. The second new step,"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"81cf3594_8f478350","line":42,"range":{"start_line":42,"start_character":17,"end_line":42,"end_character":79},"in_reply_to":"529b5053_09de281a","updated":"2023-09-08 19:02:36.000000000","message":"If we don\u0027t land redfish support; this entire bulletpoint needs to go. It\u0027s useless without a working backend. I\u0027d rather us work to ensure this makes the release.","commit_id":"76abc6930abe1147276cdcd6ac66831f36fc8321"},{"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":"764797f6afd1d54e270a9ea5f79c2b1a36e43b88","unresolved":false,"context_lines":[{"line_number":39,"context_line":"    support, at ``/v1/nodes/{node_ident}/firmware`` for getting firmware"},{"line_number":40,"context_line":"    information. In addition, operators can use the steps provided by the new"},{"line_number":41,"context_line":"    ``FirmwareInterface`` their firmware upgrade process for step-based"},{"line_number":42,"context_line":"    automations. Initial support in this release is limited to redfish hardware."},{"line_number":43,"context_line":"  - Additional steps have been added to enable operators to control the flow of"},{"line_number":44,"context_line":"    the automated process. The first new step, ``wait``, which waits for a"},{"line_number":45,"context_line":"    specified amount of time or until the next heartbeat. The second new step,"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"49de2104_e26e227e","line":42,"range":{"start_line":42,"start_character":17,"end_line":42,"end_character":79},"in_reply_to":"81cf3594_8f478350","updated":"2023-09-13 20:25:36.000000000","message":"Done","commit_id":"76abc6930abe1147276cdcd6ac66831f36fc8321"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"9f930a85fe85ec715d46bef9f73e0b44967c1a53","unresolved":true,"context_lines":[{"line_number":40,"context_line":"    information. In addition, operators can use the steps provided by the new"},{"line_number":41,"context_line":"    ``FirmwareInterface`` their firmware upgrade process for step-based"},{"line_number":42,"context_line":"    automations. Initial support in this release is limited to redfish hardware."},{"line_number":43,"context_line":"  - Additional steps have been added to enable operators to control the flow of"},{"line_number":44,"context_line":"    the automated process. The first new step, ``wait``, which waits for a"},{"line_number":45,"context_line":"    specified amount of time or until the next heartbeat. The second new step,"},{"line_number":46,"context_line":"    ``hold``, stops evaluation of steps until the ``unhold`` verb is submitted"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"b83bb791_16c922c9","line":43,"updated":"2023-09-08 07:50:00.000000000","message":"Same: \"clean/deploy/service steps\", otherwise steps is too ambiguous.","commit_id":"76abc6930abe1147276cdcd6ac66831f36fc8321"},{"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":"764797f6afd1d54e270a9ea5f79c2b1a36e43b88","unresolved":false,"context_lines":[{"line_number":40,"context_line":"    information. In addition, operators can use the steps provided by the new"},{"line_number":41,"context_line":"    ``FirmwareInterface`` their firmware upgrade process for step-based"},{"line_number":42,"context_line":"    automations. Initial support in this release is limited to redfish hardware."},{"line_number":43,"context_line":"  - Additional steps have been added to enable operators to control the flow of"},{"line_number":44,"context_line":"    the automated process. The first new step, ``wait``, which waits for a"},{"line_number":45,"context_line":"    specified amount of time or until the next heartbeat. The second new step,"},{"line_number":46,"context_line":"    ``hold``, stops evaluation of steps until the ``unhold`` verb is submitted"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"d2ac445e_ff811077","line":43,"in_reply_to":"b83bb791_16c922c9","updated":"2023-09-13 20:25:36.000000000","message":"Done","commit_id":"76abc6930abe1147276cdcd6ac66831f36fc8321"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"9f930a85fe85ec715d46bef9f73e0b44967c1a53","unresolved":true,"context_lines":[{"line_number":54,"context_line":"    ``reboot`` names have been reserved for internal Ironic use. If used as a"},{"line_number":55,"context_line":"    step name, Ironic will perform the corresponding power operation on the"},{"line_number":56,"context_line":"    node."},{"line_number":57,"context_line":"  - Conductors can now be restarted with less impact on nodes with an operation"},{"line_number":58,"context_line":"    in progress by now honoring the value of"},{"line_number":59,"context_line":"    ``[DEFAULT]/graceful_shutdown_timeout``. Operators can set this to the"},{"line_number":60,"context_line":"    number of seconds their conductors have to complete operations in"},{"line_number":61,"context_line":"    progress. Previously, all in-progress operations would fail regardless of"},{"line_number":62,"context_line":"    the value of this setting."},{"line_number":63,"context_line":"  - Ironic deployers can now set separate defaults, by CPU architecture, for"},{"line_number":64,"context_line":"    configuration of kernel and ramdisk for deploy and rescue operations."},{"line_number":65,"context_line":"  - Operators are now able to disable MD5 use in Ironic by setting "}],"source_content_type":"text/x-yaml","patch_set":2,"id":"26219423_f4566f58","line":62,"range":{"start_line":57,"start_character":0,"end_line":62,"end_character":30},"updated":"2023-09-08 07:50:00.000000000","message":"This seems release note worthy, but probably not something to highlight openstack-wide.","commit_id":"76abc6930abe1147276cdcd6ac66831f36fc8321"},{"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":"80c2b09ee5ca388358e1cde9a7b7d6fe6bca3051","unresolved":false,"context_lines":[{"line_number":54,"context_line":"    ``reboot`` names have been reserved for internal Ironic use. If used as a"},{"line_number":55,"context_line":"    step name, Ironic will perform the corresponding power operation on the"},{"line_number":56,"context_line":"    node."},{"line_number":57,"context_line":"  - Conductors can now be restarted with less impact on nodes with an operation"},{"line_number":58,"context_line":"    in progress by now honoring the value of"},{"line_number":59,"context_line":"    ``[DEFAULT]/graceful_shutdown_timeout``. Operators can set this to the"},{"line_number":60,"context_line":"    number of seconds their conductors have to complete operations in"},{"line_number":61,"context_line":"    progress. Previously, all in-progress operations would fail regardless of"},{"line_number":62,"context_line":"    the value of this setting."},{"line_number":63,"context_line":"  - Ironic deployers can now set separate defaults, by CPU architecture, for"},{"line_number":64,"context_line":"    configuration of kernel and ramdisk for deploy and rescue operations."},{"line_number":65,"context_line":"  - Operators are now able to disable MD5 use in Ironic by setting "}],"source_content_type":"text/x-yaml","patch_set":2,"id":"266e126d_5fe19b81","line":62,"range":{"start_line":57,"start_character":0,"end_line":62,"end_character":30},"in_reply_to":"26219423_f4566f58","updated":"2023-09-08 19:02:36.000000000","message":"Ack","commit_id":"76abc6930abe1147276cdcd6ac66831f36fc8321"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"9f930a85fe85ec715d46bef9f73e0b44967c1a53","unresolved":true,"context_lines":[{"line_number":60,"context_line":"    number of seconds their conductors have to complete operations in"},{"line_number":61,"context_line":"    progress. Previously, all in-progress operations would fail regardless of"},{"line_number":62,"context_line":"    the value of this setting."},{"line_number":63,"context_line":"  - Ironic deployers can now set separate defaults, by CPU architecture, for"},{"line_number":64,"context_line":"    configuration of kernel and ramdisk for deploy and rescue operations."},{"line_number":65,"context_line":"  - Operators are now able to disable MD5 use in Ironic by setting "},{"line_number":66,"context_line":"    ``[agent]/allow_md5_checksum`` to false. This option defaults to ``true`` to"},{"line_number":67,"context_line":"    preserve existing behavior."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"d94f0ad8_08402b2a","line":67,"range":{"start_line":63,"start_character":1,"end_line":67,"end_character":31},"updated":"2023-09-08 07:50:00.000000000","message":"Same for both of these. (Unless you want to announce the pending MD5 deprecation which is not obvious from the last item)","commit_id":"76abc6930abe1147276cdcd6ac66831f36fc8321"},{"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":"80c2b09ee5ca388358e1cde9a7b7d6fe6bca3051","unresolved":false,"context_lines":[{"line_number":60,"context_line":"    number of seconds their conductors have to complete operations in"},{"line_number":61,"context_line":"    progress. Previously, all in-progress operations would fail regardless of"},{"line_number":62,"context_line":"    the value of this setting."},{"line_number":63,"context_line":"  - Ironic deployers can now set separate defaults, by CPU architecture, for"},{"line_number":64,"context_line":"    configuration of kernel and ramdisk for deploy and rescue operations."},{"line_number":65,"context_line":"  - Operators are now able to disable MD5 use in Ironic by setting "},{"line_number":66,"context_line":"    ``[agent]/allow_md5_checksum`` to false. This option defaults to ``true`` to"},{"line_number":67,"context_line":"    preserve existing behavior."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"b85f9fcb_0cd9e913","line":67,"range":{"start_line":63,"start_character":1,"end_line":67,"end_character":31},"in_reply_to":"d94f0ad8_08402b2a","updated":"2023-09-08 19:02:36.000000000","message":"Ack","commit_id":"76abc6930abe1147276cdcd6ac66831f36fc8321"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"0b9415c307921b392b80972c2ad08f7155bba4ad","unresolved":true,"context_lines":[{"line_number":28,"context_line":"    a step-based action, those steps can include actions to be run on the child"},{"line_number":29,"context_line":"    node. See `parent node documentation \u003chttps://docs.openstack.org/ironic/latest/admin/cleaning.html#parent-nodes\u003e`_"},{"line_number":30,"context_line":"    for more information."},{"line_number":31,"context_line":"  - Ironic now has basic support for serviving nodes. Servicing allows"},{"line_number":32,"context_line":"    allows operators to use steps, like you would for cleaning, to perform"},{"line_number":33,"context_line":"    service on deployed nodes in ``ACTIVE`` state. Previously, Ironic would"},{"line_number":34,"context_line":"    not perform operations on active nodes. See"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"ca48cc82_287797f1","line":31,"range":{"start_line":31,"start_character":37,"end_line":31,"end_character":46},"updated":"2023-09-13 14:45:58.000000000","message":"nit: as Dmitry noted already, this is i guess a typo?","commit_id":"7fa287f59098813075d42b80d8d3bfeccd82e728"},{"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":"764797f6afd1d54e270a9ea5f79c2b1a36e43b88","unresolved":false,"context_lines":[{"line_number":28,"context_line":"    a step-based action, those steps can include actions to be run on the child"},{"line_number":29,"context_line":"    node. See `parent node documentation \u003chttps://docs.openstack.org/ironic/latest/admin/cleaning.html#parent-nodes\u003e`_"},{"line_number":30,"context_line":"    for more information."},{"line_number":31,"context_line":"  - Ironic now has basic support for serviving nodes. Servicing allows"},{"line_number":32,"context_line":"    allows operators to use steps, like you would for cleaning, to perform"},{"line_number":33,"context_line":"    service on deployed nodes in ``ACTIVE`` state. Previously, Ironic would"},{"line_number":34,"context_line":"    not perform operations on active nodes. See"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"4530d3c3_210dcba1","line":31,"range":{"start_line":31,"start_character":37,"end_line":31,"end_character":46},"in_reply_to":"ca48cc82_287797f1","updated":"2023-09-13 20:25:36.000000000","message":"Ack","commit_id":"7fa287f59098813075d42b80d8d3bfeccd82e728"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"0b9415c307921b392b80972c2ad08f7155bba4ad","unresolved":true,"context_lines":[{"line_number":28,"context_line":"    a step-based action, those steps can include actions to be run on the child"},{"line_number":29,"context_line":"    node. See `parent node documentation \u003chttps://docs.openstack.org/ironic/latest/admin/cleaning.html#parent-nodes\u003e`_"},{"line_number":30,"context_line":"    for more information."},{"line_number":31,"context_line":"  - Ironic now has basic support for serviving nodes. Servicing allows"},{"line_number":32,"context_line":"    allows operators to use steps, like you would for cleaning, to perform"},{"line_number":33,"context_line":"    service on deployed nodes in ``ACTIVE`` state. Previously, Ironic would"},{"line_number":34,"context_line":"    not perform operations on active nodes. See"},{"line_number":35,"context_line":"    `Node servicing documentation \u003chttps://docs.openstack.org/ironic/latest/admin/servicing\u003e`_"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"401aaaf9_5b4f1927","line":32,"range":{"start_line":31,"start_character":64,"end_line":32,"end_character":10},"updated":"2023-09-13 14:45:58.000000000","message":"nit: one \u0027allows\u0027 would be enough? o:)","commit_id":"7fa287f59098813075d42b80d8d3bfeccd82e728"},{"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":"764797f6afd1d54e270a9ea5f79c2b1a36e43b88","unresolved":false,"context_lines":[{"line_number":28,"context_line":"    a step-based action, those steps can include actions to be run on the child"},{"line_number":29,"context_line":"    node. See `parent node documentation \u003chttps://docs.openstack.org/ironic/latest/admin/cleaning.html#parent-nodes\u003e`_"},{"line_number":30,"context_line":"    for more information."},{"line_number":31,"context_line":"  - Ironic now has basic support for serviving nodes. Servicing allows"},{"line_number":32,"context_line":"    allows operators to use steps, like you would for cleaning, to perform"},{"line_number":33,"context_line":"    service on deployed nodes in ``ACTIVE`` state. Previously, Ironic would"},{"line_number":34,"context_line":"    not perform operations on active nodes. See"},{"line_number":35,"context_line":"    `Node servicing documentation \u003chttps://docs.openstack.org/ironic/latest/admin/servicing\u003e`_"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"3c677df4_61240380","line":32,"range":{"start_line":31,"start_character":64,"end_line":32,"end_character":10},"in_reply_to":"401aaaf9_5b4f1927","updated":"2023-09-13 20:25:36.000000000","message":"Ack","commit_id":"7fa287f59098813075d42b80d8d3bfeccd82e728"}]}
