)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":6681,"name":"Brent Eagles","email":"beagles@redhat.com","username":"beagles"},"change_message_id":"7ca7598bc3df2dd857a19cb4589e379fb521aa0d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c2c24ce9_13c1e5e2","updated":"2022-09-14 16:25:26.000000000","message":"So IIUC this basically: a.) sets the stage for making ovn the undercloud default and b.) provides and environment file for deploying it that way. Makes sense if that\u0027s the way we are going.","commit_id":"e4c5afeda23a2ef1a78dc2c0ee5f9422644da240"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"bbe767baa033aa08b64bfc58a86a00ad7b1fe5f1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"37d48e9a_698683ac","updated":"2022-09-14 12:37:47.000000000","message":"Tested OVN on Undercloud in https://review.opendev.org/c/openstack/tripleo-quickstart/+/853503\n\novb jobs are currently impacted with https://bugs.launchpad.net/tripleo/+bug/1987092 so fails randomly, but provisioning part is passing.","commit_id":"e4c5afeda23a2ef1a78dc2c0ee5f9422644da240"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"ecefe773dd30a492d149818375257bd1c878a691","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"17164520_5a79acda","updated":"2022-09-14 05:48:45.000000000","message":"check-rdo unrelated failures","commit_id":"e4c5afeda23a2ef1a78dc2c0ee5f9422644da240"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"af47ea68ba8defec0fbebc33265c9731a348b702","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"432b4178_2d0b04ce","in_reply_to":"c2c24ce9_13c1e5e2","updated":"2022-09-15 06:18:21.000000000","message":"Yes right, we will switch default later once we have migration/upgrade path evaluated. For now just allowing the deployment of OVN on Undercloud with this new env file so that we can test it in one OVB job and catch issues with it.","commit_id":"e4c5afeda23a2ef1a78dc2c0ee5f9422644da240"}],"environments/services/undercloud-neutron-ovn-dhcp.yaml":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"82a4238fc4d1cf49486986c5af972bb4b5537281","unresolved":true,"context_lines":[{"line_number":8,"context_line":"  OS::TripleO::Services::OVNMetadataAgent: ../../deployment/ovn/ovn-metadata-container-puppet.yaml"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"# Disabling Neutron services that overlap with OVN"},{"line_number":11,"context_line":"  OS::TripleO::Services::NeutronDhcpAgent: OS::Heat::None"},{"line_number":12,"context_line":"  OS::TripleO::Services::NeutronL3Agent: OS::Heat::None"},{"line_number":13,"context_line":"  OS::TripleO::Services::NeutronOvsAgent: OS::Heat::None"},{"line_number":14,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"2c789973_a3266061","line":11,"range":{"start_line":11,"start_character":2,"end_line":11,"end_character":57},"updated":"2022-09-15 11:25:24.000000000","message":"I think we still need dhcp agent for baremetal node deployment, right ?","commit_id":"e4c5afeda23a2ef1a78dc2c0ee5f9422644da240"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"b94ac02ec926c4de403f524700c23da4893e56fe","unresolved":true,"context_lines":[{"line_number":8,"context_line":"  OS::TripleO::Services::OVNMetadataAgent: ../../deployment/ovn/ovn-metadata-container-puppet.yaml"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"# Disabling Neutron services that overlap with OVN"},{"line_number":11,"context_line":"  OS::TripleO::Services::NeutronDhcpAgent: OS::Heat::None"},{"line_number":12,"context_line":"  OS::TripleO::Services::NeutronL3Agent: OS::Heat::None"},{"line_number":13,"context_line":"  OS::TripleO::Services::NeutronOvsAgent: OS::Heat::None"},{"line_number":14,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7365c7ca_05babe43","line":11,"range":{"start_line":11,"start_character":2,"end_line":11,"end_character":57},"in_reply_to":"1267651d_a5ae205d","updated":"2022-09-16 10:51:56.000000000","message":"Thanks Rabi, proposed https://review.opendev.org/c/openstack/tripleo-heat-templates/+/858093","commit_id":"e4c5afeda23a2ef1a78dc2c0ee5f9422644da240"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"0f6a6717cfa55c6220b79099ecae6d041661f1ae","unresolved":true,"context_lines":[{"line_number":8,"context_line":"  OS::TripleO::Services::OVNMetadataAgent: ../../deployment/ovn/ovn-metadata-container-puppet.yaml"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"# Disabling Neutron services that overlap with OVN"},{"line_number":11,"context_line":"  OS::TripleO::Services::NeutronDhcpAgent: OS::Heat::None"},{"line_number":12,"context_line":"  OS::TripleO::Services::NeutronL3Agent: OS::Heat::None"},{"line_number":13,"context_line":"  OS::TripleO::Services::NeutronOvsAgent: OS::Heat::None"},{"line_number":14,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"31199f6b_2e0f9344","line":11,"range":{"start_line":11,"start_character":2,"end_line":11,"end_character":57},"in_reply_to":"2c789973_a3266061","updated":"2022-09-15 11:28:12.000000000","message":"OVN22.06+ have native DHCP supports for IPv4 baremetal ports, so we don\u0027t required dhcp agent for such cases. wallaby+ we running with ovn22.06.","commit_id":"e4c5afeda23a2ef1a78dc2c0ee5f9422644da240"},{"author":{"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"},"change_message_id":"404d8d48db1e35774d642e5037d2182f5c395d7d","unresolved":true,"context_lines":[{"line_number":8,"context_line":"  OS::TripleO::Services::OVNMetadataAgent: ../../deployment/ovn/ovn-metadata-container-puppet.yaml"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"# Disabling Neutron services that overlap with OVN"},{"line_number":11,"context_line":"  OS::TripleO::Services::NeutronDhcpAgent: OS::Heat::None"},{"line_number":12,"context_line":"  OS::TripleO::Services::NeutronL3Agent: OS::Heat::None"},{"line_number":13,"context_line":"  OS::TripleO::Services::NeutronOvsAgent: OS::Heat::None"},{"line_number":14,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"91cd3586_f7f42212","line":11,"range":{"start_line":11,"start_character":2,"end_line":11,"end_character":57},"in_reply_to":"31199f6b_2e0f9344","updated":"2022-09-15 11:40:37.000000000","message":"This won\u0027t work with ipv6 provisioning network use-cases which we don\u0027t test upstream right? Should we also document that?","commit_id":"e4c5afeda23a2ef1a78dc2c0ee5f9422644da240"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"847df267a97fcef36e182297ee15434c6e589a32","unresolved":false,"context_lines":[{"line_number":8,"context_line":"  OS::TripleO::Services::OVNMetadataAgent: ../../deployment/ovn/ovn-metadata-container-puppet.yaml"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"# Disabling Neutron services that overlap with OVN"},{"line_number":11,"context_line":"  OS::TripleO::Services::NeutronDhcpAgent: OS::Heat::None"},{"line_number":12,"context_line":"  OS::TripleO::Services::NeutronL3Agent: OS::Heat::None"},{"line_number":13,"context_line":"  OS::TripleO::Services::NeutronOvsAgent: OS::Heat::None"},{"line_number":14,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"befdd8f4_86121411","line":11,"range":{"start_line":11,"start_character":2,"end_line":11,"end_character":57},"in_reply_to":"31199f6b_2e0f9344","updated":"2022-09-15 11:36:42.000000000","message":"We definitely need support for IPv6 when we make the switch because we need to support IPv6 provisioning network. However because this is still optional and is morely for testing use, I\u0027m good with using native dhcp with that limitation.","commit_id":"e4c5afeda23a2ef1a78dc2c0ee5f9422644da240"},{"author":{"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"},"change_message_id":"0e81affd6c7d4a304a3becf6d9e15be90f6bc41a","unresolved":true,"context_lines":[{"line_number":8,"context_line":"  OS::TripleO::Services::OVNMetadataAgent: ../../deployment/ovn/ovn-metadata-container-puppet.yaml"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"# Disabling Neutron services that overlap with OVN"},{"line_number":11,"context_line":"  OS::TripleO::Services::NeutronDhcpAgent: OS::Heat::None"},{"line_number":12,"context_line":"  OS::TripleO::Services::NeutronL3Agent: OS::Heat::None"},{"line_number":13,"context_line":"  OS::TripleO::Services::NeutronOvsAgent: OS::Heat::None"},{"line_number":14,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"1267651d_a5ae205d","line":11,"range":{"start_line":11,"start_character":2,"end_line":11,"end_character":57},"in_reply_to":"8e99eeae_51d36a17","updated":"2022-09-16 06:33:18.000000000","message":"We could add a comment in the environment file as a minimum.","commit_id":"e4c5afeda23a2ef1a78dc2c0ee5f9422644da240"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"06be7ef13990af4fde2a56626eb28daadfe39eed","unresolved":true,"context_lines":[{"line_number":8,"context_line":"  OS::TripleO::Services::OVNMetadataAgent: ../../deployment/ovn/ovn-metadata-container-puppet.yaml"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"# Disabling Neutron services that overlap with OVN"},{"line_number":11,"context_line":"  OS::TripleO::Services::NeutronDhcpAgent: OS::Heat::None"},{"line_number":12,"context_line":"  OS::TripleO::Services::NeutronL3Agent: OS::Heat::None"},{"line_number":13,"context_line":"  OS::TripleO::Services::NeutronOvsAgent: OS::Heat::None"},{"line_number":14,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"8e99eeae_51d36a17","line":11,"range":{"start_line":11,"start_character":2,"end_line":11,"end_character":57},"in_reply_to":"91cd3586_f7f42212","updated":"2022-09-15 14:28:08.000000000","message":"@Rabi yes right OVN do not support IPv6 provisioning yet[1], and yes we don\u0027t test that in any of the upstream/RDO jobs.\nCurrently considering the new env file makes thing optional not sure where to document unsupportibility of IPv6? I can put a follow up patch for that if required.\n\n[1] https://bugzilla.redhat.com/show_bug.cgi?id\u003d2062295","commit_id":"e4c5afeda23a2ef1a78dc2c0ee5f9422644da240"},{"author":{"_account_id":6681,"name":"Brent Eagles","email":"beagles@redhat.com","username":"beagles"},"change_message_id":"7ca7598bc3df2dd857a19cb4589e379fb521aa0d","unresolved":true,"context_lines":[{"line_number":22,"context_line":"  NeutronServicePlugins: \u0027ovn-router,segments\u0027"},{"line_number":23,"context_line":"  NeutronMechanismDrivers: [\u0027ovn\u0027, \u0027baremetal\u0027]"},{"line_number":24,"context_line":"  NeutronPluginExtensions: \"port_security,dns\""},{"line_number":25,"context_line":"  NeutronNetworkType: [\u0027geneve\u0027, \u0027vxlan\u0027, \u0027vlan\u0027, \u0027flat\u0027]"},{"line_number":26,"context_line":"  NeutronTypeDrivers: [\u0027geneve\u0027, \u0027vxlan\u0027, \u0027vlan\u0027, \u0027flat\u0027]"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"acb62e4a_1db89af8","line":25,"updated":"2022-09-14 16:25:26.000000000","message":"Not that it matters overly much but I wonder if we need all of the drivers on the undercloud.","commit_id":"e4c5afeda23a2ef1a78dc2c0ee5f9422644da240"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"18dbe062c1cccab79019ff3ab90ca5691e43ba42","unresolved":false,"context_lines":[{"line_number":22,"context_line":"  NeutronServicePlugins: \u0027ovn-router,segments\u0027"},{"line_number":23,"context_line":"  NeutronMechanismDrivers: [\u0027ovn\u0027, \u0027baremetal\u0027]"},{"line_number":24,"context_line":"  NeutronPluginExtensions: \"port_security,dns\""},{"line_number":25,"context_line":"  NeutronNetworkType: [\u0027geneve\u0027, \u0027vxlan\u0027, \u0027vlan\u0027, \u0027flat\u0027]"},{"line_number":26,"context_line":"  NeutronTypeDrivers: [\u0027geneve\u0027, \u0027vxlan\u0027, \u0027vlan\u0027, \u0027flat\u0027]"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"acbe68e6_d03a25c1","line":25,"in_reply_to":"acb62e4a_1db89af8","updated":"2022-09-15 11:14:52.000000000","message":"Yes all shouldn\u0027t be needed, but this shouldn\u0027t hurt and can be cleaned up later.","commit_id":"e4c5afeda23a2ef1a78dc2c0ee5f9422644da240"}]}