)]}'
{"id":"openstack%2Fdevstack~324570","triplet_id":"openstack%2Fdevstack~master~I352362cf59e492fa9f7725190f0243f2436ac347","project":"openstack/devstack","branch":"master","topic":"vercmp","hashtags":[],"change_id":"I352362cf59e492fa9f7725190f0243f2436ac347","subject":"Change ovs_base neutron plugin to use vercmp","status":"MERGED","created":"2016-06-02 14:35:35.000000000","updated":"2016-06-09 14:22:18.000000000","submitted":"2016-06-09 14:22:18.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"d757bd08a8be4d59a1f718d6123256cb90369a69","_number":324570,"virtual_id_number":324570,"owner":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},{"value":2,"date":"2016-06-09 14:22:18.000000000","post_submit":true,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},"all":[{"value":2,"date":"2016-06-03 15:05:09.000000000","_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2016-06-09 13:13:11.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"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":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"all":[{"value":0,"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2016-06-09 13:13:11.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-06-03 15:05:09.000000000","updated_by":{"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},"reviewer":{"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},"state":"REVIEWER"},{"updated":"2016-06-09 13:13:11.000000000","updated_by":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"reviewer":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"state":"REVIEWER"},{"updated":"2016-06-09 14:22:18.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"45442532962e1e346a0362cc8d5f1d718b2019a0","author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"date":"2016-06-02 14:35:35.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"03e037597658d184e03d9340d917171d9b3c5e76","author":{"_account_id":16376,"name":"Intel NFV CI","email":"openstack-nfv-ci@intel.com","username":"intel-nfv-ci","tags":["SERVICE_USER"]},"date":"2016-06-02 16:17:02.000000000","message":"Patch Set 1:\n\nBuild succeeded (check pipeline).\n\n- tempest-dsvm-ovsdpdk-odl http://intel-openstack-ci-logs.ovh/70/324570/1/check/tempest-dsvm-ovsdpdk-odl/2be391a : FAILURE in 1h 03m 50s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"86ba1b5089296df2366800591ffd37fbd97c90a3","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2016-06-02 16:25:36.000000000","message":"Patch Set 1:\n\nTesting completed on Citrix Xenserver. For rechecking only on the Citrix Xenserver CI,add a review comment with \"xenserver: recheck\". Contact info: openstack@citrix.com.\nFor debugging suggestions, see https://wiki.openstack.org/wiki/Debugging_XenServer_CI_failures\n\n- check-citrix-xenserver http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/70/324570/1/17353/index.html : SUCCESS in 01 h 20 m 00 s","accounts_in_message":[],"_revision_number":1},{"id":"18314803e2e4cd1c6591b9b7487be555acdb0ffe","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-06-02 17:09:23.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-tempest-dsvm-full http://logs.openstack.org/70/324570/1/check/gate-tempest-dsvm-full/a592604/ : SUCCESS in 1h 12m 25s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/70/324570/1/check/gate-tempest-dsvm-postgres-full/8e31d92/ : SUCCESS in 50m 33s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/70/324570/1/check/gate-tempest-dsvm-neutron-full/9e4c84a/ : SUCCESS in 58m 51s\n- gate-grenade-dsvm http://logs.openstack.org/70/324570/1/check/gate-grenade-dsvm/14c3dc8/ : SUCCESS in 23m 39s\n- gate-devstack-docs http://docs-draft.openstack.org/70/324570/1/check/gate-devstack-docs/d065ea4//doc/build/html/ : SUCCESS in 7m 09s\n- gate-devstack-bashate http://logs.openstack.org/70/324570/1/check/gate-devstack-bashate/f1279c1/ : SUCCESS in 2m 55s\n- gate-devstack-unit-tests http://logs.openstack.org/70/324570/1/check/gate-devstack-unit-tests/68a03a5/ : SUCCESS in 2m 47s\n- gate-tempest-dsvm-ironic-agent_ssh-nv http://logs.openstack.org/70/324570/1/check/gate-tempest-dsvm-ironic-agent_ssh-nv/49a7f77/ : SUCCESS in 40m 59s (non-voting)\n- gate-devstack-dsvm-updown http://logs.openstack.org/70/324570/1/check/gate-devstack-dsvm-updown/e40d27f/ : SUCCESS in 23m 36s\n- gate-tempest-dsvm-neutron-identity-v3-only-full-nv http://logs.openstack.org/70/324570/1/check/gate-tempest-dsvm-neutron-identity-v3-only-full-nv/e20807b/ : SUCCESS in 56m 59s (non-voting)\n- gate-swift-dsvm-functional-nv http://logs.openstack.org/70/324570/1/check/gate-swift-dsvm-functional-nv/c34869c/ : SUCCESS in 24m 41s (non-voting)\n- gate-grenade-dsvm-neutron http://logs.openstack.org/70/324570/1/check/gate-grenade-dsvm-neutron/75da4a7/ : SUCCESS in 56m 56s\n- gate-tempest-dsvm-multinode-full http://logs.openstack.org/70/324570/1/check/gate-tempest-dsvm-multinode-full/78381a2/ : SUCCESS in 1h 10m 01s (non-voting)\n- gate-tempest-dsvm-neutron-multinode-full http://logs.openstack.org/70/324570/1/check/gate-tempest-dsvm-neutron-multinode-full/6ddd6e5/ : SUCCESS in 1h 30m 36s (non-voting)\n- gate-grenade-dsvm-multinode http://logs.openstack.org/70/324570/1/check/gate-grenade-dsvm-multinode/8688ec3/ : SUCCESS in 52m 28s\n- gate-grenade-dsvm-neutron-multinode http://logs.openstack.org/70/324570/1/check/gate-grenade-dsvm-neutron-multinode/e515627/ : SUCCESS in 54m 00s\n- gate-tempest-dsvm-platform-centos7-nv http://logs.openstack.org/70/324570/1/check/gate-tempest-dsvm-platform-centos7-nv/cd4cd04/ : SUCCESS in 47m 20s (non-voting)\n- gate-tempest-dsvm-platform-fedora23-nv http://logs.openstack.org/70/324570/1/check/gate-tempest-dsvm-platform-fedora23-nv/1f153c3/ : SUCCESS in 56m 32s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"dbf878b78d292978d9e5b6ea67fa4c33ebade0e5","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2016-06-02 18:23:22.000000000","message":"Patch Set 1:\n\nTesting completed on IBM PowerKVM platform. For rechecking only on the IBM PowerKVM CI, add a review comment with pkvm-recheck. Contact info: kvmpower@linux.vnet.ibm.com. For more information, see https://wiki.openstack.org/wiki/PowerKVM\n\n- check-ibm-tempest-dsvm-full http://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/pkvmci/devstack/70/324570/1/check/check-ibm-tempest-dsvm-full/8f99f3a/ : SUCCESS in 1h 08m 46s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"78f80393b29cd3372cc6cc0638f305f8b9d24eec","author":{"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},"date":"2016-06-03 15:05:09.000000000","message":"Patch Set 1: Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"21ebd667f38033c50a167c8a0055bcd13c178eba","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2016-06-09 13:13:11.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":1},{"id":"8c1fc149badfac6a74532dbbf1eb3deecd96ba1a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-06-09 13:13:20.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":1},{"id":"1eccdda397168fe01453e2c30ad1bd8b4e63b5b3","date":"2016-06-09 14:22:18.000000000","message":"Change has been successfully merged into the git repository by Jenkins","accounts_in_message":[],"_revision_number":1},{"id":"1cf34013b7300be528629d5f9c0d3b5423e2c9b3","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-06-09 14:22:18.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-tempest-dsvm-full http://logs.openstack.org/70/324570/1/gate/gate-tempest-dsvm-full/914abec/ : SUCCESS in 37m 18s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/70/324570/1/gate/gate-tempest-dsvm-postgres-full/9fc1ac1/ : SUCCESS in 39m 33s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/70/324570/1/gate/gate-tempest-dsvm-neutron-full/a8b6a84/ : SUCCESS in 58m 48s\n- gate-grenade-dsvm http://logs.openstack.org/70/324570/1/gate/gate-grenade-dsvm/6b1e434/ : SUCCESS in 25m 43s\n- gate-devstack-docs http://docs-draft.openstack.org/70/324570/1/gate/gate-devstack-docs/130d48c//doc/build/html/ : SUCCESS in 1m 20s\n- gate-devstack-bashate http://logs.openstack.org/70/324570/1/gate/gate-devstack-bashate/06f0804/ : SUCCESS in 1m 11s\n- gate-grenade-dsvm-multinode http://logs.openstack.org/70/324570/1/gate/gate-grenade-dsvm-multinode/fc47e57/ : SUCCESS in 50m 51s\n- gate-grenade-dsvm-neutron-multinode http://logs.openstack.org/70/324570/1/gate/gate-grenade-dsvm-neutron-multinode/c5c3a28/ : SUCCESS in 59m 32s","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"09604349f89f59ee2ac07b4daa93bd1ae52d1b79","revisions":{"09604349f89f59ee2ac07b4daa93bd1ae52d1b79":{"kind":"REWORK","_number":1,"created":"2016-06-02 14:35:35.000000000","uploader":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"ref":"refs/changes/70/324570/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/70/324570/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/70/324570/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/70/324570/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/70/324570/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/70/324570/1"}}},"commit":{"parents":[{"commit":"feb828b7aadd65b6bd584a1510a1b80dbe49a332","subject":"Merge \"Remove verbose setting from l3 service plugin\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/feb828b7aadd65b6bd584a1510a1b80dbe49a332"}]}],"author":{"name":"Brian Haley","email":"brian.haley@hpe.com","date":"2016-06-02 14:29:43.000000000","tz":-240},"committer":{"name":"Brian Haley","email":"brian.haley@hpe.com","date":"2016-06-02 14:33:54.000000000","tz":-240},"subject":"Change ovs_base neutron plugin to use vercmp","message":"Change ovs_base neutron plugin to use vercmp\n\nThis plugin was using a deprecated function, vercmp_numbers(),\nthat wasn\u0027t actually working properly because the call to\n\u0027deprecated\u0027 at the beginning was causing garbage to be\nreturned to the caller.  For example, this was always in\nstack.sh.log when using OVS:\n\n.../lib/neutron_plugins/ovs_base: line 57: [: too many arguments\n\nUpdate to use vercmp() like all other users in devstack, and\nremove all the old code.\n\nChange-Id: I352362cf59e492fa9f7725190f0243f2436ac347\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/09604349f89f59ee2ac07b4daa93bd1ae52d1b79"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/09604349f89f59ee2ac07b4daa93bd1ae52d1b79"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}