)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"8ebdb479c8b13e67c41155c333a1de502c3d9fb7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"fea7a058_ff1829e1","updated":"2023-05-22 17:34:07.000000000","message":"I think we can land that and iterate over, as overall it looks okeyish, despite I don\u0027t really like reffering `neutron_` vars as is.","commit_id":"5eae150e00ea9a080401f9596fcb5841b5f1e01f"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"b3b81682492056567ffbbcb31b56ec936e7f0ee5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"f0e7cdd6_02b981d4","updated":"2024-02-06 17:55:20.000000000","message":"Needs some testing/polishing","commit_id":"30dcc1f14e88af54d850539d12cd7e50d4f5385b"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"bd885daa9ad10aa284af3e33399cf989cc69c729","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"c5bf43fb_a42c9e1f","updated":"2024-02-07 15:42:30.000000000","message":"recheck - dependent patch updated","commit_id":"f51bcbe85aff15ea0bcd96ed5b0cb5d86650aaec"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"b1c9d05427032ab984d4c656e04a3f827ff52eb3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"25fb8bd2_089da15b","updated":"2024-02-10 14:43:45.000000000","message":"Failure we see is not related with the patch I assume, since it\u0027s failing on keystone application_credentials test, which can\u0027t be affected","commit_id":"35c15317144b3f00d2e83954914f2178eb419c65"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"18c6c81208cc3dd1cb0dd1740caa68d69c22bff5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"36d78d96_e788e34c","updated":"2024-02-09 17:06:27.000000000","message":"I can confirm it works in our multinode snadbox.","commit_id":"35c15317144b3f00d2e83954914f2178eb419c65"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"a7e1311c4c46b43f304d5c95152308edd8edd4b8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"bff4fd8a_999402e8","updated":"2024-02-09 14:41:29.000000000","message":"recheck - ci failed weirdly on application credentials, that has nothing to do with patch itself","commit_id":"35c15317144b3f00d2e83954914f2178eb419c65"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"50f0861a6923e0b50b30dd4f8ff7f7cf980586e6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"d431be5b_4f35f658","updated":"2024-02-11 18:25:52.000000000","message":"recheck - see if smth has accidentally changed...","commit_id":"35c15317144b3f00d2e83954914f2178eb419c65"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"5e3d3b99221da65c30ea151448efb65725eb4e75","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"e951751d_6e21a8a1","updated":"2024-02-13 08:41:32.000000000","message":"recheck rocky upgrade","commit_id":"44dc18d6da65f13b47a66b937f9b88c4cd45f205"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"ad7e78711f0b77e6e5ee65b38f1e2555b8106b15","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"d6d32516_8b1cc496","updated":"2024-02-20 15:32:30.000000000","message":"recheck timeout","commit_id":"44dc18d6da65f13b47a66b937f9b88c4cd45f205"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"f025d0b8980fc9b89419c666593dbae1834577a4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"b9589436_33c1e140","updated":"2024-02-21 18:59:36.000000000","message":"recheck upgrade timeout","commit_id":"44dc18d6da65f13b47a66b937f9b88c4cd45f205"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"9d03b7a0c54c42354348debc4c7f0c9d30f12c77","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"e9dc987c_515f6e57","updated":"2024-02-13 14:08:19.000000000","message":"recheck upgrade timeout","commit_id":"44dc18d6da65f13b47a66b937f9b88c4cd45f205"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"5355fc71cc28bb9f828f05f400eb9862791a0132","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"b24f8e94_5185ab6d","updated":"2024-02-28 08:44:26.000000000","message":"this works for me 😊","commit_id":"44dc18d6da65f13b47a66b937f9b88c4cd45f205"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"ff8012afbb6b404ae29da35e0e06e3ba3031b49d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"e1270ad8_ce3111ba","updated":"2024-04-30 16:00:23.000000000","message":"recheck - dependent patch updated","commit_id":"a610ffb49b50bc2e64e25735a7d643f01cf00962"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"aa5ed63c506deb4f6dde25e0a5def7a7619e63b2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"3c122e6e_19b05c87","updated":"2024-05-02 09:01:36.000000000","message":"recheck - gates should be fixed now","commit_id":"a610ffb49b50bc2e64e25735a7d643f01cf00962"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"9285b212c680d54f97ea936a13dc22014b11428a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":18,"id":"c2228e32_781dc774","updated":"2024-05-16 08:03:52.000000000","message":"recheck upgrade","commit_id":"11b2e99315457280b38cadba07c1176c6c153978"}],"defaults/main.yml":[{"author":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"change_message_id":"06d70741b9668b993017f7654b69304b128c36db","unresolved":true,"context_lines":[{"line_number":326,"context_line":"neutron_ovn_pki_intermediate_chain_path: \"{{ neutron_ovn_pki_dir ~ \u0027/roots/\u0027 ~ neutron_ovn_pki_intermediate_cert_name ~ \u0027/certs/\u0027 ~ neutron_ovn_pki_intermediate_cert_name ~ \u0027-chain.crt\u0027 }}\""},{"line_number":327,"context_line":"neutron_ovn_pki_regen_cert: \"\""},{"line_number":328,"context_line":"neutron_ovn_pki_certificates:"},{"line_number":329,"context_line":"  - name: \"neutron_ovn_{{ ansible_facts[\u0027hostname\u0027] }}\""},{"line_number":330,"context_line":"    provider: ownca"},{"line_number":331,"context_line":"    cn: \"{{ ansible_facts[\u0027hostname\u0027] }}\""},{"line_number":332,"context_line":"    san: \"{{ \u0027DNS:\u0027 ~ ansible_facts[\u0027hostname\u0027] ~ \u0027,IP:\u0027 ~ neutron_ovn_node_address }}\""}],"source_content_type":"text/x-yaml","patch_set":9,"id":"f53e0410_228c9ff9","line":329,"range":{"start_line":329,"start_character":4,"end_line":329,"end_character":8},"updated":"2023-05-23 08:55:48.000000000","message":"this is going to re-use the certificate created by os_neutron for a metal deploy where ansible_facts[\u0027hostname\u0027] is the same for neutron/octavia.\n\nI think it is best to namespace the variables and certificates correctly otherwise this is going to be pretty confusing.","commit_id":"5eae150e00ea9a080401f9596fcb5841b5f1e01f"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"66fc42ce6d97eecd39c1402d66df27f9bdc6721e","unresolved":false,"context_lines":[{"line_number":326,"context_line":"neutron_ovn_pki_intermediate_chain_path: \"{{ neutron_ovn_pki_dir ~ \u0027/roots/\u0027 ~ neutron_ovn_pki_intermediate_cert_name ~ \u0027/certs/\u0027 ~ neutron_ovn_pki_intermediate_cert_name ~ \u0027-chain.crt\u0027 }}\""},{"line_number":327,"context_line":"neutron_ovn_pki_regen_cert: \"\""},{"line_number":328,"context_line":"neutron_ovn_pki_certificates:"},{"line_number":329,"context_line":"  - name: \"neutron_ovn_{{ ansible_facts[\u0027hostname\u0027] }}\""},{"line_number":330,"context_line":"    provider: ownca"},{"line_number":331,"context_line":"    cn: \"{{ ansible_facts[\u0027hostname\u0027] }}\""},{"line_number":332,"context_line":"    san: \"{{ \u0027DNS:\u0027 ~ ansible_facts[\u0027hostname\u0027] ~ \u0027,IP:\u0027 ~ neutron_ovn_node_address }}\""}],"source_content_type":"text/x-yaml","patch_set":9,"id":"dbf82c40_19a98eac","line":329,"range":{"start_line":329,"start_character":4,"end_line":329,"end_character":8},"in_reply_to":"f53e0410_228c9ff9","updated":"2024-02-06 18:11:12.000000000","message":"Done","commit_id":"5eae150e00ea9a080401f9596fcb5841b5f1e01f"}],"tasks/main.yml":[{"author":{"_account_id":14288,"name":"Matthew Thode","display_name":"prometheanfire","email":"mthode@mthode.org","username":"prometheanfire"},"change_message_id":"d982dc63d842ca463b4976a785344e8a99014a44","unresolved":true,"context_lines":[{"line_number":140,"context_line":"    - octavia-config"},{"line_number":141,"context_line":"    - octavia-install"},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"- name: Copy Neutron OVN certificates"},{"line_number":144,"context_line":"  ansible.builtin.copy:"},{"line_number":145,"context_line":"    src: \"{{ ovn_certificate }}\""},{"line_number":146,"context_line":"    dest: /etc/octavia/certs/"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"c45fdd2f_79fb768f","line":143,"updated":"2023-02-23 20:33:44.000000000","message":"the octavia container can\u0027t access the neutron venv to copy the keys, got `could not find or access \u0027/openstack/venvs/.../etc/neutron/neutron_ovn-ca.pem\u0027.\n\nMy testing was applying this patch and the related openstack-ansible patch (setting neutron_bin) on the stable/zed branch.","commit_id":"65f203453e463b77349c7d086a4d1f884af4f7e5"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"66fc42ce6d97eecd39c1402d66df27f9bdc6721e","unresolved":false,"context_lines":[{"line_number":140,"context_line":"    - octavia-config"},{"line_number":141,"context_line":"    - octavia-install"},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"- name: Copy Neutron OVN certificates"},{"line_number":144,"context_line":"  ansible.builtin.copy:"},{"line_number":145,"context_line":"    src: \"{{ ovn_certificate }}\""},{"line_number":146,"context_line":"    dest: /etc/octavia/certs/"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"acf5b60e_e69da897","line":143,"in_reply_to":"23dd25d4_1cfcf29d","updated":"2024-02-06 18:11:12.000000000","message":"Done","commit_id":"65f203453e463b77349c7d086a4d1f884af4f7e5"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"a43d17450000c90c711c690264f8714a375fa446","unresolved":true,"context_lines":[{"line_number":140,"context_line":"    - octavia-config"},{"line_number":141,"context_line":"    - octavia-install"},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"- name: Copy Neutron OVN certificates"},{"line_number":144,"context_line":"  ansible.builtin.copy:"},{"line_number":145,"context_line":"    src: \"{{ ovn_certificate }}\""},{"line_number":146,"context_line":"    dest: /etc/octavia/certs/"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"23dd25d4_1cfcf29d","line":143,"in_reply_to":"c45fdd2f_79fb768f","updated":"2023-02-28 09:22:05.000000000","message":"Hm, I\u0027d say we should copy certificates not from neutron venv, but from pki host, which is localhost by default - we have certificates we need there.","commit_id":"65f203453e463b77349c7d086a4d1f884af4f7e5"}],"templates/octavia.conf.j2":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"c0131863df6b91736a5448d003c9677a1339418f","unresolved":true,"context_lines":[{"line_number":192,"context_line":"{% if octavia_ovn_enabled %}"},{"line_number":193,"context_line":"{%   if neutron_plugin_type \u003d\u003d \u0027ml2.ovn\u0027 %}"},{"line_number":194,"context_line":"[driver_agent]"},{"line_number":195,"context_line":"enabled_provider_agents \u003d ovn"},{"line_number":196,"context_line":""},{"line_number":197,"context_line":"[ovn]"},{"line_number":198,"context_line":"ovn_nb_connection \u003d {{ neutron_ovn_nb_connection }}"}],"source_content_type":"text/x-jinja2","patch_set":5,"id":"4f329a78_bc44c5bc","line":195,"updated":"2023-01-17 19:21:57.000000000","message":"nit: You might want to make this a variable to accommodate other provider drivers that may need the driver agent as well. Much like you do on line 19 above to enable the provider drivers in the API.","commit_id":"a051a48c4e940efc05c88e37e76de90cd02c9659"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"c9bc26a5d5fcd0d65706afb7eaea53dcf6020b88","unresolved":false,"context_lines":[{"line_number":192,"context_line":"{% if octavia_ovn_enabled %}"},{"line_number":193,"context_line":"{%   if neutron_plugin_type \u003d\u003d \u0027ml2.ovn\u0027 %}"},{"line_number":194,"context_line":"[driver_agent]"},{"line_number":195,"context_line":"enabled_provider_agents \u003d ovn"},{"line_number":196,"context_line":""},{"line_number":197,"context_line":"[ovn]"},{"line_number":198,"context_line":"ovn_nb_connection \u003d {{ neutron_ovn_nb_connection }}"}],"source_content_type":"text/x-jinja2","patch_set":5,"id":"cad51c2a_483f6e57","line":195,"in_reply_to":"4f329a78_bc44c5bc","updated":"2023-02-23 18:08:37.000000000","message":"Thanks for the input! I\u0027ve made it as a varaible as it seems to be useful even outside of OVN. Though I\u0027m not fully understanding value of driver agents yet.","commit_id":"a051a48c4e940efc05c88e37e76de90cd02c9659"}]}