)]}'
{"tasks/main.yml":[{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"8b1bb58ab805bdaddde29680191b059fb402e74a","unresolved":true,"context_lines":[{"line_number":218,"context_line":"        - \"{{ neutron_services[\u0027neutron-metadata-agent\u0027] }}\""},{"line_number":219,"context_line":"        - \"{{ neutron_services[\u0027neutron-dhcp-agent\u0027] }}\""},{"line_number":220,"context_line":""},{"line_number":221,"context_line":"# TODO(hamburgler): Can be removed once pre Caracal"},{"line_number":222,"context_line":"# support is deprecated. (Bobcat and prior)"},{"line_number":223,"context_line":"- name: Set ovn-metadata-proxy folder user/group"},{"line_number":224,"context_line":"  become: true"},{"line_number":225,"context_line":"  block:"},{"line_number":226,"context_line":"    - name: Check if ovn-metadata-proxy folder exists"},{"line_number":227,"context_line":"      ansible.builtin.stat:"},{"line_number":228,"context_line":"        path: /var/lib/neutron/ovn-metadata-proxy"},{"line_number":229,"context_line":"      register: ovn_metadata_proxy_stat"},{"line_number":230,"context_line":""},{"line_number":231,"context_line":"    - name: Chown ovn-metadata-proxy folder to neutron:neutron"},{"line_number":232,"context_line":"      ansible.builtin.file:"},{"line_number":233,"context_line":"        path: /var/lib/neutron/ovn-metadata-proxy"},{"line_number":234,"context_line":"        owner: neutron"},{"line_number":235,"context_line":"        group: neutron"},{"line_number":236,"context_line":"        mode: \u00270755\u0027"},{"line_number":237,"context_line":"        state: directory"},{"line_number":238,"context_line":"      when: ovn_metadata_proxy_stat.stat.exists"},{"line_number":239,"context_line":"  tags:"},{"line_number":240,"context_line":"    - neutron-config"},{"line_number":241,"context_line":"    - systemd-service"},{"line_number":242,"context_line":""},{"line_number":243,"context_line":"- name: Run the systemd service role"},{"line_number":244,"context_line":"  import_role:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"01a41364_d814eca8","line":241,"range":{"start_line":221,"start_character":0,"end_line":241,"end_character":21},"updated":"2024-08-26 16:39:07.000000000","message":"Maybe we should simplify that and just move to https://opendev.org/openstack/openstack-ansible-os_neutron/src/branch/master/tasks/neutron_pre_install.yml#L73-L74 ?\n\nOne way could be to have `neutron_system_home_folder` ownership recursed, but I\u0027m kinda afraid to do so due to many other things residing there.\nBut adding just another record when `neutron_plugin_type \u003d\u003d \u0027ml2.ovn\u0027` and host in `neutron_ovn_controller` group.","commit_id":"bbe046c77088fa49772a86e4d4e4495f9027f29f"},{"author":{"_account_id":35831,"name":"James Black","display_name":"James Black","email":"jamesjordanblack604@gmail.com","username":"hamburgler"},"change_message_id":"c2d8486ef812407405e17af85ab29d4aaef1a1a5","unresolved":false,"context_lines":[{"line_number":218,"context_line":"        - \"{{ neutron_services[\u0027neutron-metadata-agent\u0027] }}\""},{"line_number":219,"context_line":"        - \"{{ neutron_services[\u0027neutron-dhcp-agent\u0027] }}\""},{"line_number":220,"context_line":""},{"line_number":221,"context_line":"# TODO(hamburgler): Can be removed once pre Caracal"},{"line_number":222,"context_line":"# support is deprecated. (Bobcat and prior)"},{"line_number":223,"context_line":"- name: Set ovn-metadata-proxy folder user/group"},{"line_number":224,"context_line":"  become: true"},{"line_number":225,"context_line":"  block:"},{"line_number":226,"context_line":"    - name: Check if ovn-metadata-proxy folder exists"},{"line_number":227,"context_line":"      ansible.builtin.stat:"},{"line_number":228,"context_line":"        path: /var/lib/neutron/ovn-metadata-proxy"},{"line_number":229,"context_line":"      register: ovn_metadata_proxy_stat"},{"line_number":230,"context_line":""},{"line_number":231,"context_line":"    - name: Chown ovn-metadata-proxy folder to neutron:neutron"},{"line_number":232,"context_line":"      ansible.builtin.file:"},{"line_number":233,"context_line":"        path: /var/lib/neutron/ovn-metadata-proxy"},{"line_number":234,"context_line":"        owner: neutron"},{"line_number":235,"context_line":"        group: neutron"},{"line_number":236,"context_line":"        mode: \u00270755\u0027"},{"line_number":237,"context_line":"        state: directory"},{"line_number":238,"context_line":"      when: ovn_metadata_proxy_stat.stat.exists"},{"line_number":239,"context_line":"  tags:"},{"line_number":240,"context_line":"    - neutron-config"},{"line_number":241,"context_line":"    - systemd-service"},{"line_number":242,"context_line":""},{"line_number":243,"context_line":"- name: Run the systemd service role"},{"line_number":244,"context_line":"  import_role:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"68327c50_277af31d","line":241,"range":{"start_line":221,"start_character":0,"end_line":241,"end_character":21},"in_reply_to":"01a41364_d814eca8","updated":"2024-08-26 19:35:39.000000000","message":"Your https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/927183 looks good. I can close this PR of mine off?","commit_id":"bbe046c77088fa49772a86e4d4e4495f9027f29f"}]}