)]}'
{".zuul.yaml":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"add2e7783f47f1c5197d28e06772d121c2d76b0a","unresolved":false,"context_lines":[{"line_number":4,"context_line":"      - openstack-python3-train-jobs"},{"line_number":5,"context_line":"    check:"},{"line_number":6,"context_line":"      jobs:"},{"line_number":7,"context_line":"        - openstack-tox-py37"},{"line_number":8,"context_line":"    gate:"},{"line_number":9,"context_line":"      jobs:"},{"line_number":10,"context_line":"        - openstack-tox-py37"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"3fa7e38b_9cbc2708","line":10,"range":{"start_line":7,"start_character":4,"end_line":10,"end_character":28},"updated":"2019-12-23 20:44:17.000000000","message":"I can remove these in favour of template use at L4","commit_id":"514af2295255014451613a47601546ac215b010f"}],"requirements.txt":[{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"ab5aae8da345bbd1aee215312cb5803460b48ea2","unresolved":false,"context_lines":[{"line_number":2,"context_line":"# of appearance. Changing the order has an impact on the overall integration"},{"line_number":3,"context_line":"# process, which may cause wedges in the gate later."},{"line_number":4,"context_line":"pbr\u003e\u003d2.0.0"},{"line_number":5,"context_line":"sphinx\u003e\u003d1.5.1,!\u003d1.6.6,!\u003d1.6.7,\u003c2.0.0;python_version\u003d\u003d\u00272.7\u0027  # BSD"},{"line_number":6,"context_line":"sphinx\u003e\u003d1.5.1,!\u003d1.6.6,!\u003d1.6.7,!\u003d2.1.0;python_version\u003e\u003d\u00273.4\u0027  # BSD"},{"line_number":7,"context_line":"sphinxcontrib-blockdiag"},{"line_number":8,"context_line":"openstackdocstheme\u003e\u003d1.20.0 # Apache-2.0"}],"source_content_type":"text/plain","patch_set":3,"id":"3fa7e38b_f148e462","side":"PARENT","line":5,"updated":"2019-12-23 18:24:55.000000000","message":"This repo doesn\u0027t seem to run the requirements job but in other projects we\u0027ve seen that removing this breaks the requirements check, are you sure we need to remove this?","commit_id":"b90f7c87021fb3eb9cb093a5af9ef9670e0af347"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f5193c82667344e25329a694d5930050051c6288","unresolved":false,"context_lines":[{"line_number":2,"context_line":"# of appearance. Changing the order has an impact on the overall integration"},{"line_number":3,"context_line":"# process, which may cause wedges in the gate later."},{"line_number":4,"context_line":"pbr\u003e\u003d2.0.0"},{"line_number":5,"context_line":"sphinx\u003e\u003d1.5.1,!\u003d1.6.6,!\u003d1.6.7,\u003c2.0.0;python_version\u003d\u003d\u00272.7\u0027  # BSD"},{"line_number":6,"context_line":"sphinx\u003e\u003d1.5.1,!\u003d1.6.6,!\u003d1.6.7,!\u003d2.1.0;python_version\u003e\u003d\u00273.4\u0027  # BSD"},{"line_number":7,"context_line":"sphinxcontrib-blockdiag"},{"line_number":8,"context_line":"openstackdocstheme\u003e\u003d1.20.0 # Apache-2.0"}],"source_content_type":"text/plain","patch_set":3,"id":"3fa7e38b_bceec31e","side":"PARENT","line":5,"in_reply_to":"3fa7e38b_f148e462","updated":"2019-12-23 20:43:23.000000000","message":"this is ok as requirement-check job has been updated to adjust not to check the py2 matches. \n\n- https://review.opendev.org/#/c/693631/","commit_id":"b90f7c87021fb3eb9cb093a5af9ef9670e0af347"}],"tox.ini":[{"author":{"_account_id":27621,"name":"Vishakha Agarwal","email":"agarwalvishakha18@gmail.com","username":"Vishakha"},"change_message_id":"f1239279b7f705e427038e7bf170a1029093820a","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[tox]"},{"line_number":2,"context_line":"minversion \u003d 1.6"},{"line_number":3,"context_line":"envlist \u003d docs,py27,py37"},{"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":3,"id":"3fa7e38b_ece1182d","side":"PARENT","line":3,"range":{"start_line":3,"start_character":20,"end_line":3,"end_character":24},"updated":"2019-12-19 09:24:25.000000000","message":"Since we are running py37 now it shouldn\u0027t be removed?","commit_id":"b90f7c87021fb3eb9cb093a5af9ef9670e0af347"},{"author":{"_account_id":27621,"name":"Vishakha Agarwal","email":"agarwalvishakha18@gmail.com","username":"Vishakha"},"change_message_id":"f1239279b7f705e427038e7bf170a1029093820a","unresolved":false,"context_lines":[{"line_number":5,"context_line":"ignore_basepython_conflict \u003d True"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[testenv]"},{"line_number":8,"context_line":"basepython \u003d python"},{"line_number":9,"context_line":"usedevelop \u003d True"},{"line_number":10,"context_line":"setenv \u003d VIRTUAL_ENV\u003d{envdir}"},{"line_number":11,"context_line":"install_command \u003d pip install -U {opts} {packages}"}],"source_content_type":"text/x-properties","patch_set":3,"id":"3fa7e38b_6cee8822","line":8,"range":{"start_line":8,"start_character":13,"end_line":8,"end_character":19},"updated":"2019-12-19 09:24:25.000000000","message":"it Should be python3.","commit_id":"514af2295255014451613a47601546ac215b010f"},{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"ab5aae8da345bbd1aee215312cb5803460b48ea2","unresolved":false,"context_lines":[{"line_number":5,"context_line":"ignore_basepython_conflict \u003d True"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[testenv]"},{"line_number":8,"context_line":"basepython \u003d python"},{"line_number":9,"context_line":"usedevelop \u003d True"},{"line_number":10,"context_line":"setenv \u003d VIRTUAL_ENV\u003d{envdir}"},{"line_number":11,"context_line":"install_command \u003d pip install -U {opts} {packages}"}],"source_content_type":"text/x-properties","patch_set":3,"id":"3fa7e38b_913730dc","line":8,"range":{"start_line":8,"start_character":13,"end_line":8,"end_character":19},"in_reply_to":"3fa7e38b_6cee8822","updated":"2019-12-23 18:24:55.000000000","message":"Agreed, but based on the addition of ignore_basepython_conflict above I\u0027m guessing this is intentional? Ghanshyam, can you explain?","commit_id":"514af2295255014451613a47601546ac215b010f"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f5193c82667344e25329a694d5930050051c6288","unresolved":false,"context_lines":[{"line_number":5,"context_line":"ignore_basepython_conflict \u003d True"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[testenv]"},{"line_number":8,"context_line":"basepython \u003d python"},{"line_number":9,"context_line":"usedevelop \u003d True"},{"line_number":10,"context_line":"setenv \u003d VIRTUAL_ENV\u003d{envdir}"},{"line_number":11,"context_line":"install_command \u003d pip install -U {opts} {packages}"}],"source_content_type":"text/x-properties","patch_set":3,"id":"3fa7e38b_318e7cd1","line":8,"range":{"start_line":8,"start_character":13,"end_line":8,"end_character":19},"in_reply_to":"3fa7e38b_913730dc","updated":"2019-12-23 20:43:23.000000000","message":"nice catch, it needs to be python3 otherwise it might take py2 if env has no py3.\n\nignore_basepython_conflict is for specific version env like py37 or 36 env.","commit_id":"514af2295255014451613a47601546ac215b010f"}]}
