)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13642,"name":"Vikram Hosakote","email":"vikram.ganesh@gmail.com","username":"vhosakot"},"change_message_id":"8f78e8e2ebf9378260b8bd45416d1f936b3d0723","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"fa6399be_834b24b0","line":13,"updated":"2016-10-10 20:33:34.000000000","message":"Can you file a bug with errors seen when upper constraints are not used, and add the bug ID here?","commit_id":"a9d660c688bd84718340a724244519558daa8cef"}],"tox.ini":[{"author":{"_account_id":13039,"name":"Martin André","email":"m.andre@redhat.com","username":"mandre"},"change_message_id":"0a453ddc34bb7010c9cb8c6883b6fb64f7a241f8","unresolved":false,"context_lines":[{"line_number":7,"context_line":"usedevelop\u003dTrue"},{"line_number":8,"context_line":"whitelist_externals \u003d find"},{"line_number":9,"context_line":"                      rm"},{"line_number":10,"context_line":"install_command \u003d pip install -c{env:UPPER_CONSTRAINTS_FILE:https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt} {opts} {packages}"},{"line_number":11,"context_line":"deps \u003d -r{toxinidir}/requirements.txt"},{"line_number":12,"context_line":"       -r{toxinidir}/test-requirements.txt"},{"line_number":13,"context_line":"passenv \u003d http_proxy HTTP_PROXY https_proxy HTTPS_PROXY no_proxy NO_PROXY"}],"source_content_type":"text/x-properties","patch_set":2,"id":"fa6399be_734afead","line":10,"updated":"2016-10-19 14:28:59.000000000","message":"Was the -U flag removed accidentally? Please restore it or explain why it was removed.","commit_id":"bcd3868275b134dcbaaebb38617237c80fc89afc"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"6941daddc5e8269b7c4eb796284c3c7a29f3de59","unresolved":false,"context_lines":[{"line_number":7,"context_line":"usedevelop\u003dTrue"},{"line_number":8,"context_line":"whitelist_externals \u003d find"},{"line_number":9,"context_line":"                      rm"},{"line_number":10,"context_line":"install_command \u003d pip install -c{env:UPPER_CONSTRAINTS_FILE:https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt} {opts} {packages}"},{"line_number":11,"context_line":"deps \u003d -r{toxinidir}/requirements.txt"},{"line_number":12,"context_line":"       -r{toxinidir}/test-requirements.txt"},{"line_number":13,"context_line":"passenv \u003d http_proxy HTTP_PROXY https_proxy HTTPS_PROXY no_proxy NO_PROXY"}],"source_content_type":"text/x-properties","patch_set":2,"id":"fa6399be_08c6ec48","line":10,"updated":"2016-10-19 15:05:11.000000000","message":"seems other project (nova, keystone) use such pattern w/o -U","commit_id":"bcd3868275b134dcbaaebb38617237c80fc89afc"},{"author":{"_account_id":13039,"name":"Martin André","email":"m.andre@redhat.com","username":"mandre"},"change_message_id":"132ad316eda22deb1fb733dcf7046299c6bda8e5","unresolved":false,"context_lines":[{"line_number":7,"context_line":"usedevelop\u003dTrue"},{"line_number":8,"context_line":"whitelist_externals \u003d find"},{"line_number":9,"context_line":"                      rm"},{"line_number":10,"context_line":"install_command \u003d pip install -c{env:UPPER_CONSTRAINTS_FILE:https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt} {opts} {packages}"},{"line_number":11,"context_line":"deps \u003d -r{toxinidir}/requirements.txt"},{"line_number":12,"context_line":"       -r{toxinidir}/test-requirements.txt"},{"line_number":13,"context_line":"passenv \u003d http_proxy HTTP_PROXY https_proxy HTTPS_PROXY no_proxy NO_PROXY"}],"source_content_type":"text/x-properties","patch_set":2,"id":"fa6399be_fb879db2","line":10,"in_reply_to":"fa6399be_08c6ec48","updated":"2016-10-20 13:54:52.000000000","message":"Right, but even if that was the case it doesn\u0027t belong to this patch which should just ensure the pip install command respects the upper constraints.","commit_id":"bcd3868275b134dcbaaebb38617237c80fc89afc"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"80a5a5fd5d82f3a9b900114f8a00076367c61da3","unresolved":false,"context_lines":[{"line_number":7,"context_line":"usedevelop\u003dTrue"},{"line_number":8,"context_line":"whitelist_externals \u003d find"},{"line_number":9,"context_line":"                      rm"},{"line_number":10,"context_line":"install_command \u003d pip install -c{env:UPPER_CONSTRAINTS_FILE:https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt} {opts} {packages}"},{"line_number":11,"context_line":"deps \u003d -r{toxinidir}/requirements.txt"},{"line_number":12,"context_line":"       -r{toxinidir}/test-requirements.txt"},{"line_number":13,"context_line":"passenv \u003d http_proxy HTTP_PROXY https_proxy HTTPS_PROXY no_proxy NO_PROXY"}],"source_content_type":"text/x-properties","patch_set":2,"id":"da6895a0_df966a4b","line":10,"in_reply_to":"fa6399be_fb879db2","updated":"2016-10-25 14:51:11.000000000","message":"I think it is OK to remove -U in this patch. w/ constraints, packages will be upgraded if its version is not satisfied in constraints.txt file.","commit_id":"bcd3868275b134dcbaaebb38617237c80fc89afc"}]}
