)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"6f0ad88adb098ac79a47198e11550798067b1a8b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"f9b21bb0_040c73af","updated":"2022-02-21 11:33:31.000000000","message":"we should add task to deploy/remove service-redrect.j2 in https://opendev.org/openstack/openstack-ansible-haproxy_server/src/branch/master/tasks/haproxy_service_config.yml","commit_id":"d30bb2e6d12233a5a20a9b739c46e40cbabc5bf9"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"39eda85f367046fd4d02cf33e8c9ab2705012cd3","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"631259f5_b91c9d6d","in_reply_to":"1ff98ef4_6127c621","updated":"2022-02-22 15:23:52.000000000","message":"Tbh I\u0027m not sure how it will behave. As basically, what we do is assemble configs into one thing: https://opendev.org/openstack/openstack-ansible-haproxy_server/src/branch/master/handlers/main.yml#L24-L30\n\nSo I\u0027d say it depends on where we place that redirect file.","commit_id":"d30bb2e6d12233a5a20a9b739c46e40cbabc5bf9"},{"author":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"change_message_id":"7f4236921932b17410577a7983909016d727e9e2","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"1ff98ef4_6127c621","in_reply_to":"5242dcb3_7d293fe6","updated":"2022-02-22 15:06:14.000000000","message":"This doesnt template out a seperate config though - it\u0027s a conditional jinja include into the existing service template when item.service.haproxy_tcp_upgrade_frontend is true.\n\nI don\u0027t think a cleanup task is required becasue when the flag is set to false and the haproxy role is re-run, the http-\u003ehttps migration redirects are removed.","commit_id":"d30bb2e6d12233a5a20a9b739c46e40cbabc5bf9"},{"author":{"_account_id":31749,"name":"James Gibson","email":"james.gibson@bbc.co.uk","username":"jamesgibo"},"change_message_id":"793cf407117fa78a88970efac532b11f90832f22","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"5242dcb3_7d293fe6","in_reply_to":"f9b21bb0_040c73af","updated":"2022-02-21 12:46:02.000000000","message":"Do you mean change this from an include inside the service.j2 template to a separate config file that is included into main haproxy.cfg file?\nI thought it was better to include this in the service.j2 template so that the redirect is placed next to the main frontend in the file, which would not be the case if added as a task.","commit_id":"d30bb2e6d12233a5a20a9b739c46e40cbabc5bf9"}]}