)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"3f08fc7ff4b4847114508dae6535fd6834d17a53","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1e9553c7_4f1a5cba","updated":"2025-04-09 00:54:10.000000000","message":"AFAICT this isn\u0027t working either:\n\n```\ntest/s3api/test_request_target_style.py::TestRequestTargetStyle::test_absolute_url SKIPPED [ 93%]\n```\n\nhttps://zuul.opendev.org/t/openstack/build/74d89d51887142bc8dcb75244230d334/console","commit_id":"405440a2ee1f4c8ddda1651c7aaf070232fecd7c"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"d74a9eb10232a226b8483606ee7c49b4588f8e75","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"fcac89e3_339b5f64","updated":"2025-04-09 09:00:20.000000000","message":"I\u0027d still also like https://review.opendev.org/c/openstack/swift/+/946283","commit_id":"405440a2ee1f4c8ddda1651c7aaf070232fecd7c"},{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"8ed18ad24fa2a4e5427308022692a2be649562c5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"28844a28_5681088e","updated":"2025-04-09 14:03:06.000000000","message":"yeah, I think `-ra` might have been a better default than `-rfE`\n\nhttps://docs.pytest.org/en/6.2.x/usage.html#detailed-summary-report\n\nTBH tho I find the difference between `-rx` and `-xr` a little confusing; I don\u0027t think I actually understand what `-r` on it\u0027s own is doing. \n\nI\u0027d be happy to see this merged in addition to 946283: Add skip reason to tox pytest output | https://review.opendev.org/c/openstack/swift/+/946283 as this might have been benefit of being more robust if we manage to forget to plumb a tox_environment in the .zuul.yaml for some new job that hopefully is still using tox w/ pytest posargs - you know, belts and braces - b/c not having the skip reason was sort of bollocks)","commit_id":"405440a2ee1f4c8ddda1651c7aaf070232fecd7c"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"8abb440d72060b2ab094b63db7b88f3b0bfa9e96","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"756896c7_dd216f56","in_reply_to":"1e9553c7_4f1a5cba","updated":"2025-04-09 02:44:44.000000000","message":"Don\u0027t look at [the test](https://zuul.opendev.org/t/openstack/build/74d89d51887142bc8dcb75244230d334/log/job-output.txt#2071), look at [the summary](https://zuul.opendev.org/t/openstack/build/74d89d51887142bc8dcb75244230d334/log/job-output.txt#2092).","commit_id":"405440a2ee1f4c8ddda1651c7aaf070232fecd7c"},{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"8ed18ad24fa2a4e5427308022692a2be649562c5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1edfcd7e_0a485940","in_reply_to":"756896c7_dd216f56","updated":"2025-04-09 14:03:06.000000000","message":"I think it would be really useful if the test *name* was included in the summary on SKIPPED (same as on `FAILED`) but I\u0027m sure any hint would have been better that truncating the skip reason.\n```\n2025-04-08 20:11:16.854597 | centos-9-stream | SKIPPED [1] test/s3api/test_request_target_style.py:40: Absolute URL test requires https\n```","commit_id":"405440a2ee1f4c8ddda1651c7aaf070232fecd7c"}],"tox.ini":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"d74a9eb10232a226b8483606ee7c49b4588f8e75","unresolved":true,"context_lines":[{"line_number":6,"context_line":"  virtualenv\u003c20.22"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"[pytest]"},{"line_number":9,"context_line":"addopts \u003d --verbose -p no:requests_mock -r a"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"[testenv]"},{"line_number":12,"context_line":"usedevelop \u003d True"}],"source_content_type":"text/x-properties","patch_set":1,"id":"016d9b88_cda37f6f","line":9,"updated":"2025-04-09 09:00:20.000000000","message":"OIC, so the `-r` adds summary at the finale rather than extra output per test","commit_id":"405440a2ee1f4c8ddda1651c7aaf070232fecd7c"},{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"8ed18ad24fa2a4e5427308022692a2be649562c5","unresolved":true,"context_lines":[{"line_number":6,"context_line":"  virtualenv\u003c20.22"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"[pytest]"},{"line_number":9,"context_line":"addopts \u003d --verbose -p no:requests_mock -r a"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"[testenv]"},{"line_number":12,"context_line":"usedevelop \u003d True"}],"source_content_type":"text/x-properties","patch_set":1,"id":"97148cc8_80e6fb30","line":9,"in_reply_to":"016d9b88_cda37f6f","updated":"2025-04-09 14:03:06.000000000","message":"and I guess somehow it applies to any job where we include say `{posargs:...}`\n\ne.g.\n\n```\ncommands \u003d\n  pytest {posargs:test/s3api}\n```\n\nhttps://tox.wiki/en/latest/config.html#positional-argument-reference\n\nwhich might be better than a new pytest.ini file, although I\u0027m not sure which is easier to grok...\n\nhttps://docs.pytest.org/en/stable/reference/customize.html#pytest-ini","commit_id":"405440a2ee1f4c8ddda1651c7aaf070232fecd7c"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"d23cf9a38f1e0ffd677d3a83183c4a34af835756","unresolved":true,"context_lines":[{"line_number":6,"context_line":"  virtualenv\u003c20.22"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"[pytest]"},{"line_number":9,"context_line":"addopts \u003d --verbose -p no:requests_mock -r a"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"[testenv]"},{"line_number":12,"context_line":"usedevelop \u003d True"}],"source_content_type":"text/x-properties","patch_set":1,"id":"6c7b1083_cfc96905","line":9,"in_reply_to":"97148cc8_80e6fb30","updated":"2025-04-09 17:54:38.000000000","message":"FWIW, I think there were two main reasons I used a `[pytest]` section within `tox.ini` [originally](https://github.com/openstack/swift/commit/ef155bd74aaf4ffce11956b2e5bd921dfb5d8809):\n\n1. That was where we had been configuring `nosetests` (even if it _was_ with env vars -- blech!)\n2. I felt like we have more than enough dot files and config files at the top level already\n\nThat\u0027s also my biggest complaint about working in `pyproject.toml` support -- I\u0027ve yet to see a proposal that would actually remove our need for `setup.cfg`, `requirements.txt`, or anything else -- it seems to always be additive and just makes the project/dev/testing environment metadata _more_ diffuse.","commit_id":"405440a2ee1f4c8ddda1651c7aaf070232fecd7c"}]}