)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"586d0fa40b8617891eadfa4f71f026b52e3b81c1","unresolved":true,"context_lines":[{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Even though there is only one \"dvr_snat\" node in the job, it keeps"},{"line_number":16,"context_line":"\"l3_ha\" option set to \"true\" in Neutron\u0027s config. That way we are still"},{"line_number":17,"context_line":"testing \"ha\" code path of the centralized part of the router but also"},{"line_number":18,"context_line":"job is now closer to the real life, and supported deployments topology."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Additionally this patch adds nodeset for 3 nodes with Ubuntu Jammy and"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"b030ab84_858b123e","line":17,"range":{"start_line":17,"start_character":13,"end_line":17,"end_character":46},"updated":"2023-02-22 08:35:02.000000000","message":"qq: should we propose to use a 4 node set? With 2 controllers + 2 compute nodes, in order to check that the HA functionality works in case of redundancy (routers, dhcp).","commit_id":"169ef0558926656fe9ef38fea42dcada2786ae0d"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"9b483790c788112b4834881bde421f101d0b69f7","unresolved":false,"context_lines":[{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Even though there is only one \"dvr_snat\" node in the job, it keeps"},{"line_number":16,"context_line":"\"l3_ha\" option set to \"true\" in Neutron\u0027s config. That way we are still"},{"line_number":17,"context_line":"testing \"ha\" code path of the centralized part of the router but also"},{"line_number":18,"context_line":"job is now closer to the real life, and supported deployments topology."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Additionally this patch adds nodeset for 3 nodes with Ubuntu Jammy and"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"519dfe30_05f19fb2","line":17,"range":{"start_line":17,"start_character":13,"end_line":17,"end_character":46},"in_reply_to":"7efc9928_ebb38141","updated":"2023-02-28 09:45:25.000000000","message":"yeah... you are right, using 4 nodes for a CI job could be too much","commit_id":"169ef0558926656fe9ef38fea42dcada2786ae0d"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"858f5102fe54103930cb5cddae65d1f8a15b18a7","unresolved":true,"context_lines":[{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Even though there is only one \"dvr_snat\" node in the job, it keeps"},{"line_number":16,"context_line":"\"l3_ha\" option set to \"true\" in Neutron\u0027s config. That way we are still"},{"line_number":17,"context_line":"testing \"ha\" code path of the centralized part of the router but also"},{"line_number":18,"context_line":"job is now closer to the real life, and supported deployments topology."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Additionally this patch adds nodeset for 3 nodes with Ubuntu Jammy and"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"7efc9928_ebb38141","line":17,"range":{"start_line":17,"start_character":13,"end_line":17,"end_character":46},"in_reply_to":"b030ab84_858b123e","updated":"2023-02-28 08:23:42.000000000","message":"I\u0027m not sure if it\u0027s not too much for infra and also we don\u0027t have tests which really checks e.g. failover in such ha case","commit_id":"169ef0558926656fe9ef38fea42dcada2786ae0d"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"51f192e2c11b6dfca187390faba1927625e39caf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5c312ed6_47da4704","updated":"2023-02-21 12:05:34.000000000","message":"We need to check [1] test_schedule_to_all_nodes\n\nTraceback (most recent call last):\n  File \"/opt/stack/tempest/tempest/common/utils/__init__.py\", line 70, in wrapper\n    return f(*func_args, **func_kwargs)\n  File \"/opt/stack/tempest/tempest/scenario/test_server_multinode.py\", line 55, in test_schedule_to_all_nodes\n    raise exceptions.InvalidConfiguration(\ntempest.lib.exceptions.InvalidConfiguration: Invalid Configuration\nDetails: Host list [{\u0027zone\u0027: \u0027nova\u0027, \u0027host_name\u0027: \u0027np0033195998\u0027}, {\u0027zone\u0027: \u0027nova\u0027, \u0027host_name\u0027: \u0027np0033195997\u0027}] is shorter than min_compute_nodes. Did a compute worker not boot correctly?\n\n[1]https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_c61/874536/1/check/neutron-ovs-tempest-dvr-ha-multinode-full/c615db4/testr_results.html","commit_id":"f134c79d7736377a6ccb9fe46adb981e930dbc32"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"c258ae882aaa0203a84c561768b5acffb74bce7a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1eb37411_5d5fe157","in_reply_to":"00aa3b38_c7d6a866","updated":"2023-02-21 13:04:12.000000000","message":"Lets try it now :)","commit_id":"f134c79d7736377a6ccb9fe46adb981e930dbc32"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"1da0a99dab190b7d50567c3bf35d897a8cd8796a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"00aa3b38_c7d6a866","in_reply_to":"5c312ed6_47da4704","updated":"2023-02-21 12:46:24.000000000","message":"Ok, I think I found the problem [1]. We should add only the compute nodes here, not the controllers. I\u0027ll check that with gmann.\n\n[1]https://opendev.org/openstack/tempest/src/branch/master/zuul.d/base.yaml#L22","commit_id":"f134c79d7736377a6ccb9fe46adb981e930dbc32"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"fe9bb9ec829308f6c4fad99b24f5f8ef3af381c0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"82e2a077_8ce7e4ac","updated":"2023-02-22 08:40:12.000000000","message":"-1 just for visibility","commit_id":"169ef0558926656fe9ef38fea42dcada2786ae0d"}],"zuul.d/tempest-multinode.yaml":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"29b6b53c7ca6f318a5f5ea7eb33534e12912d567","unresolved":true,"context_lines":[{"line_number":1,"context_line":"- job:"},{"line_number":2,"context_line":"    name: neutron-ovs-tempest-dvr-ha-multinode-full"},{"line_number":3,"context_line":"    parent: tempest-multinode-full-py3"},{"line_number":4,"context_line":"    nodeset: openstack-three-node-focal"},{"line_number":5,"context_line":"    timeout: 10800"},{"line_number":6,"context_line":"    roles:"},{"line_number":7,"context_line":"      - zuul: openstack/neutron-tempest-plugin"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"27bbda13_f5be8e03","line":4,"updated":"2023-02-21 14:13:28.000000000","message":"wouldnt you better also use a different nodeset that doesn\u0027t put the controller into the compute group?","commit_id":"131c5fa00aec6deb3801c95db38f2f327c57b07f"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"858f5102fe54103930cb5cddae65d1f8a15b18a7","unresolved":false,"context_lines":[{"line_number":1,"context_line":"- job:"},{"line_number":2,"context_line":"    name: neutron-ovs-tempest-dvr-ha-multinode-full"},{"line_number":3,"context_line":"    parent: tempest-multinode-full-py3"},{"line_number":4,"context_line":"    nodeset: openstack-three-node-focal"},{"line_number":5,"context_line":"    timeout: 10800"},{"line_number":6,"context_line":"    roles:"},{"line_number":7,"context_line":"      - zuul: openstack/neutron-tempest-plugin"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"f47b378f_cd8fb44a","line":4,"in_reply_to":"27bbda13_f5be8e03","updated":"2023-02-28 08:23:42.000000000","message":"Done","commit_id":"131c5fa00aec6deb3801c95db38f2f327c57b07f"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"29b6b53c7ca6f318a5f5ea7eb33534e12912d567","unresolved":true,"context_lines":[{"line_number":219,"context_line":"        test-config:"},{"line_number":220,"context_line":"          $TEMPEST_CONFIG:"},{"line_number":221,"context_line":"            compute:"},{"line_number":222,"context_line":"              min_compute_nodes: 2"},{"line_number":223,"context_line":"            network-feature-enabled:"},{"line_number":224,"context_line":"              qos_placement_physnet: public"},{"line_number":225,"context_line":"              qos_min_bw_and_pps: True"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"17270f15_51682621","line":222,"updated":"2023-02-21 14:13:28.000000000","message":"with the above, this override also wouldn\u0027t be necessary","commit_id":"131c5fa00aec6deb3801c95db38f2f327c57b07f"}]}