)]}'
{"ansible/roles/ironic/tasks/config.yml":[{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"ea8c9fdd01714bdbea182036e45c7c8529eef896","unresolved":true,"context_lines":[{"line_number":164,"context_line":"    - inventory_hostname in groups[service.group]"},{"line_number":165,"context_line":"    - service.enabled | bool"},{"line_number":166,"context_line":"    - not enable_ironic_pxe_uefi | bool"},{"line_number":167,"context_line":"    - not enable_ironic_ipxe | bool"},{"line_number":168,"context_line":"  notify:"},{"line_number":169,"context_line":"    - Restart ironic-pxe container"},{"line_number":170,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"19ca47b2_e5a36d22","side":"PARENT","line":167,"updated":"2021-07-19 12:29:57.000000000","message":"Let\u0027s refactor this fix out.","commit_id":"e63d985ccb111eb8aeb3b02a41d8ce04acdb7016"}],"doc/source/reference/bare-metal/ironic-guide.rst":[{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"248a8d5e4f1df88ca688093b5ce7ef21662da081","unresolved":true,"context_lines":[{"line_number":78,"context_line":""},{"line_number":79,"context_line":"When iPXE booting is enabled, this will enable deployment of a docker"},{"line_number":80,"context_line":"container, called ``ironic_ipxe``, running the web server which iPXE uses to"},{"line_number":81,"context_line":"obtain its boot images. The server may also used by the ``direct`` deploy"},{"line_number":82,"context_line":"interface to serve disk images."},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"The port used for the iPXE webserver is controlled via ``ironic_ipxe_port`` in"}],"source_content_type":"text/x-rst","patch_set":1,"id":"83bbe6c5_87ac56d8","line":81,"range":{"start_line":81,"start_character":43,"end_line":81,"end_character":44},"updated":"2021-07-19 09:19:58.000000000","message":"be","commit_id":"a5ac90908fb03f5dd01a43d671327e5525fab203"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"ea8c9fdd01714bdbea182036e45c7c8529eef896","unresolved":true,"context_lines":[{"line_number":88,"context_line":""},{"line_number":89,"context_line":"    ironic_ipxe_port: \"8089\""},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"In order to enable the iPXE driver in ironic, set the ``[DEFAULT]"},{"line_number":92,"context_line":"enabled_boot_interfaces`` option in ``/etc/kolla/config/ironic.conf``:"},{"line_number":93,"context_line":""},{"line_number":94,"context_line":".. code-block:: yaml"}],"source_content_type":"text/x-rst","patch_set":1,"id":"010aeb0d_f506e9c5","line":91,"range":{"start_line":91,"start_character":38,"end_line":91,"end_character":44},"updated":"2021-07-19 12:29:57.000000000","message":"nit: Ironic","commit_id":"a5ac90908fb03f5dd01a43d671327e5525fab203"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"248a8d5e4f1df88ca688093b5ce7ef21662da081","unresolved":true,"context_lines":[{"line_number":88,"context_line":""},{"line_number":89,"context_line":"    ironic_ipxe_port: \"8089\""},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"In order to enable the iPXE driver in ironic, set the ``[DEFAULT]"},{"line_number":92,"context_line":"enabled_boot_interfaces`` option in ``/etc/kolla/config/ironic.conf``:"},{"line_number":93,"context_line":""},{"line_number":94,"context_line":".. code-block:: yaml"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"   [DEFAULT]"},{"line_number":97,"context_line":"   enabled_boot_interfaces \u003d ipxe"},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"The following changes will occur if iPXE booting is enabled:"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"- The inspection ramdisk and kernel will be loaded via iPXE"}],"source_content_type":"text/x-rst","patch_set":1,"id":"cf698508_e1f4f96c","line":98,"range":{"start_line":91,"start_character":0,"end_line":98,"end_character":0},"updated":"2021-07-19 09:19:58.000000000","message":"I would move this below the next list.\nAlso, why does not it happen by default with \u0027yes\u0027?","commit_id":"a5ac90908fb03f5dd01a43d671327e5525fab203"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"ea8c9fdd01714bdbea182036e45c7c8529eef896","unresolved":true,"context_lines":[{"line_number":88,"context_line":""},{"line_number":89,"context_line":"    ironic_ipxe_port: \"8089\""},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"In order to enable the iPXE driver in ironic, set the ``[DEFAULT]"},{"line_number":92,"context_line":"enabled_boot_interfaces`` option in ``/etc/kolla/config/ironic.conf``:"},{"line_number":93,"context_line":""},{"line_number":94,"context_line":".. code-block:: yaml"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"   [DEFAULT]"},{"line_number":97,"context_line":"   enabled_boot_interfaces \u003d ipxe"},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"The following changes will occur if iPXE booting is enabled:"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"- The inspection ramdisk and kernel will be loaded via iPXE"}],"source_content_type":"text/x-rst","patch_set":1,"id":"40a939b4_25925ce1","line":98,"range":{"start_line":91,"start_character":0,"end_line":98,"end_character":0},"in_reply_to":"c28100f1_54e05e48","updated":"2021-07-19 12:29:57.000000000","message":"1) The paragraph below is applicable to setting ``enable_ironic_ipxe: yes`` (and not overriding ``enabled_boot_interfaces``). Hence I suggested the move.\n\n2) Hmm, but setting ``enable_ironic_ipxe: yes`` hijacks ``tftp_root`` so I assume pxe booting is b0rken.","commit_id":"a5ac90908fb03f5dd01a43d671327e5525fab203"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"296f8a3da054a45beec13f434a06b692e861b7fc","unresolved":true,"context_lines":[{"line_number":88,"context_line":""},{"line_number":89,"context_line":"    ironic_ipxe_port: \"8089\""},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"In order to enable the iPXE driver in ironic, set the ``[DEFAULT]"},{"line_number":92,"context_line":"enabled_boot_interfaces`` option in ``/etc/kolla/config/ironic.conf``:"},{"line_number":93,"context_line":""},{"line_number":94,"context_line":".. code-block:: yaml"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"   [DEFAULT]"},{"line_number":97,"context_line":"   enabled_boot_interfaces \u003d ipxe"},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"The following changes will occur if iPXE booting is enabled:"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"- The inspection ramdisk and kernel will be loaded via iPXE"}],"source_content_type":"text/x-rst","patch_set":1,"id":"c28100f1_54e05e48","line":98,"range":{"start_line":91,"start_character":0,"end_line":98,"end_character":0},"in_reply_to":"cf698508_e1f4f96c","updated":"2021-07-19 10:14:53.000000000","message":"I considered changing the order, but decided it made more sense to state that it is not enabled by default.\n\nIt doesn\u0027t happen by default because we don\u0027t try to manage all the lists of enabled and default drivers. We do it in kayobe and I wish we didn\u0027t. There is also some history, in that originally ipxe was enabled by setting ipxe_enabled \u003d True in ironic.conf. Then it became a separate boot interface to allow mixing pxe and ipxe nodes in the same deployment.","commit_id":"a5ac90908fb03f5dd01a43d671327e5525fab203"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"ea8c9fdd01714bdbea182036e45c7c8529eef896","unresolved":true,"context_lines":[{"line_number":87,"context_line":""},{"line_number":88,"context_line":"The following changes will occur if iPXE booting is enabled:"},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"- Ironic will be configured with the ``ipxe_enabled`` configuration option set"},{"line_number":91,"context_line":"  to true"},{"line_number":92,"context_line":"- The inspection ramdisk and kernel will be loaded via iPXE"},{"line_number":93,"context_line":"- The DHCP servers will be configured to chainload iPXE from an existing PXE"}],"source_content_type":"text/x-rst","patch_set":2,"id":"65a4015b_d6a1fae5","side":"PARENT","line":90,"updated":"2021-07-19 12:29:57.000000000","message":"And perhaps refactor this docfix.","commit_id":"e63d985ccb111eb8aeb3b02a41d8ce04acdb7016"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"893106af53c126793bc4ab29ee0be50bebdbce51","unresolved":true,"context_lines":[{"line_number":75,"context_line":""},{"line_number":76,"context_line":"   enable_ironic_ipxe: \"yes\""},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"When iPXE booting is enabled, this will enable use of a docker container,"},{"line_number":79,"context_line":"called ``ironic_ipxe``, running the web server which iPXE uses to obtain its"},{"line_number":80,"context_line":"boot images. The server may also be used by the ``direct`` deploy interface to"},{"line_number":81,"context_line":"serve disk images."},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"The port used for the iPXE webserver is controlled via ``ironic_ipxe_port`` in"},{"line_number":84,"context_line":"``/etc/kolla/globals.yml``:"}],"source_content_type":"text/x-rst","patch_set":5,"id":"04a93ee4_1bc457d5","line":81,"range":{"start_line":78,"start_character":0,"end_line":81,"end_character":18},"updated":"2021-07-21 16:12:34.000000000","message":"This is confusing as it suggests this enables the server and it is required by the \u0027direct\u0027 deployment interface and that\u0027s not the case. Mayhaps:\n\n\"When iPXE booting is enabled, the ``ironic_ipxe`` container is used to serve the iPXE boot images as described below. Regardless of the setting above, the same container is used to support the ``direct`` deploy interface.","commit_id":"2c9e3ae10593761806ac94fa3e9b5775409e452c"}],"releasenotes/notes/enable-ipxe-cf461344bdb99881.yaml":[{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"893106af53c126793bc4ab29ee0be50bebdbce51","unresolved":true,"context_lines":[{"line_number":2,"context_line":"upgrade:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    An HTTP server is now always deployed for Ironic conductor, while"},{"line_number":5,"context_line":"    previously it was only deployed when iPXE is enabled."},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"    In the Xena release, Ironic removed the iSCSI driver. The recommended"},{"line_number":8,"context_line":"    deploy driver is ``direct``, which uses HTTP to transfer the disk image."}],"source_content_type":"text/x-yaml","patch_set":5,"id":"ddf12689_33e8e7d7","line":5,"range":{"start_line":5,"start_character":46,"end_line":5,"end_character":49},"updated":"2021-07-21 16:12:34.000000000","message":"was","commit_id":"2c9e3ae10593761806ac94fa3e9b5775409e452c"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"ece1bd23cc9cd88a83d4f294978312674cfa1b17","unresolved":true,"context_lines":[{"line_number":2,"context_line":"upgrade:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    An HTTP server is now always deployed for Ironic conductor, while"},{"line_number":5,"context_line":"    previously it was only deployed when iPXE is enabled."},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"    In the Xena release, Ironic removed the iSCSI driver. The recommended"},{"line_number":8,"context_line":"    deploy driver is ``direct``, which uses HTTP to transfer the disk image."}],"source_content_type":"text/x-yaml","patch_set":5,"id":"c36301ec_2ac902f1","line":5,"range":{"start_line":5,"start_character":46,"end_line":5,"end_character":49},"in_reply_to":"ddf12689_33e8e7d7","updated":"2021-07-22 08:46:26.000000000","message":"I don\u0027t think this is in the same tense as the previous behaviour of kolla ansible. It\u0027s a logical boolean variable.","commit_id":"2c9e3ae10593761806ac94fa3e9b5775409e452c"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"893106af53c126793bc4ab29ee0be50bebdbce51","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"    In the Xena release, Ironic removed the iSCSI driver. The recommended"},{"line_number":8,"context_line":"    deploy driver is ``direct``, which uses HTTP to transfer the disk image."},{"line_number":9,"context_line":"    This requires an HTTP server, and the simplest option is to use the one"},{"line_number":10,"context_line":"    previously deployed when ``enable_ironic_ipxe`` is set to ``true``."}],"source_content_type":"text/x-yaml","patch_set":5,"id":"e945ad86_ddd15b28","line":9,"range":{"start_line":9,"start_character":58,"end_line":9,"end_character":61},"updated":"2021-07-21 16:12:34.000000000","message":"was","commit_id":"2c9e3ae10593761806ac94fa3e9b5775409e452c"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"ece1bd23cc9cd88a83d4f294978312674cfa1b17","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"    In the Xena release, Ironic removed the iSCSI driver. The recommended"},{"line_number":8,"context_line":"    deploy driver is ``direct``, which uses HTTP to transfer the disk image."},{"line_number":9,"context_line":"    This requires an HTTP server, and the simplest option is to use the one"},{"line_number":10,"context_line":"    previously deployed when ``enable_ironic_ipxe`` is set to ``true``."}],"source_content_type":"text/x-yaml","patch_set":5,"id":"e2d4a20d_bbf23dd6","line":9,"range":{"start_line":9,"start_character":58,"end_line":9,"end_character":61},"in_reply_to":"e945ad86_ddd15b28","updated":"2021-07-22 08:46:26.000000000","message":"still is","commit_id":"2c9e3ae10593761806ac94fa3e9b5775409e452c"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"893106af53c126793bc4ab29ee0be50bebdbce51","unresolved":true,"context_lines":[{"line_number":7,"context_line":"    In the Xena release, Ironic removed the iSCSI driver. The recommended"},{"line_number":8,"context_line":"    deploy driver is ``direct``, which uses HTTP to transfer the disk image."},{"line_number":9,"context_line":"    This requires an HTTP server, and the simplest option is to use the one"},{"line_number":10,"context_line":"    previously deployed when ``enable_ironic_ipxe`` is set to ``true``."}],"source_content_type":"text/x-yaml","patch_set":5,"id":"1b7f3340_204f2e93","line":10,"range":{"start_line":10,"start_character":52,"end_line":10,"end_character":55},"updated":"2021-07-21 16:12:34.000000000","message":"was","commit_id":"2c9e3ae10593761806ac94fa3e9b5775409e452c"}]}
