)]}'
{"id":"openstack%2Fdevstack-gate~416584","triplet_id":"openstack%2Fdevstack-gate~master~I4512946fe0fd907b94e1401175e4e544fdc4c9a8","project":"openstack/devstack-gate","branch":"master","topic":"ara-callback","hashtags":[],"change_id":"I4512946fe0fd907b94e1401175e4e544fdc4c9a8","subject":"Do not assume python version for ARA callback","status":"ABANDONED","created":"2017-01-04 15:16:33.000000000","updated":"2017-09-05 18:11:31.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"10e9c1985e2adbfa0dc3a9c5c3b66bf2d2ae4aac","_number":416584,"virtual_id_number":416584,"owner":{"_account_id":7979,"name":"Mikhail S Medvedev","email":"mihailmed@gmail.com","username":"msmedved"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"value":-1,"date":"2017-08-25 22:05:10.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},{"date":"2017-02-17 14:42:40.000000000","_account_id":7979,"name":"Mikhail S Medvedev","email":"mihailmed@gmail.com","username":"msmedved"}],"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":{"recommended":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2017-01-04 15:19:35.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":7979,"name":"Mikhail S Medvedev","email":"mihailmed@gmail.com","username":"msmedved"}],"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":[{"date":"2017-02-22 14:56:37.000000000","_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"date":"2017-02-17 13:59:01.000000000","_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},{"date":"2017-02-17 13:58:49.000000000","_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":7979,"name":"Mikhail S Medvedev","email":"mihailmed@gmail.com","username":"msmedved"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},{"_account_id":7979,"name":"Mikhail S Medvedev","email":"mihailmed@gmail.com","username":"msmedved"},{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2017-01-04 15:19:35.000000000","updated_by":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"reviewer":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"state":"REVIEWER"},{"updated":"2017-02-17 13:58:49.000000000","updated_by":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"reviewer":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"state":"REVIEWER"},{"updated":"2017-02-17 13:59:01.000000000","updated_by":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"reviewer":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"state":"REVIEWER"},{"updated":"2017-02-22 14:56:37.000000000","updated_by":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"reviewer":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"state":"REVIEWER"},{"updated":"2017-08-25 22:05:10.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"1407e47427c4d3870eb63da02497a443701c805f","author":{"_account_id":7979,"name":"Mikhail S Medvedev","email":"mihailmed@gmail.com","username":"msmedved"},"date":"2017-01-04 15:16:33.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"174e487c44c24f5a396c1c4d6082529bd2740238","author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"date":"2017-01-04 15:19:35.000000000","message":"Patch Set 1: Code-Review+1\n\nClever and good idea.","accounts_in_message":[],"_revision_number":1},{"id":"14196238231044081ed7207ac0b2fb2341f22c17","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-01-04 16:26:44.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-tempest-dsvm-neutron-full-ubuntu-xenial http://logs.openstack.org/84/416584/1/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/a43432a/ : SUCCESS in 56m 28s\n- gate-grenade-dsvm-neutron-ubuntu-xenial http://logs.openstack.org/84/416584/1/check/gate-grenade-dsvm-neutron-ubuntu-xenial/a9b9b1f/ : SUCCESS in 34m 08s\n- gate-devstack-gate-unittests http://logs.openstack.org/84/416584/1/check/gate-devstack-gate-unittests/1108f55/ : SUCCESS in 1m 33s\n- gate-devstack-gate-bashate http://logs.openstack.org/84/416584/1/check/gate-devstack-gate-bashate/7450117/ : SUCCESS in 1m 56s\n- gate-dg-hooks-dsvm http://logs.openstack.org/84/416584/1/check/gate-dg-hooks-dsvm/ae0d0d0/ : SUCCESS in 7m 45s\n- gate-grenade-dsvm-neutron-multinode-ubuntu-xenial http://logs.openstack.org/84/416584/1/check/gate-grenade-dsvm-neutron-multinode-ubuntu-xenial/f6197fb/ : SUCCESS in 38m 03s\n- gate-neutron-dsvm-functional-ubuntu-xenial-nv http://logs.openstack.org/84/416584/1/check/gate-neutron-dsvm-functional-ubuntu-xenial-nv/429f76a/ : SUCCESS in 14m 48s (non-voting)\n- gate-swift-dsvm-functional-ubuntu-xenial-nv http://logs.openstack.org/84/416584/1/check/gate-swift-dsvm-functional-ubuntu-xenial-nv/bf3a77b/ : SUCCESS in 53m 08s (non-voting)\n- gate-tempest-dsvm-full-ubuntu-trusty-mitaka-nv http://logs.openstack.org/84/416584/1/check/gate-tempest-dsvm-full-ubuntu-trusty-mitaka-nv/191e73f/ : SUCCESS in 40m 47s (non-voting)\n- gate-tempest-dsvm-neutron-full-ubuntu-trusty-mitaka-nv http://logs.openstack.org/84/416584/1/check/gate-tempest-dsvm-neutron-full-ubuntu-trusty-mitaka-nv/9b56ad0/ : SUCCESS in 51m 49s (non-voting)\n- gate-tempest-dsvm-full-ubuntu-xenial-newton-nv http://logs.openstack.org/84/416584/1/check/gate-tempest-dsvm-full-ubuntu-xenial-newton-nv/ad8a8af/ : SUCCESS in 47m 52s (non-voting)\n- gate-tempest-dsvm-neutron-full-ubuntu-xenial-newton-nv http://logs.openstack.org/84/416584/1/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial-newton-nv/ab6b106/ : SUCCESS in 54m 34s (non-voting)\n- gate-tempest-dsvm-platform-centos7-nv http://logs.openstack.org/84/416584/1/check/gate-tempest-dsvm-platform-centos7-nv/cb99460/ : FAILURE in 1h 05m 56s (non-voting)\n- gate-tempest-dsvm-multinode-full-ubuntu-xenial-nv http://logs.openstack.org/84/416584/1/check/gate-tempest-dsvm-multinode-full-ubuntu-xenial-nv/52c5ed2/ : SUCCESS in 1h 00m 54s (non-voting)\n- gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial-nv http://logs.openstack.org/84/416584/1/check/gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial-nv/94c9025/ : SUCCESS in 59m 10s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"98dc77af5bb1cc20217f817947bc3ee03854ebe7","author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"date":"2017-01-04 18:26:25.000000000","message":"Patch Set 1:\n\nAdded to ARA documentation in https://review.openstack.org/#/c/416684/","accounts_in_message":[],"_revision_number":1},{"id":"6c5c58fd5107755244183e33db55f3ec7028fa18","author":{"_account_id":7979,"name":"Mikhail S Medvedev","email":"mihailmed@gmail.com","username":"msmedved"},"date":"2017-02-17 14:42:40.000000000","message":"Patch Set 1:\n\nExample of an empty ARA report http://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/pkvmci/devstack/24/435224/4/check/tempest-dsvm-full-xenial/1aaf05f/ .","accounts_in_message":[],"_revision_number":1},{"id":"9ec7065723172da423b7360bec4a15efd5c42307","author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"date":"2017-02-22 14:56:37.000000000","message":"Patch Set 1:\n\nCan you provide more detail on where this breaks? `python` is supposed to point to `python2` according to a Python PEP. Most distros honor this and due to python2.6\u0027s age this means `python` is essentially `python2.7` everywhere.\n\nSince this change uses `python` I\u0027m not sure how this is different than the old code.","accounts_in_message":[],"_revision_number":1},{"id":"665f7eead01c89324ef67983c456e1427fb90b4f","author":{"_account_id":7979,"name":"Mikhail S Medvedev","email":"mihailmed@gmail.com","username":"msmedved"},"date":"2017-02-22 16:23:54.000000000","message":"Patch Set 1:\n\n@Clark, I do not think there is a place in the log where it shows the problem. The logs that I provided above show empty ara report as a manifestation of the problem. In my case, ARA gets installed under /tmp/ansible/lib/python3.5/site-packages/ara/.\n\n`callback_plugins` being set to use the hardcoded /tmp/ansible/lib/python2.7/site-packages/ara/ is wrong. I would argue we should not hardcode something like that, even if `python` is supposed to be aliased to `python2`.","accounts_in_message":[],"_revision_number":1},{"id":"4d23c846d0260ab4ee1545bed63f6534819fc943","author":{"_account_id":7979,"name":"Mikhail S Medvedev","email":"mihailmed@gmail.com","username":"msmedved"},"date":"2017-02-22 17:02:34.000000000","message":"Patch Set 1:\n\nTo answer the second part, this change does not use `python`, it uses `/tmp/ansible/bin/python`. There are no guarantees it would end up being python2.7. And in case when it does, yes, the code is not different.","accounts_in_message":[],"_revision_number":1},{"id":"09b5e3640748447025ed5a9f5ac51657deefade3","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-08-25 22:05:10.000000000","message":"Patch Set 1: Verified-1\n\nMerge Failed.\n\nThis change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.","accounts_in_message":[],"_revision_number":1},{"id":"096b33bcb519cbd2a760d1da2d5b8a95d1c390b8","author":{"_account_id":7979,"name":"Mikhail S Medvedev","email":"mihailmed@gmail.com","username":"msmedved"},"date":"2017-09-05 18:11:31.000000000","message":"Abandoned\n\nreview process swamp strikes again","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"549a87cd6bb05c6e09fbda362ff138e32b22d6d9","revisions":{"549a87cd6bb05c6e09fbda362ff138e32b22d6d9":{"kind":"REWORK","_number":1,"created":"2017-01-04 15:16:33.000000000","uploader":{"_account_id":7979,"name":"Mikhail S Medvedev","email":"mihailmed@gmail.com","username":"msmedved"},"ref":"refs/changes/84/416584/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-gate","ref":"refs/changes/84/416584/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/84/416584/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/84/416584/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/84/416584/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-gate refs/changes/84/416584/1"}}},"commit":{"parents":[{"commit":"8d680c44de55cd00a01968ecd453b8ac57da7ab4","subject":"pass-through USE_PYTHON3 to devstack","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/8d680c44de55cd00a01968ecd453b8ac57da7ab4"}]}],"author":{"name":"Mikhail S Medvedev","email":"mmedvede@us.ibm.com","date":"2017-01-04 14:55:33.000000000","tz":-360},"committer":{"name":"Mikhail S Medvedev","email":"mmedvede@us.ibm.com","date":"2017-01-04 14:55:33.000000000","tz":-360},"subject":"Do not assume python version for ARA callback","message":"Do not assume python version for ARA callback\n\nWe are not guaranteed to get ARA installed under a specific python\nversion. When it is not under python2.7, no data would be collected,\nand we would get an empty ARA report.\n\nThis change replaces hard coded ara module path. Note that ARA still\nmight not work quite well under python \u003e 3, but at least it would be\npossible to test it.\n\nChange-Id: I4512946fe0fd907b94e1401175e4e544fdc4c9a8\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/549a87cd6bb05c6e09fbda362ff138e32b22d6d9"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/549a87cd6bb05c6e09fbda362ff138e32b22d6d9"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}