)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7822,"name":"Chris Hoge (not involved in upstream development)","email":"chris@hogepodge.com","username":"hogepodge"},"change_message_id":"67b2af6bf7720e1aae2f896d6ab613ff799f5958","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Use same venv created in WORKDIR for installing tempest"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently setup_env is creating two .venv one under refstack-client"},{"line_number":10,"context_line":"project and another with in .tempest. It does not make sense to create"},{"line_number":11,"context_line":"two .venv. we can use the same .venv for all operations."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I2938b83b176cdc4c9e87703dd059ca58873419d7"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1f1a1f67_60231f07","line":11,"range":{"start_line":10,"start_character":38,"end_line":11,"end_character":56},"updated":"2017-07-21 01:48:54.000000000","message":"Can you please explain why you feel this does \"not make sense\"?","commit_id":"beb4e5202e103762302fca865f3a17391129a2f3"}],"setup_env":[{"author":{"_account_id":10526,"name":"Catherine Diep","email":"cdiep@us.ibm.com","username":"cdiep"},"change_message_id":"0afcee2177ad9a3a8cf83e974f4a7eff60b86f8e","unresolved":false,"context_lines":[{"line_number":158,"context_line":"rm -rf virtualenv-${VENV_VERSION}"},{"line_number":159,"context_line":"rm virtualenv-${VENV_VERSION}.tar.gz"},{"line_number":160,"context_line":"${WORKDIR}/.venv/bin/python -m pip install -r ${WORKDIR}/requirements.txt"},{"line_number":161,"context_line":"${TEMPEST_DIR}/.venv/bin/python -m pip install -r ${TEMPEST_DIR}/requirements.txt"},{"line_number":162,"context_line":""},{"line_number":163,"context_line":"# Add additional packages to find more tests by tempest"},{"line_number":164,"context_line":"# Note: Since there are no requirements in tempest-additional-requirements.txt by default,"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"1f1a1f67_4197ee91","side":"PARENT","line":161,"range":{"start_line":161,"start_character":1,"end_line":161,"end_character":15},"updated":"2017-07-21 16:23:11.000000000","message":"The reasons for using 2 virtual environments:\n\n1) Code and dependency isolation between RefStack and Tempest.\n2) Independent Tempest environment that can run with or without RefStack for test debug purposes.\n3) Enable RefStack to test with an existing Tempest environment in the future.","commit_id":"a361287d713a6d03951671ead31ee7cb04743d4c"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"ee8cce904f85c76b43b7183794f7f29796aaa49a","unresolved":false,"context_lines":[{"line_number":158,"context_line":"rm -rf virtualenv-${VENV_VERSION}"},{"line_number":159,"context_line":"rm virtualenv-${VENV_VERSION}.tar.gz"},{"line_number":160,"context_line":"${WORKDIR}/.venv/bin/python -m pip install -r ${WORKDIR}/requirements.txt"},{"line_number":161,"context_line":"${TEMPEST_DIR}/.venv/bin/python -m pip install -r ${TEMPEST_DIR}/requirements.txt"},{"line_number":162,"context_line":""},{"line_number":163,"context_line":"# Add additional packages to find more tests by tempest"},{"line_number":164,"context_line":"# Note: Since there are no requirements in tempest-additional-requirements.txt by default,"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"ff346bd7_a43116a5","side":"PARENT","line":161,"range":{"start_line":161,"start_character":1,"end_line":161,"end_character":15},"in_reply_to":"1f1a1f67_4197ee91","updated":"2017-07-25 05:23:41.000000000","message":"Thanks catherine and Chris, Thanks for the pointers, I will abandon my patch, 3 point as a feature looks interesting.","commit_id":"a361287d713a6d03951671ead31ee7cb04743d4c"}]}