)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"0fc507d048e0128bc341917bd2d94738774dedb7","unresolved":false,"context_lines":[{"line_number":7,"context_line":"tempest py26 setup"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Simplifying the steps between devstack install and tempest run, by"},{"line_number":10,"context_line":" * installing tox\u003d\u003d1.6.1 since it is latest known working version"},{"line_number":11,"context_line":" * installing and patching \u0027discover\u0027 when the python version is 2.6"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I8008837d2fae6cebeeeb051b63066279ca477e01"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAAXH%2F%2FLdA%3D","line":10,"updated":"2014-04-23 14:31:19.000000000","message":"Update this comment to match the code, or update the code to match this","commit_id":"4c6cb23cd17adb68c0c4b666ec160cdbb0e06f67"}],"files/unittest2-discover.patch":[{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"39a689b3cd8769ef04ecf35b929489d0ebe654e8","unresolved":false,"context_lines":[{"line_number":1,"context_line":"diff -r b2efb7df637b discover.py"},{"line_number":2,"context_line":"--- a/discover.py\tThu Mar 24 00:31:02 2011 -0400"},{"line_number":3,"context_line":"+++ b/discover.py\tThu Nov 28 12:02:19 2013 +0000"},{"line_number":4,"context_line":"@@ -82,7 +82,11 @@"}],"source_content_type":"text/x-diff","patch_set":1,"id":"AAAAW3%2F%2Fegw%3D","line":1,"updated":"2014-04-14 11:10:06.000000000","message":"we should probably have a files/patches/ directory for things like this.","commit_id":"8f31cfaa9f18d638e7f1dc4d9174d0d839b190ba"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"bdf6a5f8a5f060d419680f7adc55540435e0c9f7","unresolved":false,"context_lines":[{"line_number":1,"context_line":"diff -r b2efb7df637b discover.py"},{"line_number":2,"context_line":"--- a/discover.py\tThu Mar 24 00:31:02 2011 -0400"},{"line_number":3,"context_line":"+++ b/discover.py\tThu Nov 28 12:02:19 2013 +0000"},{"line_number":4,"context_line":"@@ -82,7 +82,11 @@"}],"source_content_type":"text/x-diff","patch_set":1,"id":"AAAAXH%2F%2FjJs%3D","line":1,"in_reply_to":"AAAAW3%2F%2FXmc%3D","updated":"2014-04-19 16:45:34.000000000","message":"It is reported at upstream since long: https://code.google.com/p/unittest-ext/issues/detail?id\u003d79.","commit_id":"8f31cfaa9f18d638e7f1dc4d9174d0d839b190ba"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"461aef7567bd3c50e1000f24fad200d5ca12cbf8","unresolved":false,"context_lines":[{"line_number":1,"context_line":"diff -r b2efb7df637b discover.py"},{"line_number":2,"context_line":"--- a/discover.py\tThu Mar 24 00:31:02 2011 -0400"},{"line_number":3,"context_line":"+++ b/discover.py\tThu Nov 28 12:02:19 2013 +0000"},{"line_number":4,"context_line":"@@ -82,7 +82,11 @@"}],"source_content_type":"text/x-diff","patch_set":1,"id":"AAAAW3%2F%2FXmc%3D","line":1,"in_reply_to":"AAAAW3%2F%2Fegw%3D","updated":"2014-04-15 05:13:33.000000000","message":"IMO I think this is a bad precedent; why is this not incorporated upstream? (that should be explained above the patch quite clearly if this is to go ahead anyway)","commit_id":"8f31cfaa9f18d638e7f1dc4d9174d0d839b190ba"}],"lib/tempest":[{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"39a689b3cd8769ef04ecf35b929489d0ebe654e8","unresolved":false,"context_lines":[{"line_number":403,"context_line":"# install_tempest() - Collect source and prepare"},{"line_number":404,"context_line":"function install_tempest {"},{"line_number":405,"context_line":"    git_clone $TEMPEST_REPO $TEMPEST_DIR $TEMPEST_BRANCH"},{"line_number":406,"context_line":"    pip_install tox\u003d\u003d1.6.1"},{"line_number":407,"context_line":"    if python --version 2\u003e\u00261 | grep -q \u0027Python 2.6\u0027; then"},{"line_number":408,"context_line":"        install_package python-unittest2 patch"},{"line_number":409,"context_line":"        pip_install discover"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAAW3%2F%2FehA%3D","line":406,"updated":"2014-04-14 11:10:06.000000000","message":"can we do tox\u003c1.7 instead","commit_id":"8f31cfaa9f18d638e7f1dc4d9174d0d839b190ba"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"39a689b3cd8769ef04ecf35b929489d0ebe654e8","unresolved":false,"context_lines":[{"line_number":404,"context_line":"function install_tempest {"},{"line_number":405,"context_line":"    git_clone $TEMPEST_REPO $TEMPEST_DIR $TEMPEST_BRANCH"},{"line_number":406,"context_line":"    pip_install tox\u003d\u003d1.6.1"},{"line_number":407,"context_line":"    if python --version 2\u003e\u00261 | grep -q \u0027Python 2.6\u0027; then"},{"line_number":408,"context_line":"        install_package python-unittest2 patch"},{"line_number":409,"context_line":"        pip_install discover"},{"line_number":410,"context_line":"        (cd /usr/lib/python2.6/site-packages/; sudo patch \u003c\"$FILES/unittest2-discover.patch\" || echo \u0027Assume already applied\u0027)"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAAW3%2F%2FehM%3D","line":407,"updated":"2014-04-14 11:10:06.000000000","message":"honestly, I\u0027d rather this goes into the work arounds for RHEL in the install_fixups","commit_id":"8f31cfaa9f18d638e7f1dc4d9174d0d839b190ba"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"0fc507d048e0128bc341917bd2d94738774dedb7","unresolved":false,"context_lines":[{"line_number":412,"context_line":"# install_tempest() - Collect source and prepare"},{"line_number":413,"context_line":"function install_tempest {"},{"line_number":414,"context_line":"    git_clone $TEMPEST_REPO $TEMPEST_DIR $TEMPEST_BRANCH"},{"line_number":415,"context_line":"    pip_install \"tox\u003c1.7\""},{"line_number":416,"context_line":"}"},{"line_number":417,"context_line":""},{"line_number":418,"context_line":"# init_tempest() - Initialize ec2 images"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"AAAAXH%2F%2FLew%3D","line":415,"updated":"2014-04-23 14:31:19.000000000","message":"Is this requirement specific to tempest? or to RHEL6?  It doesn\u0027t seem to be included in _any_ of our requirements files.  I guess I am wondering if this shouldn\u0027t also be in fixup_stuff.sh?","commit_id":"4c6cb23cd17adb68c0c4b666ec160cdbb0e06f67"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"6d0dada5ef0e4bea2d6a4f51c1067842423f2f43","unresolved":false,"context_lines":[{"line_number":412,"context_line":"# install_tempest() - Collect source and prepare"},{"line_number":413,"context_line":"function install_tempest {"},{"line_number":414,"context_line":"    git_clone $TEMPEST_REPO $TEMPEST_DIR $TEMPEST_BRANCH"},{"line_number":415,"context_line":"    pip_install \"tox\u003c1.7\""},{"line_number":416,"context_line":"}"},{"line_number":417,"context_line":""},{"line_number":418,"context_line":"# init_tempest() - Initialize ec2 images"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"AAAAXH%2F%2FLTI%3D","line":415,"in_reply_to":"AAAAXH%2F%2FLew%3D","updated":"2014-04-23 14:42:33.000000000","message":"It is not RHEL  specific.\ntox above 1.7 is not compatible with the tempests tox.ini.\n\nI usually recommend to use tempest together with tox, as it happens in the gate jobs. \n\ntox is not runtime hard dependency, it is an utility for managing the venv.","commit_id":"4c6cb23cd17adb68c0c4b666ec160cdbb0e06f67"}]}