)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"dbaf05ee861b04f187d77a420bce47bd05d4cd98","unresolved":false,"context_lines":[{"line_number":26,"context_line":"   required project except lower-constraint job. Let\u0027s make that also"},{"line_number":27,"context_line":"   install neutron. Also add neutron to requirements.txt."},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"Note(elod.illes): by removing editable git installation of neutron from"},{"line_number":30,"context_line":"tox and adding it to requirements.txt involves that tests in Zuul will"},{"line_number":31,"context_line":"run against latest stable state, but locally the test will run against"},{"line_number":32,"context_line":"latest release from stable/* of neutron. So in short: they test"},{"line_number":33,"context_line":"different things."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"Change-Id: Ic4d97db6c947264806e5ef9e35f21f764cbecf46"},{"line_number":36,"context_line":"(cherry picked from commit a00fc731b14bd3ef953fe4cffdc74c8e441c0e13)"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"3fa7e38b_b7f55586","line":33,"range":{"start_line":29,"start_character":0,"end_line":33,"end_character":17},"updated":"2020-01-22 15:31:44.000000000","message":"Okay, this needs to be a big fat note in the relevant sections of tox.ini.","commit_id":"998e0ffba9f9c63c3aab4a0ba6ade90b8b05b7e7"}],"tox.ini":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"d18d7d62cdbf674a23bcd11db1bd480fb539e32c","unresolved":false,"context_lines":[{"line_number":22,"context_line":"  -c{env:UPPER_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/train}"},{"line_number":23,"context_line":"  -r{toxinidir}/requirements.txt"},{"line_number":24,"context_line":"  -r{toxinidir}/test-requirements.txt"},{"line_number":25,"context_line":"  -egit+https://opendev.org/openstack/neutron#egg\u003dneutron"},{"line_number":26,"context_line":"  -rhttps://opendev.org/openstack/neutron/raw/branch/master/test-requirements.txt"},{"line_number":27,"context_line":"whitelist_externals \u003d"},{"line_number":28,"context_line":"  bash"}],"source_content_type":"text/x-properties","patch_set":2,"id":"3fa7e38b_a0ea5c9d","side":"PARENT","line":25,"range":{"start_line":25,"start_character":0,"end_line":25,"end_character":57},"updated":"2020-01-15 09:28:09.000000000","message":"This is the cause of conflict as editable git install cannot be constrained.\n\n(It\u0027s just another thing that here we install neutron from latest master instead of latest stable/train (which would be ...neutron@stable/train#egg\u003dneutron)","commit_id":"58706cecd911271cd20c4d5c91ef9a5747d1b07c"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"dbaf05ee861b04f187d77a420bce47bd05d4cd98","unresolved":false,"context_lines":[{"line_number":22,"context_line":"  -c{env:UPPER_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/train}"},{"line_number":23,"context_line":"  -r{toxinidir}/requirements.txt"},{"line_number":24,"context_line":"  -r{toxinidir}/test-requirements.txt"},{"line_number":25,"context_line":"  -egit+https://opendev.org/openstack/neutron#egg\u003dneutron"},{"line_number":26,"context_line":"  -rhttps://opendev.org/openstack/neutron/raw/branch/master/test-requirements.txt"},{"line_number":27,"context_line":"whitelist_externals \u003d"},{"line_number":28,"context_line":"  bash"},{"line_number":29,"context_line":"  find"}],"source_content_type":"text/x-properties","patch_set":5,"id":"3fa7e38b_1732295e","side":"PARENT","line":26,"range":{"start_line":25,"start_character":0,"end_line":26,"end_character":81},"updated":"2020-01-22 15:31:44.000000000","message":"IIUC if a person wanted to test locally in the same way as the gate, they could temporarily re-add these lines, and then remember to remove them before committing? That should be included in the big fat note. Along with a TODO to find a better way to make it work. This can\u0027t be the only project with this issue.","commit_id":"58706cecd911271cd20c4d5c91ef9a5747d1b07c"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"61bfaab07a40a344fc4aa0b05dee41c72c9bd774","unresolved":false,"context_lines":[{"line_number":22,"context_line":"  -c{env:UPPER_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/train}"},{"line_number":23,"context_line":"  -r{toxinidir}/requirements.txt"},{"line_number":24,"context_line":"  -r{toxinidir}/test-requirements.txt"},{"line_number":25,"context_line":"  -egit+https://opendev.org/openstack/neutron#egg\u003dneutron"},{"line_number":26,"context_line":"  -rhttps://opendev.org/openstack/neutron/raw/branch/master/test-requirements.txt"},{"line_number":27,"context_line":"whitelist_externals \u003d"},{"line_number":28,"context_line":"  bash"},{"line_number":29,"context_line":"  find"}],"source_content_type":"text/x-properties","patch_set":5,"id":"3fa7e38b_833e9ce0","side":"PARENT","line":26,"range":{"start_line":25,"start_character":0,"end_line":26,"end_character":81},"in_reply_to":"3fa7e38b_1732295e","updated":"2020-01-28 12:23:46.000000000","message":"Done","commit_id":"58706cecd911271cd20c4d5c91ef9a5747d1b07c"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"e7f1ee04db92048b9e764685e41943c632612f21","unresolved":false,"context_lines":[{"line_number":22,"context_line":"  -c{env:UPPER_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/train}"},{"line_number":23,"context_line":"  -r{toxinidir}/requirements.txt"},{"line_number":24,"context_line":"  -r{toxinidir}/test-requirements.txt"},{"line_number":25,"context_line":"  -egit+https://opendev.org/openstack/neutron#egg\u003dneutron"},{"line_number":26,"context_line":"  -rhttps://opendev.org/openstack/neutron/raw/branch/master/test-requirements.txt"},{"line_number":27,"context_line":"whitelist_externals \u003d"},{"line_number":28,"context_line":"  bash"},{"line_number":29,"context_line":"  find"}],"source_content_type":"text/x-properties","patch_set":5,"id":"3fa7e38b_52966ff1","side":"PARENT","line":26,"range":{"start_line":25,"start_character":0,"end_line":26,"end_character":81},"in_reply_to":"3fa7e38b_1732295e","updated":"2020-01-22 16:23:41.000000000","message":"I saw some projects that do tests against released neutron only (i.e. they don\u0027t add neutron to \u0027required projects\u0027 in .zuul.yaml)\n\nAnother complicated use case for local tests is to:\n* add *dev tox target with the git link AND\n* set UPPER_CONSTRAINTS_FILE environment variable to an edited upper-constraints.txt where neutron is removed\n\ne.g. \nUPPER_CONSTRAINTS_FILE\u003dupper-constraints-edited.txt tox -e py37-dev\n\n... or something like this.","commit_id":"58706cecd911271cd20c4d5c91ef9a5747d1b07c"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"7c84c679a36b0986712a889da8fe6c84992a945c","unresolved":false,"context_lines":[{"line_number":29,"context_line":"  # * zuul test: tests with latest state of stable/* neutron (due to"},{"line_number":30,"context_line":"  #   neutron listed in \u0027required-projects\u0027)"},{"line_number":31,"context_line":"  # To test the same way as in zuul, uncomment the below lines and use"},{"line_number":32,"context_line":"  # an edited UPPER_CONTRAINT_FILE that does not contain neutron."},{"line_number":33,"context_line":"  # -egit+https://opendev.org/openstack/neutron@stable/train#egg\u003dneutron"},{"line_number":34,"context_line":"  # -rhttps://opendev.org/openstack/neutron/raw/branch/stable/train/test-requirements.txt"},{"line_number":35,"context_line":"whitelist_externals \u003d"}],"source_content_type":"text/x-properties","patch_set":6,"id":"3fa7e38b_4f24450e","line":32,"updated":"2020-01-28 15:20:42.000000000","message":"++","commit_id":"cfa217e9b1c9a346b2e7165110e4195d773268f3"}]}