)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34271,"name":"Miro Tomaska","display_name":"Miro Tomaska","email":"mtomaska@redhat.com","username":"mtomaska"},"change_message_id":"6b1213e9af3e2750f9703514fc40d8f7c5e27a1a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"a31789e6_a608e390","updated":"2022-07-25 19:09:50.000000000","message":"just a small nit besides what was already set. I will review again when you push new changes.","commit_id":"8f4cb8c1d14575485fc63b85337a09b2337c7cac"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"aa3eba2387cb0f31e1215e4e118a0c161f56856b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"6cf647f3_b7fda183","updated":"2022-07-22 14:46:37.000000000","message":"leaving a few early comments. Also have we done any testing about upgrade path ? AFAIK even after we switched to networking v2 we need to keep the neutron resources in undercloud and we need to migrate resources from ml2+ovs to ml2+ovn.\n\nIf that upgrade path is not yet implemented in undercloud upgrade then we should not switch the default at this moment, IMHO.","commit_id":"8f4cb8c1d14575485fc63b85337a09b2337c7cac"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"eb1ff343ca09ab36902c696b72e23b7d11b1426d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"894077cf_ce37996d","in_reply_to":"4959df5b_4cc29014","updated":"2022-08-17 13:50:35.000000000","message":"Proposed https://review.opendev.org/c/openstack/python-tripleoclient/+/853410 to do not allow upgrade if plugin is changed during undercloud upgrade. migration part will be taken care seperately.","commit_id":"8f4cb8c1d14575485fc63b85337a09b2337c7cac"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"1835cb7f24fca7aa01d4341ff62511ce3dd26925","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"4959df5b_4cc29014","in_reply_to":"6cf647f3_b7fda183","updated":"2022-07-22 15:04:28.000000000","message":"No no testing is done for undercloud yet, all migration steps available only for overcloud and we expected same will apply to undercloud as well.\n\nAnd yes you are right resources needs migration when switching ml2 ovs to ovn, We are still considering migration path, i thought i had added details in releasenotes/commit message but missed. Thanks for raising it so can be discussed here itself.\n\nWe were thinking of keeping migration thing seperate effort as follow up but before zed release, and i think same was done when default was switched in overcloud.","commit_id":"8f4cb8c1d14575485fc63b85337a09b2337c7cac"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"4f58357b7b6c99102104b8aa07bc37b7d271c5c8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"80db9956_18a23b55","updated":"2022-08-17 14:19:45.000000000","message":"I think https://review.opendev.org/c/openstack/tripleo-quickstart/+/853503 needs to be incorporated into this change to cover FFU upgrades impact","commit_id":"c7b81f39d78dada633e5580b98698cdceed3f622"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"ee7ee6140ca0a732dc385ed2990f9c367def829a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"131495b6_df296fa3","in_reply_to":"80db9956_18a23b55","updated":"2022-08-17 14:24:43.000000000","message":"Sorry but i didn\u0027t got, can you please elaborate on what \"cover FFU upgrades impact\" means? And how to incorporate that job specific change here in THT?","commit_id":"c7b81f39d78dada633e5580b98698cdceed3f622"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"5d3cdc9ef38490f6e92eb314cb70da0d1daa3bd9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"3cdbba83_23e472a0","updated":"2022-08-18 09:24:40.000000000","message":"-W until understand/clears bogdan\u0027s concern","commit_id":"d8c96160ba130afbf095a719f8f972ee8c14f2ac"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"ddab4228b88a74e90f4a38473b6f82ea5f9bc5b8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"fb27e3e0_20f54785","updated":"2022-08-24 14:47:56.000000000","message":"ok, now with https://review.opendev.org/c/openstack/python-tripleoclient/+/853411 I think this is fine to have this change in CI. Just please make sure the cases when users will require to manually add that env file to their CLI commands are documented and explained, not leaving this a CI-only change not covered in product and tripleo.","commit_id":"d8c96160ba130afbf095a719f8f972ee8c14f2ac"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"d5650b2f59481abb71ec2c1455b8463940a37aca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"9c58202f_6f116e44","updated":"2022-09-12 04:29:58.000000000","message":"IMO we should not switch the default plugin and keep ml2+ovn until we implement the proper upgrade path.\n\nI understand you are providing the option to keep ml2+ovs but without any migration path then we are likely to see bunch of deployments with ml2+ovs upgraded from the old version. Then removing test coverage of ml2+ovs in undercloud is quite concerning.","commit_id":"6be36ab29996c5b416297aaa17bb8d98a4d16590"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"00b6ac99f9e261a005661313ef16105dc5f05c6f","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":9,"id":"4bebcbb7_91f1be48","updated":"2023-10-02 23:19:31.000000000","message":"Should we abandon this now, as TripleO has been deprecated ?","commit_id":"6be36ab29996c5b416297aaa17bb8d98a4d16590"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"f0794272fd0c368382c4a6a3a8949c0e89ac8f23","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"aec87c1f_f5e395fc","updated":"2022-09-13 13:50:04.000000000","message":"Thanks Everyone, Taking alternative path https://review.opendev.org/c/openstack/tripleo-heat-templates/+/857436\n\nAnd once we evaluated upgrade/migration path for Undercloud networking plugin we can revisit switching default. Till then we can test Undercloud with Ml2 OVN on some jobs.","commit_id":"6be36ab29996c5b416297aaa17bb8d98a4d16590"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"0272a6bebd8dc4d5157bb0441968b44cfb2ec383","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"8d6a2fff_3abb0296","updated":"2022-09-12 03:36:19.000000000","message":"check-rdo overcloud-images up to date with latest kernel","commit_id":"6be36ab29996c5b416297aaa17bb8d98a4d16590"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"fcffae4cfc612e52f768d3c33395c877d80d646e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"3168835e_3ff471ba","updated":"2022-09-12 06:08:45.000000000","message":"check-rdo tripleo-mount-image failure","commit_id":"6be36ab29996c5b416297aaa17bb8d98a4d16590"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"38671ebcc0d750dec2819437c3c6f7288182f5e3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"65c53056_d46f2d0b","in_reply_to":"53ee9555_af6966fd","updated":"2022-09-12 05:54:02.000000000","message":"\u003c\u003c\u003c I understand you are providing the option to keep ml2+ovs but without any migration path then we are likely to see bunch of deployments with ml2+ovs upgraded from the old version. Then removing test coverage of ml2+ovs in undercloud is quite concerning.\n\nThanks Takashi, yes you are right ml2+ovs in Undercloud will still be there for upgrade cases atleast until we completely drop ml2+ovs[1](pre requisite for that is complete migration path for  both Undercloud/Overcloud) and this is kind of expected as we see default changed to ML2 OVN in Overcloud since Stein release, but we do still multiple ml2+ovs deployments. The change in Undercloud just got delayed but considering the future plans for ML2 OVS[1] the switch in Undercloud is quite needed now.\nFor the coverage part we can keep one job for ml2-ovs coverage like we do for Overcloud since stein release.\n\n[1] https://bugzilla.redhat.com/show_bug.cgi?id\u003d2100403","commit_id":"6be36ab29996c5b416297aaa17bb8d98a4d16590"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"d1f4235e6b88f18f256816d0d87b6fd706462921","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":9,"id":"e469c0d5_c6a6b686","in_reply_to":"65c53056_d46f2d0b","updated":"2022-09-13 05:16:22.000000000","message":"Yes we can add a few tests with ml2+ovs but I\u0027m still not sure how much gain we\u0027ll have compared to what we need to support both ml2+ovs and ml2+ovn such as the overhead we need in CI, our maintenance effort and downstream support effort.\n\nIMO everything is a kind of blocked by the migration path and we should implement that path or at least get a clear view about our plan and the feasibility before we make this switch. As I could not find any effort to complete that migration, I\u0027m concerned that we might end up with supporting these two for multiple cycles.","commit_id":"6be36ab29996c5b416297aaa17bb8d98a4d16590"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"76d96157a9bd820c3bca2d15cd6981c2bdda3a35","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"53ee9555_af6966fd","in_reply_to":"9c58202f_6f116e44","updated":"2022-09-12 04:30:36.000000000","message":"\u003e IMO we should not switch the default plugin and keep ml2+ovn until we implement the proper upgrade path.\n\nOops... I meant to say;\n\nIMO we should not switch the default plugin and keep *ml2+ovs* until we implement the proper upgrade path.","commit_id":"6be36ab29996c5b416297aaa17bb8d98a4d16590"}],"environments/undercloud.yaml":[{"author":{"_account_id":34271,"name":"Miro Tomaska","display_name":"Miro Tomaska","email":"mtomaska@redhat.com","username":"mtomaska"},"change_message_id":"6b1213e9af3e2750f9703514fc40d8f7c5e27a1a","unresolved":true,"context_lines":[{"line_number":12,"context_line":"  OS::TripleO::Services::OVNDBs: ../deployment/ovn/ovn-dbs-container-puppet.yaml"},{"line_number":13,"context_line":"  OS::TripleO::Services::OVNController: ../deployment/ovn/ovn-controller-container-puppet.yaml"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"  # Neutron DHCP agent as OVN do not support DHCP over ipv6 yet"},{"line_number":16,"context_line":"  OS::TripleO::Services::NeutronDhcpAgent: ../deployment/neutron/neutron-dhcp-container-puppet.yaml"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"# Disabling Neutron services that overlap with OVN"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"a24fd55e_39af60d3","line":15,"range":{"start_line":15,"start_character":2,"end_line":15,"end_character":63},"updated":"2022-07-25 19:09:50.000000000","message":"nit:\n\"Neutron DHCP agent as OVN\u0027s DHCP server provided via OVNController does not support ipv6 yet.\"\nJust so its clear where the \"other\" dhcp is coming from","commit_id":"8f4cb8c1d14575485fc63b85337a09b2337c7cac"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"eb1ff343ca09ab36902c696b72e23b7d11b1426d","unresolved":false,"context_lines":[{"line_number":12,"context_line":"  OS::TripleO::Services::OVNDBs: ../deployment/ovn/ovn-dbs-container-puppet.yaml"},{"line_number":13,"context_line":"  OS::TripleO::Services::OVNController: ../deployment/ovn/ovn-controller-container-puppet.yaml"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"  # Neutron DHCP agent as OVN do not support DHCP over ipv6 yet"},{"line_number":16,"context_line":"  OS::TripleO::Services::NeutronDhcpAgent: ../deployment/neutron/neutron-dhcp-container-puppet.yaml"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"# Disabling Neutron services that overlap with OVN"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"f1385dfd_bc50d9b3","line":15,"range":{"start_line":15,"start_character":2,"end_line":15,"end_character":63},"in_reply_to":"a24fd55e_39af60d3","updated":"2022-08-17 13:50:35.000000000","message":"Done","commit_id":"8f4cb8c1d14575485fc63b85337a09b2337c7cac"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"aa3eba2387cb0f31e1215e4e118a0c161f56856b","unresolved":true,"context_lines":[{"line_number":95,"context_line":"  IronicConfigureSwiftTempUrlKey: false"},{"line_number":96,"context_line":"  IronicProvisioningNetwork: \u0027ctlplane\u0027"},{"line_number":97,"context_line":"  IronicRescuingNetwork: \u0027ctlplane\u0027"},{"line_number":98,"context_line":"  NeutronServicePlugins: \u0027qos,ovn-router,trunk,segments,port_forwarding,log\u0027"},{"line_number":99,"context_line":"  NeutronMechanismDrivers: [\u0027ovn\u0027, \u0027baremetal\u0027]"},{"line_number":100,"context_line":"  NeutronNetworkVLANRanges: \u0027physnet1:1000:2999\u0027"},{"line_number":101,"context_line":"  NeutronPluginExtensions: \"qos,port_security,dns\""}],"source_content_type":"text/x-yaml","patch_set":6,"id":"7ddad491_abb7b8cc","line":98,"range":{"start_line":98,"start_character":26,"end_line":98,"end_character":29},"updated":"2022-07-22 14:46:37.000000000","message":"This adds a few plugins(qos, trunk, port_fowarding and log) but IIUC these plugins are not required in undercloud then I\u0027d keep this list minimum following what we had in the previous version.","commit_id":"8f4cb8c1d14575485fc63b85337a09b2337c7cac"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"1835cb7f24fca7aa01d4341ff62511ce3dd26925","unresolved":true,"context_lines":[{"line_number":95,"context_line":"  IronicConfigureSwiftTempUrlKey: false"},{"line_number":96,"context_line":"  IronicProvisioningNetwork: \u0027ctlplane\u0027"},{"line_number":97,"context_line":"  IronicRescuingNetwork: \u0027ctlplane\u0027"},{"line_number":98,"context_line":"  NeutronServicePlugins: \u0027qos,ovn-router,trunk,segments,port_forwarding,log\u0027"},{"line_number":99,"context_line":"  NeutronMechanismDrivers: [\u0027ovn\u0027, \u0027baremetal\u0027]"},{"line_number":100,"context_line":"  NeutronNetworkVLANRanges: \u0027physnet1:1000:2999\u0027"},{"line_number":101,"context_line":"  NeutronPluginExtensions: \"qos,port_security,dns\""}],"source_content_type":"text/x-yaml","patch_set":6,"id":"e7f72e0b_7bb0b966","line":98,"range":{"start_line":98,"start_character":26,"end_line":98,"end_character":29},"in_reply_to":"7ddad491_abb7b8cc","updated":"2022-07-22 15:04:28.000000000","message":"I copied it from ovn non-ha config, but yes agree unrequired things will be cleaned up.","commit_id":"8f4cb8c1d14575485fc63b85337a09b2337c7cac"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"eb1ff343ca09ab36902c696b72e23b7d11b1426d","unresolved":false,"context_lines":[{"line_number":95,"context_line":"  IronicConfigureSwiftTempUrlKey: false"},{"line_number":96,"context_line":"  IronicProvisioningNetwork: \u0027ctlplane\u0027"},{"line_number":97,"context_line":"  IronicRescuingNetwork: \u0027ctlplane\u0027"},{"line_number":98,"context_line":"  NeutronServicePlugins: \u0027qos,ovn-router,trunk,segments,port_forwarding,log\u0027"},{"line_number":99,"context_line":"  NeutronMechanismDrivers: [\u0027ovn\u0027, \u0027baremetal\u0027]"},{"line_number":100,"context_line":"  NeutronNetworkVLANRanges: \u0027physnet1:1000:2999\u0027"},{"line_number":101,"context_line":"  NeutronPluginExtensions: \"qos,port_security,dns\""}],"source_content_type":"text/x-yaml","patch_set":6,"id":"04e4aaf3_50405aba","line":98,"range":{"start_line":98,"start_character":26,"end_line":98,"end_character":29},"in_reply_to":"e7f72e0b_7bb0b966","updated":"2022-08-17 13:50:35.000000000","message":"Done","commit_id":"8f4cb8c1d14575485fc63b85337a09b2337c7cac"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"aa3eba2387cb0f31e1215e4e118a0c161f56856b","unresolved":true,"context_lines":[{"line_number":98,"context_line":"  NeutronServicePlugins: \u0027qos,ovn-router,trunk,segments,port_forwarding,log\u0027"},{"line_number":99,"context_line":"  NeutronMechanismDrivers: [\u0027ovn\u0027, \u0027baremetal\u0027]"},{"line_number":100,"context_line":"  NeutronNetworkVLANRanges: \u0027physnet1:1000:2999\u0027"},{"line_number":101,"context_line":"  NeutronPluginExtensions: \"qos,port_security,dns\""},{"line_number":102,"context_line":"  NeutronOVSFirewallDriver: \u0027\u0027"},{"line_number":103,"context_line":"  NeutronNetworkType: [\u0027geneve\u0027, \u0027vxlan\u0027, \u0027vlan\u0027, \u0027flat\u0027]"},{"line_number":104,"context_line":"  NeutronTunnelIdRanges: \u002720:100\u0027"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"61f7d87c_74ba2eec","line":101,"range":{"start_line":101,"start_character":28,"end_line":101,"end_character":31},"updated":"2022-07-22 14:46:37.000000000","message":"ditto","commit_id":"8f4cb8c1d14575485fc63b85337a09b2337c7cac"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"eb1ff343ca09ab36902c696b72e23b7d11b1426d","unresolved":false,"context_lines":[{"line_number":98,"context_line":"  NeutronServicePlugins: \u0027qos,ovn-router,trunk,segments,port_forwarding,log\u0027"},{"line_number":99,"context_line":"  NeutronMechanismDrivers: [\u0027ovn\u0027, \u0027baremetal\u0027]"},{"line_number":100,"context_line":"  NeutronNetworkVLANRanges: \u0027physnet1:1000:2999\u0027"},{"line_number":101,"context_line":"  NeutronPluginExtensions: \"qos,port_security,dns\""},{"line_number":102,"context_line":"  NeutronOVSFirewallDriver: \u0027\u0027"},{"line_number":103,"context_line":"  NeutronNetworkType: [\u0027geneve\u0027, \u0027vxlan\u0027, \u0027vlan\u0027, \u0027flat\u0027]"},{"line_number":104,"context_line":"  NeutronTunnelIdRanges: \u002720:100\u0027"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"a31dc81f_96a66f74","line":101,"range":{"start_line":101,"start_character":28,"end_line":101,"end_character":31},"in_reply_to":"61f7d87c_74ba2eec","updated":"2022-08-17 13:50:35.000000000","message":"Done","commit_id":"8f4cb8c1d14575485fc63b85337a09b2337c7cac"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"aa3eba2387cb0f31e1215e4e118a0c161f56856b","unresolved":true,"context_lines":[{"line_number":104,"context_line":"  NeutronTunnelIdRanges: \u002720:100\u0027"},{"line_number":105,"context_line":"  NeutronTypeDrivers: [\u0027geneve\u0027, \u0027vxlan\u0027, \u0027vlan\u0027, \u0027flat\u0027]"},{"line_number":106,"context_line":"  NeutronVniRanges: \u002710:100\u0027"},{"line_number":107,"context_line":"  NeutronEnableDVR: true"},{"line_number":108,"context_line":"  NeutronPortQuota: \u0027-1\u0027"},{"line_number":109,"context_line":"  # This allows MTU \u003e 1500 for the overcloud if local_mtu is set to 1500"},{"line_number":110,"context_line":"  # See LP#1826729"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"59beaf20_a60869ef","line":107,"range":{"start_line":107,"start_character":2,"end_line":107,"end_character":18},"updated":"2022-07-22 14:46:37.000000000","message":"Does this have any actual effect ? If no I\u0027d leave it false as we don\u0027t really expect distributed floating ip does not look valid based on our network design in undercloud.","commit_id":"8f4cb8c1d14575485fc63b85337a09b2337c7cac"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"eb1ff343ca09ab36902c696b72e23b7d11b1426d","unresolved":false,"context_lines":[{"line_number":104,"context_line":"  NeutronTunnelIdRanges: \u002720:100\u0027"},{"line_number":105,"context_line":"  NeutronTypeDrivers: [\u0027geneve\u0027, \u0027vxlan\u0027, \u0027vlan\u0027, \u0027flat\u0027]"},{"line_number":106,"context_line":"  NeutronVniRanges: \u002710:100\u0027"},{"line_number":107,"context_line":"  NeutronEnableDVR: true"},{"line_number":108,"context_line":"  NeutronPortQuota: \u0027-1\u0027"},{"line_number":109,"context_line":"  # This allows MTU \u003e 1500 for the overcloud if local_mtu is set to 1500"},{"line_number":110,"context_line":"  # See LP#1826729"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"d6f115bb_cee8d25e","line":107,"range":{"start_line":107,"start_character":2,"end_line":107,"end_character":18},"in_reply_to":"526c6a0a_6e604776","updated":"2022-08-17 13:50:35.000000000","message":"Done","commit_id":"8f4cb8c1d14575485fc63b85337a09b2337c7cac"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"1835cb7f24fca7aa01d4341ff62511ce3dd26925","unresolved":true,"context_lines":[{"line_number":104,"context_line":"  NeutronTunnelIdRanges: \u002720:100\u0027"},{"line_number":105,"context_line":"  NeutronTypeDrivers: [\u0027geneve\u0027, \u0027vxlan\u0027, \u0027vlan\u0027, \u0027flat\u0027]"},{"line_number":106,"context_line":"  NeutronVniRanges: \u002710:100\u0027"},{"line_number":107,"context_line":"  NeutronEnableDVR: true"},{"line_number":108,"context_line":"  NeutronPortQuota: \u0027-1\u0027"},{"line_number":109,"context_line":"  # This allows MTU \u003e 1500 for the overcloud if local_mtu is set to 1500"},{"line_number":110,"context_line":"  # See LP#1826729"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"526c6a0a_6e604776","line":107,"range":{"start_line":107,"start_character":2,"end_line":107,"end_character":18},"in_reply_to":"59beaf20_a60869ef","updated":"2022-07-22 15:04:28.000000000","message":"same can be cleaned up considering its not needed.","commit_id":"8f4cb8c1d14575485fc63b85337a09b2337c7cac"}],"releasenotes/notes/switch-undercloud-default-to-ovn-9322a7c2778d5406.yaml":[{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"62dd93179dea5e7f36d90ac6412548a142b5e3c3","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"upgrade:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    The default ML2 plugin on Undercloud is now ``OVN``."},{"line_number":5,"context_line":"    ``environments/services/undercloud-neutron-ovs.yaml`` environment"},{"line_number":6,"context_line":"    file can be passed to undercloud install/upgrade"},{"line_number":7,"context_line":"    to keep deploying it with ML2 ``OVS``."}],"source_content_type":"text/x-yaml","patch_set":7,"id":"2d30d3e0_ffd6e0de","line":4,"updated":"2022-08-17 14:22:15.000000000","message":"please mention the real upgrade impact, if changing from OVS to OVN, if that supported. What additional migrations will be required?","commit_id":"c7b81f39d78dada633e5580b98698cdceed3f622"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"9c1ac136cc47ed7c3de20086793b1cce405259b2","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"upgrade:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    The default ML2 plugin on Undercloud is now ``OVN``."},{"line_number":5,"context_line":"    ``environments/services/undercloud-neutron-ovs.yaml`` environment"},{"line_number":6,"context_line":"    file can be passed to undercloud install/upgrade"},{"line_number":7,"context_line":"    to keep deploying it with ML2 ``OVS``."}],"source_content_type":"text/x-yaml","patch_set":7,"id":"9039132d_74d08678","line":4,"in_reply_to":"2d30d3e0_ffd6e0de","updated":"2022-08-17 14:30:49.000000000","message":"Actually OVN is default in overcloud since stein release and migration process available for that(i need to check if there are upstream docs for that), but no upstream CI job exist for those. Migration of ovs to ovn on undercloud is not yet evaluated, that will be done follow up to the default switch. \n\nBut yes good point i will update the release note about ovs to ovn upgrade not allowed[1], only possible way is migrate to ovn and then upgrade.\n[1] https://review.opendev.org/c/openstack/python-tripleoclient/+/853411/","commit_id":"c7b81f39d78dada633e5580b98698cdceed3f622"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"ddab4228b88a74e90f4a38473b6f82ea5f9bc5b8","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"upgrade:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    The default ML2 plugin on Undercloud is now ``OVN``."},{"line_number":5,"context_line":"    ``environments/services/undercloud-neutron-ovs.yaml`` environment"},{"line_number":6,"context_line":"    file can be passed to undercloud install/upgrade"},{"line_number":7,"context_line":"    to keep deploying it with ML2 ``OVS``."}],"source_content_type":"text/x-yaml","patch_set":7,"id":"05b324b8_5bf8493d","line":4,"in_reply_to":"9039132d_74d08678","updated":"2022-08-24 14:47:56.000000000","message":"ok, now with https://review.opendev.org/c/openstack/python-tripleoclient/+/853411 I think this OK. Just please make sure the upgrade story for undercloud is properly documented to cover this change","commit_id":"c7b81f39d78dada633e5580b98698cdceed3f622"}]}