)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"b1b925ecd33447f2215edd403dfacfe999f573fa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"60616ab7_57ce85e0","updated":"2024-04-30 12:02:09.000000000","message":"I\u0027m not against this, but see comment inline.","commit_id":"4caf01352888a22ea15546ec54a116e1ed309830"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"15437589ddebd5f1c6fac85aa2d4e886470b4a80","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"95bac48f_0eed8f0e","updated":"2023-08-01 14:06:29.000000000","message":"Thanks for your patch!\n\nSomething similar was abandoned in the past (https://review.opendev.org/c/openstack/glance/+/787006). I honestly don\u0027t have a strong opinion on this, since I always pass my env manually to tox using -e. Let\u0027s see what the others think.","commit_id":"4caf01352888a22ea15546ec54a116e1ed309830"}],"tox.ini":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"b1b925ecd33447f2215edd403dfacfe999f573fa","unresolved":true,"context_lines":[{"line_number":1,"context_line":"[tox]"},{"line_number":2,"context_line":"minversion \u003d 3.18.0"},{"line_number":3,"context_line":"# python runtimes: https://governance.openstack.org/tc/reference/runtimes/ussuri.html"},{"line_number":4,"context_line":"envlist \u003d functional-py39,py3,pep8"},{"line_number":5,"context_line":"skip_missing_interpreters \u003d true"},{"line_number":6,"context_line":"# this allows tox to infer the base python from the environment name"},{"line_number":7,"context_line":"# and override any basepython configured in this file"}],"source_content_type":"text/x-properties","patch_set":1,"id":"856a54d0_54f15292","line":4,"range":{"start_line":4,"start_character":10,"end_line":4,"end_character":25},"updated":"2024-04-30 12:02:09.000000000","message":"so this is still requiring that a py39 interpreter be available.  Probably you should also change this one to \u0027functional-py3\u0027 and make a change at line 45 to define the testenv.","commit_id":"4caf01352888a22ea15546ec54a116e1ed309830"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"b1b925ecd33447f2215edd403dfacfe999f573fa","unresolved":true,"context_lines":[{"line_number":42,"context_line":"  TEST_PATH \u003d ./glance/tests/functional"},{"line_number":43,"context_line":"commands \u003d stestr run {posargs}"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"[testenv:functional-py{38,39,310,311}]"},{"line_number":46,"context_line":"setenv \u003d {[testenv:functional]setenv}"},{"line_number":47,"context_line":"commands \u003d {[testenv:functional]commands}"},{"line_number":48,"context_line":""}],"source_content_type":"text/x-properties","patch_set":1,"id":"2059b366_b230ce6b","line":45,"range":{"start_line":45,"start_character":20,"end_line":45,"end_character":37},"updated":"2024-04-30 12:02:09.000000000","message":"if you take my suggestion above, change this to:\n\n[testenv:functional-py{3,38,39,310,311}]","commit_id":"4caf01352888a22ea15546ec54a116e1ed309830"}]}