)]}'
{"/PATCHSET_LEVEL":[{"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":"0af176d5b4da3adb574bd2e0c88f1b1f3938ed00","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"5ef8bf9e_334f776b","updated":"2022-09-28 16:43:11.000000000","message":"Perhaps we should have the family variable.","commit_id":"9d5f9080dae018855b074a4daf09e7c62a51dffb"}],"ansible/roles/ironic/templates/ironic-tftp.json.j2":[{"author":{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},"change_message_id":"c36e6839fcc8768cf5e387b38f366f86f854809f","unresolved":true,"context_lines":[{"line_number":1,"context_line":"{% set pxe_dir \u003d (\u0027/var/lib/ironic/tftpboot/grub\u0027 if kolla_base_distro in [\u0027ubuntu\u0027, \u0027debian\u0027] else \u0027/var/lib/ironic/tftpboot/EFI/centos\u0027 if kolla_base_distro \u003d\u003d \u0027centos\u0027 else \u0027/var/lib/ironic/tftpboot/EFI/rocky\u0027 if kolla_base_distro \u003d\u003d \u0027rocky\u0027) if enable_ironic_pxe_uefi | bool else \u0027/var/lib/ironic/tftpboot/pxelinux.cfg\u0027 %}"},{"line_number":2,"context_line":"{% set pxe_cfg \u003d \u0027grub.cfg\u0027 if enable_ironic_pxe_uefi | bool else \u0027default\u0027 %}"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"{"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"6b4ea162_f502d265","line":1,"updated":"2022-09-28 13:18:30.000000000","message":"can we get that line rewritten into something more readable?\n\n{% if %} {% elif %} {% else %} etc","commit_id":"defcf5d7c46c5754f6f91ece88e0abb0068dca9d"},{"author":{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},"change_message_id":"4d5fa6a1fda1ed83641412885a683bc0ff182bdd","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{% set pxe_dir \u003d (\u0027/var/lib/ironic/tftpboot/grub\u0027 if kolla_base_distro in [\u0027ubuntu\u0027, \u0027debian\u0027] else \u0027/var/lib/ironic/tftpboot/EFI/centos\u0027 if kolla_base_distro \u003d\u003d \u0027centos\u0027 else \u0027/var/lib/ironic/tftpboot/EFI/rocky\u0027 if kolla_base_distro \u003d\u003d \u0027rocky\u0027) if enable_ironic_pxe_uefi | bool else \u0027/var/lib/ironic/tftpboot/pxelinux.cfg\u0027 %}"},{"line_number":2,"context_line":"{% set pxe_cfg \u003d \u0027grub.cfg\u0027 if enable_ironic_pxe_uefi | bool else \u0027default\u0027 %}"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"{"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"7ed3ef2f_ee6e19ed","line":1,"in_reply_to":"6b4ea162_f502d265","updated":"2022-09-29 08:35:02.000000000","message":"Done","commit_id":"defcf5d7c46c5754f6f91ece88e0abb0068dca9d"},{"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":"0af176d5b4da3adb574bd2e0c88f1b1f3938ed00","unresolved":true,"context_lines":[{"line_number":1,"context_line":"{% if enable_ironic_pxe_uefi | bool %}"},{"line_number":2,"context_line":"  {% if kolla_base_distro in [\u0027debian\u0027, \u0027ubuntu\u0027] %}"},{"line_number":3,"context_line":"    {% set pxe_dir \u003d \u0027/var/lib/ironic/tftpboot/grub\u0027 %}"},{"line_number":4,"context_line":"  {% elif kolla_base_distro in [\u0027centos\u0027, \u0027rocky\u0027] %}"}],"source_content_type":"text/x-jinja2","patch_set":4,"id":"12a3f90f_a0325b52","line":1,"range":{"start_line":1,"start_character":6,"end_line":1,"end_character":29},"updated":"2022-09-28 16:43:11.000000000","message":"this was deprecated for removal in Zed, so let\u0027s do the removal next :D keeping this for backporting","commit_id":"9d5f9080dae018855b074a4daf09e7c62a51dffb"},{"author":{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},"change_message_id":"4d5fa6a1fda1ed83641412885a683bc0ff182bdd","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{% if enable_ironic_pxe_uefi | bool %}"},{"line_number":2,"context_line":"  {% if kolla_base_distro in [\u0027debian\u0027, \u0027ubuntu\u0027] %}"},{"line_number":3,"context_line":"    {% set pxe_dir \u003d \u0027/var/lib/ironic/tftpboot/grub\u0027 %}"},{"line_number":4,"context_line":"  {% elif kolla_base_distro in [\u0027centos\u0027, \u0027rocky\u0027] %}"}],"source_content_type":"text/x-jinja2","patch_set":4,"id":"0e90b5ea_09aecead","line":1,"range":{"start_line":1,"start_character":6,"end_line":1,"end_character":29},"in_reply_to":"12a3f90f_a0325b52","updated":"2022-09-29 08:35:02.000000000","message":"Done","commit_id":"9d5f9080dae018855b074a4daf09e7c62a51dffb"}],"tests/templates/globals-default.j2":[{"author":{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},"change_message_id":"c36e6839fcc8768cf5e387b38f366f86f854809f","unresolved":true,"context_lines":[{"line_number":144,"context_line":"{% if base_distro \u003d\u003d \"ubuntu\" or base_distro \u003d\u003d \"debian\" %}"},{"line_number":145,"context_line":"openstack_cacert: \"/etc/ssl/certs/ca-certificates.crt\""},{"line_number":146,"context_line":"{% endif %}"},{"line_number":147,"context_line":"{% if base_distro \u003d\u003d \"centos\" or base_distro \u003d\u003d \"rocky\" %}"},{"line_number":148,"context_line":"openstack_cacert: \"/etc/pki/tls/certs/ca-bundle.crt\""},{"line_number":149,"context_line":"{% endif %}"},{"line_number":150,"context_line":"kolla_admin_openrc_cacert: \"{% raw %}{{ kolla_certificates_dir }}{% endraw %}/ca/root.crt\""}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"bfd83081_4c6ac6bb","line":147,"updated":"2022-09-28 13:18:30.000000000","message":"Can we get \"if in\" like in rest of patch? And same for debian/ubuntu above?","commit_id":"defcf5d7c46c5754f6f91ece88e0abb0068dca9d"},{"author":{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},"change_message_id":"4d5fa6a1fda1ed83641412885a683bc0ff182bdd","unresolved":false,"context_lines":[{"line_number":144,"context_line":"{% if base_distro \u003d\u003d \"ubuntu\" or base_distro \u003d\u003d \"debian\" %}"},{"line_number":145,"context_line":"openstack_cacert: \"/etc/ssl/certs/ca-certificates.crt\""},{"line_number":146,"context_line":"{% endif %}"},{"line_number":147,"context_line":"{% if base_distro \u003d\u003d \"centos\" or base_distro \u003d\u003d \"rocky\" %}"},{"line_number":148,"context_line":"openstack_cacert: \"/etc/pki/tls/certs/ca-bundle.crt\""},{"line_number":149,"context_line":"{% endif %}"},{"line_number":150,"context_line":"kolla_admin_openrc_cacert: \"{% raw %}{{ kolla_certificates_dir }}{% endraw %}/ca/root.crt\""}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"e5d52d9f_1326e990","line":147,"in_reply_to":"bfd83081_4c6ac6bb","updated":"2022-09-29 08:35:02.000000000","message":"Done","commit_id":"defcf5d7c46c5754f6f91ece88e0abb0068dca9d"}]}