)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"e8c341a3b40802985ee16213d3add28f8b079061","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"630b9c37_eb299d03","updated":"2022-12-15 16:39:06.000000000","message":"And it seems lots of other namespace commands aren\u0027t working like ping, curl, neutron-keepalived-state-change\n\nIt\u0027s like this file isn\u0027t getting loaded, since afaik you just need a *Filter defined for the binary in order to be used in a namespace:\n\nneutron/tests/contrib/testing.filters","commit_id":"f323554782b629e0033028bb80707e33460a60ea"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"75b8263330a8bfe10ed1a7b3a12cbbfe5cb05dae","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"587e2d03_5dadfdcf","updated":"2022-12-15 15:50:35.000000000","message":"So the functional job failed with:\n\nUnauthorized command: ip netns exec test-72a6d3c5-60be-48a8-abe6-8d0e0db8de56 ping -W 1 -c 3 192.178.0.2 (no filter matched)\n\nBut this is in rootwrap.filters (or at least the default one):\n\n# EXECUTE COMMANDS IN A NAMESPACE\nip: IpFilter, ip, root\nip_exec: IpNetnsExecFilter, ip, root\n\nThat last line should allow it, right? Or does it only allow \u0027ip netns exec foo ip ...\u0027?","commit_id":"f323554782b629e0033028bb80707e33460a60ea"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"cca0ddc9305dcd55a2a4df1619f1f13e821b642f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"acbb4a8e_78b610c5","in_reply_to":"630b9c37_eb299d03","updated":"2022-12-15 18:03:03.000000000","message":"I think we are not installing the rootwrap filters (that should be when the package is installed)\n\nIn any case, I\u0027m following https://github.com/tox-dev/tox/issues/2730","commit_id":"f323554782b629e0033028bb80707e33460a60ea"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"fdf6966828dadf9e4c829a6a34cd827fea6adb32","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"8e1787a2_9f680ba9","updated":"2022-12-15 19:41:01.000000000","message":"So my other theory is OS_SUDO_TESTING isn\u0027t getting set to 1, so tools/deploy_rootwrap.sh isn\u0027t installing all the filters. I\u0027ll add a -x to it so it will at least print what it\u0027s doing.","commit_id":"41886feaa7fa8a162df21751db4f8fad737bf871"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"8b3ef4bfa67ea1e1f281be7c1f17679a9e66a4fb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"9636faf6_61897a4b","updated":"2022-12-15 22:17:03.000000000","message":"My hunch was wrong, OS_SUDO_TESTING\u003d1 in the script so it did get passed correctly from tox.ini 😞\n\nhttps://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_af4/867554/7/check/neutron-functional-with-uwsgi/af4701e/job-output.txt","commit_id":"f3e21528885adcf48603a6478f46261476d91bec"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"42862880b09ecc8f0a21221a894880335ea0e9fd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"01bf5bf0_214bf70f","updated":"2023-02-06 10:44:00.000000000","message":"Brian is right, I see only tox 3.x in the logs of the dependent patch, so something is not clear","commit_id":"a5988aca049d907a772fc4d7bae4f11b61ea837a"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"716ecd7eb0e7eb7056468aa29212c09e196100ca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"43ee804c_c58cedf1","updated":"2023-01-25 21:17:48.000000000","message":"Do we also have to undo the playbook changes made in https://review.opendev.org/c/openstack/neutron/+/867977 to run tox4? I think this is only using tox3","commit_id":"a5988aca049d907a772fc4d7bae4f11b61ea837a"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"8430b1d4e8e489733470c7ac7371ca39681a5b23","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"fc96bb77_1f3ed108","in_reply_to":"01bf5bf0_214bf70f","updated":"2023-02-06 11:29:52.000000000","message":"This is because [1] was merged before. I\u0027ve pushed a new version of [2]. Regardless of the test results, you\u0027ll see how \"deploy_rootwrap\" is called with relative paths, instead of absolute.\n\n[1]https://review.opendev.org/c/openstack/neutron/+/868518\n[2]https://review.opendev.org/c/openstack/neutron/+/868380","commit_id":"a5988aca049d907a772fc4d7bae4f11b61ea837a"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"5fe249bd62c7871a44af26858cc582d6dea4ef9e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"4853bb6a_d4f16baa","in_reply_to":"43ee804c_c58cedf1","updated":"2023-01-27 07:15:34.000000000","message":"+1","commit_id":"a5988aca049d907a772fc4d7bae4f11b61ea837a"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"cfffbde0af5e9757e9da7a9a72436f0eb7e7f608","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"bb4edc37_0cc99b84","in_reply_to":"4853bb6a_d4f16baa","updated":"2023-01-27 15:33:06.000000000","message":"No, we are still not going to move to tox4. This is a patch needed to start investigating the issue with fullstack and functional tests and tox4.","commit_id":"a5988aca049d907a772fc4d7bae4f11b61ea837a"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"327246f4cb1e5fb03d8021c1bcbfe548a2e88e17","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"a12c31ce_ea579f81","in_reply_to":"8172eea5_1e862734","updated":"2023-02-01 14:38:29.000000000","message":"I\u0027ll let you keep debugging, but the functional log for that change only shows tox3 so I must be missing something.\n\ncontroller | Collecting tox\u003c4\ncontroller |   Downloading https://mirror.iad3.inmotion.opendev.org/pypifiles/packages/31/c7/69ec9b8bbb4fe649b4bc960a53b837b4dace889137be5f23b4236a3e9f7c/tox-3.28.0-py2.py3-none-any.whl (86 kB)","commit_id":"a5988aca049d907a772fc4d7bae4f11b61ea837a"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"4c48ac465b3f0f8aec25429c580448e5f4b1d7ea","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"8172eea5_1e862734","in_reply_to":"9176aa00_70f7164c","updated":"2023-02-01 07:37:23.000000000","message":"We need this change for the tox4 testing that is being done in https://review.opendev.org/c/openstack/neutron/+/868380/4.","commit_id":"a5988aca049d907a772fc4d7bae4f11b61ea837a"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"e1d9c303c05f3e0e13d0ec5e7b9433ba004109ef","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"9176aa00_70f7164c","in_reply_to":"bb4edc37_0cc99b84","updated":"2023-01-31 22:09:55.000000000","message":"Right, but it seemed this was testing with tox3 still, so didn\u0027t think it would see any tox4 failures.","commit_id":"a5988aca049d907a772fc4d7bae4f11b61ea837a"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"66f36b280ed0ba219ad9d6fa94598b5ce168a0cc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"82ff2683_ce816761","in_reply_to":"fc96bb77_1f3ed108","updated":"2023-02-08 08:21:43.000000000","message":"ack, thanks","commit_id":"a5988aca049d907a772fc4d7bae4f11b61ea837a"}],"tox.ini":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"dc33b753523afe731522994e15a8bf6aab8d44b0","unresolved":true,"context_lines":[{"line_number":21,"context_line":"          no_proxy"},{"line_number":22,"context_line":"          NO_PROXY"},{"line_number":23,"context_line":"          TOX_ENV_SRC_MODULES"},{"line_number":24,"context_line":"use_develop \u003d True"},{"line_number":25,"context_line":"deps \u003d"},{"line_number":26,"context_line":"  -c{env:TOX_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master}"},{"line_number":27,"context_line":"  -r{toxinidir}/requirements.txt"}],"source_content_type":"text/x-properties","patch_set":1,"id":"620b5d6b_494cfa9e","line":24,"updated":"2022-12-14 01:33:06.000000000","message":"From what I\u0027ve seen it\u0027s \u0027usedevelop\u0027","commit_id":"d7bc45b37513cdd68a764fbc33d8fe274b1b6b42"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"ed8aac5de54978cfd1922ff759e7b94cc7c34855","unresolved":true,"context_lines":[{"line_number":21,"context_line":"          no_proxy"},{"line_number":22,"context_line":"          NO_PROXY"},{"line_number":23,"context_line":"          TOX_ENV_SRC_MODULES"},{"line_number":24,"context_line":"use_develop \u003d True"},{"line_number":25,"context_line":"deps \u003d"},{"line_number":26,"context_line":"  -c{env:TOX_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master}"},{"line_number":27,"context_line":"  -r{toxinidir}/requirements.txt"}],"source_content_type":"text/x-properties","patch_set":1,"id":"a699c815_30033160","line":24,"in_reply_to":"620b5d6b_494cfa9e","updated":"2022-12-14 08:36:56.000000000","message":"https://tox.wiki/en/latest/config.html#use_develop","commit_id":"d7bc45b37513cdd68a764fbc33d8fe274b1b6b42"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"6b456260427297d5e15c19a5b3a84ca3ef61ccf6","unresolved":true,"context_lines":[{"line_number":21,"context_line":"          no_proxy"},{"line_number":22,"context_line":"          NO_PROXY"},{"line_number":23,"context_line":"          TOX_ENV_SRC_MODULES"},{"line_number":24,"context_line":"use_develop \u003d True"},{"line_number":25,"context_line":"deps \u003d"},{"line_number":26,"context_line":"  -c{env:TOX_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master}"},{"line_number":27,"context_line":"  -r{toxinidir}/requirements.txt"}],"source_content_type":"text/x-properties","patch_set":1,"id":"e8e8bf1f_7e562b16","line":24,"in_reply_to":"a699c815_30033160","updated":"2022-12-14 13:30:44.000000000","message":"That has both side-by-side, so think they both do the same thing. Of course that doesn\u0027t mean they won\u0027t deprecate one :-/","commit_id":"d7bc45b37513cdd68a764fbc33d8fe274b1b6b42"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"6b456260427297d5e15c19a5b3a84ca3ef61ccf6","unresolved":true,"context_lines":[{"line_number":1,"context_line":"[tox]"},{"line_number":2,"context_line":"envlist \u003d docs,py38,pep8"},{"line_number":3,"context_line":"minversion \u003d 3.18.0"},{"line_number":4,"context_line":"skipsdist \u003d True"},{"line_number":5,"context_line":"ignore_basepython_conflict \u003d True"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[testenv]"}],"source_content_type":"text/x-properties","patch_set":4,"id":"24a95b3e_b92448e8","line":4,"updated":"2022-12-14 13:30:44.000000000","message":"Getting rid of this seems to do the same as -e, at least in the neutron-lib change I created.","commit_id":"3dec744f63e7d9d3fcebab03baff67c9080ba740"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"81d347a67a9032875fe90e2ea6000057a6550180","unresolved":true,"context_lines":[{"line_number":1,"context_line":"[tox]"},{"line_number":2,"context_line":"envlist \u003d docs,py38,pep8"},{"line_number":3,"context_line":"minversion \u003d 3.18.0"},{"line_number":4,"context_line":"skipsdist \u003d True"},{"line_number":5,"context_line":"ignore_basepython_conflict \u003d True"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[testenv]"}],"source_content_type":"text/x-properties","patch_set":4,"id":"265fdd81_a449996d","line":4,"in_reply_to":"24a95b3e_b92448e8","updated":"2022-12-15 08:48:03.000000000","message":"Actually I don\u0027t know why we are using it here [1]. This should be disabled by default in case of having a setup.py file to install the project.\n\n[1]https://stackoverflow.com/questions/18962403/how-do-i-run-tox-in-a-project-that-has-no-setup-py","commit_id":"3dec744f63e7d9d3fcebab03baff67c9080ba740"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"d9d406f12d6f7a86c9084ab178a981af2fc816d8","unresolved":true,"context_lines":[{"line_number":165,"context_line":"  -c{env:TOX_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master}"},{"line_number":166,"context_line":"  -r{toxinidir}/doc/requirements.txt"},{"line_number":167,"context_line":"  -r{toxinidir}/requirements.txt"},{"line_number":168,"context_line":"  -e ."},{"line_number":169,"context_line":"commands \u003d sphinx-build -W -b html doc/source doc/build/html"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":"[testenv:pdf-docs]"}],"source_content_type":"text/x-properties","patch_set":4,"id":"a25c7c3c_bba6632e","line":168,"updated":"2022-12-14 10:34:25.000000000","message":"In case that use_develop \u003d False and we use -e . Is that going to create issue?","commit_id":"3dec744f63e7d9d3fcebab03baff67c9080ba740"}]}