)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":36670,"name":"Jay Jahns","email":"jayjahns@gmail.com","username":"jayjahns"},"change_message_id":"faa9e95af8d51b79bcf598b1c3af395fc184fd3e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e42ae349_1f7c6cad","updated":"2026-05-08 14:15:03.000000000","message":"I\u0027m going to suggest +1 this. \n\nHaving run into this in production, I can state that if you run ironic in an air-gapped configuration, where bare metal nodes are on a private unroutable subnet, customizing the TFTP server address to a different value than the api_interface_address is necessary.\n\nThat ensures that the TFTP server is on the right network reachable to the baremetal nodes its trying to deploy.\n\nSomewhere up the chain - the defaults can be specified to api_interface_address to maintain a semblance of consistency. \n\nBut this is necessary, especially if people have a specifically secure posture.","commit_id":"fdc13f56d38fc5c84b9b1eb39b1b7a061d153bbb"},{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"9cbbe2e42490e490121bafc3cb314eb8addef84e","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b4c99939_1fba42be","updated":"2024-08-28 15:33:35.000000000","message":"While I think this does indirectly default to the same value, I think this should have a reno in case someone is relying on setting ``api_interface_address`` for this currently.","commit_id":"fdc13f56d38fc5c84b9b1eb39b1b7a061d153bbb"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"01073f6be6444f7229dba0c0eec0e0b303d062a3","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"9061d2dc_344025ec","updated":"2024-08-30 17:56:36.000000000","message":"also reno is absent","commit_id":"fdc13f56d38fc5c84b9b1eb39b1b7a061d153bbb"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"01073f6be6444f7229dba0c0eec0e0b303d062a3","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c673c36b_4c23aaa4","in_reply_to":"b4c99939_1fba42be","updated":"2024-08-30 17:56:36.000000000","message":"not really the same, tftp service should be only one, not behind the haproxy, and it can be placed not on the 1st controller as usual.","commit_id":"fdc13f56d38fc5c84b9b1eb39b1b7a061d153bbb"}],"ansible/roles/ironic/templates/ironic-dnsmasq.conf.j2":[{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"01073f6be6444f7229dba0c0eec0e0b303d062a3","unresolved":true,"context_lines":[{"line_number":19,"context_line":"{# different options must be used here #}"},{"line_number":20,"context_line":"{% else %}{# ipv4 #}"},{"line_number":21,"context_line":"dhcp-option\u003doption:tftp-server,{{ ironic_tftp_listen_address }}"},{"line_number":22,"context_line":"dhcp-option\u003doption:server-ip-address,{{ ironic_tftp_listen_address }}"},{"line_number":23,"context_line":"dhcp-option\u003d210,/var/lib/ironic/tftpboot/"},{"line_number":24,"context_line":"{% if ironic_dnsmasq_serve_ipxe | bool %}"},{"line_number":25,"context_line":"dhcp-match\u003dipxe,175"}],"source_content_type":"text/x-jinja2","patch_set":3,"id":"256f188d_ae320a78","line":22,"range":{"start_line":22,"start_character":40,"end_line":22,"end_character":66},"updated":"2024-08-30 17:56:36.000000000","message":"its ok for tftp, but imho this should be api_interface_address as is","commit_id":"fdc13f56d38fc5c84b9b1eb39b1b7a061d153bbb"}]}
