)]}'
{"setup_env":[{"author":{"_account_id":9626,"name":"Paul Van Eck","email":"pvaneck@us.ibm.com","username":"pvaneck"},"change_message_id":"b2f6a49cfe59b43cb100ff712d94d18a3f68fd4c","unresolved":false,"context_lines":[{"line_number":153,"context_line":"${WORKDIR}/.tempest/.venv/bin/pip install -r ${WORKDIR}/.tempest/requirements.txt"},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"# Add additional packages to find more tests by tempest"},{"line_number":156,"context_line":"# Note: commenting the following line out because the TC wants all tests in tempest https://governance.openstack.org/resolutions/20160504-defcore-test-location.html"},{"line_number":157,"context_line":"# Since we have nothing in the tempest-additional-requirements.txt."},{"line_number":158,"context_line":"# This line always return errors and confuses the user."},{"line_number":159,"context_line":"#${WORKDIR}/.tempest/.venv/bin/pip install -r ${WORKDIR}/tempest-additional-requirements.txt"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"9a89bdaa_d3ba54c0","line":156,"updated":"2016-09-02 21:41:05.000000000","message":"I think we should remove this line of the comment regarding TC because refstack-client can still be used to test out-of-tempest non-defcore tests regardless. The impetus for commenting out the below line is to prevent confusing errors when the file is empty (which it is by default).\n\nSo I think a better comment would be:\n\nNote: Since there are no requirements in tempest-additional-requirements.txt by default, this line is commented out to prevent errors from being returned. Uncomment this line if there are requirements in tempest-additonal-requirements.txt.","commit_id":"cf2a84c9d826cf0f1ef05059c69b6f01b60e0568"}]}