)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"35978f0afd6eeba28ad3d51181ca63c0559dda2f","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Build wheel house during nodepool devstack caching"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This commit adds calling tools/build_wheelhosue to the cache devstack"},{"line_number":10,"context_line":"step in the nodepool scripts."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I9064d7d9b9b879a05d665c3b002a631fcc953f52"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"ba7be1f8_cdb6e8f8","line":9,"updated":"2015-03-03 18:36:57.000000000","message":"nit: filename is wrong, it is tools/build_wheels.sh","commit_id":"75c46bd3863554471ed962c8ed4f74b97db228c3"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"4cd249c04c03b60af015a430b93330e8161f48f4","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Build wheel house during nodepool devstack caching"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This commit adds calling tools/build_wheelhosue to the cache devstack"},{"line_number":10,"context_line":"step in the nodepool scripts."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I9064d7d9b9b879a05d665c3b002a631fcc953f52"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"ba7be1f8_3046f7c0","line":9,"in_reply_to":"ba7be1f8_cdb6e8f8","updated":"2015-03-03 18:58:36.000000000","message":"Done","commit_id":"75c46bd3863554471ed962c8ed4f74b97db228c3"}],"nodepool/elements/cache-devstack/extra-data.d/55-cache-devstack-repos":[{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"35978f0afd6eeba28ad3d51181ca63c0559dda2f","unresolved":false,"context_lines":[{"line_number":121,"context_line":"    build_wheels_path \u003d os.path.join(basedir, \u0027tools/build_wheels.sh\u0027)"},{"line_number":122,"context_line":"    env \u003d os.environ"},{"line_number":123,"context_line":"    # Makes output dir for wheelhouse CACHEDIR/wheelhouse"},{"line_number":124,"context_line":"    env[\u0027WHEELHOUSE\u0027] \u003d os.path.join(\u0027/home/jenkins/cache/files/\u0027,"},{"line_number":125,"context_line":"                                     \u0027wheelhouse\u0027)"},{"line_number":126,"context_line":"    if os.path.isfile(build_wheels_path):"},{"line_number":127,"context_line":"        run_local([\u0027bash\u0027, \u0027tools/build_wheels.sh\u0027], cwd\u003dDEVSTACK, env\u003denv)"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"ba7be1f8_4da758a7","line":124,"updated":"2015-03-03 18:36:57.000000000","message":"Why not use \"/home/jenkins/cache/files/wheelhouse\" directly?","commit_id":"75c46bd3863554471ed962c8ed4f74b97db228c3"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"4cd249c04c03b60af015a430b93330e8161f48f4","unresolved":false,"context_lines":[{"line_number":121,"context_line":"    build_wheels_path \u003d os.path.join(basedir, \u0027tools/build_wheels.sh\u0027)"},{"line_number":122,"context_line":"    env \u003d os.environ"},{"line_number":123,"context_line":"    # Makes output dir for wheelhouse CACHEDIR/wheelhouse"},{"line_number":124,"context_line":"    env[\u0027WHEELHOUSE\u0027] \u003d os.path.join(\u0027/home/jenkins/cache/files/\u0027,"},{"line_number":125,"context_line":"                                     \u0027wheelhouse\u0027)"},{"line_number":126,"context_line":"    if os.path.isfile(build_wheels_path):"},{"line_number":127,"context_line":"        run_local([\u0027bash\u0027, \u0027tools/build_wheels.sh\u0027], cwd\u003dDEVSTACK, env\u003denv)"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"ba7be1f8_103d732b","line":124,"in_reply_to":"ba7be1f8_4da758a7","updated":"2015-03-03 18:58:36.000000000","message":"Mostly, because in previous revs I was using CACHEDIR, which it turns out is different in the elements. So I just replaced CACHEDIR with the path I expected that to really be not thinking about it.","commit_id":"75c46bd3863554471ed962c8ed4f74b97db228c3"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"255dbc149361b85c5f88efb3fb58c4f1ffcbf8fa","unresolved":false,"context_lines":[{"line_number":165,"context_line":""},{"line_number":166,"context_line":"        branch_data[\u0027images\u0027] \u003d images"},{"line_number":167,"context_line":"        branches.append(branch_data)"},{"line_number":168,"context_line":"        _build_wheel_house(DEVSTACK)"},{"line_number":169,"context_line":"    return branches"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":4,"id":"9a80dd14_2c1d5610","line":168,"updated":"2015-03-04 22:01:52.000000000","message":"This doesn\u0027t run in the dib chroot so all wheels will end up being built for Ubuntu Trusty. We need to delay this wheelhouse building until the install.d so that it happens in the chroot.","commit_id":"e6963aecf5f8ea269bbe7c7471ed4979d33ec9d6"}],"nodepool/elements/cache-devstack/install.d/98-build-wheelhouse":[{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"12733e65e0c269d7ec39f5533844a36e44a6be7e","unresolved":false,"context_lines":[{"line_number":29,"context_line":"    fi"},{"line_number":30,"context_line":"    git checkout $branch"},{"line_number":31,"context_line":"    git pull --ff-only origin"},{"line_number":32,"context_line":"    bash tools/build_wheels.sh"},{"line_number":33,"context_line":"done"},{"line_number":34,"context_line":"popd"}],"source_content_type":"application/x-shellscript","patch_set":5,"id":"9a80dd14_6503e10f","line":32,"updated":"2015-03-04 23:22:58.000000000","message":"should we check if this exists first? Does this file exist on all branches?\n\nAlso do we need to specify WHEELHOUSE?","commit_id":"6df41d64953d6bdc854b1e8de341cd9b0192698c"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"d231455164685548d61e3daea51fc9d9a8dc0eda","unresolved":false,"context_lines":[{"line_number":29,"context_line":"    fi"},{"line_number":30,"context_line":"    git checkout $branch"},{"line_number":31,"context_line":"    git pull --ff-only origin"},{"line_number":32,"context_line":"    bash tools/build_wheels.sh"},{"line_number":33,"context_line":"done"},{"line_number":34,"context_line":"popd"}],"source_content_type":"application/x-shellscript","patch_set":5,"id":"9a80dd14_a0ba27df","line":32,"in_reply_to":"9a80dd14_6503e10f","updated":"2015-03-04 23:35:26.000000000","message":"Good call on the existence check, I\u0027ll respin it shortly.\n\nWHEELHOUSE is specified on L23.","commit_id":"6df41d64953d6bdc854b1e8de341cd9b0192698c"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"3a994ef1ae0610fa0d7e7d73de79da853ca195bd","unresolved":false,"context_lines":[{"line_number":27,"context_line":"    if [[ $(echo $branch | grep \u0027^-\u003e\u0027) || $(echo $branch | grep \"^origin\")  || $(echo $branch | grep \u0027HEAD\u0027) ]]; then"},{"line_number":28,"context_line":"        continue"},{"line_number":29,"context_line":"    fi"},{"line_number":30,"context_line":"    git checkout $branch"},{"line_number":31,"context_line":"    git pull --ff-only origin"},{"line_number":32,"context_line":"    if [[ -f $DEVSTACK/tools/build_wheels.sh ]] ; then"},{"line_number":33,"context_line":"        bash tools/build_wheels.sh"}],"source_content_type":"application/x-shellscript","patch_set":6,"id":"9a80dd14_138e78d0","line":30,"updated":"2015-03-06 21:58:54.000000000","message":"This may require sudo, depends on user used by dib in chroot which should be root.","commit_id":"995927e485fef6bca8405887378b726530060219"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"c2e6d6c7c35f832ec4f9ba22a62a04d0f9e77532","unresolved":false,"context_lines":[{"line_number":27,"context_line":"    if [[ $(echo $branch | grep \u0027^-\u003e\u0027) || $(echo $branch | grep \"^origin\")  || $(echo $branch | grep \u0027HEAD\u0027) ]]; then"},{"line_number":28,"context_line":"        continue"},{"line_number":29,"context_line":"    fi"},{"line_number":30,"context_line":"    git checkout $branch"},{"line_number":31,"context_line":"    git pull --ff-only origin"},{"line_number":32,"context_line":"    if [[ -f $DEVSTACK/tools/build_wheels.sh ]] ; then"},{"line_number":33,"context_line":"        bash tools/build_wheels.sh"}],"source_content_type":"application/x-shellscript","patch_set":6,"id":"9a80dd14_fc90d1ee","line":30,"in_reply_to":"9a80dd14_138e78d0","updated":"2015-03-06 23:28:21.000000000","message":"Done","commit_id":"995927e485fef6bca8405887378b726530060219"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"3a994ef1ae0610fa0d7e7d73de79da853ca195bd","unresolved":false,"context_lines":[{"line_number":28,"context_line":"        continue"},{"line_number":29,"context_line":"    fi"},{"line_number":30,"context_line":"    git checkout $branch"},{"line_number":31,"context_line":"    git pull --ff-only origin"},{"line_number":32,"context_line":"    if [[ -f $DEVSTACK/tools/build_wheels.sh ]] ; then"},{"line_number":33,"context_line":"        bash tools/build_wheels.sh"},{"line_number":34,"context_line":"    fi"}],"source_content_type":"application/x-shellscript","patch_set":6,"id":"9a80dd14_d3d940f4","line":31,"updated":"2015-03-06 21:58:54.000000000","message":"Do not pull here. The repo is already up to date.","commit_id":"995927e485fef6bca8405887378b726530060219"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"c2e6d6c7c35f832ec4f9ba22a62a04d0f9e77532","unresolved":false,"context_lines":[{"line_number":28,"context_line":"        continue"},{"line_number":29,"context_line":"    fi"},{"line_number":30,"context_line":"    git checkout $branch"},{"line_number":31,"context_line":"    git pull --ff-only origin"},{"line_number":32,"context_line":"    if [[ -f $DEVSTACK/tools/build_wheels.sh ]] ; then"},{"line_number":33,"context_line":"        bash tools/build_wheels.sh"},{"line_number":34,"context_line":"    fi"}],"source_content_type":"application/x-shellscript","patch_set":6,"id":"9a80dd14_1c8e85d0","line":31,"in_reply_to":"9a80dd14_d3d940f4","updated":"2015-03-06 23:28:21.000000000","message":"Done","commit_id":"995927e485fef6bca8405887378b726530060219"}]}