)]}'
{"tox.ini":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"581b6650561ce8b9d4e83848b7f379d3adfc5aa3","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[tox]"},{"line_number":2,"context_line":"minversion \u003d 2.0"},{"line_number":3,"context_line":"envlist \u003d py3,py27,py37,pep8"},{"line_number":4,"context_line":"skipsdist \u003d True"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"[testenv]"}],"source_content_type":"text/x-properties","patch_set":1,"id":"9fb8cfa7_8a04be4b","line":3,"range":{"start_line":3,"start_character":10,"end_line":3,"end_character":13},"updated":"2019-06-25 11:25:17.000000000","message":"let\u0027s make this py36 explicitly","commit_id":"a932f1913bc1f7a352823dc850a6d1f0b902248d"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"bf06448910521edca44e8b2b644f9d8a5b0e736b","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[tox]"},{"line_number":2,"context_line":"minversion \u003d 2.0"},{"line_number":3,"context_line":"envlist \u003d py27,py36,py37,pep8"},{"line_number":4,"context_line":"skipsdist \u003d True"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"[testenv]"}],"source_content_type":"text/x-properties","patch_set":2,"id":"7faddb67_25f0c1c9","line":3,"updated":"2019-07-06 19:33:53.000000000","message":"Corey, there seems to be a major confusion here, and I\u0027m sad the projects blindly change this list. This list is NOT about \"testing\". This list is what developers will get when running \u0027tox\u0027 locally. This change will not make them magically use the right version, it will just make the tests fail in an ugly way. Things like that stop people from testing locally, relying on the CI instead.","commit_id":"4644abcc002800fde633055aa42d98b881266e5b"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"c5b8985c436709537d71e1e7bbca0e9cd484eb61","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[tox]"},{"line_number":2,"context_line":"minversion \u003d 2.0"},{"line_number":3,"context_line":"envlist \u003d py27,py36,py37,pep8"},{"line_number":4,"context_line":"skipsdist \u003d True"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"[testenv]"}],"source_content_type":"text/x-properties","patch_set":2,"id":"9fb8cfa7_488089c0","line":3,"range":{"start_line":3,"start_character":10,"end_line":3,"end_character":24},"updated":"2019-07-03 14:13:46.000000000","message":"Well, first we really don\u0027t want py27 to be first on the list. And removing py3 which will pickup any python3 version seems wrong. I think the right action is to apply the higher level train python3 template to the job which should force a test with python 3.7.","commit_id":"4644abcc002800fde633055aa42d98b881266e5b"},{"author":{"_account_id":11805,"name":"Corey Bryant","email":"corey.bryant@canonical.com","username":"coreycb"},"change_message_id":"0a4a07f53328eb0ed94d6c6fb7817c4ad200300c","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[tox]"},{"line_number":2,"context_line":"minversion \u003d 2.0"},{"line_number":3,"context_line":"envlist \u003d py27,py36,py37,pep8"},{"line_number":4,"context_line":"skipsdist \u003d True"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"[testenv]"}],"source_content_type":"text/x-properties","patch_set":2,"id":"7faddb67_3021ecfc","line":3,"range":{"start_line":3,"start_character":10,"end_line":3,"end_character":24},"in_reply_to":"9fb8cfa7_488089c0","updated":"2019-07-05 14:31:15.000000000","message":"Thanks for the comments. I\u0027m not sure why order matters (?). py27/py36/py37 are the supported train runtimes decided by the TC, which means they\u0027re available on distros that are targeted by openstack. It seems you\u0027d want to run at least a subset of those in the default tox target. Nonetheless I interpret the stretch goals in the TC goal (tox.ini changes in https://governance.openstack.org/tc/goals/train/python3-updates.html) to be optional for train, so I think we can abandon this change if it doesn\u0027t fit for ironic. Fwiw though most projects have moved to testing py27,py37 in the default tox envlist.","commit_id":"4644abcc002800fde633055aa42d98b881266e5b"}]}
