)]}'
{"id":"openstack%2Fdevstack~226973","triplet_id":"openstack%2Fdevstack~master~I88e7d2554e8a1d6dcfea71fc1fb8e9fb2491d8b7","project":"openstack/devstack","branch":"master","topic":"bug/1499029","hashtags":[],"change_id":"I88e7d2554e8a1d6dcfea71fc1fb8e9fb2491d8b7","subject":"Propagate OVS_DATAPATH_TYPE to ml2_conf.ini","status":"MERGED","created":"2015-09-23 20:15:03.000000000","updated":"2015-09-28 21:06:19.000000000","submitted":"2015-09-28 21:06:18.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"f50e05b7cdf3f85ac911977fc9ff0c013c5bd973","_number":226973,"virtual_id_number":226973,"owner":{"_account_id":17292,"name":"Michal Ptacek","email":"michalx.ptacek@intel.com","username":"mptacekx"},"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":0,"date":"2015-09-24 17:40:31.000000000","_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},{"value":2,"date":"2015-09-28 21:06:18.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"value":0,"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"value":0,"date":"2015-09-25 18:56:48.000000000","_account_id":17292,"name":"Michal Ptacek","email":"michalx.ptacek@intel.com","username":"mptacekx"},{"value":0,"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"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":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"recommended":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"all":[{"value":1,"date":"2015-09-28 19:51:51.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},{"value":0,"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":1,"date":"2015-09-25 14:23:12.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"value":0,"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"value":0,"_account_id":17292,"name":"Michal Ptacek","email":"michalx.ptacek@intel.com","username":"mptacekx"},{"value":2,"date":"2015-09-28 00:33:21.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"value":2,"date":"2015-09-28 19:52:43.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":"","value":1,"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":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"value":0,"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"value":0,"_account_id":17292,"name":"Michal Ptacek","email":"michalx.ptacek@intel.com","username":"mptacekx"},{"value":0,"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"value":1,"date":"2015-09-28 19:52:43.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":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"value":0,"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"value":0,"_account_id":17292,"name":"Michal Ptacek","email":"michalx.ptacek@intel.com","username":"mptacekx"},{"value":0,"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"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"},{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"_account_id":17292,"name":"Michal Ptacek","email":"michalx.ptacek@intel.com","username":"mptacekx"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2015-09-23 21:43:34.000000000","updated_by":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"reviewer":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"state":"REVIEWER"},{"updated":"2015-09-24 11:22:01.000000000","updated_by":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"reviewer":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"state":"REVIEWER"},{"updated":"2015-09-24 17:40:31.000000000","updated_by":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"reviewer":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2015-09-25 14:23:12.000000000","updated_by":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"reviewer":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"state":"REVIEWER"},{"updated":"2015-09-28 00:33:21.000000000","updated_by":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"reviewer":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"state":"REVIEWER"},{"updated":"2015-09-28 19:51:51.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":"2015-09-28 19:52:43.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":"2015-09-28 21:06:18.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"2ca44e222350bb578731309a2f9df8394ccc2a4b","author":{"_account_id":17292,"name":"Michal Ptacek","email":"michalx.ptacek@intel.com","username":"mptacekx"},"date":"2015-09-23 20:15:03.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"c7ff7033949dc1c38b58b76e1029280cc1303e05","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2015-09-23 21:15:09.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/73/226973/1/12143/index.html : SUCCESS in 57 m 41 s","accounts_in_message":[],"_revision_number":1},{"id":"2026a6579ccd30941080680cab13ffb088d0e907","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-09-23 21:43:21.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-tempest-dsvm-full http://logs.openstack.org/73/226973/1/check/gate-tempest-dsvm-full/8160036/ : SUCCESS in 41m 55s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/73/226973/1/check/gate-tempest-dsvm-postgres-full/6b910ad/ : SUCCESS in 44m 20s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/73/226973/1/check/gate-tempest-dsvm-neutron-full/20cfe35/ : SUCCESS in 1h 20m 13s\n- gate-grenade-dsvm http://logs.openstack.org/73/226973/1/check/gate-grenade-dsvm/48fd49e/ : SUCCESS in 39m 33s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/73/226973/1/check/gate-tempest-dsvm-large-ops/f2f8cb2/ : SUCCESS in 34m 04s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/73/226973/1/check/gate-tempest-dsvm-neutron-large-ops/9b79fc1/ : SUCCESS in 32m 01s\n- gate-devstack-bashate http://logs.openstack.org/73/226973/1/check/gate-devstack-bashate/fa1b6b3/ : SUCCESS in 24s\n- gate-devstack-unit-tests http://logs.openstack.org/73/226973/1/check/gate-devstack-unit-tests/18f8047/ : SUCCESS in 32s\n- gate-grenade-dsvm-partial-ncpu http://logs.openstack.org/73/226973/1/check/gate-grenade-dsvm-partial-ncpu/0644d57/ : SUCCESS in 54m 36s\n- gate-tempest-dsvm-ironic-agent_ssh http://logs.openstack.org/73/226973/1/check/gate-tempest-dsvm-ironic-agent_ssh/d4607c3/ : FAILURE in 38m 01s\n- gate-devstack-dsvm-updown http://logs.openstack.org/73/226973/1/check/gate-devstack-dsvm-updown/561b02e/ : SUCCESS in 34m 40s\n- gate-tempest-dsvm-f21 http://logs.openstack.org/73/226973/1/check/gate-tempest-dsvm-f21/da65dff/ : SUCCESS in 45m 22s (non-voting)\n- gate-tempest-dsvm-centos7 http://logs.openstack.org/73/226973/1/check/gate-tempest-dsvm-centos7/5514c94/ : SUCCESS in 33m 19s (non-voting)\n- gate-devstack-publish-docs http://docs-draft.openstack.org/73/226973/1/check/gate-devstack-publish-docs/a66e8dd//doc/build/html/ : SUCCESS in 1m 40s\n- gate-swift-dsvm-functional-nv http://logs.openstack.org/73/226973/1/check/gate-swift-dsvm-functional-nv/d311201/ : SUCCESS in 45m 41s (non-voting)\n- gate-grenade-dsvm-neutron http://logs.openstack.org/73/226973/1/check/gate-grenade-dsvm-neutron/f93699d/ : SUCCESS in 1h 03m 26s\n- gate-tempest-dsvm-multinode-full http://logs.openstack.org/73/226973/1/check/gate-tempest-dsvm-multinode-full/5a4a78e/ : SUCCESS in 50m 56s (non-voting)\n- gate-tempest-dsvm-neutron-multinode-full http://logs.openstack.org/73/226973/1/check/gate-tempest-dsvm-neutron-multinode-full/3dea116/ : SUCCESS in 1h 07m 04s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"af70fa2eaa04165d7927265fca581d7cfb037445","author":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"date":"2015-09-23 21:43:34.000000000","message":"Patch Set 1:\n\nI noticed jenkins failed, I think you hit bug(s):\n\n- gate-tempest-dsvm-ironic-agent_ssh: https://bugs.launchpad.net/bugs/1393099\n\nIf you believe we\u0027ve correctly identified the failure, feel free to leave a \u0027recheck\u0027 comment to run the tests again.\nFor more details on this and other bugs, please see http://status.openstack.org/elastic-recheck/","accounts_in_message":[],"_revision_number":1},{"id":"693c5760534a8df345450681e635449071a13987","author":{"_account_id":17292,"name":"Michal Ptacek","email":"michalx.ptacek@intel.com","username":"mptacekx"},"date":"2015-09-24 07:12:51.000000000","message":"Patch Set 1:\n\nrecheck","accounts_in_message":[],"_revision_number":1},{"id":"85129f32ac0084e435c3872883480dc5a7caf5a6","author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"date":"2015-09-24 08:17:06.000000000","message":"Patch Set 1: Code-Review-1\n\nI saw this change[1]. This change expect users to use dpdk and netdev as datapath. However, neutron doesn\u0027t receive \"dpdk\". This patch and devstack also will assign datapath_type\u003d\"\" as default value[2] but the default is \"system\" in neutron.\n\n[1]: https://review.openstack.org/#/c/99447/\n[2]: https://github.com/openstack-dev/devstack/blob/master/lib/neutron_plugins/ovs_base#L11","accounts_in_message":[],"_revision_number":1},{"id":"f2273bf68fe87bbbb2d1ff5e6441e556868ce98c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-09-24 08:30:06.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-tempest-dsvm-full http://logs.openstack.org/73/226973/1/check/gate-tempest-dsvm-full/84a8417/ : SUCCESS in 42m 15s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/73/226973/1/check/gate-tempest-dsvm-postgres-full/4acbb09/ : SUCCESS in 55m 33s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/73/226973/1/check/gate-tempest-dsvm-neutron-full/8cec7e5/ : SUCCESS in 1h 15m 28s\n- gate-grenade-dsvm http://logs.openstack.org/73/226973/1/check/gate-grenade-dsvm/d833d40/ : SUCCESS in 45m 12s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/73/226973/1/check/gate-tempest-dsvm-large-ops/7bbb728/ : SUCCESS in 34m 20s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/73/226973/1/check/gate-tempest-dsvm-neutron-large-ops/7030c1e/ : SUCCESS in 34m 59s\n- gate-devstack-bashate http://logs.openstack.org/73/226973/1/check/gate-devstack-bashate/fcb79ff/ : SUCCESS in 13s\n- gate-devstack-unit-tests http://logs.openstack.org/73/226973/1/check/gate-devstack-unit-tests/b5d4283/ : SUCCESS in 17s\n- gate-grenade-dsvm-partial-ncpu http://logs.openstack.org/73/226973/1/check/gate-grenade-dsvm-partial-ncpu/d98009f/ : SUCCESS in 40m 14s\n- gate-tempest-dsvm-ironic-agent_ssh http://logs.openstack.org/73/226973/1/check/gate-tempest-dsvm-ironic-agent_ssh/72ea0cb/ : SUCCESS in 41m 06s\n- gate-devstack-dsvm-updown http://logs.openstack.org/73/226973/1/check/gate-devstack-dsvm-updown/eebd5ff/ : SUCCESS in 23m 34s\n- gate-tempest-dsvm-f21 http://logs.openstack.org/73/226973/1/check/gate-tempest-dsvm-f21/f17476e/ : SUCCESS in 46m 20s (non-voting)\n- gate-tempest-dsvm-centos7 http://logs.openstack.org/73/226973/1/check/gate-tempest-dsvm-centos7/01e2529/ : SUCCESS in 38m 32s (non-voting)\n- gate-devstack-publish-docs http://docs-draft.openstack.org/73/226973/1/check/gate-devstack-publish-docs/37782d7//doc/build/html/ : SUCCESS in 1m 51s\n- gate-swift-dsvm-functional-nv http://logs.openstack.org/73/226973/1/check/gate-swift-dsvm-functional-nv/161060b/ : SUCCESS in 34m 14s (non-voting)\n- gate-grenade-dsvm-neutron http://logs.openstack.org/73/226973/1/check/gate-grenade-dsvm-neutron/d72d362/ : SUCCESS in 40m 53s\n- gate-tempest-dsvm-multinode-full http://logs.openstack.org/73/226973/1/check/gate-tempest-dsvm-multinode-full/7482e25/ : SUCCESS in 54m 04s (non-voting)\n- gate-tempest-dsvm-neutron-multinode-full http://logs.openstack.org/73/226973/1/check/gate-tempest-dsvm-neutron-multinode-full/36bfd85/ : SUCCESS in 1h 05m 38s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"644cc2773acc5b6ef4a8508d4d3122ed8d09592f","author":{"_account_id":17292,"name":"Michal Ptacek","email":"michalx.ptacek@intel.com","username":"mptacekx"},"date":"2015-09-24 08:46:24.000000000","message":"Patch Set 1:\n\nIf OVS_DATAPATH_TYPE is not specified by user (default value is empty), handling is still the same as earlier, so no \ndatapath_type\u003d\"\" in ml2_conf.ini","accounts_in_message":[],"_revision_number":1},{"id":"f2fa733432d31f44339854c39cb78582296c08d6","author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"date":"2015-09-24 08:51:46.000000000","message":"Patch Set 1:\n\nYou\u0027re right about the point. However, a case if user sets \"dpdk\"?","accounts_in_message":[],"_revision_number":1},{"id":"2c2adcf05638e855515c2955ad5c03b181333269","author":{"_account_id":17292,"name":"Michal Ptacek","email":"michalx.ptacek@intel.com","username":"mptacekx"},"date":"2015-09-24 09:12:17.000000000","message":"Patch Set 1:\n\nYou\u0027re pointing out potential misconfiguration if one specify to use DPDK datapath not working in standard OVS agent ?\nI think this will no longer be issue, upstream agent will be used also for DPDK deployments, I hope that it\u0027s covered in following:\nhttps://review.openstack.org/#/c/197210/","accounts_in_message":[],"_revision_number":1},{"id":"3c29f545f9c715faa96fb57cfc6bd2768d2e84f2","author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"date":"2015-09-24 10:40:55.000000000","message":"Patch Set 1:\n\nI don\u0027t mean misconfiguration but I misunderstand a little. I tought that ovs-dpdk users sets \"dpdk\" to OVS_DATAPATH_TYPE. I looked at networking-ovs-dpdk tree and found it uses netdev, it also seems to use special agent.\n\nHowever, I feel strange that this patch is merged straight forward because  OVS_DATAPATH_TYPE is \"system\" as default value in neutron. Therefore, we can change the default value to \"system\" in devstack.\n\nFurthermore, can you add the comments about datapath_type to place  OVS_DATAPATH_TYPE is defined?","accounts_in_message":[],"_revision_number":1},{"id":"9d49417a2a4769490efb78b38b83d3c7e65b2395","author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"date":"2015-09-24 11:22:01.000000000","message":"Patch Set 1:\n\n@ Hirofumi Ichihara\novs-dpdk use a datapath type of netdev not dpdk.\n\nthe dpdk datapath was used by ovdk intels deprecated fork of ovs which is not integrated with openstack.\n\nthe dpdk datapath type has never been used with upstream openvswitch and is not a valid configuration option for the current OVS_DATAPATH_TYPE in devstack or the datapath_type option in the ml2_conf.ini","accounts_in_message":[],"_revision_number":1},{"id":"6b90b6d0f6a8ce760cf14873d9b01af86e779b3f","author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"date":"2015-09-24 11:56:18.000000000","message":"Patch Set 1:\n\n@sean mooney I traced history about OVS_DATAPATH_TYPE and datapath_type option and I understood already :)\n\nMy point is default value of OVS_DTAPATH_TYPE only.","accounts_in_message":[],"_revision_number":1},{"id":"da69b37b4f58571bff5d7c57f4558110a7a67344","author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"date":"2015-09-24 12:01:56.000000000","message":"Patch Set 1:\n\nhi yes i think it would be appropriate to update the default in devstack to track that of neutron.\n\n+1 on also documenting the valid options where OVS_DATAPATH_TYPE is defined in devstack.","accounts_in_message":[],"_revision_number":1},{"id":"f2edca00d383c4e2f8bf994e82f86ccc9d94e7f2","author":{"_account_id":17292,"name":"Michal Ptacek","email":"michalx.ptacek@intel.com","username":"mptacekx"},"date":"2015-09-24 14:31:48.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"90df2b3ca8c60f8586a71f67e51f11ad52d41c07","author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"date":"2015-09-24 15:01:07.000000000","message":"Patch Set 2: Code-Review-1\n\n(1 comment)\n\nThnak you for your update but I still have a nit comment.","accounts_in_message":[],"_revision_number":2},{"id":"9617345ec0e264abafe9a779e644848221212e50","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2015-09-24 15:29:55.000000000","message":"Patch Set 2:\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/73/226973/2/12200/index.html : FAILURE in 54 m 46 s","accounts_in_message":[],"_revision_number":2},{"id":"acf497d849e79ae958214ea318faf998a4308989","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-09-24 16:15:02.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-tempest-dsvm-full http://logs.openstack.org/73/226973/2/check/gate-tempest-dsvm-full/320ce66/ : SUCCESS in 1h 13m 15s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/73/226973/2/check/gate-tempest-dsvm-postgres-full/7b8ced6/ : SUCCESS in 1h 04m 08s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/73/226973/2/check/gate-tempest-dsvm-neutron-full/fb7284d/ : SUCCESS in 1h 31m 13s\n- gate-grenade-dsvm http://logs.openstack.org/73/226973/2/check/gate-grenade-dsvm/fbbe00a/ : SUCCESS in 54m 50s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/73/226973/2/check/gate-tempest-dsvm-large-ops/12194c7/ : SUCCESS in 42m 10s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/73/226973/2/check/gate-tempest-dsvm-neutron-large-ops/5924253/ : SUCCESS in 28m 43s\n- gate-devstack-bashate http://logs.openstack.org/73/226973/2/check/gate-devstack-bashate/6645f46/ : SUCCESS in 25s\n- gate-devstack-unit-tests http://logs.openstack.org/73/226973/2/check/gate-devstack-unit-tests/a58e565/ : SUCCESS in 19s\n- gate-grenade-dsvm-partial-ncpu http://logs.openstack.org/73/226973/2/check/gate-grenade-dsvm-partial-ncpu/b0ed6a3/ : SUCCESS in 1h 02m 53s\n- gate-tempest-dsvm-ironic-agent_ssh http://logs.openstack.org/73/226973/2/check/gate-tempest-dsvm-ironic-agent_ssh/c42f387/ : SUCCESS in 56m 35s\n- gate-devstack-dsvm-updown http://logs.openstack.org/73/226973/2/check/gate-devstack-dsvm-updown/7754566/ : SUCCESS in 42m 56s\n- gate-tempest-dsvm-f21 http://logs.openstack.org/73/226973/2/check/gate-tempest-dsvm-f21/513dc1b/ : SUCCESS in 46m 18s (non-voting)\n- gate-tempest-dsvm-centos7 http://logs.openstack.org/73/226973/2/check/gate-tempest-dsvm-centos7/6a7a3d6/ : SUCCESS in 41m 14s (non-voting)\n- gate-devstack-publish-docs http://docs-draft.openstack.org/73/226973/2/check/gate-devstack-publish-docs/8281501//doc/build/html/ : SUCCESS in 1m 56s\n- gate-swift-dsvm-functional-nv http://logs.openstack.org/73/226973/2/check/gate-swift-dsvm-functional-nv/53da009/ : SUCCESS in 47m 39s (non-voting)\n- gate-grenade-dsvm-neutron http://logs.openstack.org/73/226973/2/check/gate-grenade-dsvm-neutron/36f15be/ : SUCCESS in 59m 21s\n- gate-tempest-dsvm-multinode-full http://logs.openstack.org/73/226973/2/check/gate-tempest-dsvm-multinode-full/eb85680/ : SUCCESS in 55m 41s (non-voting)\n- gate-tempest-dsvm-neutron-multinode-full http://logs.openstack.org/73/226973/2/check/gate-tempest-dsvm-neutron-multinode-full/d419422/ : SUCCESS in 1h 38m 55s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"bce811b97c2ba8fff4fd28f47a5b37c44d8f4760","author":{"_account_id":17292,"name":"Michal Ptacek","email":"michalx.ptacek@intel.com","username":"mptacekx"},"date":"2015-09-24 16:44:25.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"3dc6cfeac5835b08de514912135ccaa8e67a1463","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2015-09-24 17:40:31.000000000","message":"Patch Set 3:\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/73/226973/3/12212/index.html : SUCCESS in 53 m 01 s","accounts_in_message":[],"_revision_number":3},{"id":"3730236db3597c4644dac3b7cd1a4b3669ae11cc","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-09-24 17:54:04.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-tempest-dsvm-full http://logs.openstack.org/73/226973/3/check/gate-tempest-dsvm-full/d2d573c/ : SUCCESS in 43m 58s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/73/226973/3/check/gate-tempest-dsvm-postgres-full/6316680/ : SUCCESS in 40m 46s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/73/226973/3/check/gate-tempest-dsvm-neutron-full/bec8e8a/ : SUCCESS in 52m 50s\n- gate-grenade-dsvm http://logs.openstack.org/73/226973/3/check/gate-grenade-dsvm/eb57f8d/ : SUCCESS in 43m 00s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/73/226973/3/check/gate-tempest-dsvm-large-ops/88a8f11/ : SUCCESS in 27m 24s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/73/226973/3/check/gate-tempest-dsvm-neutron-large-ops/aa30dbb/ : SUCCESS in 25m 46s\n- gate-devstack-bashate http://logs.openstack.org/73/226973/3/check/gate-devstack-bashate/0f0e54e/ : SUCCESS in 41s\n- gate-devstack-unit-tests http://logs.openstack.org/73/226973/3/check/gate-devstack-unit-tests/316ac90/ : SUCCESS in 18s\n- gate-grenade-dsvm-partial-ncpu http://logs.openstack.org/73/226973/3/check/gate-grenade-dsvm-partial-ncpu/423c3ad/ : SUCCESS in 41m 12s\n- gate-tempest-dsvm-ironic-agent_ssh http://logs.openstack.org/73/226973/3/check/gate-tempest-dsvm-ironic-agent_ssh/9ed727b/ : SUCCESS in 39m 04s\n- gate-devstack-dsvm-updown http://logs.openstack.org/73/226973/3/check/gate-devstack-dsvm-updown/f9b3268/ : SUCCESS in 26m 23s\n- gate-tempest-dsvm-f21 http://logs.openstack.org/73/226973/3/check/gate-tempest-dsvm-f21/f96014a/ : SUCCESS in 42m 38s (non-voting)\n- gate-tempest-dsvm-centos7 http://logs.openstack.org/73/226973/3/check/gate-tempest-dsvm-centos7/a2208a7/ : SUCCESS in 30m 31s (non-voting)\n- gate-devstack-publish-docs http://docs-draft.openstack.org/73/226973/3/check/gate-devstack-publish-docs/f0d806e//doc/build/html/ : SUCCESS in 1m 46s\n- gate-swift-dsvm-functional-nv http://logs.openstack.org/73/226973/3/check/gate-swift-dsvm-functional-nv/f7c57c4/ : SUCCESS in 26m 53s (non-voting)\n- gate-grenade-dsvm-neutron http://logs.openstack.org/73/226973/3/check/gate-grenade-dsvm-neutron/7b543d3/ : SUCCESS in 39m 35s\n- gate-tempest-dsvm-multinode-full http://logs.openstack.org/73/226973/3/check/gate-tempest-dsvm-multinode-full/6ad8a4c/ : SUCCESS in 50m 21s (non-voting)\n- gate-tempest-dsvm-neutron-multinode-full http://logs.openstack.org/73/226973/3/check/gate-tempest-dsvm-neutron-multinode-full/a19cdfa/ : SUCCESS in 1h 00m 19s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"c135afe2d2f5d6ab279db9b3b6e5abb3d039e9c0","author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"date":"2015-09-25 14:23:12.000000000","message":"Patch Set 3: Code-Review+1\n\nLooks good.","accounts_in_message":[],"_revision_number":3},{"id":"88964d3e6e4c493b3820c53dd40863387c3717fb","author":{"_account_id":17292,"name":"Michal Ptacek","email":"michalx.ptacek@intel.com","username":"mptacekx"},"date":"2015-09-25 18:56:48.000000000","message":"Patch Set 3:\n\n@Hirofumi: thanks for your valuable comments","accounts_in_message":[],"_revision_number":3},{"id":"32806b76fdf9d358bca6c2eb81385f058a6c87cb","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2015-09-28 00:33:21.000000000","message":"Patch Set 3: Code-Review+2","accounts_in_message":[],"_revision_number":3},{"id":"565a646bf73eb4a9bfb3d1b9d12ce619da99cbf3","author":{"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},"date":"2015-09-28 19:51:51.000000000","message":"Patch Set 3: Code-Review+1\n\nAt least now we have a comment for what acceptable values there are for this option.","accounts_in_message":[],"_revision_number":3},{"id":"40708452849ea681d6e91766218bdc4fd1c50436","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2015-09-28 19:52:43.000000000","message":"Patch Set 3: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":3},{"id":"151592e3435ba98f1d02d3088c47e10adbb80681","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-09-28 19:52:48.000000000","message":"Patch Set 3: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":3},{"id":"c2998b56a7282408fa1e7deeb48e4adbd163eda6","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-09-28 21:06:18.000000000","message":"Patch Set 3: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-tempest-dsvm-full http://logs.openstack.org/73/226973/3/gate/gate-tempest-dsvm-full/d5c1ed3/ : SUCCESS in 42m 18s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/73/226973/3/gate/gate-tempest-dsvm-postgres-full/48a8639/ : SUCCESS in 49m 02s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/73/226973/3/gate/gate-tempest-dsvm-neutron-full/d62668a/ : SUCCESS in 46m 14s\n- gate-grenade-dsvm http://logs.openstack.org/73/226973/3/gate/gate-grenade-dsvm/c936d6c/ : SUCCESS in 45m 06s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/73/226973/3/gate/gate-tempest-dsvm-large-ops/637e688/ : SUCCESS in 33m 01s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/73/226973/3/gate/gate-tempest-dsvm-neutron-large-ops/d4d806c/ : SUCCESS in 27m 01s\n- gate-devstack-bashate http://logs.openstack.org/73/226973/3/gate/gate-devstack-bashate/4ddb773/ : SUCCESS in 15s\n- gate-grenade-dsvm-partial-ncpu http://logs.openstack.org/73/226973/3/gate/gate-grenade-dsvm-partial-ncpu/29986dc/ : SUCCESS in 41m 39s","accounts_in_message":[],"_revision_number":3},{"id":"dffa3adf04a9aa4063e3b25356830e2322142e10","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-09-28 21:06:19.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"c1605550d94736f5698d3f6ea7bc1e0b0914cb4e","revisions":{"d15703c5e8622038c407487cb8226f872639c891":{"kind":"REWORK","_number":1,"created":"2015-09-23 20:15:03.000000000","uploader":{"_account_id":17292,"name":"Michal Ptacek","email":"michalx.ptacek@intel.com","username":"mptacekx"},"ref":"refs/changes/73/226973/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/73/226973/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/73/226973/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/73/226973/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/73/226973/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/73/226973/1"}}},"commit":{"parents":[{"commit":"b6249dc2ad630ecb9a231d0ce65d0f14f2116598","subject":"Merge \"Add manila to devstack plugin registry\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b6249dc2ad630ecb9a231d0ce65d0f14f2116598"}]}],"author":{"name":"Michal Ptacek","email":"michalx.ptacek@intel.com","date":"2015-09-23 20:02:02.000000000","tz":60},"committer":{"name":"Michal Ptacek","email":"michalx.ptacek@intel.com","date":"2015-09-23 20:15:24.000000000","tz":60},"subject":"Propagate OVS_DATAPATH_TYPE to ml2_conf.ini","message":"Propagate OVS_DATAPATH_TYPE to ml2_conf.ini\n\nIf OVS_DATAPATH_TYPE is configured it should be visible in ML2 config\n\nCloses-Bug: 1499029\nChange-Id: I88e7d2554e8a1d6dcfea71fc1fb8e9fb2491d8b7\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/d15703c5e8622038c407487cb8226f872639c891"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/d15703c5e8622038c407487cb8226f872639c891"}]},"branch":"refs/heads/master"},"282d9cfa4a226d13532c81faf251f0eda7263c1b":{"kind":"REWORK","_number":2,"created":"2015-09-24 14:31:48.000000000","uploader":{"_account_id":17292,"name":"Michal Ptacek","email":"michalx.ptacek@intel.com","username":"mptacekx"},"ref":"refs/changes/73/226973/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/73/226973/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/73/226973/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/73/226973/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/73/226973/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/73/226973/2"}}},"commit":{"parents":[{"commit":"b6249dc2ad630ecb9a231d0ce65d0f14f2116598","subject":"Merge \"Add manila to devstack plugin registry\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b6249dc2ad630ecb9a231d0ce65d0f14f2116598"}]}],"author":{"name":"Michal Ptacek","email":"michalx.ptacek@intel.com","date":"2015-09-23 20:02:02.000000000","tz":60},"committer":{"name":"Michal Ptacek","email":"michalx.ptacek@intel.com","date":"2015-09-24 14:31:02.000000000","tz":60},"subject":"Propagate OVS_DATAPATH_TYPE to ml2_conf.ini","message":"Propagate OVS_DATAPATH_TYPE to ml2_conf.ini\n\nIf OVS_DATAPATH_TYPE is configured it should be visible in ML2 config\n\nChanging OVS_DATAPATH_TYPE default value to \u0027system\u0027 from \u0027\u0027\n\nCloses-Bug: 1499029\nChange-Id: I88e7d2554e8a1d6dcfea71fc1fb8e9fb2491d8b7\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/282d9cfa4a226d13532c81faf251f0eda7263c1b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/282d9cfa4a226d13532c81faf251f0eda7263c1b"}]},"branch":"refs/heads/master"},"c1605550d94736f5698d3f6ea7bc1e0b0914cb4e":{"kind":"REWORK","_number":3,"created":"2015-09-24 16:44:25.000000000","uploader":{"_account_id":17292,"name":"Michal Ptacek","email":"michalx.ptacek@intel.com","username":"mptacekx"},"ref":"refs/changes/73/226973/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/73/226973/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/73/226973/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/73/226973/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/73/226973/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/73/226973/3"}}},"commit":{"parents":[{"commit":"b6249dc2ad630ecb9a231d0ce65d0f14f2116598","subject":"Merge \"Add manila to devstack plugin registry\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b6249dc2ad630ecb9a231d0ce65d0f14f2116598"}]}],"author":{"name":"Michal Ptacek","email":"michalx.ptacek@intel.com","date":"2015-09-23 20:02:02.000000000","tz":60},"committer":{"name":"Michal Ptacek","email":"michalx.ptacek@intel.com","date":"2015-09-24 16:45:08.000000000","tz":60},"subject":"Propagate OVS_DATAPATH_TYPE to ml2_conf.ini","message":"Propagate OVS_DATAPATH_TYPE to ml2_conf.ini\n\nIf OVS_DATAPATH_TYPE is configured it should be visible in ML2 config\n\nChanging OVS_DATAPATH_TYPE default value to \u0027system\u0027 from \u0027\u0027\n\nCloses-Bug: 1499029\nChange-Id: I88e7d2554e8a1d6dcfea71fc1fb8e9fb2491d8b7\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/c1605550d94736f5698d3f6ea7bc1e0b0914cb4e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/c1605550d94736f5698d3f6ea7bc1e0b0914cb4e"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}