)]}'
{"id":"openstack%2Fdevstack~242624","triplet_id":"openstack%2Fdevstack~master~I4630afb6f010a4c2cb146a79264c480c64c6e4b7","project":"openstack/devstack","branch":"master","topic":"bug/1393099","hashtags":[],"change_id":"I4630afb6f010a4c2cb146a79264c480c64c6e4b7","subject":"Ironic: Explicitly allow DHCP ports","status":"MERGED","created":"2015-11-06 20:40:43.000000000","updated":"2015-11-12 18:43:40.000000000","submitted":"2015-11-12 18:43:38.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"ceabceeb1bdf11e2c4f7d35c38d5dad0f927589b","_number":242624,"virtual_id_number":242624,"owner":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"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":"2015-11-12 18:43:38.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"recommended":{"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},"all":[{"value":1,"date":"2015-11-09 14:05:45.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2015-11-12 14:53:27.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":2,"date":"2015-11-09 14:13:04.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","value":1,"default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"all":[{"value":0,"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2015-11-12 14:53:27.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2015-11-09 14:05:45.000000000","updated_by":{"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},"reviewer":{"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},"state":"REVIEWER"},{"updated":"2015-11-09 14:13:04.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-11-12 14:53:27.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-11-12 18:43:38.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"a88e16bdd8bbd52d75536598b96ea38aa8514c2f","author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"date":"2015-11-06 20:40:43.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"593cd2ec34b6dd4e2f85f268e4a81b219e2e5491","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-11-06 22:44:47.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-tempest-dsvm-full http://logs.openstack.org/24/242624/1/check/gate-tempest-dsvm-full/c4c851b/ : SUCCESS in 43m 55s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/24/242624/1/check/gate-tempest-dsvm-postgres-full/676af7d/ : SUCCESS in 41m 41s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/24/242624/1/check/gate-tempest-dsvm-neutron-full/26a9f1e/ : SUCCESS in 58m 08s\n- gate-grenade-dsvm http://logs.openstack.org/24/242624/1/check/gate-grenade-dsvm/324dc1f/ : SUCCESS in 43m 43s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/24/242624/1/check/gate-tempest-dsvm-large-ops/00bfe5b/ : SUCCESS in 33m 21s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/24/242624/1/check/gate-tempest-dsvm-neutron-large-ops/009ce4e/ : SUCCESS in 29m 44s\n- gate-devstack-bashate http://logs.openstack.org/24/242624/1/check/gate-devstack-bashate/3f034eb/ : SUCCESS in 0s\n- gate-devstack-unit-tests http://logs.openstack.org/24/242624/1/check/gate-devstack-unit-tests/31f3b9a/ : SUCCESS in 6s\n- gate-grenade-dsvm-partial-ncpu http://logs.openstack.org/24/242624/1/check/gate-grenade-dsvm-partial-ncpu/23b6e6d/ : SUCCESS in 44m 50s\n- gate-tempest-dsvm-ironic-agent_ssh http://logs.openstack.org/24/242624/1/check/gate-tempest-dsvm-ironic-agent_ssh/25738bd/ : SUCCESS in 41m 03s\n- gate-devstack-dsvm-updown http://logs.openstack.org/24/242624/1/check/gate-devstack-dsvm-updown/facbbb4/ : SUCCESS in 22m 27s\n- gate-tempest-dsvm-f21 http://logs.openstack.org/24/242624/1/check/gate-tempest-dsvm-f21/a241c5f/ : NOT_REGISTERED (non-voting)\n- gate-tempest-dsvm-centos7 http://logs.openstack.org/24/242624/1/check/gate-tempest-dsvm-centos7/af20faf/ : SUCCESS in 38m 12s (non-voting)\n- gate-devstack-publish-docs http://docs-draft.openstack.org/24/242624/1/check/gate-devstack-publish-docs/f1be371//doc/build/html/ : SUCCESS in 1m 08s\n- gate-swift-dsvm-functional-nv http://logs.openstack.org/24/242624/1/check/gate-swift-dsvm-functional-nv/f73e997/ : SUCCESS in 30m 09s (non-voting)\n- gate-grenade-dsvm-neutron http://logs.openstack.org/24/242624/1/check/gate-grenade-dsvm-neutron/b1ae200/ : SUCCESS in 44m 24s\n- gate-tempest-dsvm-multinode-full http://logs.openstack.org/24/242624/1/check/gate-tempest-dsvm-multinode-full/4d53ed0/ : FAILURE in 43m 12s (non-voting)\n- gate-tempest-dsvm-neutron-multinode-full http://logs.openstack.org/24/242624/1/check/gate-tempest-dsvm-neutron-multinode-full/4b77bee/ : FAILURE in 43m 15s (non-voting)\n- gate-grenade-dsvm-multinode-nv http://logs.openstack.org/24/242624/1/check/gate-grenade-dsvm-multinode-nv/077f6f7/ : FAILURE in 38m 06s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"e26f33c0e6bddb8c47132b9dd7bd0e3ecc53c8b1","author":{"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},"date":"2015-11-09 14:05:45.000000000","message":"Patch Set 1: Code-Review+1","accounts_in_message":[],"_revision_number":1},{"id":"c61b3cf0eb375cca3dedb1d278a767e09b9c81b4","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2015-11-09 14:13:04.000000000","message":"Patch Set 1: Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"7ba64537790ced064723c905daa13e131730d2e1","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2015-11-12 14:53:27.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":1},{"id":"f5df00a1b49a78f735066094b89f8fe245685e97","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-11-12 14:53:33.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":1},{"id":"ddd2ea0941b041dd57c551cebeb8494b191b1e79","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-11-12 18:43:38.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-tempest-dsvm-full http://logs.openstack.org/24/242624/1/gate/gate-tempest-dsvm-full/33b6fc6/ : SUCCESS in 1h 02m 21s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/24/242624/1/gate/gate-tempest-dsvm-postgres-full/8d2b55c/ : SUCCESS in 46m 59s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/24/242624/1/gate/gate-tempest-dsvm-neutron-full/09987af/ : SUCCESS in 1h 14m 03s\n- gate-grenade-dsvm http://logs.openstack.org/24/242624/1/gate/gate-grenade-dsvm/cfe65e1/ : SUCCESS in 53m 49s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/24/242624/1/gate/gate-tempest-dsvm-large-ops/c702065/ : SUCCESS in 29m 17s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/24/242624/1/gate/gate-tempest-dsvm-neutron-large-ops/b323a02/ : SUCCESS in 37m 48s\n- gate-devstack-bashate http://logs.openstack.org/24/242624/1/gate/gate-devstack-bashate/8a78a7b/ : SUCCESS in 18s\n- gate-grenade-dsvm-partial-ncpu http://logs.openstack.org/24/242624/1/gate/gate-grenade-dsvm-partial-ncpu/499074a/ : SUCCESS in 43m 16s","accounts_in_message":[],"_revision_number":1},{"id":"51359cbe8002bb4be9e1b22ee58e0ffac4930ac1","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-11-12 18:43:40.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"63cac536efa3474af40ea24603fca5e1d0a74e13","revisions":{"63cac536efa3474af40ea24603fca5e1d0a74e13":{"kind":"REWORK","_number":1,"created":"2015-11-06 20:40:43.000000000","uploader":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"ref":"refs/changes/24/242624/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/24/242624/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/24/242624/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/24/242624/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/24/242624/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/24/242624/1"}}},"commit":{"parents":[{"commit":"988389a45a9a660b65eb7110377c44320b350069","subject":"Merge \"Zookeeper for DLM scenarios\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/988389a45a9a660b65eb7110377c44320b350069"}]}],"author":{"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","date":"2015-11-06 20:37:32.000000000","tz":-480},"committer":{"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","date":"2015-11-06 20:37:32.000000000","tz":-480},"subject":"Ironic: Explicitly allow DHCP ports","message":"Ironic: Explicitly allow DHCP ports\n\nThis adds an iptables rule to allow ports 67 and 68. We see\noccassionally dropped DHCP packets, which may be causing PXE failures in\nironic jobs.\n\nI\u0027m not 100% confident this fixes the issue, however I don\u0027t think it\ncan break anything and it rules out one theory.\n\nChange-Id: I4630afb6f010a4c2cb146a79264c480c64c6e4b7\nRelated-Bug: #1393099\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/63cac536efa3474af40ea24603fca5e1d0a74e13"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/63cac536efa3474af40ea24603fca5e1d0a74e13"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}