)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":35896,"name":"Bruno Drugowick Muniz","display_name":"Bruno","email":"bruno.muniz@encora.com","username":"brunomuniz"},"change_message_id":"ccc3d2291adf90e7a9e94bf8e4f1a30f4562920f","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c7a5d630_51606356","updated":"2023-08-08 22:14:28.000000000","message":"@Daniel, can you kindly test this? I know you were having a similar problem.\n\nThis should work now if you want to use a virtual environment. Here\u0027s an example of how to configure the virtual env and run tox with this commit:\n\n```\nrm -rf venv\npython3 -m venv venv\nsource venv/bin/activate\npip install -r doc/requirements.txt\ntox\n```","commit_id":"824851a6fdc926600bae1cdfde2dcaa467eec96d"},{"author":{"_account_id":30539,"name":"Ron Stone","email":"ronald.stone@windriver.com","username":"ronstone2000"},"change_message_id":"c554a99f1379ed1c18e9639d0e9b33d8da93cc93","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4e4c567a_f0bf18fc","updated":"2023-08-10 10:50:43.000000000","message":"An alternative approach:\n\nThe core docs team runs a custom docker to avoid these sorts of conflicts. With a bit of work I could share the files someplace. It\u0027s been useful for avoiding dependency errors. We\u0027ve also been testing major upgrades of both tox and sphinx - they are both really old. We would like to do this in the near term and I\u0027d like to do them together.","commit_id":"824851a6fdc926600bae1cdfde2dcaa467eec96d"},{"author":{"_account_id":35896,"name":"Bruno Drugowick Muniz","display_name":"Bruno","email":"bruno.muniz@encora.com","username":"brunomuniz"},"change_message_id":"681d1930a2a4c3c6ba910a3d2b0436d5a8c48ef9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"cbaaa45b_b468465f","updated":"2023-08-09 14:12:32.000000000","message":"Great! Since Daniel\u0027s setup also works now, I\u0027m adding the Docs team.","commit_id":"824851a6fdc926600bae1cdfde2dcaa467eec96d"},{"author":{"_account_id":35896,"name":"Bruno Drugowick Muniz","display_name":"Bruno","email":"bruno.muniz@encora.com","username":"brunomuniz"},"change_message_id":"587449cb06e10127008f49eb396410642bc373ed","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"014a33d7_607bb346","in_reply_to":"4e4c567a_f0bf18fc","updated":"2023-08-10 18:10:06.000000000","message":"Interesting. Thanks for the updates. Indeed that would be yet another approach. I believe some opinionated directions in the README.rst (with the dockerized approach, for example) would help casual contributors.","commit_id":"824851a6fdc926600bae1cdfde2dcaa467eec96d"},{"author":{"_account_id":35896,"name":"Bruno Drugowick Muniz","display_name":"Bruno","email":"bruno.muniz@encora.com","username":"brunomuniz"},"change_message_id":"8bdb690e3138f5e9373aadb51c33c90a12a7ef5f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4654107f_f9f3c3b2","in_reply_to":"80b5aa7b_1b40d328","updated":"2023-08-09 14:12:48.000000000","message":"Done","commit_id":"824851a6fdc926600bae1cdfde2dcaa467eec96d"},{"author":{"_account_id":35920,"name":"dcaires","display_name":"Daniel Caires","email":"daniel.caires@encora.com","username":"Daniel.Caires"},"change_message_id":"3613001cabf3f6ede6a3ca68b0307a89c69cae62","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"80b5aa7b_1b40d328","in_reply_to":"c7a5d630_51606356","updated":"2023-08-09 11:25:41.000000000","message":"Great! I just tested it and it works perfectly","commit_id":"824851a6fdc926600bae1cdfde2dcaa467eec96d"},{"author":{"_account_id":35896,"name":"Bruno Drugowick Muniz","display_name":"Bruno","email":"bruno.muniz@encora.com","username":"brunomuniz"},"change_message_id":"7408acf816bd4ccbdb1c7c1295bd57b0674a5644","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c421b1e2_46e6fe0a","updated":"2023-09-01 17:32:02.000000000","message":"Well, there\u0027s an open discussion on this Review with my proposal, which fixes errors when dealing with `tox` locally. It also fixed for others. \n\nI ask the core reviewers: please, feel free to abandon this change if it won\u0027t get merged for any reason.\n\nThanks!","commit_id":"129b7c74056509ba52696a292e10cc14f318b38b"}],"doc/requirements.txt":[{"author":{"_account_id":30539,"name":"Ron Stone","email":"ronald.stone@windriver.com","username":"ronstone2000"},"change_message_id":"c554a99f1379ed1c18e9639d0e9b33d8da93cc93","unresolved":true,"context_lines":[{"line_number":2,"context_line":"openstackdocstheme\u003e\u003d2.2.1,\u003c\u003d2.3.1 # Apache-2.0"},{"line_number":3,"context_line":"docutils\u003d\u003d0.17.1"},{"line_number":4,"context_line":"PyYAML\u003d\u003d6.0"},{"line_number":5,"context_line":"tox\u003d\u003d2.9.1"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"# API Reference Guide"},{"line_number":8,"context_line":"os-api-ref\u003e\u003d1.5.0 # Apache-2.0"}],"source_content_type":"text/plain","patch_set":1,"id":"3324f963_e0d857a9","line":5,"updated":"2023-08-10 10:50:43.000000000","message":"minver is set at the top of tox.ini. It was pinned several months ago when an upgrade to tox used by zuul broke builds on some old syntax. The breakage happens in much more recent versions that 2.9.1, so this bump should be OK, but I think it belongs there.","commit_id":"824851a6fdc926600bae1cdfde2dcaa467eec96d"},{"author":{"_account_id":35896,"name":"Bruno Drugowick Muniz","display_name":"Bruno","email":"bruno.muniz@encora.com","username":"brunomuniz"},"change_message_id":"587449cb06e10127008f49eb396410642bc373ed","unresolved":true,"context_lines":[{"line_number":2,"context_line":"openstackdocstheme\u003e\u003d2.2.1,\u003c\u003d2.3.1 # Apache-2.0"},{"line_number":3,"context_line":"docutils\u003d\u003d0.17.1"},{"line_number":4,"context_line":"PyYAML\u003d\u003d6.0"},{"line_number":5,"context_line":"tox\u003d\u003d2.9.1"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"# API Reference Guide"},{"line_number":8,"context_line":"os-api-ref\u003e\u003d1.5.0 # Apache-2.0"}],"source_content_type":"text/plain","patch_set":1,"id":"4fa14b4b_250a044b","line":5,"in_reply_to":"3324f963_e0d857a9","updated":"2023-08-10 18:10:06.000000000","message":"Got it. Then I\u0027d argue that it needs, on top of a `minversion`, something like `requires \u003d tox \u003c\u003d 3.23.0`, because the problem is that with newer versions of tox (`4.x` it appears), it won\u0027t work. Then we get a proper error message saying that the required version is not present, like in the example below where required is `\u003c\u003d 3.23.0` and I have `4.7.0`:\n\n```shell\ndocs ((HEAD detached at FETCH_HEAD))\n \u003e\u003e python3 -m venv venv\ndocs ((HEAD detached at FETCH_HEAD))\n \u003e\u003e source venv/bin/activate\n(venv) docs ((HEAD detached at FETCH_HEAD))\n \u003e\u003e pip install -r doc/requirements.txt\n\n... long text removed for brevity ...\n\n(venv) docs ((HEAD detached at FETCH_HEAD))\n \u003e\u003e tox\nROOT: will run in automatically provisioned tox, host /usr/bin/python3 is missing [requires (has)]: tox\u003c\u003d3.23.0 (4.7.0)\n```","commit_id":"824851a6fdc926600bae1cdfde2dcaa467eec96d"}]}