)]}'
{"id":"openstack%2Fopenstack-ansible-tests~409867","triplet_id":"openstack%2Fopenstack-ansible-tests~master~I73ae4b3a6600cb9df2add0a3d6b2199fd7725890","project":"openstack/openstack-ansible-tests","branch":"master","hashtags":[],"change_id":"I73ae4b3a6600cb9df2add0a3d6b2199fd7725890","subject":"Install required apt packages for pip builds","status":"ABANDONED","created":"2016-12-12 17:08:10.000000000","updated":"2016-12-12 18:45:50.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"0b169da08fc5fb50ad5cf8b46ae68665d4c5aa62","_number":409867,"virtual_id_number":409867,"owner":{"_account_id":2799,"name":"Andy McCrae","email":"andy.mccrae@googlemail.com","username":"andrew-mccrae"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"_account_id":17799,"name":"Logan V","email":"logan2211@gmail.com","username":"Logan2211"},{"_account_id":12807,"name":"Steve Lewis (stevelle)","email":"stevelle@gmail.com","username":"stevelle"},{"value":-1,"date":"2016-12-12 18:06:53.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":-1,"default_value":0,"optional":true},"Code-Review":{"disliked":{"_account_id":12807,"name":"Steve Lewis (stevelle)","email":"stevelle@gmail.com","username":"stevelle"},"all":[{"value":0,"date":"2016-12-12 18:45:48.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":17799,"name":"Logan V","email":"logan2211@gmail.com","username":"Logan2211"},{"value":-1,"date":"2016-12-12 18:25:45.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":12807,"name":"Steve Lewis (stevelle)","email":"stevelle@gmail.com","username":"stevelle"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","value":-1,"default_value":0,"optional":true},"Workflow":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":17799,"name":"Logan V","email":"logan2211@gmail.com","username":"Logan2211"},{"_account_id":12807,"name":"Steve Lewis (stevelle)","email":"stevelle@gmail.com","username":"stevelle"},{"_account_id":3,"name":"Jenkins","username":"jenkins"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Backport-Candidate":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":17799,"name":"Logan V","email":"logan2211@gmail.com","username":"Logan2211"},{"_account_id":12807,"name":"Steve Lewis (stevelle)","email":"stevelle@gmail.com","username":"stevelle"},{"_account_id":3,"name":"Jenkins","username":"jenkins"}],"values":{"-1":"Not A Backport Candidate"," 0":"Backport Review Needed","+1":"Proposed Backport"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":12807,"name":"Steve Lewis (stevelle)","email":"stevelle@gmail.com","username":"stevelle"},{"_account_id":17799,"name":"Logan V","email":"logan2211@gmail.com","username":"Logan2211"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-12-12 18:06:53.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2016-12-12 18:25:45.000000000","updated_by":{"_account_id":12807,"name":"Steve Lewis (stevelle)","email":"stevelle@gmail.com","username":"stevelle"},"reviewer":{"_account_id":12807,"name":"Steve Lewis (stevelle)","email":"stevelle@gmail.com","username":"stevelle"},"state":"REVIEWER"},{"updated":"2016-12-12 18:45:48.000000000","updated_by":{"_account_id":17799,"name":"Logan V","email":"logan2211@gmail.com","username":"Logan2211"},"reviewer":{"_account_id":17799,"name":"Logan V","email":"logan2211@gmail.com","username":"Logan2211"},"state":"REVIEWER"}],"messages":[{"id":"5586c5dc7f0de612fc1e82bd075dc523216995e1","author":{"_account_id":2799,"name":"Andy McCrae","email":"andy.mccrae@googlemail.com","username":"andrew-mccrae"},"date":"2016-12-12 17:08:10.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"327fefda1e75675f5f866244474c007b9d4b0132","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-12-12 17:20:28.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-openstack-ansible-tests-docs-ubuntu-xenial http://docs-draft.openstack.org/67/409867/1/check/gate-openstack-ansible-tests-docs-ubuntu-xenial/ea35789//doc/build/html/ : SUCCESS in 1m 12s\n- gate-openstack-ansible-tests-linters-ubuntu-xenial http://logs.openstack.org/67/409867/1/check/gate-openstack-ansible-tests-linters-ubuntu-xenial/82f11b7/ : SUCCESS in 1m 33s\n- gate-openstack-ansible-tests-ansible-func-centos-7 http://logs.openstack.org/67/409867/1/check/gate-openstack-ansible-tests-ansible-func-centos-7/ac29b8e/ : FAILURE in 11m 17s\n- gate-openstack-ansible-tests-ansible-func-ubuntu-xenial http://logs.openstack.org/67/409867/1/check/gate-openstack-ansible-tests-ansible-func-ubuntu-xenial/4b8081c/ : FAILURE in 4m 37s","accounts_in_message":[],"_revision_number":1},{"id":"81b3b0af54e9bf8de2ace7111af98b844011d818","author":{"_account_id":2799,"name":"Andy McCrae","email":"andy.mccrae@googlemail.com","username":"andrew-mccrae"},"date":"2016-12-12 17:22:03.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"519ac150ac0c729107638c8ef46e9d661b1ad26e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-12-12 17:43:48.000000000","message":"Patch Set 2: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-openstack-ansible-tests-docs-ubuntu-xenial http://docs-draft.openstack.org/67/409867/2/check/gate-openstack-ansible-tests-docs-ubuntu-xenial/3297f73//doc/build/html/ : SUCCESS in 1m 01s\n- gate-openstack-ansible-tests-linters-ubuntu-xenial http://logs.openstack.org/67/409867/2/check/gate-openstack-ansible-tests-linters-ubuntu-xenial/7a5d9fd/ : SUCCESS in 2m 48s\n- gate-openstack-ansible-tests-ansible-func-centos-7 http://logs.openstack.org/67/409867/2/check/gate-openstack-ansible-tests-ansible-func-centos-7/61c6c5c/ : SUCCESS in 21m 04s\n- gate-openstack-ansible-tests-ansible-func-ubuntu-xenial http://logs.openstack.org/67/409867/2/check/gate-openstack-ansible-tests-ansible-func-ubuntu-xenial/fb654fa/ : FAILURE in 4m 44s","accounts_in_message":[],"_revision_number":2},{"id":"a70a2b5dce7ab3d71004bb26fcb8c1a188888292","author":{"_account_id":2799,"name":"Andy McCrae","email":"andy.mccrae@googlemail.com","username":"andrew-mccrae"},"date":"2016-12-12 18:00:50.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"59fa013cd60a80a43a441fd3f7778d168788db47","author":{"_account_id":17799,"name":"Logan V","email":"logan2211@gmail.com","username":"Logan2211"},"date":"2016-12-12 18:01:10.000000000","message":"Patch Set 3: Code-Review+2","accounts_in_message":[],"_revision_number":3},{"id":"e309c290df0a69ca91a9c78564ec8422915928aa","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-12-12 18:06:53.000000000","message":"Patch Set 3: Verified-1\n\nThis change depends on a change that failed to merge.","accounts_in_message":[],"_revision_number":3},{"id":"e4990734ff3a7924eb2c7dd27a24e91019f3b0c9","author":{"_account_id":12807,"name":"Steve Lewis (stevelle)","email":"stevelle@gmail.com","username":"stevelle"},"date":"2016-12-12 18:25:45.000000000","message":"Patch Set 3: Code-Review-1\n\nIsn\u0027t this a problem with installing the pip packages for the affected roles? Why then solve this in the centralized testing role? I don\u0027t like an actual deploy requirement to depend on the support of role testing scaffolding.\n\nA straight revert of Ia3fe82b78ef06d8b8d90eb5071bf0e284dba38b9 is less controversial all around, even if it adds something unnecessary to the integrated on Xenial.\n\nBetter would be to solve this for the roles in another way without bloating the integrated build but not linking it to the centralized testing role.","accounts_in_message":[],"_revision_number":3},{"id":"aebf4a31c787cf53f861ddd9667ccc91fe0a4ee7","author":{"_account_id":17799,"name":"Logan V","email":"logan2211@gmail.com","username":"Logan2211"},"date":"2016-12-12 18:45:48.000000000","message":"Patch Set 3: -Code-Review\n\nNot needed anymore as the flag is going to default to enabled","accounts_in_message":[],"_revision_number":3},{"id":"0b169da08fc5fb50ad5cf8b46ae68665d4c5aa62","author":{"_account_id":2799,"name":"Andy McCrae","email":"andy.mccrae@googlemail.com","username":"andrew-mccrae"},"date":"2016-12-12 18:45:50.000000000","message":"Abandoned\n\nabandoning - changing default behaviour to yes - set to no in repo server builds","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"4cf65d1656e7b672d99c2c2203fef1073bcb54b9","revisions":{"966784cd6aededf3e50d4d8912c79a35f21ea5de":{"kind":"REWORK","_number":1,"created":"2016-12-12 17:08:10.000000000","uploader":{"_account_id":2799,"name":"Andy McCrae","email":"andy.mccrae@googlemail.com","username":"andrew-mccrae"},"ref":"refs/changes/67/409867/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-ansible-tests","ref":"refs/changes/67/409867/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-ansible-tests refs/changes/67/409867/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-ansible-tests refs/changes/67/409867/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-ansible-tests refs/changes/67/409867/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-ansible-tests refs/changes/67/409867/1"}}},"commit":{"parents":[{"commit":"04dec8d74a00e0510baa0077265759a04f0704d7","subject":"Merge \"Custom ansible playbooks checker using linter\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-tests/commit/04dec8d74a00e0510baa0077265759a04f0704d7"}]}],"author":{"name":"Andy McCrae","email":"andy.mccrae@gmail.com","date":"2016-12-12 17:07:13.000000000","tz":0},"committer":{"name":"Andy McCrae","email":"andy.mccrae@gmail.com","date":"2016-12-12 17:07:13.000000000","tz":0},"subject":"Install required apt packages for pip builds","message":"Install required apt packages for pip builds\n\nSince https://review.openstack.org/#/c/409468/ we need to install the\nrequired apt packages needed to build pip packages. These are already\npresent on repo_servers but in the centralised role testing we don\u0027t use\na repo_server.\n\nChange-Id: I73ae4b3a6600cb9df2add0a3d6b2199fd7725890\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-tests/commit/966784cd6aededf3e50d4d8912c79a35f21ea5de"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-tests/commit/966784cd6aededf3e50d4d8912c79a35f21ea5de"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"04dec8d74a00e0510baa0077265759a04f0704d7","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"b6a3edd99191fe79e9d881e3592ba349c3e6dcdc":{"kind":"REWORK","_number":2,"created":"2016-12-12 17:22:03.000000000","uploader":{"_account_id":2799,"name":"Andy McCrae","email":"andy.mccrae@googlemail.com","username":"andrew-mccrae"},"ref":"refs/changes/67/409867/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-ansible-tests","ref":"refs/changes/67/409867/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-ansible-tests refs/changes/67/409867/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-ansible-tests refs/changes/67/409867/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-ansible-tests refs/changes/67/409867/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-ansible-tests refs/changes/67/409867/2"}}},"commit":{"parents":[{"commit":"04dec8d74a00e0510baa0077265759a04f0704d7","subject":"Merge \"Custom ansible playbooks checker using linter\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-tests/commit/04dec8d74a00e0510baa0077265759a04f0704d7"}]}],"author":{"name":"Andy McCrae","email":"andy.mccrae@gmail.com","date":"2016-12-12 17:07:13.000000000","tz":0},"committer":{"name":"Andy McCrae","email":"andy.mccrae@gmail.com","date":"2016-12-12 17:22:03.000000000","tz":0},"subject":"Install required apt packages for pip builds","message":"Install required apt packages for pip builds\n\nSince https://review.openstack.org/#/c/409468/ we need to install the\nrequired apt packages needed to build pip packages. These are already\npresent on repo_servers but in the centralised role testing we don\u0027t use\na repo_server.\n\nChange-Id: I73ae4b3a6600cb9df2add0a3d6b2199fd7725890\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-tests/commit/b6a3edd99191fe79e9d881e3592ba349c3e6dcdc"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-tests/commit/b6a3edd99191fe79e9d881e3592ba349c3e6dcdc"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"04dec8d74a00e0510baa0077265759a04f0704d7","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"4cf65d1656e7b672d99c2c2203fef1073bcb54b9":{"kind":"REWORK","_number":3,"created":"2016-12-12 18:00:50.000000000","uploader":{"_account_id":2799,"name":"Andy McCrae","email":"andy.mccrae@googlemail.com","username":"andrew-mccrae"},"ref":"refs/changes/67/409867/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-ansible-tests","ref":"refs/changes/67/409867/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-ansible-tests refs/changes/67/409867/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-ansible-tests refs/changes/67/409867/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-ansible-tests refs/changes/67/409867/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-ansible-tests refs/changes/67/409867/3"}}},"commit":{"parents":[{"commit":"04dec8d74a00e0510baa0077265759a04f0704d7","subject":"Merge \"Custom ansible playbooks checker using linter\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-tests/commit/04dec8d74a00e0510baa0077265759a04f0704d7"}]}],"author":{"name":"Andy McCrae","email":"andy.mccrae@gmail.com","date":"2016-12-12 18:00:08.000000000","tz":0},"committer":{"name":"Andy McCrae","email":"andy.mccrae@gmail.com","date":"2016-12-12 18:00:08.000000000","tz":0},"subject":"Install required apt packages for pip builds","message":"Install required apt packages for pip builds\n\nSince https://review.openstack.org/#/c/409468/ we need to install the\nrequired apt packages needed to build pip packages. These are already\npresent on repo_servers but in the centralised role testing we don\u0027t use\na repo_server.\n\nDepends-On: If484e14a49c97467a38f9ab6e4b0e3351f60d4ed\nChange-Id: I73ae4b3a6600cb9df2add0a3d6b2199fd7725890\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-tests/commit/4cf65d1656e7b672d99c2c2203fef1073bcb54b9"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-tests/commit/4cf65d1656e7b672d99c2c2203fef1073bcb54b9"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"04dec8d74a00e0510baa0077265759a04f0704d7","is_merged_in_target_branch":true}],"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}