)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"9ea91fe9e9e2f694e87121f05fc71849d6f37599","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Move /{tftp,http}boot to /var/lib/ironic"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"There is no guarantee that /var is not on a separate filesystem. Also"},{"line_number":10,"context_line":"we should not pollute the root directory with our stuff."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Removes the wildly outdated statement about purging the image cache"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"af84b353_4dc1259a","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":64},"updated":"2021-12-23 10:16:35.000000000","message":"An old one :) https://bugs.launchpad.net/ironic/+bug/1507894\n\nWe hit that when running in docker with volumes (bind mounts).\n\nI went the other way and put the TFTP master path in /httpboot. I could probably switch the mount points around and go with this approach, although we do have a few hardcoded /httpboot\u0027s in kayobe.\n\nhttps://opendev.org/openstack/kolla-ansible/src/branch/master/ansible/roles/bifrost/templates/bifrost.yml.j2#L17\n\nI\u0027ll preempt this change by forcing the /*tpboot paths in kolla-ansible. I\u0027ll share a link to a patch you can depend on.","commit_id":"b7bcad0ace0a4debf775837c0672ca227dfb60d0"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"27739064544e4c85fd528407f5143b2f588aaeaf","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Move /{tftp,http}boot to /var/lib/ironic"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"There is no guarantee that /var is not on a separate filesystem. Also"},{"line_number":10,"context_line":"we should not pollute the root directory with our stuff."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Removes the wildly outdated statement about purging the image cache"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"74c662b0_842f4626","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":64},"in_reply_to":"af84b353_4dc1259a","updated":"2021-12-23 10:24:01.000000000","message":"https://review.opendev.org/c/openstack/kolla-ansible/+/822811","commit_id":"b7bcad0ace0a4debf775837c0672ca227dfb60d0"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":14885,"name":"Daniel Mellado","email":"dmellado@redhat.com","username":"daniel-mellado"},"change_message_id":"02058633c17923dd912b61a2f47d1a29a44a6cde","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"8903d683_bc6be5a3","updated":"2021-12-22 16:42:12.000000000","message":"Thanks!","commit_id":"b7bcad0ace0a4debf775837c0672ca227dfb60d0"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"13121ce21f3171a1426f949ef5de618c077538c0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4b3ca1cf_71199489","updated":"2021-12-23 08:42:20.000000000","message":"recheck","commit_id":"b7bcad0ace0a4debf775837c0672ca227dfb60d0"}],"releasenotes/notes/httpboot-f3891f6343c96914.yaml":[{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"9ea91fe9e9e2f694e87121f05fc71849d6f37599","unresolved":true,"context_lines":[{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    The location of the PXE and HTTP boot directories have been changed to"},{"line_number":5,"context_line":"    ``/var/lib/ironic/tftpboot`` and ``/var/lib/ironic/httpboot`` respectively."},{"line_number":6,"context_line":"    Any custom content will not be migrated, please migrate it manually if"},{"line_number":7,"context_line":"    needed. You may remove the old locations ``/tftpboot`` and ``/httpboot``"},{"line_number":8,"context_line":"    after the upgrade."},{"line_number":9,"context_line":"fixes:"},{"line_number":10,"context_line":"  - |"},{"line_number":11,"context_line":"    Makes sure the image cache directories are on the same filesystem as"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"4b75ced1_7baa5e9e","line":8,"range":{"start_line":6,"start_character":4,"end_line":8,"end_character":22},"updated":"2021-12-23 10:16:35.000000000","message":"When you say custom content, that includes PXE config created by ironic during deployment, right? I suppose the deployment_image.qcow2 and IPA images will get rebuilt in the new location if you run ansible.","commit_id":"b7bcad0ace0a4debf775837c0672ca227dfb60d0"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"357abc993c282c9aa56d4432691733a7d0943aa5","unresolved":true,"context_lines":[{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    The location of the PXE and HTTP boot directories have been changed to"},{"line_number":5,"context_line":"    ``/var/lib/ironic/tftpboot`` and ``/var/lib/ironic/httpboot`` respectively."},{"line_number":6,"context_line":"    Any custom content will not be migrated, please migrate it manually if"},{"line_number":7,"context_line":"    needed. You may remove the old locations ``/tftpboot`` and ``/httpboot``"},{"line_number":8,"context_line":"    after the upgrade."},{"line_number":9,"context_line":"fixes:"},{"line_number":10,"context_line":"  - |"},{"line_number":11,"context_line":"    Makes sure the image cache directories are on the same filesystem as"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"23e1c214_55682eb8","line":8,"range":{"start_line":6,"start_character":4,"end_line":8,"end_character":22},"in_reply_to":"4b75ced1_7baa5e9e","updated":"2022-01-05 15:01:23.000000000","message":"Hmmm, this is worth rephrasing","commit_id":"b7bcad0ace0a4debf775837c0672ca227dfb60d0"}]}
