)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":17716,"name":"Luz Cazares","email":"luz.cazares@intel.com","username":"luzcazares"},"change_message_id":"b7c372b4073a5a6f633290a6a1c3cdc12f317bf7","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Use upper-constraints for tox envs"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Pin tox environments to upper-constraints to avoid conflicts with"},{"line_number":10,"context_line":"library releases."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I6e564f3035e2fdc5461f57f15168f498c99fc164"},{"line_number":13,"context_line":"Closes-Bug:#1628597"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3a461143_3d13edb9","line":10,"range":{"start_line":10,"start_character":16,"end_line":10,"end_character":17},"updated":"2017-01-27 15:26:49.000000000","message":"nit: Probably add details about why the helper script is needed, maybe: \"Since libraries themselves are listed in upper-constraints.txt. It leads to errors that you can\u0027t install a specific version and a\nconstrained version.\nThis change adds constraints support by adding a helper script to\nremove tempest constraints.\"","commit_id":"35c0157ed3aa2f544cfb6c820c918d8ec9bf9034"}],"tools/tox_install.sh":[{"author":{"_account_id":17716,"name":"Luz Cazares","email":"luz.cazares@intel.com","username":"luzcazares"},"change_message_id":"b7c372b4073a5a6f633290a6a1c3cdc12f317bf7","unresolved":false,"context_lines":[{"line_number":5,"context_line":"# the constraints file before applying it for from-source installation."},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"ZUUL_CLONER\u003d/usr/zuul-env/bin/zuul-cloner"},{"line_number":8,"context_line":"BRANCH_NAME\u003dmaster"},{"line_number":9,"context_line":"CLIENT_NAME\u003dtempest"},{"line_number":10,"context_line":"requirements_installed\u003d$(echo \"import openstack_requirements\" | python 2\u003e/dev/null ; echo $?)"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"set -e"}],"source_content_type":"text/x-sh","patch_set":2,"id":"3a461143_404e1070","line":9,"range":{"start_line":8,"start_character":0,"end_line":9,"end_character":19},"updated":"2017-01-27 15:26:49.000000000","message":"Should these be moved to tox.ini file, under setenv ?","commit_id":"35c0157ed3aa2f544cfb6c820c918d8ec9bf9034"},{"author":{"_account_id":17716,"name":"Luz Cazares","email":"luz.cazares@intel.com","username":"luzcazares"},"change_message_id":"b7c372b4073a5a6f633290a6a1c3cdc12f317bf7","unresolved":false,"context_lines":[{"line_number":14,"context_line":"CONSTRAINTS_FILE\u003d$1"},{"line_number":15,"context_line":"shift"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"install_cmd\u003d\"pip install\""},{"line_number":18,"context_line":"mydir\u003d$(mktemp -dt \"$CLIENT_NAME-tox_install-XXXXXXX\")"},{"line_number":19,"context_line":"trap \"rm -rf $mydir\" EXIT"},{"line_number":20,"context_line":"localfile\u003d$mydir/upper-constraints.txt"}],"source_content_type":"text/x-sh","patch_set":2,"id":"3a461143_0057a8ce","line":17,"range":{"start_line":17,"start_character":0,"end_line":17,"end_character":25},"updated":"2017-01-27 15:26:49.000000000","message":"would removed this line and directly use \"pip install\" on line 25","commit_id":"35c0157ed3aa2f544cfb6c820c918d8ec9bf9034"}]}