)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":28048,"name":"Will Szumski","email":"will@stackhpc.com","username":"jovial"},"change_message_id":"c2224e5bb104eb44acd1b34bdc139ab31393b52e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"98f4c82d_4a5bfab9","updated":"2024-02-06 12:29:56.000000000","message":"Should probably at least document this option. Its a little weird since there is enable_neutron_ovs in kolla. Instead, openswitch is the default (neutron_plugin_agent: \"openvswitch\"). Normally these kolla_enable_ feature flags correspond to an equivalent variable in kolla. e.g kolla_enable_ironic maps to enable_ironic.","commit_id":"fe429f78d3554130125cac18191cf716cd629d22"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"5c9c5470359dc42517deb5505eef4fc8171a5f38","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"32959f6c_e4f4545e","in_reply_to":"1cb8aae1_b39dd971","updated":"2024-02-08 09:26:47.000000000","message":"I agree. As this only impacts neutron ovs, going with kolla_build_neutron_ovs sounds better to me. I\u0027ll change the variable name and also document this under /doc","commit_id":"fe429f78d3554130125cac18191cf716cd629d22"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"caf217035ecebbe65500f5d9f3f1c17cf7b93fd6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b8ebe7d2_b59f5cff","in_reply_to":"32959f6c_e4f4545e","updated":"2024-02-12 14:55:51.000000000","message":"Done","commit_id":"fe429f78d3554130125cac18191cf716cd629d22"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"bf774cf97b5db35fd07df052e06deef2b8276968","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1cb8aae1_b39dd971","in_reply_to":"98f4c82d_4a5bfab9","updated":"2024-02-07 09:02:48.000000000","message":"The name of this variable is confusing. It impacts only the image build, so it should be different to kolla_enable_* variables.\n\nWhat do you think of kolla_build_neutron_ovs or kolla_build_all_neutron_images?","commit_id":"fe429f78d3554130125cac18191cf716cd629d22"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"ea7edde3c44a82f669a8337b145242cb783238b5","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e553b262_a11b9b90","updated":"2024-02-12 15:50:06.000000000","message":"reno would be nice","commit_id":"69f851183f4ac645223bac4281d1f0911ccfcd1c"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"49926c022cb96406192c13c944ff08a4442c7894","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"3c30a4c5_aa80a4a2","in_reply_to":"e553b262_a11b9b90","updated":"2024-02-13 09:07:08.000000000","message":"It\u0027s added.","commit_id":"69f851183f4ac645223bac4281d1f0911ccfcd1c"},{"author":{"_account_id":28048,"name":"Will Szumski","email":"will@stackhpc.com","username":"jovial"},"change_message_id":"3abe762ab75103b71b3c8291ada4fab5a11f473c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"5fc1d6d3_b8819455","updated":"2024-02-15 15:43:15.000000000","message":"New variable name looks good, thanks.","commit_id":"ec5bbf398c9eb4d8763c4de8afd0aca4ae5b42c8"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"503074968370f2fadc77280497d0949eba5c6563","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"dc257c86_7afeb94a","updated":"2024-02-19 15:02:14.000000000","message":"Thanks","commit_id":"ec5bbf398c9eb4d8763c4de8afd0aca4ae5b42c8"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"67d98d24bd771a4f7a4c0f35fc99779bb42c18a1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"b154e627_451ddb9a","updated":"2024-02-13 15:56:15.000000000","message":"recheck ci failure unrelated to recent change","commit_id":"ec5bbf398c9eb4d8763c4de8afd0aca4ae5b42c8"}],"doc/source/deployment.rst":[{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"947edfb123c48a09dc043beea0731e64e529e570","unresolved":true,"context_lines":[{"line_number":534,"context_line":"    (kayobe) $ kayobe overcloud container image build ironic- nova-api"},{"line_number":535,"context_line":""},{"line_number":536,"context_line":"When your environment uses OVN, OVS images will not be built. If you want to"},{"line_number":537,"context_line":"build all images at the same time, extra variable ``kolla_build_neutron_ovs``"},{"line_number":538,"context_line":"needs to be set to ``true``::"},{"line_number":539,"context_line":""},{"line_number":540,"context_line":"    (kayobe) $ kayobe overcloud container image build -e kolla_build_neutron_ovs\u003dtrue"}],"source_content_type":"text/x-rst","patch_set":4,"id":"4302185d_0160d408","line":537,"range":{"start_line":537,"start_character":6,"end_line":537,"end_character":16},"updated":"2024-02-13 09:51:42.000000000","message":"nitpick: this isn\u0027t fully clear to me on when all the Neutron images are built. I\u0027d suggest you say\n```\nIf you want to build all Neutron images when OVN is enabled, the extra variable ``kolla_build_neutron_ovs`` needs to be set to ``true``:\n```","commit_id":"4fa8516fe722d44e91a01034662da3a3f4df9265"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"7d764e6e9aacfce15708d33b6d838b6db873ff71","unresolved":false,"context_lines":[{"line_number":534,"context_line":"    (kayobe) $ kayobe overcloud container image build ironic- nova-api"},{"line_number":535,"context_line":""},{"line_number":536,"context_line":"When your environment uses OVN, OVS images will not be built. If you want to"},{"line_number":537,"context_line":"build all images at the same time, extra variable ``kolla_build_neutron_ovs``"},{"line_number":538,"context_line":"needs to be set to ``true``::"},{"line_number":539,"context_line":""},{"line_number":540,"context_line":"    (kayobe) $ kayobe overcloud container image build -e kolla_build_neutron_ovs\u003dtrue"}],"source_content_type":"text/x-rst","patch_set":4,"id":"123f8193_80590ee8","line":537,"range":{"start_line":537,"start_character":6,"end_line":537,"end_character":16},"in_reply_to":"4302185d_0160d408","updated":"2024-02-13 11:54:31.000000000","message":"Sounds good. I made the change.","commit_id":"4fa8516fe722d44e91a01034662da3a3f4df9265"}]}
