)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"fcea46630953801777912324a99fc09331f5c540","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"a048e3d3_4cf617dd","updated":"2024-01-03 14:03:56.000000000","message":"Sorry, I went on and proposed slightly alternative patch: https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/904572","commit_id":"592d8bd343681a852a99338cf4838353dd94ceba"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"12a2bfd1b14acfae304e5596a46e752250c7d7c7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"6a0aab7f_f1b0daea","updated":"2023-12-07 14:43:14.000000000","message":"what is actually a usecase, where gateway is part of multiple AZs?","commit_id":"592d8bd343681a852a99338cf4838353dd94ceba"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"31238d357bf5b91845d6ffb48312e6e87e08ed19","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"8fed3943_eab8f528","in_reply_to":"6a0aab7f_f1b0daea","updated":"2023-12-07 15:08:38.000000000","message":"I guess what I meant to ask was more like - if `neutron_default_availability_zones` should be a list instead?\nAs it seems that is a valid list for both OVN/OVS/LXB scenarios?\n\nAnd then `neutron_availability_zone` can be used only for OVS/LXB? As it seems that availability_zone_hints will be same for OVN as availability_zones anyway...\n\nBut I\u0027m not really sure here...","commit_id":"592d8bd343681a852a99338cf4838353dd94ceba"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"91b1354b9955d83b28e6d4a1d1852934f42b3700","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"ddeb5cde_7ffee825","in_reply_to":"8fed3943_eab8f528","updated":"2023-12-07 15:11:28.000000000","message":"So for OVS we have following overrides basically to make AZs work for us:\n\n```\nneutron_neutron_conf_overrides:\n  DEFAULT:\n    default_availability_zones: \"az1,az2,az3\"\n    router_scheduler_driver: neutron.scheduler.l3_agent_scheduler.AZLeastRoutersScheduler\n    network_scheduler_driver: neutron.scheduler.dhcp_agent_scheduler.AZAwareWeightScheduler\n    \n\nneutron_l3_agent_ini_overrides:\n  AGENT:\n    availability_zone: \"{{ az_name }}\"\n\nneutron_dhcp_agent_ini_overrides:\n  AGENT:\n    availability_zone: \"{{ az_name }}\"\n```\n\nSo if, if, for OVN it doesn\u0027t matter what default is - maybe same value can be set for both `default_availability_zones` and in ovsdb?","commit_id":"592d8bd343681a852a99338cf4838353dd94ceba"}],"defaults/main.yml":[{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"d33e13d798a37afc09bf850b788e899b40d0bee1","unresolved":true,"context_lines":[{"line_number":319,"context_line":"###"},{"line_number":320,"context_line":"### Availability zones"},{"line_number":321,"context_line":"###"},{"line_number":322,"context_line":"neutron_availability_zones: nova"},{"line_number":323,"context_line":"neutron_default_availability_zones: \"{{ neutron_availability_zones }}\""},{"line_number":324,"context_line":""},{"line_number":325,"context_line":"###"},{"line_number":326,"context_line":"### Telemetry integration"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"391604a7_9f4df857","line":323,"range":{"start_line":322,"start_character":0,"end_line":323,"end_character":70},"updated":"2022-12-03 19:17:49.000000000","message":"So in neutron lxb/ovs default AZ is a list - you can say az1,az2 here which would mean that L3 and DHCP agents will be spawned cross-AZ.\n\nCan ovs-vsctl also treat that as a list? As I guess it can\u0027t? As it feels like it\u0027s different in logic and intention.","commit_id":"2d405526116fd467ed73dd91047cebc5147cd05b"},{"author":{"_account_id":34653,"name":"Danila Balagansky","email":"dbalagansky@me.com","username":"dbalagansky"},"change_message_id":"c74e371aa6299cbfae6b86847af7f312913f2205","unresolved":true,"context_lines":[{"line_number":319,"context_line":"###"},{"line_number":320,"context_line":"### Availability zones"},{"line_number":321,"context_line":"###"},{"line_number":322,"context_line":"neutron_availability_zones: nova"},{"line_number":323,"context_line":"neutron_default_availability_zones: \"{{ neutron_availability_zones }}\""},{"line_number":324,"context_line":""},{"line_number":325,"context_line":"###"},{"line_number":326,"context_line":"### Telemetry integration"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"6182eee3_c91679f6","line":323,"range":{"start_line":322,"start_character":0,"end_line":323,"end_character":70},"in_reply_to":"391604a7_9f4df857","updated":"2023-06-19 14:45:40.000000000","message":"According to the doc [1], it can.\n\n[1] https://docs.openstack.org/neutron/latest/admin/ovn/availability_zones.html","commit_id":"2d405526116fd467ed73dd91047cebc5147cd05b"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"0a9f78a03f4e2e2580dd69080e16956f0165328b","unresolved":true,"context_lines":[{"line_number":337,"context_line":"###"},{"line_number":338,"context_line":"### Availability zones"},{"line_number":339,"context_line":"###"},{"line_number":340,"context_line":"neutron_availability_zones:"},{"line_number":341,"context_line":"  - nova"},{"line_number":342,"context_line":"neutron_default_availability_zones: \"{{ neutron_availability_zones }}\""},{"line_number":343,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":4,"id":"4b6b0d54_eabe6750","line":340,"range":{"start_line":340,"start_character":0,"end_line":340,"end_character":26},"updated":"2023-12-14 13:41:52.000000000","message":"And make that a string instead of the list? As everywhere except OVN we just take `| first` out of them?\n\nSo using `neutron_default_availability_zones` as a list and having it as a string makes more sense I guess, and don\u0027t use `neutron_availability_zones` for ovn...","commit_id":"592d8bd343681a852a99338cf4838353dd94ceba"}],"tasks/providers/setup_ovs_ovn.yml":[{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"0a9f78a03f4e2e2580dd69080e16956f0165328b","unresolved":true,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"- name: Set CMS Options for Gateway Scheduling"},{"line_number":24,"context_line":"  changed_when: false"},{"line_number":25,"context_line":"  command: \"ovs-vsctl set Open_vSwitch . external-ids:ovn-cms-options\u003denable-chassis-as-gw,availability-zones\u003d{{ neutron_availability_zones | join(\u0027:\u0027) }}\""},{"line_number":26,"context_line":"  when:"},{"line_number":27,"context_line":"    - \u0027\"neutron_ovn_gateway\" in group_names\u0027"},{"line_number":28,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":4,"id":"de898e6e_b13ba6c0","line":25,"range":{"start_line":25,"start_character":113,"end_line":25,"end_character":139},"updated":"2023-12-14 13:41:52.000000000","message":"so maybe here better to use `neutron_default_availability_zones`?","commit_id":"592d8bd343681a852a99338cf4838353dd94ceba"}]}