)]}'
{".zuul.yaml":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"66b1cf508a5c09abdc4ea5e5ec0e8d2905c35a12","unresolved":false,"context_lines":[{"line_number":7,"context_line":"      - neutron-tempest-plugin-jobs"},{"line_number":8,"context_line":"      - openstack-cover-jobs"},{"line_number":9,"context_line":"      - openstack-lower-constraints-jobs"},{"line_number":10,"context_line":"      - openstack-python-jobs"},{"line_number":11,"context_line":"      - openstack-python3-ussuri-jobs"},{"line_number":12,"context_line":"      - publish-openstack-docs-pti"},{"line_number":13,"context_line":"      - periodic-stable-jobs"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"3fa7e38b_570d1e3c","line":10,"updated":"2019-10-25 14:57:01.000000000","message":"I think we can remove this now as the job on L11 has pep8 now and no python2 :)\n\nhttps://review.opendev.org/#/c/688997/","commit_id":"b704d0197d64897c37e01062daa7eaf9a8297504"}],"etc/neutron/rootwrap.d/l3.filters":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"994bdf4507fb9d76a3cbd9dd82041f654df4249b","unresolved":false,"context_lines":[{"line_number":64,"context_line":"# command of the process to be killed is python."},{"line_number":65,"context_line":"# TODO(mlavalle) These kill filters will be updated once we come up with a"},{"line_number":66,"context_line":"# mechanism to kill using the name of the script being executed by Python"},{"line_number":67,"context_line":"# TODO(haleyb) Python 2.7 entry can eventually be removed"},{"line_number":68,"context_line":"kill_keepalived_monitor_py: KillFilter, root, python, -15"},{"line_number":69,"context_line":"kill_keepalived_monitor_py27: KillFilter, root, python2.7, -15"},{"line_number":70,"context_line":"kill_keepalived_monitor_py3: KillFilter, root, python3, -15"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"3fa7e38b_131ea8ba","line":67,"updated":"2019-10-11 19:51:55.000000000","message":"I\u0027m actually going to go ahead and remove this.","commit_id":"e0e863294fb86223cdfd8397fa1869132b657020"}],"playbooks/legacy/neutron-grenade/run.yaml":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"72883db77094228946c59325355fccc1eb284dc3","unresolved":false,"context_lines":[{"line_number":45,"context_line":"          export PYTHONUNBUFFERED\u003dtrue"},{"line_number":46,"context_line":"          export DEVSTACK_GATE_TEMPEST\u003d1"},{"line_number":47,"context_line":"          export DEVSTACK_GATE_GRENADE\u003dpullup"},{"line_number":48,"context_line":"          export DEVSTACK_GATE_USE_PYTHON3\u003dTrue"},{"line_number":49,"context_line":"          export DEVSTACK_GATE_NEUTRON\u003d1"},{"line_number":50,"context_line":"          export BRANCH_OVERRIDE\u003ddefault"},{"line_number":51,"context_line":"          if [ \"$BRANCH_OVERRIDE\" !\u003d \"default\" ] ; then"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"3fa7e38b_e6a42ca5","line":48,"updated":"2019-10-16 18:42:06.000000000","message":"Devstack is being changed to always use python3, so this isn\u0027t needed.","commit_id":"809c372a8b21a0f4c2c8b04fa558c08bd57a22b9"}],"tox.ini":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"06e025ec20798eb082919bfc75e1a4a59b09a430","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[tox]"},{"line_number":2,"context_line":"envlist \u003d docs,py37,pep8"},{"line_number":3,"context_line":"minversion \u003d 2.3.2"},{"line_number":4,"context_line":"basepython \u003d python3"},{"line_number":5,"context_line":"skipsdist \u003d True"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[testenv]"}],"source_content_type":"text/x-properties","patch_set":1,"id":"3fa7e38b_6dbe7bd5","line":4,"updated":"2019-10-07 12:49:42.000000000","message":"This should actually be down in [testenv], which maybe explains some of the other pep8 issues","commit_id":"ade2f535b42f27431dd90fd6ea61898669495db4"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"6303f8eb8151e5f47985cdcfa23e51d045d116ba","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[tox]"},{"line_number":2,"context_line":"envlist \u003d docs,py27,py37,pep8"},{"line_number":3,"context_line":"minversion \u003d 2.3.2"},{"line_number":4,"context_line":"basepython \u003d python3"},{"line_number":5,"context_line":"skipsdist \u003d True"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[testenv]"}],"source_content_type":"text/x-properties","patch_set":2,"id":"3fa7e38b_d0cb6da2","line":4,"updated":"2019-10-08 12:31:16.000000000","message":"Oops, forgot to remove","commit_id":"d6a5e25ade30bc7bb123957573613cf9cb889da1"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"c9b03794058b2662ade84ddf5686fae6c068a31a","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[tox]"},{"line_number":2,"context_line":"envlist \u003d docs,py27,py37,pep8"},{"line_number":3,"context_line":"minversion \u003d 2.3.2"},{"line_number":4,"context_line":"skipsdist \u003d True"},{"line_number":5,"context_line":""}],"source_content_type":"text/x-properties","patch_set":3,"id":"3fa7e38b_1d358eda","line":2,"range":{"start_line":2,"start_character":15,"end_line":2,"end_character":19},"updated":"2019-10-09 06:54:36.000000000","message":"shouldn\u0027t You remove py27 from this list too?","commit_id":"fa4205114ddb22c36e86edf1975337e1d0c5295d"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"1214f6ede18ae1566266e6e4b7acb7bcccefd5f5","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[tox]"},{"line_number":2,"context_line":"envlist \u003d docs,py27,py37,pep8"},{"line_number":3,"context_line":"minversion \u003d 2.3.2"},{"line_number":4,"context_line":"skipsdist \u003d True"},{"line_number":5,"context_line":""}],"source_content_type":"text/x-properties","patch_set":3,"id":"3fa7e38b_973dd68f","line":2,"range":{"start_line":2,"start_character":15,"end_line":2,"end_character":19},"in_reply_to":"3fa7e38b_1d358eda","updated":"2019-10-09 12:21:13.000000000","message":"I had the same comment for Kuba in the networking-ovn change, but we actually can\u0027t because there is still the openstack-tox-py27 job that is running, so until that is removed from the inherited job template this has to stay.  The failures in PS1 confirmed this :(","commit_id":"fa4205114ddb22c36e86edf1975337e1d0c5295d"},{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"929205bd54df51e3b01dbd24f86b17961e7bf854","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[tox]"},{"line_number":2,"context_line":"envlist \u003d docs,py27,py37,pep8"},{"line_number":3,"context_line":"minversion \u003d 2.3.2"},{"line_number":4,"context_line":"skipsdist \u003d True"},{"line_number":5,"context_line":""}],"source_content_type":"text/x-properties","patch_set":3,"id":"3fa7e38b_fd9aeb11","line":2,"range":{"start_line":2,"start_character":15,"end_line":2,"end_character":19},"in_reply_to":"3fa7e38b_973dd68f","updated":"2019-10-09 14:24:54.000000000","message":"In the meantime we can add a TODO item so we are \"cleanup ready\"","commit_id":"fa4205114ddb22c36e86edf1975337e1d0c5295d"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"e689740efb4368d5fea8f56f640db666d99ac1cc","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[tox]"},{"line_number":2,"context_line":"envlist \u003d docs,py27,py37,pep8"},{"line_number":3,"context_line":"minversion \u003d 2.3.2"},{"line_number":4,"context_line":"skipsdist \u003d True"},{"line_number":5,"context_line":""}],"source_content_type":"text/x-properties","patch_set":3,"id":"3fa7e38b_b2c1dc56","line":2,"range":{"start_line":2,"start_character":15,"end_line":2,"end_character":19},"in_reply_to":"3fa7e38b_cd25ecc3","updated":"2019-10-25 16:44:45.000000000","message":"\u0027py27\u0027 is defined in tox internally, so you can drop py27 from here. Having py27 here is just to clarify we use py27 target.","commit_id":"fa4205114ddb22c36e86edf1975337e1d0c5295d"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"cd9d24b438fa2f08fc3548fe80356caca6ea07cc","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[tox]"},{"line_number":2,"context_line":"envlist \u003d docs,py27,py37,pep8"},{"line_number":3,"context_line":"minversion \u003d 2.3.2"},{"line_number":4,"context_line":"skipsdist \u003d True"},{"line_number":5,"context_line":""}],"source_content_type":"text/x-properties","patch_set":3,"id":"3fa7e38b_cd25ecc3","line":2,"range":{"start_line":2,"start_character":15,"end_line":2,"end_character":19},"in_reply_to":"3fa7e38b_fd9aeb11","updated":"2019-10-10 14:05:53.000000000","message":"Done","commit_id":"fa4205114ddb22c36e86edf1975337e1d0c5295d"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"fd4023252a2116ae93dd0c9483458b0e0f84cbc8","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[tox]"},{"line_number":2,"context_line":"# TODO(haleyb) remove py27 from here once openstack-tox-py27 job goes away"},{"line_number":3,"context_line":"envlist \u003d docs,py27,py37,pep8"},{"line_number":4,"context_line":"minversion \u003d 2.3.2"},{"line_number":5,"context_line":"skipsdist \u003d True"},{"line_number":6,"context_line":""}],"source_content_type":"text/x-properties","patch_set":4,"id":"3fa7e38b_0d1e8bba","line":3,"updated":"2019-10-11 17:38:16.000000000","message":"Maybe it was some other issue that prevented me from removing py27 here, i\u0027m running a test on the octavia version to see what happens.","commit_id":"e0e863294fb86223cdfd8397fa1869132b657020"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"994bdf4507fb9d76a3cbd9dd82041f654df4249b","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[tox]"},{"line_number":2,"context_line":"# TODO(haleyb) remove py27 from here once openstack-tox-py27 job goes away"},{"line_number":3,"context_line":"envlist \u003d docs,py27,py37,pep8"},{"line_number":4,"context_line":"minversion \u003d 2.3.2"},{"line_number":5,"context_line":"skipsdist \u003d True"},{"line_number":6,"context_line":""}],"source_content_type":"text/x-properties","patch_set":4,"id":"3fa7e38b_736a7c1a","line":3,"in_reply_to":"3fa7e38b_0d1e8bba","updated":"2019-10-11 19:51:55.000000000","message":"So it doesn\u0027t look like we need to keep this, and since tox -e py27 can still be run anyways I\u0027ll remove it.","commit_id":"e0e863294fb86223cdfd8397fa1869132b657020"}]}
