)]}'
{"lib/tempest":[{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"62d862ff7ef098cf25d62cfdc3049d45c653ba3b","unresolved":false,"context_lines":[{"line_number":664,"context_line":"    # NOTE(mtreinish) Respect constraints in the tempest full venv, things that"},{"line_number":665,"context_line":"    # are using a tox job other than full will not be respecting constraints but"},{"line_number":666,"context_line":"    # running pip install -U on tempest requirements"},{"line_number":667,"context_line":"    $TEMPEST_DIR/.tox/tempest/bin/pip install -c $REQUIREMENTS_DIR/upper-constraints.txt -r requirements.txt"},{"line_number":668,"context_line":"    PROJECT_VENV[\"tempest\"]\u003d${TEMPEST_DIR}/.tox/tempest"},{"line_number":669,"context_line":"    popd"},{"line_number":670,"context_line":"}"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"9fdfeff1_767ef3e3","line":667,"updated":"2019-02-27 11:14:13.000000000","message":"But then, does it make sense to use the requirements from $REQUIREMENTS_DIR? In another part of the code the requirements come from master, maybe there is a reason","commit_id":"0d83e09464551a8eb4ee5c66dc82de7053b14d70"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"5b7cd21a82c0eff3f9fd5f39cd3d363b0cc559b9","unresolved":false,"context_lines":[{"line_number":664,"context_line":"    # NOTE(mtreinish) Respect constraints in the tempest full venv, things that"},{"line_number":665,"context_line":"    # are using a tox job other than full will not be respecting constraints but"},{"line_number":666,"context_line":"    # running pip install -U on tempest requirements"},{"line_number":667,"context_line":"    $TEMPEST_DIR/.tox/tempest/bin/pip install -c $REQUIREMENTS_DIR/upper-constraints.txt -r requirements.txt"},{"line_number":668,"context_line":"    PROJECT_VENV[\"tempest\"]\u003d${TEMPEST_DIR}/.tox/tempest"},{"line_number":669,"context_line":"    popd"},{"line_number":670,"context_line":"}"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"9fdfeff1_79790480","line":667,"in_reply_to":"9fdfeff1_767ef3e3","updated":"2019-02-27 11:31:08.000000000","message":"we can change this also but this is not issue as this will be overridden by configure_tempest() L590, so this is no effect. \n\nI will make this also to use master constraint (but separate patch which does not need to be backported) to avoid confusion and duplicate install/uninstall.\n\nCurrent change pass the sahara test job - https://review.openstack.org/#/c/639530/","commit_id":"0d83e09464551a8eb4ee5c66dc82de7053b14d70"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"1e03454685ebb45680500d70d5c6cfe7558b082f","unresolved":false,"context_lines":[{"line_number":664,"context_line":"    # NOTE(mtreinish) Respect constraints in the tempest full venv, things that"},{"line_number":665,"context_line":"    # are using a tox job other than full will not be respecting constraints but"},{"line_number":666,"context_line":"    # running pip install -U on tempest requirements"},{"line_number":667,"context_line":"    $TEMPEST_DIR/.tox/tempest/bin/pip install -c $REQUIREMENTS_DIR/upper-constraints.txt -r requirements.txt"},{"line_number":668,"context_line":"    PROJECT_VENV[\"tempest\"]\u003d${TEMPEST_DIR}/.tox/tempest"},{"line_number":669,"context_line":"    popd"},{"line_number":670,"context_line":"}"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"9fdfeff1_19a5589f","line":667,"in_reply_to":"9fdfeff1_79790480","updated":"2019-02-27 11:51:18.000000000","message":"https://review.openstack.org/#/c/639637/2","commit_id":"0d83e09464551a8eb4ee5c66dc82de7053b14d70"}]}