)]}'
{"id":"openstack%2Fdevstack~329933","triplet_id":"openstack%2Fdevstack~master~I5781cb1ec4cfc1699e61dbc324d0bdb824b56be1","project":"openstack/devstack","branch":"master","topic":"bug/missing-l3-conf-parameter","hashtags":[],"change_id":"I5781cb1ec4cfc1699e61dbc324d0bdb824b56be1","subject":"lib/neutron-plugins: Pass conf file parameter","status":"MERGED","created":"2016-06-15 13:33:48.000000000","updated":"2016-06-17 21:33:06.000000000","submitted":"2016-06-17 21:33:06.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"a13d86b47c9fb9fcc4845194a75e1c55feba8434","_number":329933,"virtual_id_number":329933,"owner":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},{"value":2,"date":"2016-06-17 21:33:06.000000000","post_submit":true,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},"all":[{"value":2,"date":"2016-06-17 20:19:15.000000000","_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2016-06-15 23:30:09.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},"all":[{"value":1,"date":"2016-06-17 20:19:15.000000000","_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"}],"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":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-06-15 23:30:09.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":"2016-06-17 20:19:15.000000000","updated_by":{"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},"reviewer":{"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},"state":"REVIEWER"},{"updated":"2016-06-17 21:33:06.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"293f409e01019b9efff7d7a33a25b294c860edb7","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2016-06-15 13:33:48.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"ab8d3c95c782b2ce7dfaecb5a276af2141209770","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2016-06-15 13:34:21.000000000","message":"Topic set to bug/missing-l3-conf-parameter","accounts_in_message":[],"_revision_number":1},{"id":"23f10bac9b8bbf7fa3d9a00840fdcd7b9b6e3261","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2016-06-15 13:47:04.000000000","message":"Patch Set 2: Published edit on patch set 1","accounts_in_message":[],"_revision_number":2},{"id":"841c0f0c28734a17b3263429463dd6e9c4d84b43","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2016-06-15 14:52:16.000000000","message":"Patch Set 2:\n\nTesting completed on IBM PowerKVM platform. For rechecking only on the IBM PowerKVM CI, add a review comment with pkvm: recheck. Contact info: kvmpower@linux.vnet.ibm.com. For more information, see https://wiki.openstack.org/wiki/PowerKVM\n\n- check-ibm-tempest-dsvm-full-xenial http://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/pkvmci/devstack/33/329933/2/check/check-ibm-tempest-dsvm-full-xenial/ac49dea/ : SUCCESS in 1h 04m 56s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"3dddf7735c18fb5ddce2c7eb7cfb5a9717904521","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2016-06-15 15:00:09.000000000","message":"Patch Set 2:\n\nTesting completed on Citrix Xenserver. For rechecking only on the Citrix Xenserver CI,add a review comment with \"xenserver: recheck\". Contact info: openstack@citrix.com.\nFor debugging suggestions, see https://wiki.openstack.org/wiki/Debugging_XenServer_CI_failures\n\n- check-citrix-xenserver http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/33/329933/2/19549/index.html : SUCCESS in 01 h 06 m 48 s","accounts_in_message":[],"_revision_number":2},{"id":"03d22c97cee10f954a73ea9047a5143707f6b081","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-06-15 15:01:01.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-tempest-dsvm-full http://logs.openstack.org/33/329933/2/check/gate-tempest-dsvm-full/9908c72/ : SUCCESS in 43m 55s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/33/329933/2/check/gate-tempest-dsvm-postgres-full/1374b2b/ : SUCCESS in 31m 31s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/33/329933/2/check/gate-tempest-dsvm-neutron-full/685def5/ : SUCCESS in 1h 10m 34s\n- gate-grenade-dsvm http://logs.openstack.org/33/329933/2/check/gate-grenade-dsvm/8ec651f/ : SUCCESS in 40m 07s\n- gate-devstack-docs http://docs-draft.openstack.org/33/329933/2/check/gate-devstack-docs/27f6440//doc/build/html/ : SUCCESS in 4m 26s\n- gate-devstack-bashate http://logs.openstack.org/33/329933/2/check/gate-devstack-bashate/a34933c/ : SUCCESS in 3m 49s\n- gate-devstack-unit-tests http://logs.openstack.org/33/329933/2/check/gate-devstack-unit-tests/d13b02b/ : SUCCESS in 4m 36s\n- gate-tempest-dsvm-ironic-agent_ssh-nv http://logs.openstack.org/33/329933/2/check/gate-tempest-dsvm-ironic-agent_ssh-nv/cae7bc0/ : FAILURE in 21m 54s (non-voting)\n- gate-devstack-dsvm-updown http://logs.openstack.org/33/329933/2/check/gate-devstack-dsvm-updown/bd4bdee/ : SUCCESS in 13m 09s\n- gate-tempest-dsvm-neutron-identity-v3-only-full-nv http://logs.openstack.org/33/329933/2/check/gate-tempest-dsvm-neutron-identity-v3-only-full-nv/00288e1/ : SUCCESS in 59m 29s (non-voting)\n- gate-swift-dsvm-functional-nv http://logs.openstack.org/33/329933/2/check/gate-swift-dsvm-functional-nv/e004fcf/ : SUCCESS in 24m 08s (non-voting)\n- gate-grenade-dsvm-neutron http://logs.openstack.org/33/329933/2/check/gate-grenade-dsvm-neutron/e903244/ : SUCCESS in 32m 00s\n- gate-tempest-dsvm-multinode-full http://logs.openstack.org/33/329933/2/check/gate-tempest-dsvm-multinode-full/fbb729a/ : SUCCESS in 1h 02m 20s (non-voting)\n- gate-tempest-dsvm-neutron-multinode-full http://logs.openstack.org/33/329933/2/check/gate-tempest-dsvm-neutron-multinode-full/e600b2c/ : SUCCESS in 1h 07m 31s (non-voting)\n- gate-grenade-dsvm-multinode http://logs.openstack.org/33/329933/2/check/gate-grenade-dsvm-multinode/3e245ee/ : SUCCESS in 54m 16s\n- gate-grenade-dsvm-neutron-multinode http://logs.openstack.org/33/329933/2/check/gate-grenade-dsvm-neutron-multinode/cc40a49/ : SUCCESS in 53m 26s\n- gate-tempest-dsvm-platform-centos7-nv http://logs.openstack.org/33/329933/2/check/gate-tempest-dsvm-platform-centos7-nv/a73f022/ : SUCCESS in 53m 31s (non-voting)\n- gate-tempest-dsvm-platform-fedora23-nv http://logs.openstack.org/33/329933/2/check/gate-tempest-dsvm-platform-fedora23-nv/1a7e562/ : SUCCESS in 55m 57s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"9feae8fd8b9a72b0a5c35b7418954fe1da563b43","author":{"_account_id":16376,"name":"Intel NFV CI","email":"openstack-nfv-ci@intel.com","username":"intel-nfv-ci","tags":["SERVICE_USER"]},"date":"2016-06-15 19:48:31.000000000","message":"Patch Set 2:\n\nBuild succeeded (check pipeline).\n\n- tempest-dsvm-ovsdpdk-odl http://intel-openstack-ci-logs.ovh/33/329933/2/check/tempest-dsvm-ovsdpdk-odl/e1261a5 : FAILURE in 59m 13s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"d7fda522002c8c3562b06d1204986f2164931e10","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2016-06-15 23:30:09.000000000","message":"Patch Set 2: Code-Review+2","accounts_in_message":[],"_revision_number":2},{"id":"6e3a796e1197a0b77094af47b61ed3415c9c2f22","author":{"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},"date":"2016-06-17 20:19:15.000000000","message":"Patch Set 2: Workflow+1 Code-Review+2\n\nSometimes I am astounded at some of the stupid things I do.","accounts_in_message":[],"_revision_number":2},{"id":"54879ddc1fa8d392a7039ddac8d4c5b896a923b9","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-06-17 20:19:26.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"bed383ac91290c49ee6cad9e99d1f6f98e27a23a","date":"2016-06-17 21:33:06.000000000","message":"Change has been successfully merged into the git repository by Jenkins","accounts_in_message":[],"_revision_number":2},{"id":"a13d86b47c9fb9fcc4845194a75e1c55feba8434","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-06-17 21:33:06.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-tempest-dsvm-full http://logs.openstack.org/33/329933/2/gate/gate-tempest-dsvm-full/bd0ca1c/ : SUCCESS in 55m 30s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/33/329933/2/gate/gate-tempest-dsvm-postgres-full/9fff21f/ : SUCCESS in 43m 52s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/33/329933/2/gate/gate-tempest-dsvm-neutron-full/33734ef/ : SUCCESS in 59m 42s\n- gate-grenade-dsvm http://logs.openstack.org/33/329933/2/gate/gate-grenade-dsvm/8a1476b/ : SUCCESS in 41m 59s\n- gate-devstack-docs http://docs-draft.openstack.org/33/329933/2/gate/gate-devstack-docs/8b785ac//doc/build/html/ : SUCCESS in 2m 51s\n- gate-devstack-bashate http://logs.openstack.org/33/329933/2/gate/gate-devstack-bashate/00b9572/ : SUCCESS in 2m 16s\n- gate-grenade-dsvm-multinode http://logs.openstack.org/33/329933/2/gate/gate-grenade-dsvm-multinode/b886f81/ : SUCCESS in 42m 29s\n- gate-grenade-dsvm-neutron-multinode http://logs.openstack.org/33/329933/2/gate/gate-grenade-dsvm-neutron-multinode/f9c8719/ : SUCCESS in 51m 17s","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"24e29f2265e6fd496addf00adfcc13ba8533b912","revisions":{"48b87a42b0cca40d909f2c2359b9257302e0b76e":{"kind":"REWORK","_number":1,"created":"2016-06-15 13:33:48.000000000","uploader":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"ref":"refs/changes/33/329933/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/33/329933/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/33/329933/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/33/329933/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/33/329933/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/33/329933/1"}}},"commit":{"parents":[{"commit":"5fc597289b9ace6cb3fed83c0fd696d5daa59e50","subject":"Merge \"Enable Swift Account Management\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/5fc597289b9ace6cb3fed83c0fd696d5daa59e50"}]}],"author":{"name":"Stephen Finucane","email":"stephen.finucane@intel.com","date":"2016-06-15 13:31:51.000000000","tz":60},"committer":{"name":"Stephen Finucane","email":"stephen.finucane@intel.com","date":"2016-06-15 13:31:51.000000000","tz":60},"subject":"lib/neutron-plugins: Pass conf file parameter","message":"lib/neutron-plugins: Pass conf file parameter\n\nThe \u0027neutron_plugin_configure_l3_agent\u0027 function expects a path to a\nconfiguration file as a paramter. This was not done in one place,\nresulting in the generation of a \u0027DEFAULT\u0027 file in the DevStack\ndirectory along with an invalid L3 configuration file. Resolve this.\n\nChange-Id: I5781cb1ec4cfc1699e61dbc324d0bdb824b56be1\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/48b87a42b0cca40d909f2c2359b9257302e0b76e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/48b87a42b0cca40d909f2c2359b9257302e0b76e"}]},"branch":"refs/heads/master"},"24e29f2265e6fd496addf00adfcc13ba8533b912":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2016-06-15 13:47:04.000000000","uploader":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"ref":"refs/changes/33/329933/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/33/329933/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/33/329933/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/33/329933/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/33/329933/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/33/329933/2"}}},"commit":{"parents":[{"commit":"5fc597289b9ace6cb3fed83c0fd696d5daa59e50","subject":"Merge \"Enable Swift Account Management\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/5fc597289b9ace6cb3fed83c0fd696d5daa59e50"}]}],"author":{"name":"Stephen Finucane","email":"stephen.finucane@intel.com","date":"2016-06-15 13:31:51.000000000","tz":60},"committer":{"name":"Stephen Finucane","email":"stephen.finucane@intel.com","date":"2016-06-15 13:35:47.000000000","tz":0},"subject":"lib/neutron-plugins: Pass conf file parameter","message":"lib/neutron-plugins: Pass conf file parameter\n\nThe \u0027neutron_plugin_configure_l3_agent\u0027 function expects a path to a\nconfiguration file as a parameter. This was not done for one call,\nresulting in the generation of a \u0027DEFAULT\u0027 file in the DevStack\ndirectory along with an invalid L3 configuration file. Resolve this.\n\nChange-Id: I5781cb1ec4cfc1699e61dbc324d0bdb824b56be1\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/24e29f2265e6fd496addf00adfcc13ba8533b912"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/24e29f2265e6fd496addf00adfcc13ba8533b912"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}