)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":14394,"name":"Dale Smith","email":"dale@catalystcloud.nz","username":"dalees"},"change_message_id":"6487b582c08f64549fc74bf708e0141908459a05","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"276260f8_c22d3d4e","updated":"2025-03-26 10:47:26.000000000","message":"I support this change, but have a couple of questions before merging.","commit_id":"c54597a56d700b3fee6a99af97f5d6937b61fa84"},{"author":{"_account_id":8064,"name":"Jake Yip","email":"jake.yip@ardc.edu.au","username":"jake"},"change_message_id":"884bdce284977bad9ed45aa9c9fed9c7d932ebb6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"209a854b_142da83f","updated":"2025-03-27 10:53:55.000000000","message":"sorry I must have commented on the wrong PS","commit_id":"aa06aafa8b725218b086106091f46d817c1d1ba7"}],"tox.ini":[{"author":{"_account_id":14394,"name":"Dale Smith","email":"dale@catalystcloud.nz","username":"dalees"},"change_message_id":"6487b582c08f64549fc74bf708e0141908459a05","unresolved":true,"context_lines":[{"line_number":12,"context_line":"       -r{toxinidir}/test-requirements.txt"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"[doc8]"},{"line_number":15,"context_line":"ignore-path\u003dspecs/pre-ocata,specs/pike,specs/queens,specs/ussuri,specs/bobcat,specs/caracal"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"[testenv:venv]"},{"line_number":18,"context_line":"commands \u003d {posargs}"}],"source_content_type":"text/x-properties","patch_set":3,"id":"454f8cfc_0f96d797","line":15,"updated":"2025-03-26 10:47:26.000000000","message":"did these fail doc8? these all seem to pass in my local tests but perhaps I\u0027m not seeing the same errors.\n\nIf you intended to ignore all the old specs regardless `ocata` and `stein` are missing from this list.","commit_id":"c54597a56d700b3fee6a99af97f5d6937b61fa84"},{"author":{"_account_id":14394,"name":"Dale Smith","email":"dale@catalystcloud.nz","username":"dalees"},"change_message_id":"0714410e6f970787f13a1419c597afb2dd806848","unresolved":false,"context_lines":[{"line_number":12,"context_line":"       -r{toxinidir}/test-requirements.txt"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"[doc8]"},{"line_number":15,"context_line":"ignore-path\u003dspecs/pre-ocata,specs/pike,specs/queens,specs/ussuri,specs/bobcat,specs/caracal"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"[testenv:venv]"},{"line_number":18,"context_line":"commands \u003d {posargs}"}],"source_content_type":"text/x-properties","patch_set":3,"id":"f7b7d138_b88da16d","line":15,"in_reply_to":"130821e4_2e0219a8","updated":"2025-03-26 19:24:45.000000000","message":"Jake, see https://review.opendev.org/c/openstack/magnum-specs/+/945007/comment/64fca834_db3c2ea6/\n\nThe new patchset does not test this path, so I think we\u0027re good here?","commit_id":"c54597a56d700b3fee6a99af97f5d6937b61fa84"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"78a9fb0df9ee643fc57dde5a53cc71f9aaf9a855","unresolved":true,"context_lines":[{"line_number":12,"context_line":"       -r{toxinidir}/test-requirements.txt"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"[doc8]"},{"line_number":15,"context_line":"ignore-path\u003dspecs/pre-ocata,specs/pike,specs/queens,specs/ussuri,specs/bobcat,specs/caracal"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"[testenv:venv]"},{"line_number":18,"context_line":"commands \u003d {posargs}"}],"source_content_type":"text/x-properties","patch_set":3,"id":"965d4c30_85fd6628","line":15,"in_reply_to":"454f8cfc_0f96d797","updated":"2025-03-26 10:50:42.000000000","message":"Those did fail on previous CI runs I think","commit_id":"c54597a56d700b3fee6a99af97f5d6937b61fa84"},{"author":{"_account_id":8064,"name":"Jake Yip","email":"jake.yip@ardc.edu.au","username":"jake"},"change_message_id":"e4a8849f371398ac4145a2723e2cfe8cf9bb98df","unresolved":true,"context_lines":[{"line_number":12,"context_line":"       -r{toxinidir}/test-requirements.txt"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"[doc8]"},{"line_number":15,"context_line":"ignore-path\u003dspecs/pre-ocata,specs/pike,specs/queens,specs/ussuri,specs/bobcat,specs/caracal"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"[testenv:venv]"},{"line_number":18,"context_line":"commands \u003d {posargs}"}],"source_content_type":"text/x-properties","patch_set":3,"id":"130821e4_2e0219a8","line":15,"in_reply_to":"965d4c30_85fd6628","updated":"2025-03-26 13:35:09.000000000","message":"I get an error on a second run unless I add `doc/build` to this list.","commit_id":"c54597a56d700b3fee6a99af97f5d6937b61fa84"},{"author":{"_account_id":14394,"name":"Dale Smith","email":"dale@catalystcloud.nz","username":"dalees"},"change_message_id":"6487b582c08f64549fc74bf708e0141908459a05","unresolved":true,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"[testenv:docs]"},{"line_number":21,"context_line":"commands \u003d"},{"line_number":22,"context_line":"  doc8 doc specs"},{"line_number":23,"context_line":"  sphinx-build -W -b html doc/source doc/build/html"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"[testenv:spelling]"}],"source_content_type":"text/x-properties","patch_set":3,"id":"64fca834_db3c2ea6","line":22,"updated":"2025-03-26 10:47:26.000000000","message":"if `doc/build/html` is built, this throws a lot of errors. Perhaps change to `doc8 doc/source/ specs/`?\n\nThis will help pass a local run of `tox`.","commit_id":"c54597a56d700b3fee6a99af97f5d6937b61fa84"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"48855908786212229b4d30da8ae8c9f97c26acdc","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"[testenv:docs]"},{"line_number":21,"context_line":"commands \u003d"},{"line_number":22,"context_line":"  doc8 doc specs"},{"line_number":23,"context_line":"  sphinx-build -W -b html doc/source doc/build/html"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"[testenv:spelling]"}],"source_content_type":"text/x-properties","patch_set":3,"id":"e7045c3c_3a6d085d","line":22,"in_reply_to":"64fca834_db3c2ea6","updated":"2025-03-26 10:51:45.000000000","message":"Done","commit_id":"c54597a56d700b3fee6a99af97f5d6937b61fa84"}]}
