)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":37313,"name":"Alma MAILLET-CONTOZ","display_name":"Alma MAILLET-CONTOZ","email":"alma.maillet-contoz@infomaniak.com","username":"nicolasmc","status":"Intern SRE at Infomaniak"},"change_message_id":"90489420c4186c0750d7d57c074ed89738f39838","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"4b2df3e2_71c150e6","updated":"2026-07-08 08:52:54.000000000","message":"LGTM","commit_id":"658dfb62a41439ac012b156c9c0ed5b3f49ec43e"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"18249e097a64dc3c9725aef80bc8c257c6b5f40c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"2fa7febb_1300a02c","updated":"2026-07-08 09:13:14.000000000","message":"Sorry for the change of course, but I analysed Neutron wsgi app code a bit more","commit_id":"658dfb62a41439ac012b156c9c0ed5b3f49ec43e"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"1088be0c3447cbd04df7d8062581cfe8e8ede732","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"20058a1f_8c1d6fa9","updated":"2026-07-09 20:39:55.000000000","message":"recheck not related","commit_id":"1300c91ab12cf5d07bae86341570aebad32d06b2"}],"ansible/roles/neutron/defaults/main.yml":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"e2349805aab2eb175a3059b8fe16a9939fa5eb4c","unresolved":true,"context_lines":[{"line_number":255,"context_line":"  - generator: \"/neutron/etc/oslo-config-generator/dhcp_agent.ini\""},{"line_number":256,"context_line":"    config: \"/etc/neutron/dhcp_agent.ini\""},{"line_number":257,"context_line":""},{"line_number":258,"context_line":"neutron_config_files: \"neutron.conf;plugins/ml2/ml2_conf.ini{% if enable_neutron_vpnaas | bool %};neutron_vpnaas.conf{% endif %}\""},{"line_number":259,"context_line":""},{"line_number":260,"context_line":"####################"},{"line_number":261,"context_line":"# Database"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"56f92cdf_ce3656fe","line":258,"updated":"2026-07-10 09:16:34.000000000","message":"Maybe make that a dict with enabled: \"{{ enable_neutron_vpnaas | bool }}\" and use it in both config.jsons and the env variable?","commit_id":"1300c91ab12cf5d07bae86341570aebad32d06b2"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"5399396951e7bd0b0932bec24ddaa9fe5447ae95","unresolved":true,"context_lines":[{"line_number":255,"context_line":"  - generator: \"/neutron/etc/oslo-config-generator/dhcp_agent.ini\""},{"line_number":256,"context_line":"    config: \"/etc/neutron/dhcp_agent.ini\""},{"line_number":257,"context_line":""},{"line_number":258,"context_line":"neutron_config_files: \"neutron.conf;plugins/ml2/ml2_conf.ini{% if enable_neutron_vpnaas | bool %};neutron_vpnaas.conf{% endif %}\""},{"line_number":259,"context_line":""},{"line_number":260,"context_line":"####################"},{"line_number":261,"context_line":"# Database"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"6b730cba_36fd4369","line":258,"in_reply_to":"56f92cdf_ce3656fe","updated":"2026-07-10 13:35:23.000000000","message":"Haha, I expected that question. I was trying to keep it simple, but I can change it of course...","commit_id":"1300c91ab12cf5d07bae86341570aebad32d06b2"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"be018d5cb414b239895f6d84f65a3622aea9ba67","unresolved":true,"context_lines":[{"line_number":255,"context_line":"  - generator: \"/neutron/etc/oslo-config-generator/dhcp_agent.ini\""},{"line_number":256,"context_line":"    config: \"/etc/neutron/dhcp_agent.ini\""},{"line_number":257,"context_line":""},{"line_number":258,"context_line":"neutron_config_files: \"neutron.conf;plugins/ml2/ml2_conf.ini{% if enable_neutron_vpnaas | bool %};neutron_vpnaas.conf{% endif %}\""},{"line_number":259,"context_line":""},{"line_number":260,"context_line":"####################"},{"line_number":261,"context_line":"# Database"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"9291ae5e_33cd4875","line":258,"in_reply_to":"6b730cba_36fd4369","updated":"2026-07-13 13:13:28.000000000","message":"@mnasiadka@gmail.com hmm, but it can\u0027t be the same variable as confis use --config-file FOO --config-file BAR ....etc And ENV uing names of files delimited by an ; ...\n\nAre u sure that it can\u0027t be as it is now ?","commit_id":"1300c91ab12cf5d07bae86341570aebad32d06b2"}],"ansible/roles/neutron/tasks/config.yml":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"47a77b155ef6b828fe706767427a5deccbd827d7","unresolved":false,"context_lines":[{"line_number":424,"context_line":"    service_uwsgi_config_tls_key: \"/etc/neutron/certs/neutron-key.pem\""},{"line_number":425,"context_line":"    service_uwsgi_config_uid: \"neutron\""},{"line_number":426,"context_line":"    service_uwsgi_config_workers: \"{{ neutron_api_workers }}\""},{"line_number":427,"context_line":"    service_uwsgi_config_pyargv: \u003e-"},{"line_number":428,"context_line":"      --config-file /etc/neutron/neutron.conf"},{"line_number":429,"context_line":"      --config-file /etc/neutron/plugins/ml2/ml2_conf.ini"},{"line_number":430,"context_line":"      {%- if enable_neutron_vpnaas | bool %}"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"f4228231_eb26f507","line":427,"updated":"2026-07-07 21:54:52.000000000","message":"linters: jinja[spacing][/]: Jinja2 spacing could be improved: --config-file /etc/neutron/neutron.conf --config-file /etc/neutron/plugins/ml2/ml2_conf.ini {%- if enable_neutron_vpnaas | bool %} --config-file /etc/neutron/neutron_vpnaas.conf {%- endif %} -\u003e --config-file /etc/neutron/neutron.conf --config-file /etc/neutron/plugins/ml2/ml2_conf.ini{%- if enable_neutron_vpnaas | bool %} --config-file /etc/neutron/neutron_vpnaas.conf{%- endif %} (warning)","commit_id":"9ad6bb62e39f3f39ccad7e4d3b64420ecab5cc7e"},{"author":{"_account_id":37313,"name":"Alma MAILLET-CONTOZ","display_name":"Alma MAILLET-CONTOZ","email":"alma.maillet-contoz@infomaniak.com","username":"nicolasmc","status":"Intern SRE at Infomaniak"},"change_message_id":"9ebcd496e496924962944c8c59499fb191ef9011","unresolved":true,"context_lines":[{"line_number":424,"context_line":"    service_uwsgi_config_tls_key: \"/etc/neutron/certs/neutron-key.pem\""},{"line_number":425,"context_line":"    service_uwsgi_config_uid: \"neutron\""},{"line_number":426,"context_line":"    service_uwsgi_config_workers: \"{{ neutron_api_workers }}\""},{"line_number":427,"context_line":"    service_uwsgi_config_pyargv: \u003e-"},{"line_number":428,"context_line":"      --config-file /etc/neutron/neutron.conf"},{"line_number":429,"context_line":"      --config-file /etc/neutron/plugins/ml2/ml2_conf.ini{% if enable_neutron_vpnaas | bool %}"},{"line_number":430,"context_line":"      --config-file /etc/neutron/neutron_vpnaas.conf{% endif %}"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"126b6d45_d6f36306","line":427,"updated":"2026-07-08 07:57:24.000000000","message":"Why are you not using this var in the templates ?","commit_id":"4b500aa28e51fca9485b5dccbb653642c83cc6d8"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"f47c5295bd9d422dfe42ec3dd2c4d62fcfa4a3b6","unresolved":false,"context_lines":[{"line_number":424,"context_line":"    service_uwsgi_config_tls_key: \"/etc/neutron/certs/neutron-key.pem\""},{"line_number":425,"context_line":"    service_uwsgi_config_uid: \"neutron\""},{"line_number":426,"context_line":"    service_uwsgi_config_workers: \"{{ neutron_api_workers }}\""},{"line_number":427,"context_line":"    service_uwsgi_config_pyargv: \u003e-"},{"line_number":428,"context_line":"      --config-file /etc/neutron/neutron.conf"},{"line_number":429,"context_line":"      --config-file /etc/neutron/plugins/ml2/ml2_conf.ini{% if enable_neutron_vpnaas | bool %}"},{"line_number":430,"context_line":"      --config-file /etc/neutron/neutron_vpnaas.conf{% endif %}"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"b4112d49_ee9a42d7","line":427,"in_reply_to":"126b6d45_d6f36306","updated":"2026-07-08 08:08:01.000000000","message":"Yeah, sorry I forgot to send it to gerrit together in one chain. Commits are split, one is adding option, second sets this option.\n\nAdding option to template -\u003e https://review.opendev.org/c/openstack/kolla-ansible/+/996214/1\n\nThanks for the heads-up.","commit_id":"4b500aa28e51fca9485b5dccbb653642c83cc6d8"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"18249e097a64dc3c9725aef80bc8c257c6b5f40c","unresolved":true,"context_lines":[{"line_number":424,"context_line":"    service_uwsgi_config_tls_key: \"/etc/neutron/certs/neutron-key.pem\""},{"line_number":425,"context_line":"    service_uwsgi_config_uid: \"neutron\""},{"line_number":426,"context_line":"    service_uwsgi_config_workers: \"{{ neutron_api_workers }}\""},{"line_number":427,"context_line":"    service_uwsgi_config_pyargv: \u003e-"},{"line_number":428,"context_line":"      --config-file /etc/neutron/neutron.conf"},{"line_number":429,"context_line":"      --config-file /etc/neutron/plugins/ml2/ml2_conf.ini{% if enable_neutron_vpnaas | bool %}"},{"line_number":430,"context_line":"      --config-file /etc/neutron/neutron_vpnaas.conf{% endif %}"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"b96f0d33_81b00e07","line":427,"updated":"2026-07-08 09:13:14.000000000","message":"Use https://opendev.org/openstack/neutron/src/commit/e9abc267d0911c27b2f9772e7cb08f3af3fc3a6d/neutron/server/__init__.py#L41 instead which will be universal if we decide to use some other WSGI server in future (since uWSGI is in maintenance mode)","commit_id":"658dfb62a41439ac012b156c9c0ed5b3f49ec43e"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"82d13e094f7966d94334d250c8009a0e8155897d","unresolved":false,"context_lines":[{"line_number":424,"context_line":"    service_uwsgi_config_tls_key: \"/etc/neutron/certs/neutron-key.pem\""},{"line_number":425,"context_line":"    service_uwsgi_config_uid: \"neutron\""},{"line_number":426,"context_line":"    service_uwsgi_config_workers: \"{{ neutron_api_workers }}\""},{"line_number":427,"context_line":"    service_uwsgi_config_pyargv: \u003e-"},{"line_number":428,"context_line":"      --config-file /etc/neutron/neutron.conf"},{"line_number":429,"context_line":"      --config-file /etc/neutron/plugins/ml2/ml2_conf.ini{% if enable_neutron_vpnaas | bool %}"},{"line_number":430,"context_line":"      --config-file /etc/neutron/neutron_vpnaas.conf{% endif %}"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"51a55bd6_5915fff4","line":427,"in_reply_to":"b96f0d33_81b00e07","updated":"2026-07-08 09:48:49.000000000","message":"Done","commit_id":"658dfb62a41439ac012b156c9c0ed5b3f49ec43e"}]}
