)]}'
{".zuul.yaml":[{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"22561927d0bcb0c1fe4c3d4f32dbf2635522555c","unresolved":false,"context_lines":[{"line_number":12,"context_line":"    check:"},{"line_number":13,"context_line":"      jobs:"},{"line_number":14,"context_line":"        - openstack-tox-functional"},{"line_number":15,"context_line":"        - openstack-tox-functional-py36"},{"line_number":16,"context_line":"        - placement-nova-tox-functional-py36"},{"line_number":17,"context_line":"        - placement-nested-perfload:"},{"line_number":18,"context_line":"            voting: false"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7faddb67_41760aa7","line":15,"updated":"2019-07-08 08:30:47.000000000","message":"If we\u0027re adding py37 in the unit tests, we should probably add them in functional too, since most placement tests are under functional.","commit_id":"9cce08ddd3c07d09205f09a3c3287e64200ded45"},{"author":{"_account_id":11805,"name":"Corey Bryant","email":"corey.bryant@canonical.com","username":"coreycb"},"change_message_id":"cdd64320996add80c8e8bbb80641ac78989df538","unresolved":false,"context_lines":[{"line_number":12,"context_line":"    check:"},{"line_number":13,"context_line":"      jobs:"},{"line_number":14,"context_line":"        - openstack-tox-functional"},{"line_number":15,"context_line":"        - openstack-tox-functional-py36"},{"line_number":16,"context_line":"        - placement-nova-tox-functional-py36"},{"line_number":17,"context_line":"        - placement-nested-perfload:"},{"line_number":18,"context_line":"            voting: false"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7faddb67_b5f6158f","line":15,"in_reply_to":"7faddb67_41760aa7","updated":"2019-07-08 15:46:04.000000000","message":"Please feel free to add functional tests, that would be great. I don\u0027t currently have the cycles and this TC goal is focused on enabling unit tests for train. I would love to see py37 functional tests enabled across the board though.","commit_id":"9cce08ddd3c07d09205f09a3c3287e64200ded45"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"1e99437bcaa52a17503d8eb5c07dc3cd766c987c","unresolved":false,"context_lines":[{"line_number":12,"context_line":"    check:"},{"line_number":13,"context_line":"      jobs:"},{"line_number":14,"context_line":"        - openstack-tox-functional"},{"line_number":15,"context_line":"        - openstack-tox-functional-py37"},{"line_number":16,"context_line":"        - placement-nova-tox-functional-py37"},{"line_number":17,"context_line":"        - placement-nested-perfload:"},{"line_number":18,"context_line":"            voting: false"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7faddb67_56146ea0","line":15,"updated":"2019-07-09 22:25:40.000000000","message":"@Chris, is this what you had in mind, or did you want to have *both* py36 and py37?","commit_id":"c3e7f089c29c485ec9af82cd6cb899b62fa13e1d"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"551d38c212a88ae6bb05ba4eddb914deabf3f30c","unresolved":false,"context_lines":[{"line_number":12,"context_line":"    check:"},{"line_number":13,"context_line":"      jobs:"},{"line_number":14,"context_line":"        - openstack-tox-functional"},{"line_number":15,"context_line":"        - openstack-tox-functional-py37"},{"line_number":16,"context_line":"        - placement-nova-tox-functional-py37"},{"line_number":17,"context_line":"        - placement-nested-perfload:"},{"line_number":18,"context_line":"            voting: false"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7faddb67_5c49ec88","line":15,"in_reply_to":"7faddb67_56146ea0","updated":"2019-07-10 07:36:27.000000000","message":"If it were entirely up to me, it would be just py37\n\nBut the TC and others have made a compelling case for why it should be both that comes down to \"the defaults with this tested distros\".\n\nSo both on openstack-tox-functional* as that keeps it aligned with the goals of the unit test template.\n\nI don\u0027t, however, think we need both on nova, and would prefer py37 so we see future problems (if any) early.","commit_id":"c3e7f089c29c485ec9af82cd6cb899b62fa13e1d"}],"tox.ini":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"30ac8d62de282bc5804b0bc164fe10ef4045511f","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[tox]"},{"line_number":2,"context_line":"minversion \u003d 3.1.1"},{"line_number":3,"context_line":"envlist \u003d py27,py37,functional,functional-py36,pep8"},{"line_number":4,"context_line":"skipsdist \u003d True"},{"line_number":5,"context_line":"# Automatic envs (pyXX) will use the python version appropriate to that"},{"line_number":6,"context_line":"# env and ignore basepython inherited from [testenv]. That\u0027s what we"}],"source_content_type":"text/x-properties","patch_set":1,"id":"7faddb67_9e4543b6","line":3,"updated":"2019-07-08 08:03:35.000000000","message":"I think it would be nicer just to add py37 but not remove py36","commit_id":"9cce08ddd3c07d09205f09a3c3287e64200ded45"},{"author":{"_account_id":11805,"name":"Corey Bryant","email":"corey.bryant@canonical.com","username":"coreycb"},"change_message_id":"cdd64320996add80c8e8bbb80641ac78989df538","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[tox]"},{"line_number":2,"context_line":"minversion \u003d 3.1.1"},{"line_number":3,"context_line":"envlist \u003d py27,py37,functional,functional-py36,pep8"},{"line_number":4,"context_line":"skipsdist \u003d True"},{"line_number":5,"context_line":"# Automatic envs (pyXX) will use the python version appropriate to that"},{"line_number":6,"context_line":"# env and ignore basepython inherited from [testenv]. That\u0027s what we"}],"source_content_type":"text/x-properties","patch_set":1,"id":"7faddb67_35e225d7","line":3,"in_reply_to":"7faddb67_0157b204","updated":"2019-07-08 15:46:04.000000000","message":"Correct this is only the default for \u0027tox\u0027. Without py36 in this list users can always run \u0027tox -e py36. It should save developer\u0027s cycles when running default tox is the idea.","commit_id":"9cce08ddd3c07d09205f09a3c3287e64200ded45"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"22561927d0bcb0c1fe4c3d4f32dbf2635522555c","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[tox]"},{"line_number":2,"context_line":"minversion \u003d 3.1.1"},{"line_number":3,"context_line":"envlist \u003d py27,py37,functional,functional-py36,pep8"},{"line_number":4,"context_line":"skipsdist \u003d True"},{"line_number":5,"context_line":"# Automatic envs (pyXX) will use the python version appropriate to that"},{"line_number":6,"context_line":"# env and ignore basepython inherited from [testenv]. That\u0027s what we"}],"source_content_type":"text/x-properties","patch_set":1,"id":"7faddb67_0157b204","line":3,"in_reply_to":"7faddb67_9e4543b6","updated":"2019-07-08 08:30:47.000000000","message":"As I understand things, this list is only used by development when they use \u0027tox\u0027 without any args (I generally choose specific jobs myself). If that\u0027s the case, then I think the list here should be s/36/37/ so both use and functional are running python37.","commit_id":"9cce08ddd3c07d09205f09a3c3287e64200ded45"}]}