)]}'
{"id":"openstack%2Fdevstack~20666","triplet_id":"openstack%2Fdevstack~master~Ieda16b575685071ff831c92e6b2a29737d6f849b","project":"openstack/devstack","branch":"master","topic":"bug/1084470","hashtags":[],"change_id":"Ieda16b575685071ff831c92e6b2a29737d6f849b","subject":"Pick just the first route when looking for the host interface","status":"MERGED","created":"2013-01-29 02:57:14.000000000","updated":"2013-01-30 16:37:00.000000000","submitted":"2013-01-30 16:36:59.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"75ea45e50eae83e1e2cdc4e0b316061d25f8b34c","_number":20666,"virtual_id_number":20666,"owner":{"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":2,"date":"2013-01-30 16:36:59.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},{"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"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},{"value":2,"date":"2013-01-30 15:33:36.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":2,"date":"2013-01-30 11:03:52.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"date":"2013-01-30 00:13:54.000000000","_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},{"value":1,"date":"2013-01-30 15:33:36.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"date":"2013-01-30 11:03:52.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},{"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":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-01-30 11:03:52.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":"2013-01-30 15:33:36.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":"2013-01-30 16:36:59.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"1df52bba46d3bf5069ececb1327e57a0a9a27ad5","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-01-29 03:32:06.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/20666/1/check/gate-tempest-devstack-vm/28577 : SUCCESS\n- http://logs.openstack.org/20666/1/check/gate-tempest-devstack-vm-postgres/971 : SUCCESS (non-voting)\n- http://logs.openstack.org/20666/1/check/gate-tempest-devstack-vm-quantum/3163 : SUCCESS\n","accounts_in_message":[],"_revision_number":1},{"id":"32b6fa980b73e42a9d24d137abd107547d72ca74","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-01-29 22:56:11.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\nMinor nit: can you update the commit message with the rational from the bug? Took my looking at the bug to understand why this would ever be the case.","accounts_in_message":[],"_revision_number":1},{"id":"1742c11ce0587e5622e5aa9abf39a6c89676bc24","author":{"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},"date":"2013-01-30 00:13:54.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"c0fe0f61d83ad73c447899699456757345e4da2d","author":{"_account_id":5494,"name":"Trivial Rebase","email":"trivial-rebase@review.openstack.org","username":"trivial-rebase"},"date":"2013-01-30 00:14:03.000000000","message":"Patch Set 2:\n\nNew patchset patch-id matches previous patchset, but commit message has changed.","accounts_in_message":[],"_revision_number":2},{"id":"68cd00820de14b340025cc744caed0e39841d726","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-01-30 00:50:12.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/20666/2/check/gate-tempest-devstack-vm/28945 : SUCCESS\n- http://logs.openstack.org/20666/2/check/gate-tempest-devstack-vm-postgres/992 : FAILURE (non-voting)\n- http://logs.openstack.org/20666/2/check/gate-tempest-devstack-vm-quantum/3564 : SUCCESS\n","accounts_in_message":[],"_revision_number":2},{"id":"ae98567ed99a9cad3797a985da11646ff2ac8292","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-01-30 11:03:52.000000000","message":"Patch Set 2: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":2},{"id":"21699d8c9595ed2abbbff570359fd458df81586e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-01-30 15:33:36.000000000","message":"Patch Set 2: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"b6f15b58123b44f4e46a9b04c0cd718aa25cbbfe","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-01-30 15:33:36.000000000","message":"Patch Set 2: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":2},{"id":"70f4b934757f60360d70af61de00920953e79f32","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-01-30 16:36:59.000000000","message":"Patch Set 2: Verified\n\nBuild succeeded.\n\n- http://logs.openstack.org/20666/2/gate/gate-tempest-devstack-vm/29061 : SUCCESS\n- http://logs.openstack.org/20666/2/gate/gate-tempest-devstack-vm-postgres/1014 : SUCCESS (non-voting)\n- http://logs.openstack.org/20666/2/gate/gate-tempest-devstack-vm-quantum/3729 : SUCCESS\n","accounts_in_message":[],"_revision_number":2},{"id":"f749312f2ca565f7b91565474f6817c32bdad112","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-01-30 16:37:00.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"a263ef283e422fc25e7ecff2e9c272eab1a336a6","revisions":{"578f839d2c8b011991a5bd5fb2fb23288a14aac0":{"kind":"REWORK","_number":1,"created":"2013-01-29 02:57:14.000000000","uploader":{"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},"ref":"refs/changes/66/20666/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/66/20666/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/66/20666/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/66/20666/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/66/20666/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/66/20666/1"}}},"commit":{"parents":[{"commit":"95fb0d440ceb4934c3116454cc2ff6349d39fca1","subject":"Run setup_develop for tempest.","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/95fb0d440ceb4934c3116454cc2ff6349d39fca1"}]}],"author":{"name":"Davanum Srinivas","email":"dims@linux.vnet.ibm.com","date":"2013-01-29 02:56:02.000000000","tz":-300},"committer":{"name":"Davanum Srinivas","email":"dims@linux.vnet.ibm.com","date":"2013-01-29 02:56:02.000000000","tz":-300},"subject":"Pick just the first route when looking for the host interface","message":"Pick just the first route when looking for the host interface\n\nFixes LP# 1084470\n\nChange-Id: Ieda16b575685071ff831c92e6b2a29737d6f849b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/578f839d2c8b011991a5bd5fb2fb23288a14aac0"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/578f839d2c8b011991a5bd5fb2fb23288a14aac0"}]},"branch":"refs/heads/master"},"a263ef283e422fc25e7ecff2e9c272eab1a336a6":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2013-01-30 00:13:54.000000000","uploader":{"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},"ref":"refs/changes/66/20666/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/66/20666/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/66/20666/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/66/20666/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/66/20666/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/66/20666/2"}}},"commit":{"parents":[{"commit":"95fb0d440ceb4934c3116454cc2ff6349d39fca1","subject":"Run setup_develop for tempest.","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/95fb0d440ceb4934c3116454cc2ff6349d39fca1"}]}],"author":{"name":"Davanum Srinivas","email":"dims@linux.vnet.ibm.com","date":"2013-01-29 02:56:02.000000000","tz":-300},"committer":{"name":"Davanum Srinivas","email":"dims@linux.vnet.ibm.com","date":"2013-01-30 00:12:17.000000000","tz":-300},"subject":"Pick just the first route when looking for the host interface","message":"Pick just the first route when looking for the host interface\n\nThe current code will fail miserably if multiple default routes are\npresent (perfectly normal if they have different metrics). Fixing\nthe code to pick the first/top entry\n\nFixes LP# 1084470\n\nChange-Id: Ieda16b575685071ff831c92e6b2a29737d6f849b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/a263ef283e422fc25e7ecff2e9c272eab1a336a6"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/a263ef283e422fc25e7ecff2e9c272eab1a336a6"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}