)]}'
{"id":"openstack%2Fdevstack~33632","triplet_id":"openstack%2Fdevstack~master~Ib0a0a316e849f2c2651305ea657c84820cd2bb3b","project":"openstack/devstack","branch":"master","topic":"bp/xenapi-devstack-cleanup","hashtags":[],"change_id":"Ib0a0a316e849f2c2651305ea657c84820cd2bb3b","subject":"xenapi: cleanup VM Installation","status":"MERGED","created":"2013-06-19 11:51:18.000000000","updated":"2013-07-09 00:13:31.000000000","submitted":"2013-07-09 00:13:30.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":4,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"59951db8a574cd452fea86f1889826422b28a185","_number":33632,"virtual_id_number":33632,"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-07-09 00:13:30.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":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},{"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":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},{"value":2,"date":"2013-07-05 16:25:47.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":1,"date":"2013-07-04 10:19:08.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},{"value":2,"date":"2013-07-08 19:36:29.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":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-07-05 16:25:47.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},{"value":1,"date":"2013-07-08 19:36:29.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":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":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},{"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":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"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-07-04 10:19:08.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-07-05 16:25:47.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-07-08 19:36:29.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-07-09 00:13:30.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"ac109fdb06d814b86a21922d5375c70c51ae2dbf","author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"date":"2013-06-19 11:55:06.000000000","message":"Patch Set 1: Work In Progress\n\ntesting","accounts_in_message":[],"_revision_number":1},{"id":"e504880618e1da90e63326e9f56fa6d25e5b2a2a","author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"date":"2013-06-19 12:29:27.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"e6aa3d18680b35cacf072e609d727d9e1e6a523f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-06-19 13:12:01.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/33632/2/check/gate-tempest-devstack-vm-full/24575 : SUCCESS in 39m 43s\n- http://logs.openstack.org/33632/2/check/gate-tempest-devstack-vm-postgres-full/24281 : SUCCESS in 42m 26s\n- http://logs.openstack.org/33632/2/check/gate-tempest-devstack-vm-quantum/32047 : SUCCESS in 20m 18s\n- http://logs.openstack.org/33632/2/check/gate-grenade-devstack-vm/11415 : FAILURE in 39m 05s (non-voting)\n","accounts_in_message":[],"_revision_number":2},{"id":"b6ac968f1aa73b12ff20f728f2e49c075f1fe5de","author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"date":"2013-06-19 18:14:56.000000000","message":"Patch Set 2: I would prefer that you didn\u0027t merge this\n\n(2 inline comments)\n\nI think its good, just one suggestion.","accounts_in_message":[],"_revision_number":2},{"id":"b79dd9eaed5622c1b51f9f9206b463bc73293969","author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"date":"2013-06-20 14:22:06.000000000","message":"Patch Set 2: (2 inline comments)\n\nThanks for the comments, new patch is on the way...","accounts_in_message":[],"_revision_number":2},{"id":"07ab49d0b66766b5048012513e9e13723d0ae3c3","author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"date":"2013-06-20 14:26:55.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"e1f8de41c965adb598d0e769680da47f9a84731e","author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"date":"2013-06-20 14:28:16.000000000","message":"Patch Set 3: Work In Progress\n\ntesting","accounts_in_message":[],"_revision_number":3},{"id":"2a709e0ad3c55ab54d50673fbecafdd969da6f2b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-06-20 15:17:33.000000000","message":"Patch Set 3: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/33632/3/check/gate-tempest-devstack-vm-full/24987 : SUCCESS in 46m 58s\n- http://logs.openstack.org/33632/3/check/gate-tempest-devstack-vm-postgres-full/24693 : SUCCESS in 50m 32s\n- http://logs.openstack.org/33632/3/check/gate-tempest-devstack-vm-quantum/32498 : SUCCESS in 16m 08s\n- http://logs.openstack.org/33632/3/check/gate-grenade-devstack-vm/11736 : FAILURE in 10m 53s (non-voting)\n","accounts_in_message":[],"_revision_number":3},{"id":"e7196659734d4331a9ea66a3048598b0f095354b","author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"date":"2013-06-20 20:51:53.000000000","message":"Patch Set 3: Ready For Review\n\nOkay, tests passed","accounts_in_message":[],"_revision_number":3},{"id":"341fac07aba52dfc5f4dd8b11d877c7da0187f5b","author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"date":"2013-07-04 10:19:08.000000000","message":"Patch Set 3: Looks good to me, but someone else must approve\n\nLooks like a great step forward.","accounts_in_message":[],"_revision_number":3},{"id":"23039f1b33a86ea3425bf1a18874d67e6d7c2afa","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-07-05 16:25:47.000000000","message":"Patch Set 3: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":3},{"id":"228a4a5b446724748167cb3178ed9d08b0c29bda","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-07-08 19:36:29.000000000","message":"Patch Set 3: Looks good to me (core reviewer); Approved\n\nyay for negative KLOC","accounts_in_message":[],"_revision_number":3},{"id":"3017a1f91a15b4b7eb8d94c410d51c288b5a842a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-07-08 19:36:39.000000000","message":"Patch Set 3: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":3},{"id":"5fa744cbeb9a0590d3a38c4b78c5227b01965c09","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-07-09 00:13:30.000000000","message":"Patch Set 3: Verified\n\nBuild succeeded.\n\n- http://logs.openstack.org/33632/3/gate/gate-tempest-devstack-vm-full/29451 : SUCCESS in 50m 26s\n- http://logs.openstack.org/33632/3/gate/gate-tempest-devstack-vm-postgres-full/29157 : SUCCESS in 52m 49s\n- http://logs.openstack.org/33632/3/gate/gate-tempest-devstack-vm-neutron/563 : SUCCESS in 29m 47s\n- http://logs.openstack.org/33632/3/gate/gate-grenade-devstack-vm/15680 : FAILURE in 21m 26s (non-voting)\n","accounts_in_message":[],"_revision_number":3},{"id":"7b2635eee7042ecac560c011cf89d1e7b6f719c3","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-07-09 00:13:31.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"2f524bd9053ee106daa98414bbe3b94b0cd2e43f","revisions":{"91d3f8954c95f60d5c16c1d8f61544e2ea023f55":{"kind":"REWORK","_number":1,"created":"2013-06-19 11:51:18.000000000","uploader":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"ref":"refs/changes/32/33632/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/32/33632/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/32/33632/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/32/33632/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/32/33632/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/32/33632/1"}}},"commit":{"parents":[{"commit":"b357797022eeb1344141c566f28735e0fc602528","subject":"xenapi: Get rid of internal xapi interface","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b357797022eeb1344141c566f28735e0fc602528"}]}],"author":{"name":"Mate Lakat","email":"mate.lakat@citrix.com","date":"2013-06-19 11:32:23.000000000","tz":60},"committer":{"name":"Mate Lakat","email":"mate.lakat@citrix.com","date":"2013-06-19 11:45:53.000000000","tz":60},"subject":"xenapi: cleanup VM Installation","message":"xenapi: cleanup VM Installation\n\nVM installation created a machine with multiple network interfaces. This\nis a needless complexity at that point. This change removes the network\nsetup from install-os-vpx.sh script, so that only one network interface\nis created, which is connected to the given network. This change also\nremoves a lot of non-used code from install-os-vpx.sh\n\nChange-Id: Ib0a0a316e849f2c2651305ea657c84820cd2bb3b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/91d3f8954c95f60d5c16c1d8f61544e2ea023f55"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/91d3f8954c95f60d5c16c1d8f61544e2ea023f55"}]},"branch":"refs/heads/master"},"c57b365ac68de02b569aa48608eb6bf2c89feef2":{"kind":"REWORK","_number":2,"created":"2013-06-19 12:29:27.000000000","uploader":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"ref":"refs/changes/32/33632/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/32/33632/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/32/33632/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/32/33632/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/32/33632/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/32/33632/2"}}},"commit":{"parents":[{"commit":"b357797022eeb1344141c566f28735e0fc602528","subject":"xenapi: Get rid of internal xapi interface","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b357797022eeb1344141c566f28735e0fc602528"}]}],"author":{"name":"Mate Lakat","email":"mate.lakat@citrix.com","date":"2013-06-19 11:32:23.000000000","tz":60},"committer":{"name":"Mate Lakat","email":"mate.lakat@citrix.com","date":"2013-06-19 12:19:43.000000000","tz":60},"subject":"xenapi: cleanup VM Installation","message":"xenapi: cleanup VM Installation\n\nVM installation created a machine with multiple network interfaces. This\nis a needless complexity at that point. This change removes the complex\nnetwork setup from install-os-vpx.sh script, so that only one network\ninterface is created, which is connected to the selected network. This\nchange also removes a lot of non-used code from install-os-vpx.sh\nSpecify the network to be used for ubuntu netinstall with:\n\n    UBUNTU_INST_BRIDGE_OR_NET_NAME\n\ndefaults to $MGT_BRIDGE_OR_NET_NAME.\n\nThis change also includes:\n - use precise as default\n - update readme file, as network defaults were updated previously\n\nRelated to blueprint xenapi-devstack-cleanup\n\nChange-Id: Ib0a0a316e849f2c2651305ea657c84820cd2bb3b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/c57b365ac68de02b569aa48608eb6bf2c89feef2"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/c57b365ac68de02b569aa48608eb6bf2c89feef2"}]},"branch":"refs/heads/master"},"2f524bd9053ee106daa98414bbe3b94b0cd2e43f":{"kind":"REWORK","_number":3,"created":"2013-06-20 14:26:55.000000000","uploader":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"ref":"refs/changes/32/33632/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/32/33632/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/32/33632/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/32/33632/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/32/33632/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/32/33632/3"}}},"commit":{"parents":[{"commit":"5a56cd6283f918e6022dcc47906fb86a739496dc","subject":"xenapi: Cleanup networking","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/5a56cd6283f918e6022dcc47906fb86a739496dc"}]}],"author":{"name":"Mate Lakat","email":"mate.lakat@citrix.com","date":"2013-06-19 11:32:23.000000000","tz":60},"committer":{"name":"Mate Lakat","email":"mate.lakat@citrix.com","date":"2013-06-20 14:22:58.000000000","tz":60},"subject":"xenapi: cleanup VM Installation","message":"xenapi: cleanup VM Installation\n\nVM installation created a machine with multiple network interfaces. This\nis a needless complexity at that point, we only need one interface. This\nchange removes the complex network interface setup from\ninstall-os-vpx.sh script, so that only one network interface is created,\nwhich is connected to the selected network (management by default). This\nchange also removes a lot of non-used code from install-os-vpx.sh\nSpecify the network to be used for ubuntu netinstall with:\n\n    UBUNTU_INST_BRIDGE_OR_NET_NAME\n\ndefaults to $MGT_BRIDGE_OR_NET_NAME.\n\nThis change also includes:\n - use precise as default\n\nRelated to blueprint xenapi-devstack-cleanup\n\nChange-Id: Ib0a0a316e849f2c2651305ea657c84820cd2bb3b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/2f524bd9053ee106daa98414bbe3b94b0cd2e43f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/2f524bd9053ee106daa98414bbe3b94b0cd2e43f"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}