)]}'
{"id":"openstack%2Fdevstack~33424","triplet_id":"openstack%2Fdevstack~master~I73c9751ade6f4786b8b682a1994cb87f0a624379","project":"openstack/devstack","branch":"master","topic":"bp/xenapi-devstack-cleanup","hashtags":[],"change_id":"I73c9751ade6f4786b8b682a1994cb87f0a624379","subject":"xenapi: Cleanup networking","status":"MERGED","created":"2013-06-18 10:22:04.000000000","updated":"2013-06-27 18:56:51.000000000","submitted":"2013-06-27 18:56:50.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":10,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"4e0185109713203fc32993bd3d728bf878a6c62d","_number":33424,"virtual_id_number":33424,"owner":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":2,"date":"2013-06-27 18:56:50.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},{"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"},{"value":0,"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},{"value":0,"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"}],"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,"date":"2013-06-20 21:08:50.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},{"value":2,"date":"2013-06-24 21:02:57.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":2,"date":"2013-06-27 14:15:40.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":1,"date":"2013-06-24 10:46:07.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},{"value":1,"date":"2013-06-24 10:02:16.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"}],"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":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},{"value":0,"date":"2013-06-24 21:02:57.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":1,"date":"2013-06-27 14:15:40.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},{"value":0,"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"}],"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":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},{"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"},{"value":0,"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},{"value":0,"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"}],"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":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},{"_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":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},{"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-06-24 10:02:16.000000000","updated_by":{"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"},"reviewer":{"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"},"state":"REVIEWER"},{"updated":"2013-06-24 10:46:07.000000000","updated_by":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"reviewer":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"state":"REVIEWER"},{"updated":"2013-06-24 21:02:57.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-06-27 14:15:40.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-06-27 18:56:50.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"52c75946f728d84ae07db7d73b92c0056a63d5d3","author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"date":"2013-06-18 10:22:54.000000000","message":"Patch Set 1: Work In Progress\n\ntesting","accounts_in_message":[],"_revision_number":1},{"id":"efa16d91b5a011e17e0987f83927d5e8c2600e3a","author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"date":"2013-06-18 12:41:33.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"b923ab06b98d3a2e127d9b193b9ccbaa967d250d","author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"date":"2013-06-18 12:56:49.000000000","message":"Patch Set 2: Work In Progress\n\ntesting the patch","accounts_in_message":[],"_revision_number":2},{"id":"0fdee977260a6e0be23cf346e1d5f234b12933ac","author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"date":"2013-06-18 13:09:12.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"28e8632d52971d640659b43deabf3cd53eacda0e","author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"date":"2013-06-18 13:09:45.000000000","message":"Patch Set 3: Work In Progress\n\ntesting","accounts_in_message":[],"_revision_number":3},{"id":"c183d3b98fbacad352aba97ba368b20e614738b9","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-06-18 13:59:54.000000000","message":"Patch Set 3: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/33424/3/check/gate-tempest-devstack-vm-full/24182 : SUCCESS in 38m 45s\n- http://logs.openstack.org/33424/3/check/gate-tempest-devstack-vm-postgres-full/23888 : SUCCESS in 50m 36s\n- http://logs.openstack.org/33424/3/check/gate-tempest-devstack-vm-quantum/31626 : SUCCESS in 18m 05s\n- http://logs.openstack.org/33424/3/check/gate-grenade-devstack-vm/11085 : FAILURE in 39m 54s (non-voting)\n","accounts_in_message":[],"_revision_number":3},{"id":"3ecdc887cafa88af495d4c35e2a330dd945bbd8b","author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"date":"2013-06-18 15:17:46.000000000","message":"Patch Set 3: Ready For Review\n\ntests passing","accounts_in_message":[],"_revision_number":3},{"id":"00113a454714b2a2afb491969b89c4fd40f21630","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-06-19 10:21:12.000000000","message":"Patch Set 3: Looks good to me (core reviewer)\n\nNothing that jumps out at me, would like John\u0027s POV","accounts_in_message":[],"_revision_number":3},{"id":"122ba94c9e2ce80bc2290379ab6f1a6804dc1519","author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"date":"2013-06-19 18:06:11.000000000","message":"Patch Set 3: I would prefer that you didn\u0027t merge this\n\n(5 inline comments)\n\nOK, so almost there. Most of the comments are very very minor.\n\nHowever, currently we default to using that interface you are removing:\nhttps://github.com/openstack-dev/devstack/blob/master/stackrc#L181\n\nI think BobBall suggested there was a nice way to auto-detect the management URL, rather than defaulting to something that will now never work.\n\nYou could just change stack.sh to ask the user for the correct answer, until we have the above logic in place, just like we do for passwords already.","accounts_in_message":[],"_revision_number":3},{"id":"a83b6de25b758228536647bf9e4d66cf1a62cb64","author":{"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"},"date":"2013-06-20 09:56:06.000000000","message":"Patch Set 3:\n\nWe\u0027re running install_os_domU in dom0 so we should get the IP from there and either inject it into the localrc or, more preferably, xenstore.\n\nPerhaps - since a hosts IP address could change - we should be getting it on host boot and punshing it into xenstore / the VM\u0027s xenstore-key.","accounts_in_message":[],"_revision_number":3},{"id":"0f17870fe74ca29d7bdde19ef86358a9b0dadf1d","author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"date":"2013-06-20 13:23:47.000000000","message":"Patch Set 3: (1 inline comment)\n\nOkay, so I tried out the xenstore communication, but it seems, xenstore could not be read before the xenfs is mounted, so it might not be available by the time devstack needs it. I would prefer not to modify localrc, it\u0027s already a placeholder for too many things, making it an r/w access general purpose communitation channel would not help in this situation. BobBall has strong feelings against using kernel parameters, and also keen to get back the HIMN to play.\n\nSo for now, I will remove the HIMN on eth0, and die if I don\u0027t have a xenapi url.\n\nAs soon as we know how to use HIMN to communicate with xenapi, we could start using that. But for now, let\u0027s stay only on the management network.\n\n@John: If we had a HIMN (Host Internal Management Network), and xapi would also have an IP on the OS Management Network, would that work (after making xapi calls to discover the xapi IP on OS Management net)?","accounts_in_message":[],"_revision_number":3},{"id":"dc181bb4bf851adf7754696ffcbc842de7893b23","author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"date":"2013-06-20 13:53:20.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"21bfb1069ae588663990750a775bad4f9ca19522","author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"date":"2013-06-20 14:02:42.000000000","message":"Patch Set 4: Work In Progress\n\ntesting","accounts_in_message":[],"_revision_number":4},{"id":"c0ac16af9e86ffaeaf453011f4a307872be2d932","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-06-20 14:43:23.000000000","message":"Patch Set 4: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/33424/4/check/gate-tempest-devstack-vm-full/24977 : SUCCESS in 38m 46s\n- http://logs.openstack.org/33424/4/check/gate-tempest-devstack-vm-postgres-full/24683 : SUCCESS in 49m 55s\n- http://logs.openstack.org/33424/4/check/gate-tempest-devstack-vm-quantum/32488 : SUCCESS in 20m 48s\n- http://logs.openstack.org/33424/4/check/gate-grenade-devstack-vm/11727 : FAILURE in 11m 49s (non-voting)\n","accounts_in_message":[],"_revision_number":4},{"id":"47da8df2bd76c012cfb94691f79d64c7e9986d1f","author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"date":"2013-06-20 14:48:17.000000000","message":"Patch Set 4: Ready For Review\n\ntests ran","accounts_in_message":[],"_revision_number":4},{"id":"f5f5ff391e7fab69f832d9e52154c13beabd72da","author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"date":"2013-06-20 15:33:05.000000000","message":"Patch Set 4: (1 inline comment)\n\nI think the easiest place to pass the URL into the VM is here:\nhttps://github.com/openstack-dev/devstack/blob/master/tools/xen/build_xva.sh#L143\n\nAnyway, I am OK with this change, but there are ways we could make it easier when the user doesn\u0027t want to change anything in localrc\n\nIf we change the above line in build_xva, we can probably do away with a lot of the silly settings listed in the readme as required.","accounts_in_message":[],"_revision_number":4},{"id":"5c6069c49420380299f19e7179c92b16eb129809","author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"date":"2013-06-20 21:08:50.000000000","message":"Patch Set 4: (1 inline comment)\n\nYes, you can change the generation of run.sh You just need to remember, that run.sh is your entry point from now, so that you can\u0027t run stack.sh - because your localrc would be incomplete. Anyhow, thanks for the comments.","accounts_in_message":[],"_revision_number":4},{"id":"a7b60e1acbde13aedb8b4ddbdb1c7e4873373e7d","author":{"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"},"date":"2013-06-24 10:02:16.000000000","message":"Patch Set 4: Looks good to me, but someone else must approve\n\n(1 inline comment)\n\n","accounts_in_message":[],"_revision_number":4},{"id":"671aaeeeab6cb33e492186d742474f631ba16bce","author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"date":"2013-06-24 10:46:07.000000000","message":"Patch Set 4: Looks good to me, but someone else must approve\n\n(1 inline comment)\n\nYou already have to run from run.sh, otherwise strange things can happen, but agreed that is a bit strange.\n\nI think, long term, we should ask the user questions and populate the localrc for them, just like stack.sh does, but either way, this is a good step forward.","accounts_in_message":[],"_revision_number":4},{"id":"768f6c83272883ea1d64894ed14d1553f97593ba","author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"date":"2013-06-24 12:35:50.000000000","message":"Patch Set 4:\n\nAt the moment you can run stack.sh, there is nothing special in run.sh that would cause \"strange things\".\n\nhttps://github.com/openstack-dev/devstack/blob/master/tools/xen/build_xva.sh#L139","accounts_in_message":[],"_revision_number":4},{"id":"cf34fa9d4dafdadb158ca7daf4593f5fface8874","author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"date":"2013-06-24 16:34:03.000000000","message":"Patch Set 4:\n\nBut what about these bits, if you don\u0027t happen to set them in the localrc file?\nVIRT_DRIVER\u003dxenserver FORCE\u003dyes MULTI_HOST\u003d$MULTI_HOST HOST_IP_IFACE\u003d$HOST_IP_IFACE $STACKSH_PARAMS\n\nAnyways, lets worry about this in another change. What you have here looks like a nice simple approach that is a good step forward.","accounts_in_message":[],"_revision_number":4},{"id":"e1bf7047033cad5d4dd8d8703727adbb196476dc","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-06-24 21:02:57.000000000","message":"Patch Set 4: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":4},{"id":"10a8dc3679935b45f426f4e0237757d346268beb","author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"date":"2013-06-25 22:16:19.000000000","message":"Patch Set 4: Work In Progress\n\nI need to look at this change from a quantum perspective as well.","accounts_in_message":[],"_revision_number":4},{"id":"f214ff94d229a7f927c81e2d0810d5f945e7af05","author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"date":"2013-06-27 09:50:28.000000000","message":"Patch Set 4: Ready For Review\n\nIt works with a neutron config as well","accounts_in_message":[],"_revision_number":4},{"id":"7212e68a08ab1260d3ffbba9e56954abb02f7c5b","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-06-27 14:15:40.000000000","message":"Patch Set 4: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":4},{"id":"fde2d69a9ff5f6989fe13185c05407516ef3ff59","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-06-27 14:15:42.000000000","message":"Patch Set 4: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":4},{"id":"df8a1741be8f4ca42c953bfaa468930a475c2e29","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-06-27 14:59:43.000000000","message":"Patch Set 4: Fails\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGithub#Test_Failures\n\n- http://logs.openstack.org/33424/4/gate/gate-tempest-devstack-vm-full/26802 : SUCCESS in 43m 55s\n- http://logs.openstack.org/33424/4/gate/gate-tempest-devstack-vm-postgres-full/26508 : SUCCESS in 40m 12s\n- http://logs.openstack.org/33424/4/gate/gate-tempest-devstack-vm-quantum/34490 : FAILURE in 18m 16s\n- http://logs.openstack.org/33424/4/gate/gate-grenade-devstack-vm/13316 : FAILURE in 25m 18s (non-voting)\n","accounts_in_message":[],"_revision_number":4},{"id":"87c1c1b3455e6da1a8aa3ed8da62086f0330f864","author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"date":"2013-06-27 16:23:14.000000000","message":"Patch Set 4:\n\nreverify bug 1194026.","accounts_in_message":[],"_revision_number":4},{"id":"5ba153b261e0c2342e237c280ec9d4d3cb7d3750","author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"date":"2013-06-27 17:42:22.000000000","message":"Patch Set 4:\n\nreverify bug 1194026","accounts_in_message":[],"_revision_number":4},{"id":"3a5455c00c70aa838b3abbf98036c42bec11e02f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-06-27 17:42:47.000000000","message":"Patch Set 4: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":4},{"id":"44626a6905a8d56c7640e05610fdd6cfaaaf766c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-06-27 18:56:50.000000000","message":"Patch Set 4: Verified\n\nBuild succeeded.\n\n- http://logs.openstack.org/33424/4/gate/gate-tempest-devstack-vm-full/26867 : SUCCESS in 47m 15s\n- http://logs.openstack.org/33424/4/gate/gate-tempest-devstack-vm-postgres-full/26573 : SUCCESS in 58m 18s\n- http://logs.openstack.org/33424/4/gate/gate-tempest-devstack-vm-quantum/34561 : SUCCESS in 20m 44s\n- http://logs.openstack.org/33424/4/gate/gate-grenade-devstack-vm/13378 : FAILURE in 21m 55s (non-voting)\n","accounts_in_message":[],"_revision_number":4},{"id":"1e6ca5b9b4517bac9324caa771a6172d1e42b22a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-06-27 18:56:51.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":4}],"current_revision_number":4,"current_revision":"5a56cd6283f918e6022dcc47906fb86a739496dc","revisions":{"79bfc21733d12105382d9602587c2755cc3d490a":{"kind":"REWORK","_number":1,"created":"2013-06-18 10:22:04.000000000","uploader":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"ref":"refs/changes/24/33424/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/24/33424/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/24/33424/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/24/33424/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/24/33424/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/24/33424/1"}}},"commit":{"parents":[{"commit":"8dc7a10e3da7c3b5f16c0a3e42cbcc7d803bf55c","subject":"Merge \"Fix syntax error in lib/tempest\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/8dc7a10e3da7c3b5f16c0a3e42cbcc7d803bf55c"}]}],"author":{"name":"Mate Lakat","email":"mate.lakat@citrix.com","date":"2013-06-17 12:54:43.000000000","tz":60},"committer":{"name":"Mate Lakat","email":"mate.lakat@citrix.com","date":"2013-06-18 10:17:54.000000000","tz":60},"subject":"xenapi: Get rid of internal xapi interface","message":"xenapi: Get rid of internal xapi interface\n\nOpenStack VM had 4 interfaces, from which eth0 was not used at all. This\nchange removes the non-used interface, thus devstack is providing the\nsame architecture as described in the official documentation:\n\n    http://goo.gl/dmrNF\n\nRelated to blueprint xenapi-devstack-cleanup\n\nChange-Id: I73c9751ade6f4786b8b682a1994cb87f0a624379\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/79bfc21733d12105382d9602587c2755cc3d490a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/79bfc21733d12105382d9602587c2755cc3d490a"}]},"branch":"refs/heads/master"},"47b1bc0f9cbda4bac69c53a462ba318bb3004ece":{"kind":"REWORK","_number":2,"created":"2013-06-18 12:41:33.000000000","uploader":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"ref":"refs/changes/24/33424/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/24/33424/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/24/33424/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/24/33424/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/24/33424/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/24/33424/2"}}},"commit":{"parents":[{"commit":"8dc7a10e3da7c3b5f16c0a3e42cbcc7d803bf55c","subject":"Merge \"Fix syntax error in lib/tempest\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/8dc7a10e3da7c3b5f16c0a3e42cbcc7d803bf55c"}]}],"author":{"name":"Mate Lakat","email":"mate.lakat@citrix.com","date":"2013-06-17 12:54:43.000000000","tz":60},"committer":{"name":"Mate Lakat","email":"mate.lakat@citrix.com","date":"2013-06-18 12:36:58.000000000","tz":60},"subject":"xenapi: Get rid of internal xapi interface","message":"xenapi: Get rid of internal xapi interface\n\nOpenStack VM had 4 interfaces, from which eth0 was not used at all. This\nchange removes the non-used interface, thus devstack is providing the\nsame architecture as described in the official documentation:\n\n    http://goo.gl/dmrNF\n\nRelated to blueprint xenapi-devstack-cleanup\n\nChange-Id: I73c9751ade6f4786b8b682a1994cb87f0a624379\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/47b1bc0f9cbda4bac69c53a462ba318bb3004ece"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/47b1bc0f9cbda4bac69c53a462ba318bb3004ece"}]},"branch":"refs/heads/master"},"b357797022eeb1344141c566f28735e0fc602528":{"kind":"REWORK","_number":3,"created":"2013-06-18 13:09:12.000000000","uploader":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"ref":"refs/changes/24/33424/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/24/33424/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/24/33424/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/24/33424/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/24/33424/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/24/33424/3"}}},"commit":{"parents":[{"commit":"8dc7a10e3da7c3b5f16c0a3e42cbcc7d803bf55c","subject":"Merge \"Fix syntax error in lib/tempest\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/8dc7a10e3da7c3b5f16c0a3e42cbcc7d803bf55c"}]}],"author":{"name":"Mate Lakat","email":"mate.lakat@citrix.com","date":"2013-06-17 12:54:43.000000000","tz":60},"committer":{"name":"Mate Lakat","email":"mate.lakat@citrix.com","date":"2013-06-18 13:05:26.000000000","tz":60},"subject":"xenapi: Get rid of internal xapi interface","message":"xenapi: Get rid of internal xapi interface\n\nOpenStack VM had 4 interfaces, from which eth0 was not used at all. This\nchange removes the non-used interface, thus devstack is providing the\nsame architecture as described in the official documentation:\n\n    http://goo.gl/dmrNF\n\nRelated to blueprint xenapi-devstack-cleanup\n\nChange-Id: I73c9751ade6f4786b8b682a1994cb87f0a624379\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b357797022eeb1344141c566f28735e0fc602528"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b357797022eeb1344141c566f28735e0fc602528"}]},"branch":"refs/heads/master"},"5a56cd6283f918e6022dcc47906fb86a739496dc":{"kind":"REWORK","_number":4,"created":"2013-06-20 13:53:20.000000000","uploader":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"ref":"refs/changes/24/33424/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/24/33424/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/24/33424/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/24/33424/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/24/33424/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/24/33424/4"}}},"commit":{"parents":[{"commit":"aef5fcae9c364bfa9825d71faec71d5e28c12b14","subject":"Merge \"Use heat-manage to sync heat db.\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/aef5fcae9c364bfa9825d71faec71d5e28c12b14"}]}],"author":{"name":"Mate Lakat","email":"mate.lakat@citrix.com","date":"2013-06-17 12:54:43.000000000","tz":60},"committer":{"name":"Mate Lakat","email":"mate.lakat@citrix.com","date":"2013-06-20 13:43:14.000000000","tz":60},"subject":"xenapi: Cleanup networking","message":"xenapi: Cleanup networking\n\nOpenStack VM had 4 interfaces, from which eth0 was connected to HIMN (Host\nInternal Management Network), and not used at all. The reason for not\nusing this interface according to johngarbutt is:\n - VNC console would not work\n - migrations would not work\n - many other things\nThis change removes the interface, thus devstack is providing the\nsame architecture as described in the official documentation, YAGNI\napplied on the HIMN:\n\n    http://goo.gl/dmrNF\n\nThe change also modifies some defaults, so it\u0027s easier to get started:\n - The Ubuntu VM is network-installed through the management network\n - OS VM is getting its address on the management network through DHCP\n - Default Public IP addresses aligned to stack.sh defaults\n - OS Services are listening on Management Interface\n\nRelated to blueprint xenapi-devstack-cleanup\n\nChange-Id: I73c9751ade6f4786b8b682a1994cb87f0a624379\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/5a56cd6283f918e6022dcc47906fb86a739496dc"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/5a56cd6283f918e6022dcc47906fb86a739496dc"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}