)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"0b332b111d5af6b452110c339553661623f0df60","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"e308d021_8a376aa8","updated":"2024-08-15 17:04:09.000000000","message":"Thanks for testing Bence. I guess I\u0027m fine with merging since it does at least test OVN, and we can debug the gate setup issue in the meantime.","commit_id":"e0d03fc6b2a3a6b3ff89f07d5aa0501daa0a76eb"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"4719a7c0c964ace56eb75229a89ffaa954612d5b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"5501c4fa_579881f8","updated":"2024-08-18 22:31:28.000000000","message":"The test proposed in this patch is failing in the gate with ML2/OVS and ML2/LB because we don\u0027t enable metadata with isolated networks in the DHCP agent. This patch fixes the problem: https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/926503","commit_id":"e0d03fc6b2a3a6b3ff89f07d5aa0501daa0a76eb"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"c0f0a49b766da196f110ab9171b728e9dd5e7290","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"fc70cc43_be0c1531","updated":"2024-08-21 14:31:43.000000000","message":"recheck","commit_id":"e0d03fc6b2a3a6b3ff89f07d5aa0501daa0a76eb"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"74aea728e9f39bca44c10cafdb2be928cdfa3ece","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"970fdad6_38228575","updated":"2024-08-21 22:27:39.000000000","message":"recheck test cases failures inneutron_tempest_plugin.api.test_routers.RoutersIpV6Test unrelated to this change","commit_id":"e0d03fc6b2a3a6b3ff89f07d5aa0501daa0a76eb"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"74305771c5cec6aa38670da69f0c7bd312e7d07f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"86502f8e_63e0f74b","updated":"2024-08-21 17:25:50.000000000","message":"recheck unrelated failure","commit_id":"e0d03fc6b2a3a6b3ff89f07d5aa0501daa0a76eb"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"40cf92cc564093cec2a52b90173810596acd894a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"8a516547_cc1d59ae","updated":"2024-08-13 22:23:42.000000000","message":"recheck unrelated failure in neutron_tempest_plugin.api.test_subnets.SubnetsSearchCriteriaTest.test_list_pagination_with_href_links","commit_id":"e0d03fc6b2a3a6b3ff89f07d5aa0501daa0a76eb"}],"neutron_tempest_plugin/scenario/test_metadata.py":[{"author":{"_account_id":15554,"name":"Bence Romsics","email":"bence.romsics@gmail.com","username":"ebenrom","status":"working for Ericsson, UTC+1 (+DST)"},"change_message_id":"0fabdbb3655df99952661e7e54c2ec698d68388e","unresolved":true,"context_lines":[{"line_number":212,"context_line":"        self.wait_for_server_active(server\u003dvm)"},{"line_number":213,"context_line":"        self.wait_for_guest_os_ready(vm)"},{"line_number":214,"context_line":"        console_output, pos \u003d self._wait_for_metadata_query_msg(vm)"},{"line_number":215,"context_line":"        self.assertIn(\u0027latest\u0027, console_output[pos - 100:])"}],"source_content_type":"text/x-python","patch_set":7,"id":"6ced4f56_64a4a593","line":215,"range":{"start_line":215,"start_character":23,"end_line":215,"end_character":29},"updated":"2024-08-13 16:39:12.000000000","message":"Hi Miguel,\n\nI read what you wrote in the weekly meeting, so I had a quick look what may be the cause. Did not use your test yet, just built a recent master devstack and booted two vms - one on an isolated network and another on a routed network. Each networks had a single ipv6 subnet. I did some manual testing on these. In my environment on both networks metadata worked. For example:\n\n```\nroot@ubuntu:~# curl \u0027http://[fe80::a9fe:a9fe%ens2]/\u0027\n1.0\n2007-01-19\n2007-03-01\n2007-08-29\n2007-10-10\n2007-12-15\n2008-02-01\n2008-09-01\n2009-04-04\nlatestroot@ubuntu:~# curl \u0027http://[fe80::a9fe:a9fe%ens2]/openstack\u0027\n2012-08-10\n2013-04-04\n2013-10-17\n2015-10-15\n2016-06-30\n2016-10-06\n2017-02-22\n2018-08-27\n2020-10-14\n```\n\nAs you can see there\u0027s no \u0027latest\u0027 path here - I don\u0027t know why, but this may be one reason why the test fails. Let me know if you think we have other problems and I can help.","commit_id":"d16332aa4b977e8df026f8908f634f8ebe1bcd48"},{"author":{"_account_id":15554,"name":"Bence Romsics","email":"bence.romsics@gmail.com","username":"ebenrom","status":"working for Ericsson, UTC+1 (+DST)"},"change_message_id":"5713a99a79bc12899dfc9d15eb6fa93bca0bfa67","unresolved":false,"context_lines":[{"line_number":212,"context_line":"        self.wait_for_server_active(server\u003dvm)"},{"line_number":213,"context_line":"        self.wait_for_guest_os_ready(vm)"},{"line_number":214,"context_line":"        console_output, pos \u003d self._wait_for_metadata_query_msg(vm)"},{"line_number":215,"context_line":"        self.assertIn(\u0027latest\u0027, console_output[pos - 100:])"}],"source_content_type":"text/x-python","patch_set":7,"id":"55f39d7d_0718b994","line":215,"range":{"start_line":215,"start_character":23,"end_line":215,"end_character":29},"in_reply_to":"38ce3224_12a906a5","updated":"2024-08-14 12:18:18.000000000","message":"My last comment was wrong. The \u0027latest\u0027 entry is actually there, only we have no newline after it, so I did not notice it in front of my prompt.\n\nAgain, I see ipv6-only metadata working in an ml2/ovs environment.\n\nIn a manual test:\n```\nopenstack image create --disk-format qcow2 --public --file ~/ubuntu-18.04-server-cloudimg-amd64-root-password.img u1804\n\n# isolated\nopenstack net create net0\nopenstack subnet create subnet0 --ip-version 6 --network net0 --subnet-range 2001:db8:1::/64 --dhcp --ipv6-ra-mode slaac --ipv6-address-mode slaac\n\n# routed\nopenstack net create net1\nopenstack subnet create subnet1 --ip-version 6 --network net1 --subnet-range 2001:db8:1::/64 --dhcp --ipv6-ra-mode slaac --ipv6-address-mode slaac\nopenstack router add subnet router1 subnet1\n\nopenstack server create --flavor ds1G --image u1804 --nic net-id\u003dnet0 vm0 --wait\nopenstack server create --flavor ds1G --image u1804 --nic net-id\u003dnet1 vm1 --wait\n\nsudo virsh console \"$( openstack server show vm0 -f value -c OS-EXT-SRV-ATTR:instance_name )\"\ncurl \u0027http://[fe80::a9fe:a9fe%ens2]/\u0027\n\nsudo virsh console \"$( openstack server show vm1 -f value -c OS-EXT-SRV-ATTR:instance_name )\"\ncurl \u0027http://[fe80::a9fe:a9fe%ens2]/\u0027\n```\n\nToday I also fetched this tempest test, deleted the skip-if-not-ovn decorator so I could run it in an ml2/ovs devstack. And it (ps8) passed:\n```\nscp /home/rubasov/w/cloud-images/20.04-minimal/ubuntu-20.04-minimal-cloudimg-amd64-root-password-neutron-advanced-packages.img devstack0:advanced.img\n\nsource openrc admin admin\nopenstack image create --disk-format qcow2 --public --file ~/advanced.img advanced\n\nsource ~/src/os/openstack/devstack/inc/ini-config\niniset /opt/stack/tempest/etc/tempest.conf neutron_plugin_options advanced_image_ref \"$( openstack image show advanced -f value -c id )\"\niniset /opt/stack/tempest/etc/tempest.conf neutron_plugin_options advanced_image_flavor_ref d1\niniset /opt/stack/tempest/etc/tempest.conf neutron_plugin_options advanced_image_ssh_user ubuntu\niniset /opt/stack/tempest/etc/tempest.conf neutron_plugin_options default_image_is_advanced False\niniset /opt/stack/tempest/etc/tempest.conf network public_network_id \"$( openstack net show public -f value -c id )\"\n\nsource /opt/stack/data/venv/bin/activate\n\n( mkdir /opt/stack/tempest-ws0/ \u0026\u0026 cd /opt/stack/tempest-ws0/ \u0026\u0026 tempest init )\n( cd /opt/stack/neutron-tempest-plugin \u0026\u0026 git fetch --prune \u0026\u0026 git checkout origin/review/miguel_lavalle/metadata-on-ipv6 \u0026\u0026 python3 -m pip install --force-reinstall --no-deps -e . \u0026\u0026 tempest run --workspace tempest-ws0 --config-file /opt/stack/tempest/etc/tempest.conf -r \u0027test_metadata_ipv6_only_network\u0027 )\n```\n\n```\n{0} neutron_tempest_plugin.scenario.test_metadata.MetadataTest.test_metadata_ipv6_only_network [183.876650s] ... ok\n\n\u003d\u003d\u003d\u003d\u003d\u003d\nTotals\n\u003d\u003d\u003d\u003d\u003d\u003d\nRan: 1 tests in 183.8767 sec.\n - Passed: 1\n - Skipped: 0\n - Expected Fail: 0\n - Unexpected Success: 0\n - Failed: 0\nSum of execute time for each test: 183.8767 sec.\n\n\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\nWorker Balance\n\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n - Worker 0 (1 tests) \u003d\u003e 0:03:03.876650\n\n```\n\nI believe this means that if it\u0027s not passing in the gate, then we have a problem with the environment setup in the gate and not with the feature or the test.","commit_id":"d16332aa4b977e8df026f8908f634f8ebe1bcd48"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"d1888c4067bfdd87142fa8e792706240d6a0a872","unresolved":false,"context_lines":[{"line_number":212,"context_line":"        self.wait_for_server_active(server\u003dvm)"},{"line_number":213,"context_line":"        self.wait_for_guest_os_ready(vm)"},{"line_number":214,"context_line":"        console_output, pos \u003d self._wait_for_metadata_query_msg(vm)"},{"line_number":215,"context_line":"        self.assertIn(\u0027latest\u0027, console_output[pos - 100:])"}],"source_content_type":"text/x-python","patch_set":7,"id":"52eab273_a9a290a1","line":215,"range":{"start_line":215,"start_character":23,"end_line":215,"end_character":29},"in_reply_to":"55f39d7d_0718b994","updated":"2024-08-15 00:32:54.000000000","message":"Hi Bence,\n\nThank you for following up and testing the test case locally. So we need to investigate the issue in the gate. \n\nIn the meantime we can merge the new test case with the skip in case the environment is not OVN","commit_id":"d16332aa4b977e8df026f8908f634f8ebe1bcd48"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"d1afcaa9d0409c4a5170bd9222b1d6fa39c45cec","unresolved":false,"context_lines":[{"line_number":212,"context_line":"        self.wait_for_server_active(server\u003dvm)"},{"line_number":213,"context_line":"        self.wait_for_guest_os_ready(vm)"},{"line_number":214,"context_line":"        console_output, pos \u003d self._wait_for_metadata_query_msg(vm)"},{"line_number":215,"context_line":"        self.assertIn(\u0027latest\u0027, console_output[pos - 100:])"}],"source_content_type":"text/x-python","patch_set":7,"id":"38ce3224_12a906a5","line":215,"range":{"start_line":215,"start_character":23,"end_line":215,"end_character":29},"in_reply_to":"6ced4f56_64a4a593","updated":"2024-08-13 19:54:17.000000000","message":"Hi Bence,\n\nThanks for taking the time to comment on this. In previous versions of this change, I was echoing \u0027Queried the metadata service over IPv6\u0027 and then executed the curl query. In that case, I found that \u0027latest\u0027 got mixed with cloud-init output and was separated from the rest of the paths. However, in the latest versions of this test, I echo \u0027Queried the metadata service over IPv6\u0027 after the curl command and this is what I get: https://paste.openstack.org/show/bgGkvIWeqatz8vdzXQ8G/. And indeed, in the zuul execution of the latest version of this patch, all the tests succeed, even test_metadata_routed that you wrote and searches for \u0027latest\u0027 in the curl output: https://b491ce03a38efddd7027-076a9e2443bfffb8bd944165a65c0952.ssl.cf5.rackcdn.com/925928/7/check/neutron-tempest-plugin-ovn/eec30d3/testr_results.html \n\nFor extra security, in my test case, I locate \u0027Queried the metadata service over IPv6\u0027 in the VM\u0027s console output and then I search for \u0027latest\u0027 100 characters before and all the way to the end of the string, as you can see in line 215.\n\nThe problem with the ML2/OVS and ML2/LB backends is different. In those cases, we can see curl timing out: https://paste.openstack.org/show/b6favCvPfIn9yT0kV2Ju/.","commit_id":"d16332aa4b977e8df026f8908f634f8ebe1bcd48"}]}