)]}'
{"id":"openstack%2Fdevstack~212786","triplet_id":"openstack%2Fdevstack~master~I2fbe8b4ce5a74164a4770a58a7636b9bb3dc7d3b","project":"openstack/devstack","branch":"master","hashtags":[],"change_id":"I2fbe8b4ce5a74164a4770a58a7636b9bb3dc7d3b","subject":"Provide alternate path for neutron plugin config files","status":"ABANDONED","created":"2015-08-13 21:01:32.000000000","updated":"2015-09-10 19:04:28.000000000","total_comment_count":47,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"a8793c025f707218b20bd17470b02f769c31689d","_number":212786,"virtual_id_number":212786,"owner":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":1,"date":"2015-09-08 16:50:42.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},{"_account_id":10980,"name":"Doug Wiegley","email":"dougwig@parkside.io","username":"dougw"},{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},{"date":"2015-09-08 19:01:03.000000000","_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},{"_account_id":7018,"name":"Abhishek Raut","email":"rauta@vmware.com","username":"abhiraut"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"date":"2015-09-09 02:06:43.000000000","_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"date":"2015-09-08 16:03:32.000000000","_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"_account_id":14257,"name":"Mahito Ogura","email":"m.ogura@ntt.com","username":"Mahito"},{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"_account_id":16413,"name":"Jiangjunyu","email":"double.chiang@gmail.com"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":13438,"name":"Kobi Samoray","email":"kobi.samoray@broadcom.com","username":"ksamoray"}],"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":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":10980,"name":"Doug Wiegley","email":"dougwig@parkside.io","username":"dougw"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},{"value":-1,"date":"2015-09-08 18:32:13.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":7018,"name":"Abhishek Raut","email":"rauta@vmware.com","username":"abhiraut"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":-1,"date":"2015-09-09 05:23:49.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":14257,"name":"Mahito Ogura","email":"m.ogura@ntt.com","username":"Mahito"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":16413,"name":"Jiangjunyu","email":"double.chiang@gmail.com"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":13438,"name":"Kobi Samoray","email":"kobi.samoray@broadcom.com","username":"ksamoray"}],"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":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},{"_account_id":10980,"name":"Doug Wiegley","email":"dougwig@parkside.io","username":"dougw"},{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},{"_account_id":7018,"name":"Abhishek Raut","email":"rauta@vmware.com","username":"abhiraut"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"_account_id":14257,"name":"Mahito Ogura","email":"m.ogura@ntt.com","username":"Mahito"},{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"_account_id":16413,"name":"Jiangjunyu","email":"double.chiang@gmail.com"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":13438,"name":"Kobi Samoray","email":"kobi.samoray@broadcom.com","username":"ksamoray"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":10980,"name":"Doug Wiegley","email":"dougwig@parkside.io","username":"dougw"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":7018,"name":"Abhishek Raut","email":"rauta@vmware.com","username":"abhiraut"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},{"value":0,"permitted_voting_range":{"min":-1,"max":2},"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":14257,"name":"Mahito Ogura","email":"m.ogura@ntt.com","username":"Mahito"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":16413,"name":"Jiangjunyu","email":"double.chiang@gmail.com"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":13438,"name":"Kobi Samoray","email":"kobi.samoray@broadcom.com","username":"ksamoray"}],"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":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":7018,"name":"Abhishek Raut","email":"rauta@vmware.com","username":"abhiraut"},{"_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":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"_account_id":10980,"name":"Doug Wiegley","email":"dougwig@parkside.io","username":"dougw"},{"_account_id":13438,"name":"Kobi Samoray","email":"kobi.samoray@broadcom.com","username":"ksamoray"},{"_account_id":14257,"name":"Mahito Ogura","email":"m.ogura@ntt.com","username":"Mahito"},{"_account_id":16413,"name":"Jiangjunyu","email":"double.chiang@gmail.com"},{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2015-08-14 00:17:12.000000000","updated_by":{"_account_id":14257,"name":"Mahito Ogura","email":"m.ogura@ntt.com","username":"Mahito"},"reviewer":{"_account_id":14257,"name":"Mahito Ogura","email":"m.ogura@ntt.com","username":"Mahito"},"state":"REVIEWER"},{"updated":"2015-08-24 16:49:07.000000000","updated_by":{"_account_id":13438,"name":"Kobi Samoray","email":"kobi.samoray@broadcom.com","username":"ksamoray"},"reviewer":{"_account_id":13438,"name":"Kobi Samoray","email":"kobi.samoray@broadcom.com","username":"ksamoray"},"state":"REVIEWER"},{"updated":"2015-08-30 05:44:08.000000000","updated_by":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"reviewer":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"state":"REVIEWER"},{"updated":"2015-09-01 08:12:45.000000000","updated_by":{"_account_id":16413,"name":"Jiangjunyu","email":"double.chiang@gmail.com"},"reviewer":{"_account_id":16413,"name":"Jiangjunyu","email":"double.chiang@gmail.com"},"state":"REVIEWER"},{"updated":"2015-09-01 08:13:35.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-02 10:00:40.000000000","updated_by":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"reviewer":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"state":"REVIEWER"},{"updated":"2015-09-02 14:18:49.000000000","updated_by":{"_account_id":10980,"name":"Doug Wiegley","email":"dougwig@parkside.io","username":"dougw"},"reviewer":{"_account_id":10980,"name":"Doug Wiegley","email":"dougwig@parkside.io","username":"dougw"},"state":"REVIEWER"},{"updated":"2015-09-02 17:24:54.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-02 21:46:34.000000000","updated_by":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"reviewer":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"state":"REVIEWER"},{"updated":"2015-09-02 22:30:23.000000000","updated_by":{"_account_id":7018,"name":"Abhishek Raut","email":"rauta@vmware.com","username":"abhiraut"},"reviewer":{"_account_id":7018,"name":"Abhishek Raut","email":"rauta@vmware.com","username":"abhiraut"},"state":"REVIEWER"},{"updated":"2015-09-08 03:06:31.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-08 16:03:32.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-08 16:50:42.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2015-09-08 18:32:13.000000000","updated_by":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"reviewer":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"state":"REVIEWER"},{"updated":"2015-09-09 02:06:43.000000000","updated_by":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"reviewer":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"state":"REVIEWER"}],"messages":[{"id":"260bfbc8b9b055b602d98fd5c470437515ccb4aa","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-08-13 21:01:32.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"2fb9d5ecc14f89f4defbc7b9c469e1d95c71d410","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-08-13 21:11:49.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"eb8a8a2b564dab1533bf00855e0e120320239d5c","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-08-13 21:36:30.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"70375584ef7ae8badd5168f51855d20109008808","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-08-13 22:41:48.000000000","message":"Patch Set 3: 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/86/212786/3/check/gate-tempest-dsvm-full/01adc0f/ : SUCCESS in 46m 55s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/86/212786/3/check/gate-tempest-dsvm-postgres-full/52c6c64/ : SUCCESS in 58m 33s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/86/212786/3/check/gate-tempest-dsvm-neutron-full/ee50ab3/ : FAILURE in 20m 49s\n- gate-grenade-dsvm http://logs.openstack.org/86/212786/3/check/gate-grenade-dsvm/bd14eaf/ : SUCCESS in 40m 03s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/86/212786/3/check/gate-tempest-dsvm-large-ops/f10a770/ : SUCCESS in 32m 26s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/86/212786/3/check/gate-tempest-dsvm-neutron-large-ops/bb2b24f/ : FAILURE in 17m 30s\n- gate-devstack-bashate http://logs.openstack.org/86/212786/3/check/gate-devstack-bashate/40771ef/ : FAILURE in 14s\n- gate-devstack-unit-tests http://logs.openstack.org/86/212786/3/check/gate-devstack-unit-tests/f5fe3c2/ : SUCCESS in 29s\n- gate-devstack-dsvm-cells http://logs.openstack.org/86/212786/3/check/gate-devstack-dsvm-cells/6009606/ : SUCCESS in 25m 11s\n- gate-grenade-dsvm-partial-ncpu http://logs.openstack.org/86/212786/3/check/gate-grenade-dsvm-partial-ncpu/12ecc90/ : SUCCESS in 51m 59s\n- gate-tempest-dsvm-ironic-pxe_ssh http://logs.openstack.org/86/212786/3/check/gate-tempest-dsvm-ironic-pxe_ssh/90c4cc9/ : FAILURE in 26m 42s\n- gate-devstack-dsvm-updown http://logs.openstack.org/86/212786/3/check/gate-devstack-dsvm-updown/78ea15d/ : SUCCESS in 27m 26s\n- gate-tempest-dsvm-f21 http://logs.openstack.org/86/212786/3/check/gate-tempest-dsvm-f21/f7e0849/ : FAILURE in 31m 06s (non-voting)\n- gate-tempest-dsvm-centos7 http://logs.openstack.org/86/212786/3/check/gate-tempest-dsvm-centos7/687ae63/ : SUCCESS in 32m 57s (non-voting)\n- gate-devstack-publish-docs http://docs-draft.openstack.org/86/212786/3/check/gate-devstack-publish-docs/af97161//doc/build/html/ : SUCCESS in 1m 38s\n- gate-swift-dsvm-functional-nv http://logs.openstack.org/86/212786/3/check/gate-swift-dsvm-functional-nv/0dacb04/ : SUCCESS in 30m 43s (non-voting)\n- gate-grenade-dsvm-neutron http://logs.openstack.org/86/212786/3/check/gate-grenade-dsvm-neutron/d85e0c8/ : SUCCESS in 58m 04s\n- gate-tempest-dsvm-multinode-full http://logs.openstack.org/86/212786/3/check/gate-tempest-dsvm-multinode-full/9fca7c9/ : SUCCESS in 53m 59s (non-voting)\n- gate-tempest-dsvm-neutron-multinode-full http://logs.openstack.org/86/212786/3/check/gate-tempest-dsvm-neutron-multinode-full/92b9fe0/ : FAILURE in 19m 33s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"d65d66d037a29515b7bf767d596dcc5c4d96e0f4","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-08-13 23:54:06.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"d8e836b7bb663f1093ed31e0ba7bd5062f2ce56f","author":{"_account_id":14257,"name":"Mahito Ogura","email":"m.ogura@ntt.com","username":"Mahito"},"date":"2015-08-14 00:17:12.000000000","message":"Patch Set 4:\n\n(1 comment)","accounts_in_message":[],"_revision_number":4},{"id":"3166b2d22c39e40ea6ca57cb1fdcd31b33a11691","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-08-14 00:30:46.000000000","message":"Patch Set 4:\n\n(1 comment)","accounts_in_message":[],"_revision_number":4},{"id":"8a106b9a0ce123e98252c5459a8485306b97fb0b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-08-14 00:59:45.000000000","message":"Patch Set 4: 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/86/212786/4/check/gate-tempest-dsvm-full/7457759/ : SUCCESS in 58m 30s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/86/212786/4/check/gate-tempest-dsvm-postgres-full/f624b31/ : SUCCESS in 55m 34s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/86/212786/4/check/gate-tempest-dsvm-neutron-full/7a8bd4a/ : FAILURE in 26m 56s\n- gate-grenade-dsvm http://logs.openstack.org/86/212786/4/check/gate-grenade-dsvm/5369edf/ : SUCCESS in 46m 52s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/86/212786/4/check/gate-tempest-dsvm-large-ops/ef656eb/ : SUCCESS in 33m 43s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/86/212786/4/check/gate-tempest-dsvm-neutron-large-ops/1c73629/ : FAILURE in 26m 06s\n- gate-devstack-bashate http://logs.openstack.org/86/212786/4/check/gate-devstack-bashate/7e175a9/ : SUCCESS in 26s\n- gate-devstack-unit-tests http://logs.openstack.org/86/212786/4/check/gate-devstack-unit-tests/e5b43a9/ : SUCCESS in 1m 18s\n- gate-devstack-dsvm-cells http://logs.openstack.org/86/212786/4/check/gate-devstack-dsvm-cells/bb823f7/ : SUCCESS in 30m 11s\n- gate-grenade-dsvm-partial-ncpu http://logs.openstack.org/86/212786/4/check/gate-grenade-dsvm-partial-ncpu/1e3c54b/ : SUCCESS in 49m 06s\n- gate-tempest-dsvm-ironic-pxe_ssh http://logs.openstack.org/86/212786/4/check/gate-tempest-dsvm-ironic-pxe_ssh/9cba03d/ : FAILURE in 30m 00s\n- gate-devstack-dsvm-updown http://logs.openstack.org/86/212786/4/check/gate-devstack-dsvm-updown/d6df91a/ : SUCCESS in 25m 25s\n- gate-tempest-dsvm-f21 http://logs.openstack.org/86/212786/4/check/gate-tempest-dsvm-f21/70bbb49/ : FAILURE in 32m 40s (non-voting)\n- gate-tempest-dsvm-centos7 http://logs.openstack.org/86/212786/4/check/gate-tempest-dsvm-centos7/514f5c3/ : SUCCESS in 35m 55s (non-voting)\n- gate-devstack-publish-docs http://docs-draft.openstack.org/86/212786/4/check/gate-devstack-publish-docs/4b18132//doc/build/html/ : SUCCESS in 1m 25s\n- gate-swift-dsvm-functional-nv http://logs.openstack.org/86/212786/4/check/gate-swift-dsvm-functional-nv/57733d9/ : SUCCESS in 39m 20s (non-voting)\n- gate-grenade-dsvm-neutron http://logs.openstack.org/86/212786/4/check/gate-grenade-dsvm-neutron/feb1b48/ : SUCCESS in 53m 42s\n- gate-tempest-dsvm-multinode-full http://logs.openstack.org/86/212786/4/check/gate-tempest-dsvm-multinode-full/0c14b22/ : SUCCESS in 53m 10s (non-voting)\n- gate-tempest-dsvm-neutron-multinode-full http://logs.openstack.org/86/212786/4/check/gate-tempest-dsvm-neutron-multinode-full/1c18d8a/ : FAILURE in 20m 08s (non-voting)","accounts_in_message":[],"_revision_number":4},{"id":"f1231b427589f9b36431b37e60daee1d46142d5e","author":{"_account_id":14257,"name":"Mahito Ogura","email":"m.ogura@ntt.com","username":"Mahito"},"date":"2015-08-14 01:20:35.000000000","message":"Patch Set 4: Code-Review-1\n\n(2 comments)","accounts_in_message":[],"_revision_number":4},{"id":"3a29aa01e9f2983f7cb6d9056bc3fd062e847667","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-08-14 01:33:20.000000000","message":"Patch Set 4:\n\n(1 comment)","accounts_in_message":[],"_revision_number":4},{"id":"80e500789a3ad1337c654cba1bb6ca10916fdb1e","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-08-14 01:34:50.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"4a1d1679357a0ef28c9b0cc19a5bcca1dec4f2b7","author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"date":"2015-08-14 02:19:47.000000000","message":"Patch Set 5: Code-Review-1\n\n(3 comments)\n\nThis issue is very important for neutron.\n\nAnyway, I know already there is a patch[1] includes this proposal.\n\n[1]: https://review.openstack.org/#/c/211598/","accounts_in_message":[],"_revision_number":5},{"id":"c697ace1406117fcc8bc1b65eaefe89d7d156139","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-08-14 02:47:53.000000000","message":"Patch Set 5:\n\n(3 comments)","accounts_in_message":[],"_revision_number":5},{"id":"54d0615ec4bcbcde71eafa13c743a5d8dcad975e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-08-14 02:51:21.000000000","message":"Patch Set 5: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-tempest-dsvm-full http://logs.openstack.org/86/212786/5/check/gate-tempest-dsvm-full/438a089/ : SUCCESS in 54m 03s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/86/212786/5/check/gate-tempest-dsvm-postgres-full/3d4cbf0/ : SUCCESS in 54m 11s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/86/212786/5/check/gate-tempest-dsvm-neutron-full/56345e1/ : SUCCESS in 1h 04m 55s\n- gate-grenade-dsvm http://logs.openstack.org/86/212786/5/check/gate-grenade-dsvm/815fa52/ : SUCCESS in 47m 36s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/86/212786/5/check/gate-tempest-dsvm-large-ops/3ce13c6/ : SUCCESS in 28m 50s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/86/212786/5/check/gate-tempest-dsvm-neutron-large-ops/a31a1ff/ : SUCCESS in 36m 10s\n- gate-devstack-bashate http://logs.openstack.org/86/212786/5/check/gate-devstack-bashate/7817b42/ : SUCCESS in 1m 25s\n- gate-devstack-unit-tests http://logs.openstack.org/86/212786/5/check/gate-devstack-unit-tests/dd66767/ : SUCCESS in 33s\n- gate-devstack-dsvm-cells http://logs.openstack.org/86/212786/5/check/gate-devstack-dsvm-cells/35e839b/ : SUCCESS in 27m 54s\n- gate-grenade-dsvm-partial-ncpu http://logs.openstack.org/86/212786/5/check/gate-grenade-dsvm-partial-ncpu/21c1992/ : SUCCESS in 59m 06s\n- gate-tempest-dsvm-ironic-pxe_ssh http://logs.openstack.org/86/212786/5/check/gate-tempest-dsvm-ironic-pxe_ssh/fb7f21d/ : SUCCESS in 41m 48s\n- gate-devstack-dsvm-updown http://logs.openstack.org/86/212786/5/check/gate-devstack-dsvm-updown/b0306e1/ : SUCCESS in 27m 03s\n- gate-tempest-dsvm-f21 http://logs.openstack.org/86/212786/5/check/gate-tempest-dsvm-f21/378a775/ : FAILURE in 29m 50s (non-voting)\n- gate-tempest-dsvm-centos7 http://logs.openstack.org/86/212786/5/check/gate-tempest-dsvm-centos7/d7dfad5/ : SUCCESS in 31m 18s (non-voting)\n- gate-devstack-publish-docs http://docs-draft.openstack.org/86/212786/5/check/gate-devstack-publish-docs/78c14e2//doc/build/html/ : SUCCESS in 1m 59s\n- gate-swift-dsvm-functional-nv http://logs.openstack.org/86/212786/5/check/gate-swift-dsvm-functional-nv/f7215b3/ : SUCCESS in 35m 09s (non-voting)\n- gate-grenade-dsvm-neutron http://logs.openstack.org/86/212786/5/check/gate-grenade-dsvm-neutron/8ba0d42/ : SUCCESS in 53m 46s\n- gate-tempest-dsvm-multinode-full http://logs.openstack.org/86/212786/5/check/gate-tempest-dsvm-multinode-full/4ce0433/ : SUCCESS in 1h 01m 22s (non-voting)\n- gate-tempest-dsvm-neutron-multinode-full http://logs.openstack.org/86/212786/5/check/gate-tempest-dsvm-neutron-multinode-full/7036f92/ : SUCCESS in 1h 11m 11s (non-voting)","accounts_in_message":[],"_revision_number":5},{"id":"835d85f4bf3b5446d8bfd29ae5123d60d8d3e695","author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"date":"2015-08-14 05:21:48.000000000","message":"Patch Set 5:\n\n(2 comments)","accounts_in_message":[],"_revision_number":5},{"id":"d60c0e9c8bf84eb6b6af47222b73c7d2a7b7ef4e","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-08-14 06:18:37.000000000","message":"Patch Set 5:\n\n(1 comment)","accounts_in_message":[],"_revision_number":5},{"id":"6f79039c1ea3b79434f43d4a5d5e4b6383214f02","author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"date":"2015-08-14 06:54:09.000000000","message":"Patch Set 5:\n\n(1 comment)","accounts_in_message":[],"_revision_number":5},{"id":"c313c9a7bdf41826b3628bc05b65fcd2eccbfc63","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-08-14 06:55:15.000000000","message":"Patch Set 5:\n\n(1 comment)","accounts_in_message":[],"_revision_number":5},{"id":"fb40fd6c8b5e5a55dae460496132eb73a84586e9","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-08-14 06:58:53.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"b0e4836d30133769b4e0c563b680c380fa061ae1","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-08-14 07:02:12.000000000","message":"Patch Set 7: Commit message was updated","accounts_in_message":[],"_revision_number":7},{"id":"57a401607df53b8d5bc22c4620b927740f2101bf","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-08-14 08:07:28.000000000","message":"Patch Set 7: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-tempest-dsvm-full http://logs.openstack.org/86/212786/7/check/gate-tempest-dsvm-full/f48fd1e/ : SUCCESS in 56m 15s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/86/212786/7/check/gate-tempest-dsvm-postgres-full/1cad579/ : SUCCESS in 54m 11s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/86/212786/7/check/gate-tempest-dsvm-neutron-full/7ccab07/ : SUCCESS in 56m 46s\n- gate-grenade-dsvm http://logs.openstack.org/86/212786/7/check/gate-grenade-dsvm/1c2fb84/ : SUCCESS in 41m 54s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/86/212786/7/check/gate-tempest-dsvm-large-ops/a42a497/ : SUCCESS in 29m 57s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/86/212786/7/check/gate-tempest-dsvm-neutron-large-ops/5c4a2e6/ : SUCCESS in 29m 45s\n- gate-devstack-bashate http://logs.openstack.org/86/212786/7/check/gate-devstack-bashate/e7b5b23/ : SUCCESS in 21s\n- gate-devstack-unit-tests http://logs.openstack.org/86/212786/7/check/gate-devstack-unit-tests/432727f/ : SUCCESS in 20s\n- gate-devstack-dsvm-cells http://logs.openstack.org/86/212786/7/check/gate-devstack-dsvm-cells/cae0905/ : SUCCESS in 22m 22s\n- gate-grenade-dsvm-partial-ncpu http://logs.openstack.org/86/212786/7/check/gate-grenade-dsvm-partial-ncpu/9c1fc46/ : SUCCESS in 40m 17s\n- gate-tempest-dsvm-ironic-pxe_ssh http://logs.openstack.org/86/212786/7/check/gate-tempest-dsvm-ironic-pxe_ssh/da90d12/ : SUCCESS in 35m 42s\n- gate-devstack-dsvm-updown http://logs.openstack.org/86/212786/7/check/gate-devstack-dsvm-updown/db9dd4a/ : SUCCESS in 21m 08s\n- gate-tempest-dsvm-f21 http://logs.openstack.org/86/212786/7/check/gate-tempest-dsvm-f21/59f25b6/ : FAILURE in 32m 21s (non-voting)\n- gate-tempest-dsvm-centos7 http://logs.openstack.org/86/212786/7/check/gate-tempest-dsvm-centos7/1fc18a0/ : SUCCESS in 30m 10s (non-voting)\n- gate-devstack-publish-docs http://docs-draft.openstack.org/86/212786/7/check/gate-devstack-publish-docs/964bfca//doc/build/html/ : SUCCESS in 2m 00s\n- gate-swift-dsvm-functional-nv http://logs.openstack.org/86/212786/7/check/gate-swift-dsvm-functional-nv/b2c2627/ : SUCCESS in 36m 30s (non-voting)\n- gate-grenade-dsvm-neutron http://logs.openstack.org/86/212786/7/check/gate-grenade-dsvm-neutron/6c70b61/ : SUCCESS in 41m 18s\n- gate-tempest-dsvm-multinode-full http://logs.openstack.org/86/212786/7/check/gate-tempest-dsvm-multinode-full/e6688c2/ : SUCCESS in 52m 04s (non-voting)\n- gate-tempest-dsvm-neutron-multinode-full http://logs.openstack.org/86/212786/7/check/gate-tempest-dsvm-neutron-multinode-full/b6440e3/ : SUCCESS in 1h 01m 37s (non-voting)","accounts_in_message":[],"_revision_number":7},{"id":"b0902d72a60b6a900dde4062c035fece67f84e82","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-08-17 03:48:53.000000000","message":"Uploaded patch set 8.","accounts_in_message":[],"_revision_number":8},{"id":"b15d5ca5b188ea02381d56f2b3c956c5b51f9ea9","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-08-17 03:59:57.000000000","message":"Uploaded patch set 9.","accounts_in_message":[],"_revision_number":9},{"id":"6eaafc46cd96343394cf7e6a81cb32aae4eeee20","author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"date":"2015-08-17 11:34:30.000000000","message":"Patch Set 9: Code-Review+1\n\n(1 comment)\n\nThe change works, but my feeling is that we should let the devstack plugin determine a path for the plugin conf file (which you\u0027re doing in another patch for the vmware-nsx plugin at least), and only if that variable is not defined resort to the old logic for bw compatibility.\n\nProbably this might lead to simpler logic","accounts_in_message":[],"_revision_number":9},{"id":"4ed1bb0fc8a49342887cb5ca4eedd56b5a5254c2","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-08-17 15:51:50.000000000","message":"Patch Set 9:\n\n(1 comment)\n\nHere is my thinking.\n\nBefore decomposition, all vendors\u0027 plugin config files are under neutron:etc/neutron/plugins/. So it is easier for neutron-legacy to locate those files, i.e. they are all under the same path.\n\nFor the decomposition, has neutron community decided where in vendors\u0027 repo to store their plugin config files? If yes, then it becomes very simple, we just code that in neutron-legacy. I can change the patch here accordingly.\n\nMy current design is assuming that each vendor may place their config files in different directory structure in their repo. So for neutron-legacy, they can specify the complete path (Q_PLUGIN_SRC_CONF_DIR) or relative path (Q_PLUGIN_SRC_CONF_PATH) to their config files.\n\nThe relative path will be joined with DEST by neutron-legacy when copying the source config file. The reason I choose DEST as the parent path is because it is the same one used for NEUTRON_DIR (i.e. DEST/neutron) in current neutron-legacy and also the root for all repos used by stack.sh\naccording to my understanding..\n\n\n\n\n\n\n\n\n\nThe change works, but my feeling is that we should let the devstack plugin determine a path for the plugin conf file (which you\u0027re doing in another patch for the vmware-nsx plugin at least), and only if that variable is not defined resort to the old logic for bw compatibility.","accounts_in_message":[],"_revision_number":9},{"id":"dfd5efa257e676d4aa55e9ea3535b570da67c2e9","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-08-17 15:53:29.000000000","message":"Patch Set 9:\n\nPlease ignore the last paragraph in my previous comment. It was copied from an earlier comment.","accounts_in_message":[],"_revision_number":9},{"id":"5082b228429fb223e0229c5a5446f9e7e676152f","author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"date":"2015-08-18 02:15:45.000000000","message":"Patch Set 9: Code-Review+2","accounts_in_message":[],"_revision_number":9},{"id":"6bdde8d1f91d95e40e7ba71b9a1bd4f3d1dc4a46","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-08-18 17:28:01.000000000","message":"Patch Set 10: Patch Set 9 was rebased","accounts_in_message":[],"_revision_number":9},{"id":"e98ecc92ffe436674109e8aa2b38441711792fcc","author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"date":"2015-08-18 18:55:42.000000000","message":"Patch Set 10: Code-Review-1\n\n(2 comments)","accounts_in_message":[],"_revision_number":10},{"id":"5bad55416ade08e885a5f320577221b03f7bc897","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-08-18 18:59:22.000000000","message":"Patch Set 10: 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/86/212786/10/check/gate-tempest-dsvm-full/275b9ab/ : SUCCESS in 54m 40s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/86/212786/10/check/gate-tempest-dsvm-postgres-full/8fc84f9/ : SUCCESS in 57m 08s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/86/212786/10/check/gate-tempest-dsvm-neutron-full/e3b3598/ : FAILURE in 1h 11m 51s\n- gate-grenade-dsvm http://logs.openstack.org/86/212786/10/check/gate-grenade-dsvm/d34ea3c/ : FAILURE in 43m 05s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/86/212786/10/check/gate-tempest-dsvm-large-ops/49ffd04/ : SUCCESS in 36m 02s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/86/212786/10/check/gate-tempest-dsvm-neutron-large-ops/7deb385/ : SUCCESS in 34m 55s\n- gate-devstack-bashate http://logs.openstack.org/86/212786/10/check/gate-devstack-bashate/80d3479/ : SUCCESS in 35s\n- gate-devstack-unit-tests http://logs.openstack.org/86/212786/10/check/gate-devstack-unit-tests/291eb97/ : SUCCESS in 15s\n- gate-devstack-dsvm-cells http://logs.openstack.org/86/212786/10/check/gate-devstack-dsvm-cells/6979d60/ : SUCCESS in 26m 16s\n- gate-grenade-dsvm-partial-ncpu http://logs.openstack.org/86/212786/10/check/gate-grenade-dsvm-partial-ncpu/8edc4e0/ : SUCCESS in 40m 19s\n- gate-tempest-dsvm-ironic-pxe_ssh http://logs.openstack.org/86/212786/10/check/gate-tempest-dsvm-ironic-pxe_ssh/76478f8/ : SUCCESS in 42m 35s\n- gate-devstack-dsvm-updown http://logs.openstack.org/86/212786/10/check/gate-devstack-dsvm-updown/24f425b/ : SUCCESS in 18m 48s\n- gate-tempest-dsvm-f21 http://logs.openstack.org/86/212786/10/check/gate-tempest-dsvm-f21/46a53fd/ : FAILURE in 32m 17s (non-voting)\n- gate-tempest-dsvm-centos7 http://logs.openstack.org/86/212786/10/check/gate-tempest-dsvm-centos7/ac5eabe/ : SUCCESS in 29m 31s (non-voting)\n- gate-devstack-publish-docs http://docs-draft.openstack.org/86/212786/10/check/gate-devstack-publish-docs/b32de25//doc/build/html/ : SUCCESS in 1m 30s\n- gate-swift-dsvm-functional-nv http://logs.openstack.org/86/212786/10/check/gate-swift-dsvm-functional-nv/7b81138/ : SUCCESS in 27m 48s (non-voting)\n- gate-grenade-dsvm-neutron http://logs.openstack.org/86/212786/10/check/gate-grenade-dsvm-neutron/c8f1f5a/ : SUCCESS in 42m 56s\n- gate-tempest-dsvm-multinode-full http://logs.openstack.org/86/212786/10/check/gate-tempest-dsvm-multinode-full/f47b805/ : SUCCESS in 52m 53s (non-voting)\n- gate-tempest-dsvm-neutron-multinode-full http://logs.openstack.org/86/212786/10/check/gate-tempest-dsvm-neutron-multinode-full/9bdda83/ : SUCCESS in 1h 15m 54s (non-voting)","accounts_in_message":[],"_revision_number":10},{"id":"53e1ab9f80e56f2bf8248f216e0fc2329ed4d8e0","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-08-18 20:20:55.000000000","message":"Patch Set 10:\n\n(2 comments)","accounts_in_message":[],"_revision_number":10},{"id":"103693cfba8d14740bfbbd00479f15583444a4f2","author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"date":"2015-08-19 15:41:08.000000000","message":"Patch Set 10:\n\nyour explanation makes sense.\nI think it is also ok to use $DEST as a general root directory for source files since everything that gets installed from devstack goes there.\n\nAnyway, another question:\nyou\u0027re adding Q_PLUGIN_SRC_CONF_DIR and Q_PLUGIN_SRC_CONF_PATH, but I am not able to understand how they differ.","accounts_in_message":[],"_revision_number":10},{"id":"1a7f77a04b36d726634a82ca66ba4fa9e381ccc1","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-08-19 16:59:21.000000000","message":"Patch Set 10:\n\nQ_PLUGIN_SRC_CONF_PATH is the relative path that I proposed initially, which will be rooted at DEST in neutron-legacy. For example,\n\ncp $DEST/$Q_PLUGIN_SRC_CONF_PATH/$Q_PLUGIN_CONF_FILENAME /$Q_PLUGIN_CONF_FILE\n\nQ_PLUGIN_SRC_CONF_DIR is the absolute/full path that I added later based on a review comment saying that some users may not use DEST in their setup, Q_PLUGIN_SRC_CONF_DIR will be used directly in neutron-legacy. For example,\n\ncp $Q_PLUGIN_SRC_CONF_DIR/$Q_PLUGIN_CONF_FILENAME /$Q_PLUGIN_CONF_FILE\n\nneutron-legacy will first check if Q_PLUGIN_SRC_CONF_DIR is defined. If so, then use it. Otherwise, check if Q_PLUGIN_SRC_CONF_PATH is defined.\n\nTo avoid confusion, maybe I should rename them as\n\nQ_PLUGIN_SRC_CONF_DIR -\u003e Q_PLUGIN_SRC_CONF_FULL_PATH\nQ_PLUGIN_SRC_CONF_PATH -\u003e Q_PLUGIN_SRC_CONF_REL_PATH\n\nThe other option is to keep Q_PLUGIN_SRC_CONF_PATH defining the relative path and have Q_PLUGIN_SRC_CONF_DIR default to DEST if not defined. Thus neutron-legacy will do\n\ncp $Q_PLUGIN_SRC_CONF_DIR/$Q_PLUGIN_SRC_CONF_PATH/$Q_PLUGIN_CONF_FILENAME /$Q_PLUGIN_CONF_FILE\n\nAny comment/suggestion? Thanks.","accounts_in_message":[],"_revision_number":10},{"id":"52a4b139ae9de961fbb592ba3a41c11e86dc9dbf","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-08-19 17:19:25.000000000","message":"Patch Set 10:\n\nAbout the other option in my previous comment,\nI meant to use Q_PLUGIN_SRC_CONF_DIR only defining the root directory and default to DEST.","accounts_in_message":[],"_revision_number":10},{"id":"47bf0b95a5c754026395b334a1d2561ce233d284","author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"date":"2015-08-19 17:27:18.000000000","message":"Patch Set 10:\n\n(1 comment)","accounts_in_message":[],"_revision_number":10},{"id":"bd0adf34a12bc040cea48b9021ba9c7dbb0428fd","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-08-19 17:43:55.000000000","message":"Patch Set 10:\n\n(1 comment)","accounts_in_message":[],"_revision_number":10},{"id":"200264a43f1b326a7aa69dcc2a46247894eeb3ea","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-08-19 20:50:23.000000000","message":"Uploaded patch set 11.","accounts_in_message":[],"_revision_number":11},{"id":"34ecf61a6eb07dacd4832acbc376194c9734dcb5","author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"date":"2015-08-19 21:16:20.000000000","message":"Patch Set 10:\n\n(1 comment)","accounts_in_message":[],"_revision_number":10},{"id":"2225f0205e3f48e8673295c0dbbc3fb550e3bfce","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-08-19 21:21:30.000000000","message":"Patch Set 10:\n\n(1 comment)","accounts_in_message":[],"_revision_number":10},{"id":"52a386d94734c67308460822cc55ef490c5b512f","author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"date":"2015-08-19 21:49:54.000000000","message":"Patch Set 10:\n\n(1 comment)","accounts_in_message":[],"_revision_number":10},{"id":"d1302c478b1bde244552e9df80966ed30fb2ac8c","author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"date":"2015-08-19 21:50:19.000000000","message":"Patch Set 11:\n\n(1 comment)","accounts_in_message":[],"_revision_number":11},{"id":"17937bb1b8e19968abb549787a5f06bdeabd361e","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-08-19 22:12:19.000000000","message":"Patch Set 10:\n\n(1 comment)","accounts_in_message":[],"_revision_number":10},{"id":"73d4b6b8a6d9bdb9c4e6f132bede036318855c3b","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-08-19 22:13:44.000000000","message":"Patch Set 11:\n\n(1 comment)","accounts_in_message":[],"_revision_number":11},{"id":"319745a609a1fd96313cbb1c24c7216a7b69da0a","author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"date":"2015-08-19 23:10:21.000000000","message":"Patch Set 10:\n\n(1 comment)","accounts_in_message":[],"_revision_number":10},{"id":"ca94d4676a9d3519f2b66c253ad7113451aa0c31","author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"date":"2015-08-19 23:11:15.000000000","message":"Patch Set 11:\n\n(1 comment)","accounts_in_message":[],"_revision_number":11},{"id":"c67c11d409f53e2f14753b545d2059709202ade7","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-08-19 23:16:40.000000000","message":"Patch Set 10:\n\n(1 comment)","accounts_in_message":[],"_revision_number":10},{"id":"f07e94dfd406c31e379e8c255e9a1b22898a1a94","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-08-20 00:49:39.000000000","message":"Patch Set 11: 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/86/212786/11/check/gate-tempest-dsvm-full/c3b03fa/ : SUCCESS in 1h 11m 15s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/86/212786/11/check/gate-tempest-dsvm-postgres-full/18fada3/ : SUCCESS in 44m 31s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/86/212786/11/check/gate-tempest-dsvm-neutron-full/c340f10/ : FAILURE in 1h 11m 36s\n- gate-grenade-dsvm http://logs.openstack.org/86/212786/11/check/gate-grenade-dsvm/f40a653/ : SUCCESS in 55m 19s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/86/212786/11/check/gate-tempest-dsvm-large-ops/fd499ab/ : SUCCESS in 36m 47s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/86/212786/11/check/gate-tempest-dsvm-neutron-large-ops/18076be/ : SUCCESS in 32m 52s\n- gate-devstack-bashate http://logs.openstack.org/86/212786/11/check/gate-devstack-bashate/000776e/ : SUCCESS in 0s\n- gate-devstack-unit-tests http://logs.openstack.org/86/212786/11/check/gate-devstack-unit-tests/b9d0857/ : SUCCESS in 0s\n- gate-devstack-dsvm-cells http://logs.openstack.org/86/212786/11/check/gate-devstack-dsvm-cells/e86ae13/ : SUCCESS in 29m 42s\n- gate-grenade-dsvm-partial-ncpu http://logs.openstack.org/86/212786/11/check/gate-grenade-dsvm-partial-ncpu/20939ca/ : SUCCESS in 47m 49s\n- gate-tempest-dsvm-ironic-pxe_ssh http://logs.openstack.org/86/212786/11/check/gate-tempest-dsvm-ironic-pxe_ssh/bbd78ef/ : SUCCESS in 40m 58s\n- gate-devstack-dsvm-updown http://logs.openstack.org/86/212786/11/check/gate-devstack-dsvm-updown/388e42f/ : SUCCESS in 33m 13s\n- gate-tempest-dsvm-f21 http://logs.openstack.org/86/212786/11/check/gate-tempest-dsvm-f21/7378796/ : FAILURE in 32m 14s (non-voting)\n- gate-tempest-dsvm-centos7 http://logs.openstack.org/86/212786/11/check/gate-tempest-dsvm-centos7/ae27629/ : SUCCESS in 41m 17s (non-voting)\n- gate-devstack-publish-docs http://docs-draft.openstack.org/86/212786/11/check/gate-devstack-publish-docs/4aa5726//doc/build/html/ : SUCCESS in 1m 00s\n- gate-swift-dsvm-functional-nv http://logs.openstack.org/86/212786/11/check/gate-swift-dsvm-functional-nv/15420d5/ : SUCCESS in 36m 26s (non-voting)\n- gate-grenade-dsvm-neutron http://logs.openstack.org/86/212786/11/check/gate-grenade-dsvm-neutron/6c80c04/ : SUCCESS in 59m 26s\n- gate-tempest-dsvm-multinode-full http://logs.openstack.org/86/212786/11/check/gate-tempest-dsvm-multinode-full/a4a2ae4/ : SUCCESS in 1h 01m 14s (non-voting)\n- gate-tempest-dsvm-neutron-multinode-full http://logs.openstack.org/86/212786/11/check/gate-tempest-dsvm-neutron-multinode-full/b96e769/ : FAILURE in 1h 25m 16s (non-voting)","accounts_in_message":[],"_revision_number":11},{"id":"4be1c7fc4ebf5cfa38065fd66db5d8a58975fe63","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2015-08-20 10:22:30.000000000","message":"Patch Set 11:\n\nPassed using XenAPI driver with XenServer 6.2: Logs at http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/86/212786/11/9028/index.html\n\nUse \"xenserver: recheck\" to trigger only xenserver re-check.  XenServer CI contact: openstack@citrix.com.\n\nDebugging suggestions at https://wiki.openstack.org/wiki/Debugging_XenServer_CI_failures","accounts_in_message":[],"_revision_number":11},{"id":"bda56c9f7f3091e1ba1a68c14144f6134f47f2b3","author":{"_account_id":13438,"name":"Kobi Samoray","email":"kobi.samoray@broadcom.com","username":"ksamoray"},"date":"2015-08-20 15:24:29.000000000","message":"Patch Set 11:\n\nrecheck","accounts_in_message":[],"_revision_number":11},{"id":"4b0870a5220e5a9b51c7ac4ca107677a0d4c741d","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-08-20 17:25:34.000000000","message":"Patch Set 11:\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/86/212786/11/check/gate-tempest-dsvm-full/83f8b78/ : SUCCESS in 50m 13s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/86/212786/11/check/gate-tempest-dsvm-postgres-full/006248e/ : FAILURE in 42m 13s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/86/212786/11/check/gate-tempest-dsvm-neutron-full/e20e6e8/ : SUCCESS in 1h 17m 58s\n- gate-grenade-dsvm http://logs.openstack.org/86/212786/11/check/gate-grenade-dsvm/b319453/ : SUCCESS in 1h 03m 01s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/86/212786/11/check/gate-tempest-dsvm-large-ops/c1d1e14/ : SUCCESS in 26m 03s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/86/212786/11/check/gate-tempest-dsvm-neutron-large-ops/4f55b86/ : SUCCESS in 27m 11s\n- gate-devstack-bashate http://logs.openstack.org/86/212786/11/check/gate-devstack-bashate/19f5c76/ : SUCCESS in 26s\n- gate-devstack-unit-tests http://logs.openstack.org/86/212786/11/check/gate-devstack-unit-tests/f91eea3/ : SUCCESS in 14s\n- gate-devstack-dsvm-cells http://logs.openstack.org/86/212786/11/check/gate-devstack-dsvm-cells/25a60da/ : SUCCESS in 23m 51s\n- gate-grenade-dsvm-partial-ncpu http://logs.openstack.org/86/212786/11/check/gate-grenade-dsvm-partial-ncpu/ec64335/ : SUCCESS in 46m 15s\n- gate-tempest-dsvm-ironic-pxe_ssh http://logs.openstack.org/86/212786/11/check/gate-tempest-dsvm-ironic-pxe_ssh/d116065/ : SUCCESS in 34m 36s\n- gate-devstack-dsvm-updown http://logs.openstack.org/86/212786/11/check/gate-devstack-dsvm-updown/202e719/ : SUCCESS in 19m 28s\n- gate-tempest-dsvm-f21 http://logs.openstack.org/86/212786/11/check/gate-tempest-dsvm-f21/c146a38/ : FAILURE in 34m 05s (non-voting)\n- gate-tempest-dsvm-centos7 http://logs.openstack.org/86/212786/11/check/gate-tempest-dsvm-centos7/03567ff/ : SUCCESS in 35m 54s (non-voting)\n- gate-devstack-publish-docs http://docs-draft.openstack.org/86/212786/11/check/gate-devstack-publish-docs/ef59209//doc/build/html/ : SUCCESS in 2m 01s\n- gate-swift-dsvm-functional-nv http://logs.openstack.org/86/212786/11/check/gate-swift-dsvm-functional-nv/8a58e57/ : SUCCESS in 35m 55s (non-voting)\n- gate-grenade-dsvm-neutron http://logs.openstack.org/86/212786/11/check/gate-grenade-dsvm-neutron/7d04881/ : SUCCESS in 56m 54s\n- gate-tempest-dsvm-multinode-full http://logs.openstack.org/86/212786/11/check/gate-tempest-dsvm-multinode-full/aedd3c2/ : SUCCESS in 51m 25s (non-voting)\n- gate-tempest-dsvm-neutron-multinode-full http://logs.openstack.org/86/212786/11/check/gate-tempest-dsvm-neutron-multinode-full/5e8b3ca/ : SUCCESS in 1h 02m 08s (non-voting)","accounts_in_message":[],"_revision_number":11},{"id":"55c20320b3f87dfd3c72c3771f023115b65280b0","author":{"_account_id":13438,"name":"Kobi Samoray","email":"kobi.samoray@broadcom.com","username":"ksamoray"},"date":"2015-08-20 18:53:08.000000000","message":"Patch Set 12: Patch Set 11 was rebased","accounts_in_message":[],"_revision_number":11},{"id":"e32581096143e4f55cd1f5e2b021c7f6e94bf448","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2015-08-20 19:53:32.000000000","message":"Patch Set 12:\n\nPassed using XenAPI driver with XenServer 6.2: Logs at http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/86/212786/12/9228/index.html\n\nUse \"xenserver: recheck\" to trigger only xenserver re-check.  XenServer CI contact: openstack@citrix.com.\n\nDebugging suggestions at https://wiki.openstack.org/wiki/Debugging_XenServer_CI_failures","accounts_in_message":[],"_revision_number":12},{"id":"0646659bdbc7b6db3468bb3eae9101606ac4a09c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-08-20 20:29:34.000000000","message":"Patch Set 12: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-tempest-dsvm-full http://logs.openstack.org/86/212786/12/check/gate-tempest-dsvm-full/39df067/ : SUCCESS in 55m 23s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/86/212786/12/check/gate-tempest-dsvm-postgres-full/8586a18/ : SUCCESS in 56m 36s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/86/212786/12/check/gate-tempest-dsvm-neutron-full/851070f/ : SUCCESS in 1h 10m 31s\n- gate-grenade-dsvm http://logs.openstack.org/86/212786/12/check/gate-grenade-dsvm/6c9bd9e/ : SUCCESS in 1h 03m 54s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/86/212786/12/check/gate-tempest-dsvm-large-ops/f79b179/ : SUCCESS in 34m 02s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/86/212786/12/check/gate-tempest-dsvm-neutron-large-ops/feb79e1/ : SUCCESS in 34m 49s\n- gate-devstack-bashate http://logs.openstack.org/86/212786/12/check/gate-devstack-bashate/0f480e7/ : SUCCESS in 47s\n- gate-devstack-unit-tests http://logs.openstack.org/86/212786/12/check/gate-devstack-unit-tests/583ee58/ : SUCCESS in 38s\n- gate-devstack-dsvm-cells http://logs.openstack.org/86/212786/12/check/gate-devstack-dsvm-cells/a7fdbfe/ : SUCCESS in 38m 38s\n- gate-grenade-dsvm-partial-ncpu http://logs.openstack.org/86/212786/12/check/gate-grenade-dsvm-partial-ncpu/8d7872c/ : SUCCESS in 50m 02s\n- gate-tempest-dsvm-ironic-pxe_ssh http://logs.openstack.org/86/212786/12/check/gate-tempest-dsvm-ironic-pxe_ssh/69e1d4f/ : SUCCESS in 41m 02s\n- gate-devstack-dsvm-updown http://logs.openstack.org/86/212786/12/check/gate-devstack-dsvm-updown/1b49165/ : SUCCESS in 33m 48s\n- gate-tempest-dsvm-f21 http://logs.openstack.org/86/212786/12/check/gate-tempest-dsvm-f21/47eb2a5/ : FAILURE in 30m 29s (non-voting)\n- gate-tempest-dsvm-centos7 http://logs.openstack.org/86/212786/12/check/gate-tempest-dsvm-centos7/0accee1/ : SUCCESS in 32m 47s (non-voting)\n- gate-devstack-publish-docs http://docs-draft.openstack.org/86/212786/12/check/gate-devstack-publish-docs/810b64f//doc/build/html/ : SUCCESS in 1m 25s\n- gate-swift-dsvm-functional-nv http://logs.openstack.org/86/212786/12/check/gate-swift-dsvm-functional-nv/0c9b5d7/ : SUCCESS in 34m 26s (non-voting)\n- gate-grenade-dsvm-neutron http://logs.openstack.org/86/212786/12/check/gate-grenade-dsvm-neutron/f2b6afc/ : SUCCESS in 58m 55s\n- gate-tempest-dsvm-multinode-full http://logs.openstack.org/86/212786/12/check/gate-tempest-dsvm-multinode-full/7adfb32/ : SUCCESS in 1h 05m 38s (non-voting)\n- gate-tempest-dsvm-neutron-multinode-full http://logs.openstack.org/86/212786/12/check/gate-tempest-dsvm-neutron-multinode-full/867f283/ : SUCCESS in 1h 20m 43s (non-voting)","accounts_in_message":[],"_revision_number":12},{"id":"6fc1c5376b26635990278acaf297fd2485e83b8e","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-08-24 13:40:49.000000000","message":"Patch Set 13: Commit message was updated","accounts_in_message":[],"_revision_number":13},{"id":"2dbc478279413f8bb76e24cd9ddfc6a8bad45173","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2015-08-24 14:47:44.000000000","message":"Patch Set 13:\n\nPassed using XenAPI driver with XenServer 6.2: Logs at http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/86/212786/13/9572/index.html\n\nUse \"xenserver: recheck\" to trigger only xenserver re-check.  XenServer CI contact: openstack@citrix.com.\n\nDebugging suggestions at https://wiki.openstack.org/wiki/Debugging_XenServer_CI_failures","accounts_in_message":[],"_revision_number":13},{"id":"9d4ce38ee278c43cdfb794ca2f983d4cbe8c9882","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-08-24 14:57:32.000000000","message":"Patch Set 13: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-tempest-dsvm-full http://logs.openstack.org/86/212786/13/check/gate-tempest-dsvm-full/de8eec7/ : SUCCESS in 54m 23s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/86/212786/13/check/gate-tempest-dsvm-postgres-full/f1fd40e/ : SUCCESS in 1h 05m 38s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/86/212786/13/check/gate-tempest-dsvm-neutron-full/122b39c/ : SUCCESS in 1h 12m 45s\n- gate-grenade-dsvm http://logs.openstack.org/86/212786/13/check/gate-grenade-dsvm/a9c2f70/ : SUCCESS in 47m 11s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/86/212786/13/check/gate-tempest-dsvm-large-ops/e2bb434/ : SUCCESS in 40m 06s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/86/212786/13/check/gate-tempest-dsvm-neutron-large-ops/b618665/ : SUCCESS in 32m 44s\n- gate-devstack-bashate http://logs.openstack.org/86/212786/13/check/gate-devstack-bashate/5c94af1/ : SUCCESS in 26s\n- gate-devstack-unit-tests http://logs.openstack.org/86/212786/13/check/gate-devstack-unit-tests/d3ed108/ : SUCCESS in 21s\n- gate-devstack-dsvm-cells http://logs.openstack.org/86/212786/13/check/gate-devstack-dsvm-cells/fc63c8c/ : SUCCESS in 21m 58s\n- gate-grenade-dsvm-partial-ncpu http://logs.openstack.org/86/212786/13/check/gate-grenade-dsvm-partial-ncpu/9f81671/ : SUCCESS in 49m 37s\n- gate-tempest-dsvm-ironic-pxe_ssh http://logs.openstack.org/86/212786/13/check/gate-tempest-dsvm-ironic-pxe_ssh/10cd709/ : SUCCESS in 40m 03s\n- gate-devstack-dsvm-updown http://logs.openstack.org/86/212786/13/check/gate-devstack-dsvm-updown/60b9081/ : SUCCESS in 20m 07s\n- gate-tempest-dsvm-f21 http://logs.openstack.org/86/212786/13/check/gate-tempest-dsvm-f21/9f45b9a/ : FAILURE in 41m 52s (non-voting)\n- gate-tempest-dsvm-centos7 http://logs.openstack.org/86/212786/13/check/gate-tempest-dsvm-centos7/19d9032/ : SUCCESS in 31m 03s (non-voting)\n- gate-devstack-publish-docs http://docs-draft.openstack.org/86/212786/13/check/gate-devstack-publish-docs/ff2f9ba//doc/build/html/ : SUCCESS in 1m 53s\n- gate-swift-dsvm-functional-nv http://logs.openstack.org/86/212786/13/check/gate-swift-dsvm-functional-nv/56981ce/ : SUCCESS in 28m 32s (non-voting)\n- gate-grenade-dsvm-neutron http://logs.openstack.org/86/212786/13/check/gate-grenade-dsvm-neutron/637a183/ : SUCCESS in 47m 47s\n- gate-tempest-dsvm-multinode-full http://logs.openstack.org/86/212786/13/check/gate-tempest-dsvm-multinode-full/3dd8ff2/ : SUCCESS in 1h 01m 35s (non-voting)\n- gate-tempest-dsvm-neutron-multinode-full http://logs.openstack.org/86/212786/13/check/gate-tempest-dsvm-neutron-multinode-full/746660f/ : SUCCESS in 1h 03m 25s (non-voting)","accounts_in_message":[],"_revision_number":13},{"id":"2c3fe543c27384bf1d8d8fb460d00f7988c6f2e9","author":{"_account_id":13438,"name":"Kobi Samoray","email":"kobi.samoray@broadcom.com","username":"ksamoray"},"date":"2015-08-24 16:49:07.000000000","message":"Patch Set 13: Code-Review+1","accounts_in_message":[],"_revision_number":13},{"id":"5187a8dc3825a829859b409fa43c92dcbdc63dd4","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2015-08-26 02:16:02.000000000","message":"Patch Set 13: Code-Review-1\n\n(1 comment)\n\nI would like to be convinced, see comment\n\nI also want to see that I5bd6d8f611c2a134f2e8f14c074c1a4185d9c522 integrates with this, or vice-versa","accounts_in_message":[],"_revision_number":13},{"id":"8854965a346e0552551e02d583c70ba9f8da4950","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-08-26 03:48:04.000000000","message":"Patch Set 13:\n\n(1 comment)","accounts_in_message":[],"_revision_number":13},{"id":"88f1dbed82ab9d192a4c20abdddb82cf8d6a4eb7","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2015-08-26 06:02:38.000000000","message":"Patch Set 13:\n\n(1 comment)","accounts_in_message":[],"_revision_number":13},{"id":"dc1a28b0ab8be82b94efef83072aad270a9dc1dc","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-08-26 06:38:20.000000000","message":"Patch Set 13:\n\n(1 comment)","accounts_in_message":[],"_revision_number":13},{"id":"6fe11b78099eeee6178b68a6fb3fadfae4f9c92c","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-08-28 20:46:55.000000000","message":"Patch Set 13:\n\nHi Ian, any further comment on this change regarding the -1 review? Thanks.","accounts_in_message":[],"_revision_number":13},{"id":"409c5541b22a3f62c4737614ea1b60eced0ce774","author":{"_account_id":7018,"name":"Abhishek Raut","email":"rauta@vmware.com","username":"abhiraut"},"date":"2015-08-28 21:57:33.000000000","message":"Patch Set 13:\n\n(1 comment)","accounts_in_message":[],"_revision_number":13},{"id":"e7ea561f6d859e5c414127e9493503415d20495f","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-08-28 22:26:18.000000000","message":"Patch Set 14: Commit message was updated","accounts_in_message":[],"_revision_number":14},{"id":"7cbb8c6663f10e1b1a1f78aaf6d3896228492791","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-08-28 22:43:37.000000000","message":"Patch Set 15: Commit message was updated","accounts_in_message":[],"_revision_number":15},{"id":"8ee8d420654b3eb0a1c85d6fa269b3b86e409c09","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2015-08-28 23:51:18.000000000","message":"Patch Set 15:\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/86/212786/15/10284/index.html : SUCCESS in 01 h 04 m 21 s","accounts_in_message":[],"_revision_number":15},{"id":"d5a53a3fbaa027bc3dca88ad5b34c016e1e45f09","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-08-29 00:02:45.000000000","message":"Patch Set 15: 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/86/212786/15/check/gate-tempest-dsvm-full/b3a1f44/ : SUCCESS in 57m 21s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/86/212786/15/check/gate-tempest-dsvm-postgres-full/9b9f23f/ : SUCCESS in 37m 49s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/86/212786/15/check/gate-tempest-dsvm-neutron-full/d2122cd/ : SUCCESS in 1h 12m 27s\n- gate-grenade-dsvm http://logs.openstack.org/86/212786/15/check/gate-grenade-dsvm/69d0448/ : SUCCESS in 46m 42s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/86/212786/15/check/gate-tempest-dsvm-large-ops/1de620f/ : SUCCESS in 33m 29s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/86/212786/15/check/gate-tempest-dsvm-neutron-large-ops/2ef069a/ : SUCCESS in 24m 32s\n- gate-devstack-bashate http://logs.openstack.org/86/212786/15/check/gate-devstack-bashate/9144225/ : SUCCESS in 14s\n- gate-devstack-unit-tests http://logs.openstack.org/86/212786/15/check/gate-devstack-unit-tests/84bfc9d/ : SUCCESS in 14s\n- gate-grenade-dsvm-partial-ncpu http://logs.openstack.org/86/212786/15/check/gate-grenade-dsvm-partial-ncpu/e9b80c4/ : SUCCESS in 43m 11s\n- gate-tempest-dsvm-ironic-pxe_ssh http://logs.openstack.org/86/212786/15/check/gate-tempest-dsvm-ironic-pxe_ssh/85be0b5/ : SUCCESS in 39m 31s\n- gate-devstack-dsvm-updown http://logs.openstack.org/86/212786/15/check/gate-devstack-dsvm-updown/cdccfc3/ : SUCCESS in 30m 24s\n- gate-tempest-dsvm-f21 http://logs.openstack.org/86/212786/15/check/gate-tempest-dsvm-f21/c7d67c9/ : FAILURE in 33m 29s (non-voting)\n- gate-tempest-dsvm-centos7 http://logs.openstack.org/86/212786/15/check/gate-tempest-dsvm-centos7/9085a6a/ : SUCCESS in 31m 47s (non-voting)\n- gate-devstack-publish-docs http://docs-draft.openstack.org/86/212786/15/check/gate-devstack-publish-docs/8a6b083//doc/build/html/ : SUCCESS in 2m 09s\n- gate-swift-dsvm-functional-nv http://logs.openstack.org/86/212786/15/check/gate-swift-dsvm-functional-nv/e2dbd26/ : SUCCESS in 27m 39s (non-voting)\n- gate-grenade-dsvm-neutron http://logs.openstack.org/86/212786/15/check/gate-grenade-dsvm-neutron/be9e8c6/ : FAILURE in 38m 31s\n- gate-tempest-dsvm-multinode-full http://logs.openstack.org/86/212786/15/check/gate-tempest-dsvm-multinode-full/30148b3/ : FAILURE in 55m 39s (non-voting)\n- gate-tempest-dsvm-neutron-multinode-full http://logs.openstack.org/86/212786/15/check/gate-tempest-dsvm-neutron-multinode-full/d439fcd/ : SUCCESS in 1h 12m 28s (non-voting)","accounts_in_message":[],"_revision_number":15},{"id":"be86ee45f931993b6f02da3b04c817f74ecd0681","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-08-29 04:29:24.000000000","message":"Patch Set 16: Patch Set 15 was rebased","accounts_in_message":[],"_revision_number":15},{"id":"2ea98326b6b4cab99871a020a3d8d034574263ac","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2015-08-29 05:34:43.000000000","message":"Patch Set 16:\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/86/212786/16/10292/index.html : SUCCESS in 01 h 01 m 39 s","accounts_in_message":[],"_revision_number":16},{"id":"4bf7438e5303a663b9931d2d8ae316252aac5f62","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-08-29 05:41:48.000000000","message":"Patch Set 16: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-tempest-dsvm-full http://logs.openstack.org/86/212786/16/check/gate-tempest-dsvm-full/c77c688/ : SUCCESS in 54m 01s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/86/212786/16/check/gate-tempest-dsvm-postgres-full/68f901a/ : SUCCESS in 43m 15s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/86/212786/16/check/gate-tempest-dsvm-neutron-full/002077e/ : SUCCESS in 59m 30s\n- gate-grenade-dsvm http://logs.openstack.org/86/212786/16/check/gate-grenade-dsvm/a30021c/ : SUCCESS in 50m 09s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/86/212786/16/check/gate-tempest-dsvm-large-ops/62fb05a/ : SUCCESS in 42m 07s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/86/212786/16/check/gate-tempest-dsvm-neutron-large-ops/e399a66/ : SUCCESS in 32m 20s\n- gate-devstack-bashate http://logs.openstack.org/86/212786/16/check/gate-devstack-bashate/75483ce/ : SUCCESS in 12s\n- gate-devstack-unit-tests http://logs.openstack.org/86/212786/16/check/gate-devstack-unit-tests/14e74f8/ : SUCCESS in 13s\n- gate-grenade-dsvm-partial-ncpu http://logs.openstack.org/86/212786/16/check/gate-grenade-dsvm-partial-ncpu/4741642/ : SUCCESS in 46m 32s\n- gate-tempest-dsvm-ironic-pxe_ssh http://logs.openstack.org/86/212786/16/check/gate-tempest-dsvm-ironic-pxe_ssh/ed68a48/ : SUCCESS in 39m 11s\n- gate-devstack-dsvm-updown http://logs.openstack.org/86/212786/16/check/gate-devstack-dsvm-updown/2452de3/ : SUCCESS in 20m 48s\n- gate-tempest-dsvm-f21 http://logs.openstack.org/86/212786/16/check/gate-tempest-dsvm-f21/5040401/ : FAILURE in 31m 16s (non-voting)\n- gate-tempest-dsvm-centos7 http://logs.openstack.org/86/212786/16/check/gate-tempest-dsvm-centos7/66520b7/ : SUCCESS in 31m 13s (non-voting)\n- gate-devstack-publish-docs http://docs-draft.openstack.org/86/212786/16/check/gate-devstack-publish-docs/3d6f683//doc/build/html/ : SUCCESS in 1m 19s\n- gate-swift-dsvm-functional-nv http://logs.openstack.org/86/212786/16/check/gate-swift-dsvm-functional-nv/489030e/ : SUCCESS in 28m 49s (non-voting)\n- gate-grenade-dsvm-neutron http://logs.openstack.org/86/212786/16/check/gate-grenade-dsvm-neutron/70f4375/ : SUCCESS in 44m 29s\n- gate-tempest-dsvm-multinode-full http://logs.openstack.org/86/212786/16/check/gate-tempest-dsvm-multinode-full/f247d10/ : SUCCESS in 50m 48s (non-voting)\n- gate-tempest-dsvm-neutron-multinode-full http://logs.openstack.org/86/212786/16/check/gate-tempest-dsvm-neutron-multinode-full/9801c0c/ : SUCCESS in 1h 07m 59s (non-voting)","accounts_in_message":[],"_revision_number":16},{"id":"6d446e7e56b757b5d04124fc7577a83bdc34a372","author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"date":"2015-08-30 05:44:08.000000000","message":"Patch Set 16: Code-Review+2","accounts_in_message":[],"_revision_number":16},{"id":"fca157192f0a77eef62463b479e6d12eb64107a9","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2015-09-01 08:13:35.000000000","message":"Patch Set 16:\n\nThanks for the expanded explanations; I need to parse through this fully when i\u0027ve got a bit more time.\n\nI do not think we should merge this until some of the neutron ci liasons have signed-off on this, namely from mestery, dougwig, and armax, as described in a recent email to the infra list","accounts_in_message":[],"_revision_number":16},{"id":"192541746c40757fa31be37ee640686054f526a1","author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"date":"2015-09-02 10:00:40.000000000","message":"Patch Set 16: Code-Review+1\n\nI am not neutron infra liaison, but I am quite familiar with devstack-gate, tempest, and neutron. This change does not impact any gate job as the variable used here it is not set in devstack_gate.\n\nIf a plugin library sets it, then it uses this path to copy configuration files for the plugin from the appropriate directory. It\u0027s simple as that. with a little tweak concerning whether the path is absolute or not.\n\nThe only thing on which devstack-core input would be welcome are:\n- whether it is ok to let the devstack plugin only specify where to copy the files, and then let devstack copy them, or whether we should remove the ability of doing so from neutron-legacy\n- since this change affects neutron-legacy I wonder whether this should a functionality available only to \u0027legacy\u0027 plugins. Decomposed plugins should do everything in the devstack plugin. In that case we need a solution as devstack fails to copy configuration files for decomposed neutron plugins.","accounts_in_message":[],"_revision_number":16},{"id":"203ccc68dd15d75001bd7dd26fd8479a3864d46e","author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"date":"2015-09-02 10:03:32.000000000","message":"Patch Set 16:\n\nand btw we will obviously wait for feedback from either dougwig, armax, or mestery - as that should be the process for all devstack changes","accounts_in_message":[],"_revision_number":16},{"id":"160cdc10bd4c2a5379c4e136a477cdcc18b4f033","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2015-09-02 21:46:34.000000000","message":"Patch Set 16:\n\nI\u0027d like to mention again (it\u0027s been a while) that I am really not a fan of the way the config values are handled WRT generally not using a leading \u0027/\u0027 so it is required in the code.  This is the _only_ place in DevStack that does this and it can get confusing.  My intention is still to remove that behaviour in the neutron rewrite, if I ever get back to it...","accounts_in_message":[],"_revision_number":16},{"id":"0444b64bd8fd00ee8c44e009e6122cb77e08f87a","author":{"_account_id":7018,"name":"Abhishek Raut","email":"rauta@vmware.com","username":"abhiraut"},"date":"2015-09-02 22:30:23.000000000","message":"Patch Set 16: Code-Review+1","accounts_in_message":[],"_revision_number":16},{"id":"4a8036bad26e937a62a43d59e5b6be6e38baf409","author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"date":"2015-09-04 05:41:39.000000000","message":"Patch Set 16:\n\n(3 comments)\n\nI am mainly wondering why this is really necessary and if it\u0027s an overkill. We seem to be able to integrate projects that have their own config files and we didn\u0027t require such a logic, at all.","accounts_in_message":[],"_revision_number":16},{"id":"7bd2c9a9e0ba76d204fb16dedaf8a8e142459f34","author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"date":"2015-09-04 05:45:11.000000000","message":"Patch Set 16:\n\nI would be -1, but I am not sure I understand why this scenario can\u0027t be addressed without making these changes.","accounts_in_message":[],"_revision_number":16},{"id":"73a3c1dc8621c5e99861fd376296b48ef237348e","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-09-04 07:07:16.000000000","message":"Patch Set 16:\n\n(3 comments)","accounts_in_message":[],"_revision_number":16},{"id":"bd622b95dd2d275e5587fd865c2ab6bb7d341dcf","author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"date":"2015-09-04 07:11:48.000000000","message":"Patch Set 16:\n\n(1 comment)","accounts_in_message":[],"_revision_number":16},{"id":"d9712a062962ea4b3506b3669e55d0acce840237","author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"date":"2015-09-05 15:13:58.000000000","message":"Patch Set 16:\n\n(2 comments)\n\nok...let me also look into this in more details...I am probably not seeing the actual problem as you do.","accounts_in_message":[],"_revision_number":16},{"id":"aa5cfe9c12a9eb478dc0203efb1feb8f97a5deda","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-09-06 11:57:04.000000000","message":"Patch Set 16: 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":16},{"id":"3f6f12522754750175f7c3059c0435d36af99580","author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"date":"2015-09-08 02:06:18.000000000","message":"Patch Set 16: Code-Review-1\n\nOk, I had time to look into this. If [1] yields positive results as I believe, this patch introduce complexity that is not necessary.\n\n[1] https://review.openstack.org/#/c/221137/","accounts_in_message":[],"_revision_number":16},{"id":"b562811cb6ecf1d0e30f31054c326fdf2fa65fa0","author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"date":"2015-09-08 02:57:48.000000000","message":"Patch Set 16:\n\nArmando\u0027 solution is that vendor plugin copies vendor config file into neutron directory before devstack copies the file. I feel strange that vendor plugin modifies a file in neutron directory and the method makes all plugins do.\n\nI prefer this patch over Armando\u0027s although I don\u0027t give +1 to this patch since I don\u0027t like complicated directory path.","accounts_in_message":[],"_revision_number":16},{"id":"b11e20ce03f6b08fcaa9bdf7d6b17009cc0eed15","author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"date":"2015-09-08 03:06:31.000000000","message":"Patch Set 16: Code-Review-1\n\nOops, I\u0027m sorry for my misunderstanding. Armando\u0027 solution is that vendor plugin copies vendor config file into /etc/neutron/plugins/ directly. It\u0027s very nice idea.\n\nI agree with Armando.","accounts_in_message":[],"_revision_number":16},{"id":"437420b7f092a1b9ea700e2cceb6d5ca81d68eed","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-09-08 03:52:38.000000000","message":"Patch Set 16:\n\nThe main goal of this patch is just to solve the issue caused by one line in current neutron-legacy:\n\n cp $NEUTRON_DIR/$Q_PLUGIN_CONF_FILE /$Q_PLUGIN_CONF_FILE\n\nwhere $NEUTRON_DIR/$Q_PLUGIN_CONF_FILE no longer exist after decomposition.\n\nSorry, I still don\u0027t know how the solution in [1] can solve this problem. Please teach me. Thank you.","accounts_in_message":[],"_revision_number":16},{"id":"a09e193d98411406eb26c9f560be1eefec7abedf","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-09-08 03:52:48.000000000","message":"Patch Set 17: Patch Set 16 was rebased","accounts_in_message":[],"_revision_number":16},{"id":"46b028fc84abe18548301ffc7d4de486922f3dac","author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"date":"2015-09-08 04:08:39.000000000","message":"Patch Set 17:\n\nThe solution is in [1], right in front of your eyes. I dunno how else to help you, if not doing the work myself :)","accounts_in_message":[],"_revision_number":17},{"id":"691f580fff417fe70fcfcd7cebd49d4074aa0e8c","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-09-08 04:17:03.000000000","message":"Patch Set 17:\n\nYes, I read the code. The part I understood is some codes can be done in vendor\u0027s plugin. But I think this line in neutron-legacy still needs to modified:\n\n  cp $NEUTRON_DIR/$Q_PLUGIN_CONF_FILE /$Q_PLUGIN_CONF_FILE\n\nOtherwise those decomposed plugins will break when running neutron-legacy. Do you agree?","accounts_in_message":[],"_revision_number":17},{"id":"1229fe7ded2b9b1d684c23ae75298e212440869a","author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"date":"2015-09-08 04:27:18.000000000","message":"Patch Set 17:\n\nno","accounts_in_message":[],"_revision_number":17},{"id":"344058ca4c564f77120d01266b507c7f62d40d52","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-09-08 04:35:00.000000000","message":"Patch Set 17:\n\nOK. That\u0027s the part I don\u0027t understand.\n\nIn our plugin, the old config file was in\nneutron:/etc/neutron/plugins/vmware/.\nThe new config is in vmware-nsx:vmware_nsx/etc/.\nSo either I define Q_PLUGIN_CONF_FILE as the former or the latter. The line will break.\n\n  cp $NEUTRON_DIR/$Q_PLUGIN_CONF_FILE /$Q_PLUGIN_CONF_FILE\n\nIs that right?","accounts_in_message":[],"_revision_number":17},{"id":"98cb58efc409eb973b7b142588be325f2481677a","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-09-08 04:41:08.000000000","message":"Patch Set 17:\n\nOK, I guess you are trying to make a copy in the neutron directory first, so neutron-legacy will find the file. Right?!","accounts_in_message":[],"_revision_number":17},{"id":"5f1fb1a4a003348b77d1b6d0b176d11270e367ec","author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"date":"2015-09-08 04:43:02.000000000","message":"Patch Set 17:\n\n\u003e Please teach me.\nBecause OVN devstack plugin has serious bug. He uses ml2 plugin.\n\nHowever, I agree that vender plugin prepares his config although I knew it.","accounts_in_message":[],"_revision_number":17},{"id":"01a96d31724bfb8617ddddfc984e851864b6ac17","author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"date":"2015-09-08 04:44:11.000000000","message":"Patch Set 17:\n\nShin-Hao, don\u0027t worry. You\u0027re right about it.","accounts_in_message":[],"_revision_number":17},{"id":"c5441e2ed4963ecf1cd30f819bb8c53c8afa44aa","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2015-09-08 04:57:50.000000000","message":"Patch Set 17:\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/86/212786/17/11190/index.html : SUCCESS in 01 h 00 m 09 s","accounts_in_message":[],"_revision_number":17},{"id":"997dfd600d3a37ab87453587b62ecbddd5b79526","author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"date":"2015-09-08 04:59:46.000000000","message":"Patch Set 17:\n\nI uploaded a patch[1]. I prefer such patch because it doesn\u0027t add new parameter and makes sense to other vender plugin maintainers.\n\n[1]: https://review.openstack.org/#/c/221146/","accounts_in_message":[],"_revision_number":17},{"id":"849c301c611f2ebe30e2b78e4552610f2ec75ec0","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-09-08 05:05:48.000000000","message":"Patch Set 17: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-tempest-dsvm-full http://logs.openstack.org/86/212786/17/check/gate-tempest-dsvm-full/82e5031/ : SUCCESS in 53m 45s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/86/212786/17/check/gate-tempest-dsvm-postgres-full/298c869/ : SUCCESS in 40m 59s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/86/212786/17/check/gate-tempest-dsvm-neutron-full/aebc1d5/ : SUCCESS in 1h 12m 50s\n- gate-grenade-dsvm http://logs.openstack.org/86/212786/17/check/gate-grenade-dsvm/f1db17f/ : SUCCESS in 37m 45s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/86/212786/17/check/gate-tempest-dsvm-large-ops/e7693bd/ : SUCCESS in 28m 16s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/86/212786/17/check/gate-tempest-dsvm-neutron-large-ops/9200d7f/ : SUCCESS in 26m 14s\n- gate-devstack-bashate http://logs.openstack.org/86/212786/17/check/gate-devstack-bashate/6519be7/ : SUCCESS in 34s\n- gate-devstack-unit-tests http://logs.openstack.org/86/212786/17/check/gate-devstack-unit-tests/c897f05/ : SUCCESS in 25s\n- gate-grenade-dsvm-partial-ncpu http://logs.openstack.org/86/212786/17/check/gate-grenade-dsvm-partial-ncpu/0ee4b19/ : SUCCESS in 38m 31s\n- gate-tempest-dsvm-ironic-pxe_ssh http://logs.openstack.org/86/212786/17/check/gate-tempest-dsvm-ironic-pxe_ssh/03a59cd/ : SUCCESS in 34m 57s (non-voting)\n- gate-devstack-dsvm-updown http://logs.openstack.org/86/212786/17/check/gate-devstack-dsvm-updown/2ba5532/ : SUCCESS in 21m 46s\n- gate-tempest-dsvm-f21 http://logs.openstack.org/86/212786/17/check/gate-tempest-dsvm-f21/c815192/ : SUCCESS in 44m 37s (non-voting)\n- gate-tempest-dsvm-centos7 http://logs.openstack.org/86/212786/17/check/gate-tempest-dsvm-centos7/5b1f339/ : SUCCESS in 30m 34s (non-voting)\n- gate-devstack-publish-docs http://docs-draft.openstack.org/86/212786/17/check/gate-devstack-publish-docs/019b3c8//doc/build/html/ : SUCCESS in 1m 37s\n- gate-swift-dsvm-functional-nv http://logs.openstack.org/86/212786/17/check/gate-swift-dsvm-functional-nv/42eb792/ : SUCCESS in 33m 25s (non-voting)\n- gate-grenade-dsvm-neutron http://logs.openstack.org/86/212786/17/check/gate-grenade-dsvm-neutron/465c9a3/ : SUCCESS in 44m 06s\n- gate-tempest-dsvm-multinode-full http://logs.openstack.org/86/212786/17/check/gate-tempest-dsvm-multinode-full/94184f5/ : FAILURE in 1h 00m 00s (non-voting)\n- gate-tempest-dsvm-neutron-multinode-full http://logs.openstack.org/86/212786/17/check/gate-tempest-dsvm-neutron-multinode-full/063bc8f/ : SUCCESS in 1h 08m 05s (non-voting)","accounts_in_message":[],"_revision_number":17},{"id":"1f95d9d262183946b8bcf4d1a9f00520378fdeab","author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"date":"2015-09-08 05:53:27.000000000","message":"Patch Set 17:\n\nHi Shih-Hao Li, what you really want is to get rid of all the vmware related code from [1,2] and centralize it in [3]. With the use of extras.d and external plugin support in Devstack, this code has no longer reason to exist in the devstack tree. If you do that, and follow the OVN use case I showed you, you can successfully integrate your plugin with Devstack and avoid fiddling with a logic that is labelled as \u0027legacy\u0027.\n\n[1] https://github.com/openstack-dev/devstack/tree/master/lib/neutron_plugins\n[2] https://github.com/openstack-dev/devstack/tree/master/lib/neutron_thirdparty\n[3] https://github.com/openstack/vmware-nsx/tree/master/devstack","accounts_in_message":[],"_revision_number":17},{"id":"903763a27bd8ce6ce7ed322fc889c7a600f31bd7","author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"date":"2015-09-08 05:55:16.000000000","message":"Patch Set 17:\n\n@Hirofumi: the fact that OVN uses the ml2 plugin is a bug which I identified, but has no relevance to this discussion IMO. As a matter of fact [1], address it partially.\n\n[1] https://review.openstack.org/#/c/221137/","accounts_in_message":[],"_revision_number":17},{"id":"41461da8ed088597ff4987b66ee99236a20d5833","author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"date":"2015-09-08 05:55:31.000000000","message":"Patch Set 17:\n\nml2 plugin (config file) I mean.","accounts_in_message":[],"_revision_number":17},{"id":"30752c0afa04f5f48e03443014582c7885cb8e69","author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"date":"2015-09-08 06:26:29.000000000","message":"Patch Set 17:\n\n@Armando - the OVN uses the ML2 etc file which is still in tree. If you decompose that then that too will break.\nWhat this patch essentially does, which I am sure that you guys understand, is just enable us to specify the path of the decomposed etc file.\nWe can work around this by using the same neutron conf file and just updating that.\nBut that will kind of break the broken model.\n\nI am not sure if you have tried this with the other external plugins, but all of those that use the upstream bits require this. There is/was a similar plumgrid one too.","accounts_in_message":[],"_revision_number":17},{"id":"41a61c943645271c6d0ebe52b114974dbb4ac66a","author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"date":"2015-09-08 06:31:05.000000000","message":"Patch Set 17: -Code-Review","accounts_in_message":[],"_revision_number":17},{"id":"c7fb05bc44716995ef883bb9ba0b715715d79a72","author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"date":"2015-09-08 06:31:13.000000000","message":"Patch Set 17:\n\nok, fine.","accounts_in_message":[],"_revision_number":17},{"id":"afa2c4d6a9d4e5bcc15dbc4eb1c42c1a2ef7094f","author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"date":"2015-09-08 07:11:24.000000000","message":"Patch Set 17:\n\nI\u0027m sleprical about this patch. I believe my solution is better so that neutron legacy go forward easily. But if Dean gives +2 to this patch, I can change my mind since he is main person about neutron script refactoring.","accounts_in_message":[],"_revision_number":17},{"id":"9b175e4612fa5f9c7a908c3ad7e9c03742fdc1f0","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-09-08 15:05:07.000000000","message":"Uploaded patch set 18.","accounts_in_message":[],"_revision_number":18},{"id":"8fb9b7c595f1886bd000f5910872ea3dcb8c187a","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2015-09-08 16:03:32.000000000","message":"Patch Set 18:\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/86/212786/18/11246/index.html : SUCCESS in 55 m 18 s","accounts_in_message":[],"_revision_number":18},{"id":"e0e098f4d4a4e83db5130c1badcf01aa45416aa7","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-09-08 16:50:42.000000000","message":"Patch Set 18: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-tempest-dsvm-full http://logs.openstack.org/86/212786/18/check/gate-tempest-dsvm-full/378e7cd/ : SUCCESS in 40m 34s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/86/212786/18/check/gate-tempest-dsvm-postgres-full/e57fd20/ : SUCCESS in 48m 08s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/86/212786/18/check/gate-tempest-dsvm-neutron-full/2859a58/ : SUCCESS in 1h 28m 16s\n- gate-grenade-dsvm http://logs.openstack.org/86/212786/18/check/gate-grenade-dsvm/ffbb9a9/ : SUCCESS in 57m 41s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/86/212786/18/check/gate-tempest-dsvm-large-ops/2757f94/ : SUCCESS in 43m 40s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/86/212786/18/check/gate-tempest-dsvm-neutron-large-ops/f263458/ : SUCCESS in 27m 27s\n- gate-devstack-bashate http://logs.openstack.org/86/212786/18/check/gate-devstack-bashate/236cf82/ : SUCCESS in 23s\n- gate-devstack-unit-tests http://logs.openstack.org/86/212786/18/check/gate-devstack-unit-tests/bb066ef/ : SUCCESS in 23s\n- gate-grenade-dsvm-partial-ncpu http://logs.openstack.org/86/212786/18/check/gate-grenade-dsvm-partial-ncpu/e9abc4b/ : SUCCESS in 42m 11s\n- gate-tempest-dsvm-ironic-pxe_ssh http://logs.openstack.org/86/212786/18/check/gate-tempest-dsvm-ironic-pxe_ssh/3596513/ : SUCCESS in 38m 59s (non-voting)\n- gate-devstack-dsvm-updown http://logs.openstack.org/86/212786/18/check/gate-devstack-dsvm-updown/93ffa9d/ : SUCCESS in 27m 53s\n- gate-tempest-dsvm-f21 http://logs.openstack.org/86/212786/18/check/gate-tempest-dsvm-f21/75ae6ef/ : SUCCESS in 46m 57s (non-voting)\n- gate-tempest-dsvm-centos7 http://logs.openstack.org/86/212786/18/check/gate-tempest-dsvm-centos7/46b27e5/ : SUCCESS in 35m 15s (non-voting)\n- gate-devstack-publish-docs http://docs-draft.openstack.org/86/212786/18/check/gate-devstack-publish-docs/6b91828//doc/build/html/ : SUCCESS in 1m 53s\n- gate-swift-dsvm-functional-nv http://logs.openstack.org/86/212786/18/check/gate-swift-dsvm-functional-nv/b2cf83f/ : SUCCESS in 44m 02s (non-voting)\n- gate-grenade-dsvm-neutron http://logs.openstack.org/86/212786/18/check/gate-grenade-dsvm-neutron/f4487fe/ : SUCCESS in 57m 08s\n- gate-tempest-dsvm-multinode-full http://logs.openstack.org/86/212786/18/check/gate-tempest-dsvm-multinode-full/1f6d759/ : SUCCESS in 54m 07s (non-voting)\n- gate-tempest-dsvm-neutron-multinode-full http://logs.openstack.org/86/212786/18/check/gate-tempest-dsvm-neutron-multinode-full/6d49597/ : SUCCESS in 1h 28m 35s (non-voting)","accounts_in_message":[],"_revision_number":18},{"id":"ccab90b63285fead3b1b9b9c7d84c9d19eac5a00","author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"date":"2015-09-08 17:10:30.000000000","message":"Patch Set 18:\n\n@gary et al: I have updated [1] to use [2], and demonstrated how you can have a monolithic plugin configured successfully with DevStack by keeping all of the devstack setup logic co-located with the plugin tree.\n\nI would prefer [2] because it gives you more flexibility and affects you the least, especially in face of a refactoring of the neutron-legacy file.\n\nUltimately, the choice is yours: you\u0027re the one with the +2 rights.\n\n [1] https://review.openstack.org/#/c/221137/\n [2] https://review.openstack.org/#/c/221146/","accounts_in_message":[],"_revision_number":18},{"id":"3b06062fcc7432c18171ab22c32f3bf99e2b4402","author":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"date":"2015-09-08 18:32:13.000000000","message":"Patch Set 18: Code-Review-1\n\nI\u0027d prefer https://review.openstack.org/#/c/221146/ instead of this solution.","accounts_in_message":[],"_revision_number":18},{"id":"90e0d3b0956c9dba27f4da55ab19c9e688f889a9","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-09-08 19:01:03.000000000","message":"Patch Set 18:\n\nI think both approaches will work.\n\nIn [2], each vendor plugin is responsible to copy the source config file assuming that they know where to copy it to.\n\nIn this patch, neutron-legacy is doing the copy via a new variable indicating where the source config file is in vendor\u0027s repo.","accounts_in_message":[],"_revision_number":18},{"id":"e8c4c341490405db969245dc1205cdd0e4b5879e","author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"date":"2015-09-09 02:06:43.000000000","message":"Patch Set 18:\n\n@Shih-Hao Li: agreed. But it\u0027s solely the responsibility of the plugin to configure itself. Mixing the logic in two places seems problematic.\n\n[1] was merged with that objective in mind, and it seems that is probably best to continue down that road. Gary approved that patch too, so he should have no problem agreeing with patch [2].\n\nFor this to work, you could take the step of removing the vmware_nsx support out of DevStack entirely.\n\n[1] https://github.com/openstack-dev/devstack/commit/36daecd1a362a6a5388aa4ee5c5269563a820cbf\n[2] I4997b8eae1f433b1c23f20c06ba254568ac4982b","accounts_in_message":[],"_revision_number":18},{"id":"4c111ad0fe22dff533e195321f70e3170b95edc4","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-09-09 02:37:05.000000000","message":"Patch Set 18:\n\nI am OK with either approach.\n\nIn my approach, I try to avoid needing vendor\u0027s plugin to know where the deployed root directory (i.e. DEST) is for the source config file. Thus vendor\u0027s plugin just needs to provide the relative path in their own repo and let devstack (i.e. neutron-legacy) to add the root directory when doing the copy.","accounts_in_message":[],"_revision_number":18},{"id":"fb432973e8fce46a1fcfabbb458b854f25d681da","author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"date":"2015-09-09 02:47:59.000000000","message":"Patch Set 18:\n\nunderstood; but we have been telling you that what you came up with is a convoluted logic that ties things together for no good reason.","accounts_in_message":[],"_revision_number":18},{"id":"f86a472f02a0b1d082bbe511933dc76fd5c3ffcf","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-09-09 03:06:00.000000000","message":"Patch Set 18:\n\nNot sure what you meant convoluted.\n\nI am doing the way similar to what current neutron-legacy is doing, i.e.add NEUTRON_DIR(\u003dDEST/neutron) to the source config file, so vendor\u0027s plugin doesn\u0027t need to use DEST.\n\nI think both approaches have good reasons behind them.","accounts_in_message":[],"_revision_number":18},{"id":"c7e81ba4d46bd724a876d9d7aef17629c4bf38ab","author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"date":"2015-09-09 04:51:42.000000000","message":"Patch Set 18:\n\nok, let\u0027s agree to disagree.","accounts_in_message":[],"_revision_number":18},{"id":"c62b13068f62c7fea4e84c697c5548e75911ef4b","author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"date":"2015-09-09 05:23:49.000000000","message":"Patch Set 18: Code-Review-1\n\nLooks like Gary has given +2 on [1].\n\nBoth this change and [1] can\u0027t go in together. So you\u0027d better off turning this change into a fix for bug [2] where you remove the entire vmware_nsx code from DevStack and use approach outlined in [3].\n\n[1] https://review.openstack.org/#/c/221146/\n[2] https://bugs.launchpad.net/vmware-nsx/+bug/1483453\n[3] https://review.openstack.org/#/c/221137/","accounts_in_message":[],"_revision_number":18},{"id":"d59d6f312f5c8418ef7413622b0ab847d1011978","author":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"date":"2015-09-10 17:44:14.000000000","message":"Abandoned\n\nreplaced by https://review.openstack.org/#/c/221146/","accounts_in_message":[],"_revision_number":18},{"id":"a8793c025f707218b20bd17470b02f769c31689d","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-09-10 19:04:28.000000000","message":"Patch Set 18:\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":18}],"current_revision_number":18,"current_revision":"4a8e079589b5665ab0ba52c6de079d26fb2f0e62","revisions":{"f4b6ef930c966bfa92daa81dd3f81c23c205fcca":{"kind":"REWORK","_number":1,"created":"2015-08-13 21:01:32.000000000","uploader":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"ref":"refs/changes/86/212786/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/86/212786/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/86/212786/1"}}},"commit":{"parents":[{"commit":"57036ba2b77eb1f09449266fd7510cddfc9ec321","subject":"Merge \"fix multinode guide\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/57036ba2b77eb1f09449266fd7510cddfc9ec321"}]}],"author":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-08-13 21:00:16.000000000","tz":-420},"committer":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-08-13 21:00:16.000000000","tz":-420},"subject":"Provide alternate path for neutron plugin config files","message":"Provide alternate path for neutron plugin config files\n\nSince we are in the process of decomposing vendor specific plugins,\nsome plugin config files may no longer exist in neutron repo.\nWe need to provide an alternate path for them. So neutron-leagcy\ncan still access them if not found in the previous location.\n\nThe new path is defined in Q_PLUGIN_ALT_CONF_PATH.\n\nPartial-bug: #1483453\nChange-Id: I2fbe8b4ce5a74164a4770a58a7636b9bb3dc7d3b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/f4b6ef930c966bfa92daa81dd3f81c23c205fcca"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/f4b6ef930c966bfa92daa81dd3f81c23c205fcca"}]},"branch":"refs/heads/master"},"4b7f4440e2450ef31105a64dc2ce300eabe2537b":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2015-08-13 21:11:49.000000000","uploader":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"ref":"refs/changes/86/212786/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/86/212786/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/86/212786/2"}}},"commit":{"parents":[{"commit":"57036ba2b77eb1f09449266fd7510cddfc9ec321","subject":"Merge \"fix multinode guide\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/57036ba2b77eb1f09449266fd7510cddfc9ec321"}]}],"author":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-08-13 21:11:32.000000000","tz":-420},"committer":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-08-13 21:11:32.000000000","tz":-420},"subject":"Provide alternate path for neutron plugin config files","message":"Provide alternate path for neutron plugin config files\n\nSince we are in the process of decomposing vendor specific plugins,\nsome plugin config files may no longer exist in neutron repo.\nWe need to provide an alternate path for them. So neutron-leagcy\ncan still access them if not found in the previous location.\n\nThe new path is defined in Q_PLUGIN_ALT_CONF_PATH, which should\nbe specified in vendor\u0027s neutron_plugin_configure_common function.\n\nPartial-bug: #1483453\nChange-Id: I2fbe8b4ce5a74164a4770a58a7636b9bb3dc7d3b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/4b7f4440e2450ef31105a64dc2ce300eabe2537b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/4b7f4440e2450ef31105a64dc2ce300eabe2537b"}]},"branch":"refs/heads/master"},"fc94b7606f268fd5b4ecafe882b2fe0d0be304ce":{"kind":"REWORK","_number":3,"created":"2015-08-13 21:36:30.000000000","uploader":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"ref":"refs/changes/86/212786/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/86/212786/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/86/212786/3"}}},"commit":{"parents":[{"commit":"57036ba2b77eb1f09449266fd7510cddfc9ec321","subject":"Merge \"fix multinode guide\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/57036ba2b77eb1f09449266fd7510cddfc9ec321"}]}],"author":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-08-13 21:36:25.000000000","tz":-420},"committer":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-08-13 21:36:25.000000000","tz":-420},"subject":"Provide alternate path for neutron plugin config files","message":"Provide alternate path for neutron plugin config files\n\nSince we are in the process of decomposing vendor specific plugins,\nsome plugin config files may no longer exist in neutron repo.\nWe need to provide an alternate path for them. So neutron-leagcy\ncan still access them if not found in the previous location.\n\nThe new path is defined in Q_PLUGIN_ALT_CONF_PATH, which should\nbe specified in vendor\u0027s neutron_plugin_configure_common function.\n\nPartial-bug: #1483453\nChange-Id: I2fbe8b4ce5a74164a4770a58a7636b9bb3dc7d3b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/fc94b7606f268fd5b4ecafe882b2fe0d0be304ce"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/fc94b7606f268fd5b4ecafe882b2fe0d0be304ce"}]},"branch":"refs/heads/master"},"c9af1ba19cc682be6bdc28298c84c9428485c0bc":{"kind":"REWORK","_number":4,"created":"2015-08-13 23:54:06.000000000","uploader":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"ref":"refs/changes/86/212786/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/86/212786/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/86/212786/4"}}},"commit":{"parents":[{"commit":"57036ba2b77eb1f09449266fd7510cddfc9ec321","subject":"Merge \"fix multinode guide\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/57036ba2b77eb1f09449266fd7510cddfc9ec321"}]}],"author":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-08-13 23:54:02.000000000","tz":-420},"committer":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-08-13 23:54:02.000000000","tz":-420},"subject":"Provide alternate path for neutron plugin config files","message":"Provide alternate path for neutron plugin config files\n\nSince we are in the process of decomposing vendor specific plugins,\nsome plugin config files may no longer exist in neutron repo.\nWe need to provide an alternate path for them. So neutron-leagcy\ncan still access them if not found in the previous location.\n\nThe new path is defined in Q_PLUGIN_ALT_CONF_PATH, which should\nbe specified in vendor\u0027s neutron_plugin_configure_common function.\n\nPartial-bug: #1483453\nChange-Id: I2fbe8b4ce5a74164a4770a58a7636b9bb3dc7d3b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/c9af1ba19cc682be6bdc28298c84c9428485c0bc"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/c9af1ba19cc682be6bdc28298c84c9428485c0bc"}]},"branch":"refs/heads/master"},"c8309d83f890075a03819c2e614e880e71390ad9":{"kind":"REWORK","_number":5,"created":"2015-08-14 01:34:50.000000000","uploader":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"ref":"refs/changes/86/212786/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/86/212786/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/86/212786/5"}}},"commit":{"parents":[{"commit":"57036ba2b77eb1f09449266fd7510cddfc9ec321","subject":"Merge \"fix multinode guide\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/57036ba2b77eb1f09449266fd7510cddfc9ec321"}]}],"author":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-08-14 01:34:18.000000000","tz":-420},"committer":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-08-14 01:34:18.000000000","tz":-420},"subject":"Provide alternate path for neutron plugin config files","message":"Provide alternate path for neutron plugin config files\n\nSince we are in the process of decomposing vendor specific plugins,\nsome plugin config files may no longer exist in neutron repo.\nWe need to provide an alternate path for them. So neutron-leagcy\ncan still access them if not found in the previous location.\n\nThe new path is defined in Q_PLUGIN_ALT_CONF_PATH, which should\nbe specified in vendor\u0027s neutron_plugin_configure_common function.\n\nPartial-bug: #1483453\nChange-Id: I2fbe8b4ce5a74164a4770a58a7636b9bb3dc7d3b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/c8309d83f890075a03819c2e614e880e71390ad9"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/c8309d83f890075a03819c2e614e880e71390ad9"}]},"branch":"refs/heads/master"},"e874e252ecf7d3a1ac424d2fc6adfc2072c0a036":{"kind":"REWORK","_number":6,"created":"2015-08-14 06:58:53.000000000","uploader":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"ref":"refs/changes/86/212786/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/86/212786/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/86/212786/6"}}},"commit":{"parents":[{"commit":"57036ba2b77eb1f09449266fd7510cddfc9ec321","subject":"Merge \"fix multinode guide\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/57036ba2b77eb1f09449266fd7510cddfc9ec321"}]}],"author":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-08-14 06:58:43.000000000","tz":-420},"committer":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-08-14 06:58:43.000000000","tz":-420},"subject":"Provide alternate path for neutron plugin config files","message":"Provide alternate path for neutron plugin config files\n\nSince we are in the process of decomposing vendor specific plugins,\nsome plugin config files may no longer exist in neutron repo.\nWe need to provide an alternate path for them. So neutron-leagcy\ncan still access them if not found in the previous location.\n\nThe new path is defined in Q_PLUGIN_ALT_CONF_PATH, which should\nbe specified in vendor\u0027s neutron_plugin_configure_common function.\n\nPartial-bug: #1483453\nChange-Id: I2fbe8b4ce5a74164a4770a58a7636b9bb3dc7d3b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/e874e252ecf7d3a1ac424d2fc6adfc2072c0a036"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/e874e252ecf7d3a1ac424d2fc6adfc2072c0a036"}]},"branch":"refs/heads/master"},"afb56cf711b6a4a30efb7366d4a4138ce76b386e":{"kind":"NO_CODE_CHANGE","_number":7,"created":"2015-08-14 07:02:12.000000000","uploader":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"ref":"refs/changes/86/212786/7","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/86/212786/7","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/7 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/7 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/7 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/86/212786/7"}}},"commit":{"parents":[{"commit":"57036ba2b77eb1f09449266fd7510cddfc9ec321","subject":"Merge \"fix multinode guide\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/57036ba2b77eb1f09449266fd7510cddfc9ec321"}]}],"author":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-08-14 06:58:43.000000000","tz":-420},"committer":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-08-14 07:02:12.000000000","tz":0},"subject":"Provide alternate path for neutron plugin config files","message":"Provide alternate path for neutron plugin config files\n\nSince we are in the process of decomposing vendor specific plugins,\nsome plugin config files may no longer exist in neutron repo.\nWe need to provide an alternate path for them. So neutron-legacy\ncan still access them if not found in the previous location.\n\nThe new path is defined in Q_PLUGIN_VENDOR_CONF_PATH, which should\nbe specified in vendor\u0027s neutron_plugin_configure_common function.\n\nPartial-bug: #1483453\nChange-Id: I2fbe8b4ce5a74164a4770a58a7636b9bb3dc7d3b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/afb56cf711b6a4a30efb7366d4a4138ce76b386e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/afb56cf711b6a4a30efb7366d4a4138ce76b386e"}]},"branch":"refs/heads/master"},"571a90cf4b093f3bed11d86c186a92eecaf73a69":{"kind":"REWORK","_number":8,"created":"2015-08-17 03:48:53.000000000","uploader":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"ref":"refs/changes/86/212786/8","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/86/212786/8","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/8 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/8 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/8 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/86/212786/8"}}},"commit":{"parents":[{"commit":"57036ba2b77eb1f09449266fd7510cddfc9ec321","subject":"Merge \"fix multinode guide\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/57036ba2b77eb1f09449266fd7510cddfc9ec321"}]}],"author":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-08-17 03:48:27.000000000","tz":-420},"committer":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-08-17 03:48:27.000000000","tz":-420},"subject":"Provide alternate path for neutron plugin config files","message":"Provide alternate path for neutron plugin config files\n\nSince we are in the process of decomposing vendor specific plugins,\nsome plugin config files may no longer exist in neutron repo.\nWe need to provide an alternate path for them. So neutron-legacy\ncan still access them if not found in the previous location.\n\nThe new path is defined in Q_PLUGIN_VENDOR_CONF_PATH, which should\nbe specified in vendor\u0027s neutron_plugin_configure_common function.\n\nPartial-bug: #1483453\nChange-Id: I2fbe8b4ce5a74164a4770a58a7636b9bb3dc7d3b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/571a90cf4b093f3bed11d86c186a92eecaf73a69"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/571a90cf4b093f3bed11d86c186a92eecaf73a69"}]},"branch":"refs/heads/master"},"5b785ddf00659d9f844c49aa9e81dea4fa6240ae":{"kind":"REWORK","_number":9,"created":"2015-08-17 03:59:57.000000000","uploader":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"ref":"refs/changes/86/212786/9","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/86/212786/9","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/9 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/9 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/9 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/86/212786/9"}}},"commit":{"parents":[{"commit":"57036ba2b77eb1f09449266fd7510cddfc9ec321","subject":"Merge \"fix multinode guide\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/57036ba2b77eb1f09449266fd7510cddfc9ec321"}]}],"author":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-08-17 03:59:12.000000000","tz":-420},"committer":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-08-17 03:59:12.000000000","tz":-420},"subject":"Provide alternate paths for neutron plugin config files","message":"Provide alternate paths for neutron plugin config files\n\nSince we are in the process of decomposing vendor specific plugins,\nsome plugin config files may no longer exist in neutron repo.\nWe need to provide alternate paths for neutron-legacy to locate them.\n\nThe new logic for locating the source config file is:\n1. If the full path Q_PLUGIN_SRC_CONF_DIR is defined, use it.\n2. Otherwise, try the relative path Q_PLUGIN_SRC_CONF_PATH\n   under directory DEST.\n3. If neither of them is defined, use Q_PLUGIN_CONF_PATH\n   under directory NEUTRON_DIR (original approach).\n\nPartial-bug: #1483453\nChange-Id: I2fbe8b4ce5a74164a4770a58a7636b9bb3dc7d3b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/5b785ddf00659d9f844c49aa9e81dea4fa6240ae"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/5b785ddf00659d9f844c49aa9e81dea4fa6240ae"}]},"branch":"refs/heads/master"},"12a2c98df2902b7107e964cd2ced13b95728cd2a":{"kind":"TRIVIAL_REBASE","_number":10,"created":"2015-08-18 17:28:01.000000000","uploader":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"ref":"refs/changes/86/212786/10","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/86/212786/10","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/10 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/10 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/10 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/86/212786/10"}}},"commit":{"parents":[{"commit":"5384a940276f2428b44940c103c2a54302aeb724","subject":"Merge \"Prevent spawning two VPN agents\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/5384a940276f2428b44940c103c2a54302aeb724"}]}],"author":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-08-17 03:59:12.000000000","tz":-420},"committer":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-08-18 17:28:01.000000000","tz":0},"subject":"Provide alternate paths for neutron plugin config files","message":"Provide alternate paths for neutron plugin config files\n\nSince we are in the process of decomposing vendor specific plugins,\nsome plugin config files may no longer exist in neutron repo.\nWe need to provide alternate paths for neutron-legacy to locate them.\n\nThe new logic for locating the source config file is:\n1. If the full path Q_PLUGIN_SRC_CONF_DIR is defined, use it.\n2. Otherwise, try the relative path Q_PLUGIN_SRC_CONF_PATH\n   under directory DEST.\n3. If neither of them is defined, use Q_PLUGIN_CONF_PATH\n   under directory NEUTRON_DIR (original approach).\n\nPartial-bug: #1483453\nChange-Id: I2fbe8b4ce5a74164a4770a58a7636b9bb3dc7d3b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/12a2c98df2902b7107e964cd2ced13b95728cd2a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/12a2c98df2902b7107e964cd2ced13b95728cd2a"}]},"branch":"refs/heads/master"},"09d839f15cabe88e22669eeb26ea2027afc4ab37":{"kind":"REWORK","_number":11,"created":"2015-08-19 20:50:23.000000000","uploader":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"ref":"refs/changes/86/212786/11","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/86/212786/11","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/11 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/11 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/11 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/86/212786/11"}}},"commit":{"parents":[{"commit":"57036ba2b77eb1f09449266fd7510cddfc9ec321","subject":"Merge \"fix multinode guide\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/57036ba2b77eb1f09449266fd7510cddfc9ec321"}]}],"author":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-08-19 20:50:18.000000000","tz":-420},"committer":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-08-19 20:50:18.000000000","tz":-420},"subject":"Provide alternate path for neutron plugin config files","message":"Provide alternate path for neutron plugin config files\n\nSince we are in the process of decomposing vendor specific plugins,\nsome plugin config files may no longer exist in neutron repo.\nWe need to provide alternate path for neutron-legacy to locate them.\n\nThe new logic for locating the source config file is:\n1. If Q_PLUGIN_SRC_CONF_PATH is defined, check if it is an absolute path.\n   If true, use it directly.\n   Otherwise, join it with root directory DEST.\n2. If Q_PLUGIN_SRC_CONF_PATH`` is not defined, get the source config file\n   from NEUTRON_DIR/Q_PLUGIN_CONF_FILE (original approach).\n\nPartial-bug: #1483453\nChange-Id: I2fbe8b4ce5a74164a4770a58a7636b9bb3dc7d3b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/09d839f15cabe88e22669eeb26ea2027afc4ab37"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/09d839f15cabe88e22669eeb26ea2027afc4ab37"}]},"branch":"refs/heads/master"},"b6321dd4e2d11d0ab498949f75ecf0f5e7dae9d6":{"kind":"TRIVIAL_REBASE","_number":12,"created":"2015-08-20 18:53:08.000000000","uploader":{"_account_id":13438,"name":"Kobi Samoray","email":"kobi.samoray@broadcom.com","username":"ksamoray"},"ref":"refs/changes/86/212786/12","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/86/212786/12","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/12 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/12 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/12 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/86/212786/12"}}},"commit":{"parents":[{"commit":"32e0febbc086a1accd90c3d9a9535ebb75495afe","subject":"Merge \"Cloning requirements repository before pip install\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/32e0febbc086a1accd90c3d9a9535ebb75495afe"}]}],"author":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-08-19 20:50:18.000000000","tz":-420},"committer":{"name":"Kobi Samoray","email":"ksamoray@vmware.com","date":"2015-08-20 18:53:08.000000000","tz":0},"subject":"Provide alternate path for neutron plugin config files","message":"Provide alternate path for neutron plugin config files\n\nSince we are in the process of decomposing vendor specific plugins,\nsome plugin config files may no longer exist in neutron repo.\nWe need to provide alternate path for neutron-legacy to locate them.\n\nThe new logic for locating the source config file is:\n1. If Q_PLUGIN_SRC_CONF_PATH is defined, check if it is an absolute path.\n   If true, use it directly.\n   Otherwise, join it with root directory DEST.\n2. If Q_PLUGIN_SRC_CONF_PATH`` is not defined, get the source config file\n   from NEUTRON_DIR/Q_PLUGIN_CONF_FILE (original approach).\n\nPartial-bug: #1483453\nChange-Id: I2fbe8b4ce5a74164a4770a58a7636b9bb3dc7d3b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b6321dd4e2d11d0ab498949f75ecf0f5e7dae9d6"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b6321dd4e2d11d0ab498949f75ecf0f5e7dae9d6"}]},"branch":"refs/heads/master"},"af3604559911ec7d98d131fcee724d5485c200ff":{"kind":"NO_CODE_CHANGE","_number":13,"created":"2015-08-24 13:40:49.000000000","uploader":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"ref":"refs/changes/86/212786/13","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/86/212786/13","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/13 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/13 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/13 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/86/212786/13"}}},"commit":{"parents":[{"commit":"32e0febbc086a1accd90c3d9a9535ebb75495afe","subject":"Merge \"Cloning requirements repository before pip install\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/32e0febbc086a1accd90c3d9a9535ebb75495afe"}]}],"author":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-08-19 20:50:18.000000000","tz":-420},"committer":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-08-24 13:40:49.000000000","tz":0},"subject":"Provide alternate path for neutron plugin config files","message":"Provide alternate path for neutron plugin config files\n\nSince we are in the process of decomposing vendor specific plugins,\nsome plugin config files may no longer exist in neutron repo.\nWe need to provide alternate path for neutron-legacy to locate them.\n\nThe new logic for locating the source config file is:\n1. If Q_PLUGIN_SRC_CONF_PATH is defined, check if it is an absolute path.\n   If true, use it directly.\n   Otherwise, join it with root directory DEST.\n2. If Q_PLUGIN_SRC_CONF_PATH is not defined, get the source config file\n   from NEUTRON_DIR/Q_PLUGIN_CONF_FILE (original approach).\n\nPartial-bug: #1483453\nChange-Id: I2fbe8b4ce5a74164a4770a58a7636b9bb3dc7d3b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/af3604559911ec7d98d131fcee724d5485c200ff"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/af3604559911ec7d98d131fcee724d5485c200ff"}]},"branch":"refs/heads/master"},"3c169c6126ab5e93f2f43b7ea2dc732ab0912923":{"kind":"NO_CODE_CHANGE","_number":14,"created":"2015-08-28 22:26:18.000000000","uploader":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"ref":"refs/changes/86/212786/14","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/86/212786/14","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/14 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/14 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/14 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/86/212786/14"}}},"commit":{"parents":[{"commit":"32e0febbc086a1accd90c3d9a9535ebb75495afe","subject":"Merge \"Cloning requirements repository before pip install\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/32e0febbc086a1accd90c3d9a9535ebb75495afe"}]}],"author":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-08-19 20:50:18.000000000","tz":-420},"committer":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-08-28 22:26:18.000000000","tz":0},"subject":"Provide alternate path for neutron plugin config files","message":"Provide alternate path for neutron plugin config files\n\nSince we are in the process of decomposing vendor specific plugins,\nsome plugin config files may no longer exist in neutron repo.\nWe need to provide alternate path for neutron-legacy to locate them.\n\nThe proposed new logic for locating the source config file is:\n1. If Q_PLUGIN_SRC_CONF_PATH is defined, check if it is an absolute path.\n   If true, use it directly.\n   Otherwise, join it with root directory DEST.\n2. If Q_PLUGIN_SRC_CONF_PATH is not defined, get the source config file\n   from NEUTRON_DIR/Q_PLUGIN_CONF_FILE (original approach).\n\nThe rationales behind this design are:\n1. The current neutron-legacy adds root directory NEUTRON_DIR\n   (i.e. DEST/neutron/) to Q_PLUGIN_CONF_FILE to locate the source\n   config file, where Q_PLUGIN_CONF_FILE is a relative path defined\n   in vendor\u0027s neutron_plugin_configure_common function.\n\n   Since we are decomposing vendor specific plugins from neutron repo,\n   we will leave the definitions in neutron_plugin_configure_common\n   stayed relative in vendor\u0027s own repo and let devstack to add root\n   directory to those relative paths at run-time.\n\n   Thus the new variable Q_PLUGIN_SRC_CONF_PATH we are adding in\n   neutron_plugin_configure_common function is intended to be a\n   relative path.\n\n\n\nMy point is DEST is a variable that is introduced in devstack repo and assigned a run-time value when running devstack.\nI would avoid using DEST in vendor\u0027s repo because they cross two repos. Since we are decomposing vendor plugins from neutron, I think we should try not introducing new cross-references for other stuffs too.\nSo my original idea is to keep the use of DEST inside devstack repo (i.e. use it in neutron-legacy) and let the definitions in vendor\u0027s repo (i.e. function neutron_plugin_configure_common) only define the relative path from their repo root.\n\n3. For vendors who like to define full path in their \n   neutron_plugin_configure_common, they can still assign fill path\n\n\ne are adding is Still allow users to define full path for their config files.\n\n\nPartial-bug: #1483453\nChange-Id: I2fbe8b4ce5a74164a4770a58a7636b9bb3dc7d3b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/3c169c6126ab5e93f2f43b7ea2dc732ab0912923"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/3c169c6126ab5e93f2f43b7ea2dc732ab0912923"}]},"branch":"refs/heads/master"},"0a69522f11538c7007ca9d9024aae340cb7f244c":{"kind":"NO_CODE_CHANGE","_number":15,"created":"2015-08-28 22:43:37.000000000","uploader":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"ref":"refs/changes/86/212786/15","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/86/212786/15","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/15 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/15 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/15 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/86/212786/15"}}},"commit":{"parents":[{"commit":"32e0febbc086a1accd90c3d9a9535ebb75495afe","subject":"Merge \"Cloning requirements repository before pip install\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/32e0febbc086a1accd90c3d9a9535ebb75495afe"}]}],"author":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-08-19 20:50:18.000000000","tz":-420},"committer":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-08-28 22:43:37.000000000","tz":0},"subject":"Provide alternate path for neutron plugin config files","message":"Provide alternate path for neutron plugin config files\n\nSince we are in the process of decomposing vendor specific plugins,\nsome plugin config files may no longer exist in neutron repo.\nWe need to provide alternate path for neutron-legacy to locate them.\n\nThe proposed new logic for locating the source config file is:\n1. If Q_PLUGIN_SRC_CONF_PATH is defined, check if it is an absolute path.\n   If true, use it directly.\n   Otherwise, join it with root directory DEST.\n2. If Q_PLUGIN_SRC_CONF_PATH is not defined, get the source config file\n   from NEUTRON_DIR/Q_PLUGIN_CONF_FILE (original approach).\n\nThe rationales behind this design are:\n1. The current neutron-legacy adds root directory NEUTRON_DIR\n   (i.e. DEST/neutron/) to Q_PLUGIN_CONF_PATH to locate the source\n   config file, where Q_PLUGIN_CONF_PATH is a relative path defined\n   in vendor\u0027s neutron_plugin_configure_common function.\n\n   Since we are decomposing vendor specific plugins from neutron repo,\n   we will leave the definitions in neutron_plugin_configure_common\n   stayed relative in vendor\u0027s own repo and let devstack to add root\n   directory to those relative paths at run-time.\n\n   So the new variable Q_PLUGIN_SRC_CONF_PATH we are adding in\n   neutron_plugin_configure_common function is intended to be a\n   relative path.\n\n2. After moving config files from neutron repo to vendor\u0027s repo,\n   each vendor may place them in different directories. This design\n   allow vendors to place wherever they want in their own repo.\n   Thus it is no longer tied to the original path in neutron repo,\n   i.e. the value defined in Q_PLUGIN_CONF_PATH.\n\n   In other words, Q_PLUGIN_CONF_PATH is more like the destination\n   config path (where the config files are copied to by neutron-legacy),\n   and the new variable Q_PLUGIN_SRC_CONF_PATH is like the source\n   config path (where the config files are copied from by\n   neutron-legacy). And neutron-legacy will add root directory to\n   each of them at run-time.\n\n3. To accommodate vendors who like to define full paths in their \n   neutron_plugin_configure_common, they can still do it with\n   the new variable Q_PLUGIN_SRC_CONF_PATH.\n\nPartial-bug: #1483453\nChange-Id: I2fbe8b4ce5a74164a4770a58a7636b9bb3dc7d3b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/0a69522f11538c7007ca9d9024aae340cb7f244c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/0a69522f11538c7007ca9d9024aae340cb7f244c"}]},"branch":"refs/heads/master"},"5e7ae41ab0c327a5007983c46643ea676cb7fabe":{"kind":"TRIVIAL_REBASE","_number":16,"created":"2015-08-29 04:29:24.000000000","uploader":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"ref":"refs/changes/86/212786/16","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/86/212786/16","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/16 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/16 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/16 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/86/212786/16"}}},"commit":{"parents":[{"commit":"9f52d648fbffe342b9c822f6519a7992c914f50f","subject":"Merge \"Add file creation test\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/9f52d648fbffe342b9c822f6519a7992c914f50f"}]}],"author":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-08-19 20:50:18.000000000","tz":-420},"committer":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-08-29 04:29:24.000000000","tz":0},"subject":"Provide alternate path for neutron plugin config files","message":"Provide alternate path for neutron plugin config files\n\nSince we are in the process of decomposing vendor specific plugins,\nsome plugin config files may no longer exist in neutron repo.\nWe need to provide alternate path for neutron-legacy to locate them.\n\nThe proposed new logic for locating the source config file is:\n1. If Q_PLUGIN_SRC_CONF_PATH is defined, check if it is an absolute path.\n   If true, use it directly.\n   Otherwise, join it with root directory DEST.\n2. If Q_PLUGIN_SRC_CONF_PATH is not defined, get the source config file\n   from NEUTRON_DIR/Q_PLUGIN_CONF_FILE (original approach).\n\nThe rationales behind this design are:\n1. The current neutron-legacy adds root directory NEUTRON_DIR\n   (i.e. DEST/neutron/) to Q_PLUGIN_CONF_PATH to locate the source\n   config file, where Q_PLUGIN_CONF_PATH is a relative path defined\n   in vendor\u0027s neutron_plugin_configure_common function.\n\n   Since we are decomposing vendor specific plugins from neutron repo,\n   we will leave the definitions in neutron_plugin_configure_common\n   stayed relative in vendor\u0027s own repo and let devstack to add root\n   directory to those relative paths at run-time.\n\n   So the new variable Q_PLUGIN_SRC_CONF_PATH we are adding in\n   neutron_plugin_configure_common function is intended to be a\n   relative path.\n\n2. After moving config files from neutron repo to vendor\u0027s repo,\n   each vendor may place them in different directories. This design\n   allow vendors to place wherever they want in their own repo.\n   Thus it is no longer tied to the original path in neutron repo,\n   i.e. the value defined in Q_PLUGIN_CONF_PATH.\n\n   In other words, Q_PLUGIN_CONF_PATH is more like the destination\n   config path (where the config files are copied to by neutron-legacy),\n   and the new variable Q_PLUGIN_SRC_CONF_PATH is like the source\n   config path (where the config files are copied from by\n   neutron-legacy). And neutron-legacy will add root directory to\n   each of them at run-time.\n\n3. To accommodate vendors who like to define full paths in their \n   neutron_plugin_configure_common, they can still do it with\n   the new variable Q_PLUGIN_SRC_CONF_PATH.\n\nPartial-bug: #1483453\nChange-Id: I2fbe8b4ce5a74164a4770a58a7636b9bb3dc7d3b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/5e7ae41ab0c327a5007983c46643ea676cb7fabe"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/5e7ae41ab0c327a5007983c46643ea676cb7fabe"}]},"branch":"refs/heads/master"},"b6a8e742b8c317510eddb743b725d8723fb315d0":{"kind":"TRIVIAL_REBASE","_number":17,"created":"2015-09-08 03:52:48.000000000","uploader":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"ref":"refs/changes/86/212786/17","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/86/212786/17","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/17 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/17 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/17 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/86/212786/17"}}},"commit":{"parents":[{"commit":"9451021200cf333dc624275ef832acd3f37dd553","subject":"Fix typo in _move_neutron_addresses_route()","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/9451021200cf333dc624275ef832acd3f37dd553"}]}],"author":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-08-19 20:50:18.000000000","tz":-420},"committer":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-09-08 03:52:48.000000000","tz":0},"subject":"Provide alternate path for neutron plugin config files","message":"Provide alternate path for neutron plugin config files\n\nSince we are in the process of decomposing vendor specific plugins,\nsome plugin config files may no longer exist in neutron repo.\nWe need to provide alternate path for neutron-legacy to locate them.\n\nThe proposed new logic for locating the source config file is:\n1. If Q_PLUGIN_SRC_CONF_PATH is defined, check if it is an absolute path.\n   If true, use it directly.\n   Otherwise, join it with root directory DEST.\n2. If Q_PLUGIN_SRC_CONF_PATH is not defined, get the source config file\n   from NEUTRON_DIR/Q_PLUGIN_CONF_FILE (original approach).\n\nThe rationales behind this design are:\n1. The current neutron-legacy adds root directory NEUTRON_DIR\n   (i.e. DEST/neutron/) to Q_PLUGIN_CONF_PATH to locate the source\n   config file, where Q_PLUGIN_CONF_PATH is a relative path defined\n   in vendor\u0027s neutron_plugin_configure_common function.\n\n   Since we are decomposing vendor specific plugins from neutron repo,\n   we will leave the definitions in neutron_plugin_configure_common\n   stayed relative in vendor\u0027s own repo and let devstack to add root\n   directory to those relative paths at run-time.\n\n   So the new variable Q_PLUGIN_SRC_CONF_PATH we are adding in\n   neutron_plugin_configure_common function is intended to be a\n   relative path.\n\n2. After moving config files from neutron repo to vendor\u0027s repo,\n   each vendor may place them in different directories. This design\n   allow vendors to place wherever they want in their own repo.\n   Thus it is no longer tied to the original path in neutron repo,\n   i.e. the value defined in Q_PLUGIN_CONF_PATH.\n\n   In other words, Q_PLUGIN_CONF_PATH is more like the destination\n   config path (where the config files are copied to by neutron-legacy),\n   and the new variable Q_PLUGIN_SRC_CONF_PATH is like the source\n   config path (where the config files are copied from by\n   neutron-legacy). And neutron-legacy will add root directory to\n   each of them at run-time.\n\n3. To accommodate vendors who like to define full paths in their \n   neutron_plugin_configure_common, they can still do it with\n   the new variable Q_PLUGIN_SRC_CONF_PATH.\n\nPartial-bug: #1483453\nChange-Id: I2fbe8b4ce5a74164a4770a58a7636b9bb3dc7d3b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b6a8e742b8c317510eddb743b725d8723fb315d0"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b6a8e742b8c317510eddb743b725d8723fb315d0"}]},"branch":"refs/heads/master"},"4a8e079589b5665ab0ba52c6de079d26fb2f0e62":{"kind":"REWORK","_number":18,"created":"2015-09-08 15:05:07.000000000","uploader":{"_account_id":17734,"name":"Shih-Hao Li","email":"shihli@vmware.com","username":"shli"},"ref":"refs/changes/86/212786/18","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/86/212786/18","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/18 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/18 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/86/212786/18 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/86/212786/18"}}},"commit":{"parents":[{"commit":"f86d2e1a05dbcb144bb7614932adcd56514360fd","subject":"Merge \"docs: add a blurb in the single-vm doc about cloud-init log output\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/f86d2e1a05dbcb144bb7614932adcd56514360fd"}]}],"author":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-09-08 15:05:01.000000000","tz":-420},"committer":{"name":"Shih-Hao Li","email":"shihli@vmware.com","date":"2015-09-08 15:05:01.000000000","tz":-420},"subject":"Provide alternate path for neutron plugin config files","message":"Provide alternate path for neutron plugin config files\n\nSince we are in the process of decomposing vendor specific plugins,\nsome plugin config files may no longer exist in neutron repo.\nWe need to provide alternate path for neutron-legacy to locate them.\n\nThe proposed new logic for locating the source config file is:\n1. If Q_PLUGIN_SRC_CONF_PATH is defined, join it with root directory DEST.\n2. Otherwise, do it the old way, i.e. get the source config file from\n   NEUTRON_DIR/Q_PLUGIN_CONF_FILE.\n\nThe rationales behind this design are:\n1. The current neutron-legacy adds root directory NEUTRON_DIR\n   (i.e. DEST/neutron/) to Q_PLUGIN_CONF_PATH to locate the source\n   config file, where Q_PLUGIN_CONF_PATH is a relative path defined\n   in vendor\u0027s neutron_plugin_configure_common function.\n\n   Since we are decomposing vendor specific plugins from neutron repo,\n   we will leave the definitions in neutron_plugin_configure_common\n   stayed relative in vendor\u0027s own repo and let devstack to add root\n   directory to those relative paths at run-time.\n\n   So the new variable Q_PLUGIN_SRC_CONF_PATH we are adding in\n   neutron_plugin_configure_common function is intended to be a\n   relative path.\n\n2. After moving config files from neutron repo to vendor\u0027s repo,\n   each vendor may place them in different directories. This design\n   allow vendors to place wherever they want in their own repo.\n   Thus it is no longer tied to the original path in neutron repo,\n   i.e. the value defined in Q_PLUGIN_CONF_PATH.\n\n   In other words, Q_PLUGIN_CONF_PATH is more like the destination\n   config path (where the config files are copied to by neutron-legacy),\n   and the new variable Q_PLUGIN_SRC_CONF_PATH is like the source\n   config path (where the config files are copied from by\n   neutron-legacy). And neutron-legacy will add root directory to\n   each of them at run-time.\n\nPartial-bug: #1483453\nChange-Id: I2fbe8b4ce5a74164a4770a58a7636b9bb3dc7d3b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/4a8e079589b5665ab0ba52c6de079d26fb2f0e62"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/4a8e079589b5665ab0ba52c6de079d26fb2f0e62"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}