)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5572,"name":"Don Kehn","display_name":"DEKehn","email":"dekehn@gmail.com","username":"dekehn"},"change_message_id":"a22271751dd4ccac50ae95c856b6bd2bc240a555","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a6672fc9_1ab92517","updated":"2022-10-13 15:16:26.000000000","message":"Doesn\u0027t work as described.\n\n\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\nll recreate: /opt/stack/tempest/.tox/tempest\n[1950044] /opt/stack/tempest/.tox$ /usr/bin/python3.8 -m virtualenv --no-download --python /usr/bin/python3.8 tempest \u003etempest/log/all-0.log\nall installdeps: -chttps://releases.openstack.org/constraints/upper/master, -r/opt/stack/tempest/requirements.txt\n[1950052] /opt/stack/tempest$ /opt/stack/tempest/.tox/tempest/bin/pip install -chttps://releases.openstack.org/constraints/upper/master -r/opt/stack/tempest/requirements.txt \u003e.tox/tempest/log/all-1.log\nall develop-inst: /opt/stack/tempest\nwrite config to /opt/stack/tempest/.tox/tempest/.tox-config1 as \u0027c78dac41792167274ef40c7bc3966e82651aa36138450f6e5f8afd783dd38a81 /usr/bin/python3.8\\n3.24.5 0 1 0\\n00000000000000000000000000000000 -chttps://releases.openstack.org/constraints/upper/master\\n00000000000000000000000000000000 -r/opt/stack/tempest/requirements.txt\u0027\n[1950062] /opt/stack/tempest$ /opt/stack/tempest/.tox/tempest/bin/pip install --exists-action w -e . \u003e.tox/tempest/log/all-2.log\n[1950135] /opt/stack/tempest$ /opt/stack/tempest/.tox/tempest/bin/python -m pip freeze \u003e.tox/tempest/log/all-3.log\nall installed: attrs\u003d\u003d22.1.0,autopage\u003d\u003d0.5.1,bcrypt\u003d\u003d4.0.0,certifi\u003d\u003d2022.9.24,cffi\u003d\u003d1.15.1,charset-normalizer\u003d\u003d2.1.1,cliff\u003d\u003d4.0.0,cmd2\u003d\u003d2.4.2,cryptography\u003d\u003d36.0.2,debtcollector\u003d\u003d2.5.0,defusedxml\u003d\u003d0.7.1,extras\u003d\u003d1.0.0,fasteners\u003d\u003d0.17.3,fixtures\u003d\u003d4.0.1,future\u003d\u003d0.18.2,idna\u003d\u003d3.3,importlib-metadata\u003d\u003d4.12.0,importlib-resources\u003d\u003d5.9.0,iso8601\u003d\u003d1.0.2,jsonschema\u003d\u003d4.14.0,msgpack\u003d\u003d1.0.4,netaddr\u003d\u003d0.8.0,netifaces\u003d\u003d0.11.0,oslo.concurrency\u003d\u003d5.0.1,oslo.config\u003d\u003d9.0.0,oslo.context\u003d\u003d5.0.0,oslo.i18n\u003d\u003d5.1.0,oslo.log\u003d\u003d5.0.0,oslo.serialization\u003d\u003d5.0.0,oslo.utils\u003d\u003d6.0.1,packaging\u003d\u003d21.3,paramiko\u003d\u003d2.11.0,pbr\u003d\u003d5.10.0,pkgutil_resolve_name\u003d\u003d1.3.10,prettytable\u003d\u003d3.4.1,pycparser\u003d\u003d2.21,pyinotify\u003d\u003d0.9.6,PyNaCl\u003d\u003d1.5.0,pyparsing\u003d\u003d3.0.9,pyperclip\u003d\u003d1.8.2,pyrsistent\u003d\u003d0.18.1,python-dateutil\u003d\u003d2.8.2,python-subunit\u003d\u003d1.4.0,pytz\u003d\u003d2022.2.1,PyYAML\u003d\u003d6.0,requests\u003d\u003d2.28.1,rfc3986\u003d\u003d1.5.0,six\u003d\u003d1.16.0,stestr\u003d\u003d3.2.1,stevedore\u003d\u003d4.0.0,-e git+https://opendev.org/openstack/tempest.git@ed74be1b0649729c01a70230e32f28872d16856c#egg\u003dtempest,testtools\u003d\u003d2.5.0,urllib3\u003d\u003d1.26.12,voluptuous\u003d\u003d0.13.1,wcwidth\u003d\u003d0.2.5,wrapt\u003d\u003d1.14.1,zipp\u003d\u003d3.8.1\nall run-test-pre: PYTHONHASHSEED\u003d\u00271563534724\u0027\nall run-test: commands[0] | find . -type f -name \u0027*.pyc\u0027 -delete\n[1950141] /opt/stack/tempest$ /usr/bin/find . -type f -name \u0027*.pyc\u0027 -delete\nall run-test: commands[1] | tempest run --regex designate\n[1950142] /opt/stack/tempest$ /opt/stack/tempest/.tox/tempest/bin/tempest run --regex designate\nThe specified regex doesn\u0027t match with anythingERROR: InvocationError for command /opt/stack/tempest/.tox/tempest/bin/tempest run --regex designate (exited with code 1)\n______________________________________________________________________ summary _______________________________________________________________________\nERROR:   all: commands failed\n\n\nAll repos designate, designate-tempest-plugins, and tempest are at master\n","commit_id":"e56fb0eb2ef967098630a6d18944bfe2abc412dc"},{"author":{"_account_id":5572,"name":"Don Kehn","display_name":"DEKehn","email":"dekehn@gmail.com","username":"dekehn"},"change_message_id":"43878650dc05012f524101a18934558e25d79cd9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"8523dcc0_6dfe1ef0","updated":"2022-10-13 15:17:56.000000000","message":"Doesn\u0027t work for me\n\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\nll recreate: /opt/stack/tempest/.tox/tempest\n[1950044] /opt/stack/tempest/.tox$ /usr/bin/python3.8 -m virtualenv --no-download --python /usr/bin/python3.8 tempest \u003etempest/log/all-0.log\nall installdeps: -chttps://releases.openstack.org/constraints/upper/master, -r/opt/stack/tempest/requirements.txt\n[1950052] /opt/stack/tempest$ /opt/stack/tempest/.tox/tempest/bin/pip install -chttps://releases.openstack.org/constraints/upper/master -r/opt/stack/tempest/requirements.txt \u003e.tox/tempest/log/all-1.log\nall develop-inst: /opt/stack/tempest\nwrite config to /opt/stack/tempest/.tox/tempest/.tox-config1 as \u0027c78dac41792167274ef40c7bc3966e82651aa36138450f6e5f8afd783dd38a81 /usr/bin/python3.8\\n3.24.5 0 1 0\\n00000000000000000000000000000000 -chttps://releases.openstack.org/constraints/upper/master\\n00000000000000000000000000000000 -r/opt/stack/tempest/requirements.txt\u0027\n[1950062] /opt/stack/tempest$ /opt/stack/tempest/.tox/tempest/bin/pip install --exists-action w -e . \u003e.tox/tempest/log/all-2.log\n[1950135] /opt/stack/tempest$ /opt/stack/tempest/.tox/tempest/bin/python -m pip freeze \u003e.tox/tempest/log/all-3.log\nall installed: attrs\u003d\u003d22.1.0,autopage\u003d\u003d0.5.1,bcrypt\u003d\u003d4.0.0,certifi\u003d\u003d2022.9.24,cffi\u003d\u003d1.15.1,charset-normalizer\u003d\u003d2.1.1,cliff\u003d\u003d4.0.0,cmd2\u003d\u003d2.4.2,cryptography\u003d\u003d36.0.2,debtcollector\u003d\u003d2.5.0,defusedxml\u003d\u003d0.7.1,extras\u003d\u003d1.0.0,fasteners\u003d\u003d0.17.3,fixtures\u003d\u003d4.0.1,future\u003d\u003d0.18.2,idna\u003d\u003d3.3,importlib-metadata\u003d\u003d4.12.0,importlib-resources\u003d\u003d5.9.0,iso8601\u003d\u003d1.0.2,jsonschema\u003d\u003d4.14.0,msgpack\u003d\u003d1.0.4,netaddr\u003d\u003d0.8.0,netifaces\u003d\u003d0.11.0,oslo.concurrency\u003d\u003d5.0.1,oslo.config\u003d\u003d9.0.0,oslo.context\u003d\u003d5.0.0,oslo.i18n\u003d\u003d5.1.0,oslo.log\u003d\u003d5.0.0,oslo.serialization\u003d\u003d5.0.0,oslo.utils\u003d\u003d6.0.1,packaging\u003d\u003d21.3,paramiko\u003d\u003d2.11.0,pbr\u003d\u003d5.10.0,pkgutil_resolve_name\u003d\u003d1.3.10,prettytable\u003d\u003d3.4.1,pycparser\u003d\u003d2.21,pyinotify\u003d\u003d0.9.6,PyNaCl\u003d\u003d1.5.0,pyparsing\u003d\u003d3.0.9,pyperclip\u003d\u003d1.8.2,pyrsistent\u003d\u003d0.18.1,python-dateutil\u003d\u003d2.8.2,python-subunit\u003d\u003d1.4.0,pytz\u003d\u003d2022.2.1,PyYAML\u003d\u003d6.0,requests\u003d\u003d2.28.1,rfc3986\u003d\u003d1.5.0,six\u003d\u003d1.16.0,stestr\u003d\u003d3.2.1,stevedore\u003d\u003d4.0.0,-e git+https://opendev.org/openstack/tempest.git@ed74be1b0649729c01a70230e32f28872d16856c#egg\u003dtempest,testtools\u003d\u003d2.5.0,urllib3\u003d\u003d1.26.12,voluptuous\u003d\u003d0.13.1,wcwidth\u003d\u003d0.2.5,wrapt\u003d\u003d1.14.1,zipp\u003d\u003d3.8.1\nall run-test-pre: PYTHONHASHSEED\u003d\u00271563534724\u0027\nall run-test: commands[0] | find . -type f -name \u0027*.pyc\u0027 -delete\n[1950141] /opt/stack/tempest$ /usr/bin/find . -type f -name \u0027*.pyc\u0027 -delete\nall run-test: commands[1] | tempest run --regex designate\n[1950142] /opt/stack/tempest$ /opt/stack/tempest/.tox/tempest/bin/tempest run --regex designate\nThe specified regex doesn\u0027t match with anythingERROR: InvocationError for command /opt/stack/tempest/.tox/tempest/bin/tempest run --regex designate (exited with code 1)\n______________________________________________________________________ summary _______________________________________________________________________\nERROR:   all: commands failed\n\n\nAll repos involved are at master - designate, desigante-tempest-plugin \u0026 tempest","commit_id":"e56fb0eb2ef967098630a6d18944bfe2abc412dc"},{"author":{"_account_id":30674,"name":"Lukas Piwowarski","email":"lpiwowar@redhat.com","username":"lukas-piwowarski"},"change_message_id":"0062b9bd8265a3a788869a0b783556461199e7b0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a7948ce7_d8085ffc","in_reply_to":"2f677e8f_5f7e0183","updated":"2022-10-13 15:42:16.000000000","message":"Fix: [path to designate-tempest-plugin folder]","commit_id":"e56fb0eb2ef967098630a6d18944bfe2abc412dc"},{"author":{"_account_id":30674,"name":"Lukas Piwowarski","email":"lpiwowar@redhat.com","username":"lukas-piwowarski"},"change_message_id":"9245b76afd7664335500d6c5c446127e97cca17c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2f677e8f_5f7e0183","in_reply_to":"8523dcc0_6dfe1ef0","updated":"2022-10-13 15:38:52.000000000","message":"Hi:), that\u0027s weird. Just to be sure, did you run this command before running \"tox -e all designate\"?\n\n```\ntox -e venv-tempest -- pip install [path to designate folder]\n```","commit_id":"e56fb0eb2ef967098630a6d18944bfe2abc412dc"},{"author":{"_account_id":5572,"name":"Don Kehn","display_name":"DEKehn","email":"dekehn@gmail.com","username":"dekehn"},"change_message_id":"05ae27b7eb916e3da169aa91e5ddc4c5701d8f0c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"bc4cc58f_eb099146","in_reply_to":"a7948ce7_d8085ffc","updated":"2022-10-27 20:15:47.000000000","message":"you are correct I didn\u0027t perform the extra step of setting the venv. I think there are a number of alternate ways to run tox in workspaces, venvs, etc. which I think make coding trickier, in this case I am a KISS approach until it stop working. If i recall the all-plugin vs all debate had to do with package (RPM,deb) vs pip3 installs?","commit_id":"e56fb0eb2ef967098630a6d18944bfe2abc412dc"},{"author":{"_account_id":30674,"name":"Lukas Piwowarski","email":"lpiwowar@redhat.com","username":"lukas-piwowarski"},"change_message_id":"4cfbf0c5b49265b3eaf1b24289ba92fc76420467","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a2d04b8e_80e89e65","in_reply_to":"bc4cc58f_eb099146","updated":"2022-10-31 11:01:03.000000000","message":"I get the KISS approach. I\u0027m not sure whether the conversation had to do something with (rpm,deb) vs pip3 installs [1]. But the thing is that the all-plugin environment will get removed sooner or later [2] (see gmann\u0027s comment) [3]. This patch is just making sure that things will not get broken once it will get removed.\n\nI see three solutions:\n\n1) I can change the patch so that we use all-site-plugins tox environment (instead of \u0027all\u0027 + \u0027venv-tempest\u0027). Although it seems to me that using the sitepackages\u003don is really not good practice as the tox environment is not properly isolated from the rest of the system. \n\n2) We keep the patch as it is.\n\n3) If you have some other recommendations I will follow it:). I just want to make sure that we remove the all-plugin reference. \n\n[1] https://review.opendev.org/c/openstack/tempest/+/543974\n[2] https://review.opendev.org/c/openstack/tempest/+/750375\n[3] https://opendev.org/openstack/tempest/src/commit/beda3306a25d4927cc0513d5c42c8a38b629fc9d/tox.ini#L85","commit_id":"e56fb0eb2ef967098630a6d18944bfe2abc412dc"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"d3e10619b905a8845a16253cc2555696fdbf61d6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"86959f5a_4b721e8a","updated":"2023-05-15 16:22:54.000000000","message":"Good catch, I missed this location in the other patch. Thanks!","commit_id":"d858e4e2776ee4cb6e26e19fc480f45e2d42232e"},{"author":{"_account_id":5572,"name":"Don Kehn","display_name":"DEKehn","email":"dekehn@gmail.com","username":"dekehn"},"change_message_id":"6b70e8c426643318b0c6b318802421f87dc2d89a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"3ff58e88_93c32658","updated":"2023-05-15 22:15:02.000000000","message":"LGTM - Thanks for staying with it.","commit_id":"d858e4e2776ee4cb6e26e19fc480f45e2d42232e"}]}