)]}'
{"id":"openstack%2Fdevstack~36461","triplet_id":"openstack%2Fdevstack~master~I823a8eac4f3a2ef313d06e21da0f38ed46d7386a","project":"openstack/devstack","branch":"master","topic":"xenserver-core-kpartx","hashtags":[],"change_id":"I823a8eac4f3a2ef313d06e21da0f38ed46d7386a","subject":"manage-vdi: Wait until devices have been created before mounting","status":"MERGED","created":"2013-07-10 13:07:27.000000000","updated":"2013-07-31 14:48:21.000000000","submitted":"2013-07-31 14:48:21.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":2,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"612a1a7191ebb2af624004419c82ee32f1f836d7","_number":36461,"virtual_id_number":36461,"owner":{"_account_id":7952,"name":"Euan Harris","email":"euan.harris@citrix.com","username":"euanh"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":7952,"name":"Euan Harris","email":"euan.harris@citrix.com","username":"euanh"},{"value":2,"date":"2013-07-31 14:48:21.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"},{"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,"date":"2013-07-12 15:11:34.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":7952,"name":"Euan Harris","email":"euan.harris@citrix.com","username":"euanh"},{"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-31 13:50:40.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":1,"date":"2013-07-18 07:46:55.000000000","permitted_voting_range":{"min":1,"max":1},"_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"},{"value":1,"date":"2013-07-11 15:12:53.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":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"all":[{"value":0,"_account_id":7952,"name":"Euan Harris","email":"euan.harris@citrix.com","username":"euanh"},{"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":1,"date":"2013-07-31 13:50:40.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":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"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":7952,"name":"Euan Harris","email":"euan.harris@citrix.com","username":"euanh"},{"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"},{"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"},{"_account_id":7952,"name":"Euan Harris","email":"euan.harris@citrix.com","username":"euanh"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-07-10 13:07:52.000000000","updated_by":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"reviewer":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"state":"REVIEWER"},{"updated":"2013-07-10 13:16:09.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-11 15:12:53.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-07-18 07:46:55.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-31 13:50:40.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-31 14:48:21.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"10aef03dd06019373e24d235f8b93939af661486","author":{"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"},"date":"2013-07-10 13:09:59.000000000","message":"Patch Set 1: Looks good to me, but someone else must approve\n\nImpressed this worked before!","accounts_in_message":[],"_revision_number":1},{"id":"9520f226d20181c0cac620bfd7216d7341bc5da2","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-07-10 14:02:22.000000000","message":"Patch Set 1: Doesn\u0027t seem to work\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGithub#Test_Failures\n\n- http://logs.openstack.org/36461/1/check/gate-tempest-devstack-vm-full/30317 : SUCCESS in 54m 36s\n- http://logs.openstack.org/36461/1/check/gate-tempest-devstack-vm-postgres-full/30021 : SUCCESS in 43m 34s\n- http://logs.openstack.org/36461/1/check/gate-tempest-devstack-vm-neutron/1532 : FAILURE in 21m 01s\n- http://logs.openstack.org/36461/1/check/gate-grenade-devstack-vm/16467 : FAILURE in 21m 05s (non-voting)\n","accounts_in_message":[],"_revision_number":1},{"id":"0ece83bfd4cdd71ee93b33f7a5872a91b5c1da3b","author":{"_account_id":7952,"name":"Euan Harris","email":"euan.harris@citrix.com","username":"euanh"},"date":"2013-07-10 14:43:30.000000000","message":"Patch Set 1:\n\nRecheck bug 1194026","accounts_in_message":[],"_revision_number":1},{"id":"5aa591b00aa91f20319e5197d290791287de1b83","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-07-10 15:36:52.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/36461/1/check/gate-tempest-devstack-vm-full/30360 : SUCCESS in 47m 08s\n- http://logs.openstack.org/36461/1/check/gate-tempest-devstack-vm-postgres-full/30064 : SUCCESS in 45m 43s\n- http://logs.openstack.org/36461/1/check/gate-tempest-devstack-vm-neutron/1583 : SUCCESS in 22m 56s\n- http://logs.openstack.org/36461/1/check/gate-grenade-devstack-vm/16505 : FAILURE in 22m 53s (non-voting)\n","accounts_in_message":[],"_revision_number":1},{"id":"3553f070c116618a02c356d5387834a31a82f8d3","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-07-10 16:58:30.000000000","message":"Patch Set 1: Looks good to me (core reviewer)\n\n-s must be new?","accounts_in_message":[],"_revision_number":1},{"id":"37a8c701b2d7af9e4d1880857c38492ec67382da","author":{"_account_id":7952,"name":"Euan Harris","email":"euan.harris@citrix.com","username":"euanh"},"date":"2013-07-10 17:12:40.000000000","message":"Patch Set 1:\n\nThe -s flag was added in 2010:\nhttp://git.opensvc.com/?p\u003dmultipath-tools/.git;a\u003dcommit;h\u003d8d63b33d0996e141a2451df552b062b908db15bc","accounts_in_message":[],"_revision_number":1},{"id":"2ccf98cdc22b0fcaf9e669aee0cc4133c417237a","author":{"_account_id":7952,"name":"Euan Harris","email":"euan.harris@citrix.com","username":"euanh"},"date":"2013-07-11 08:21:56.000000000","message":"Patch Set 1: Work In Progress","accounts_in_message":[],"_revision_number":1},{"id":"affc5116d01ecff55d23e309c0fc816298d37b73","author":{"_account_id":7952,"name":"Euan Harris","email":"euan.harris@citrix.com","username":"euanh"},"date":"2013-07-11 14:26:21.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"9ca4d1ca2036f4bb23328b1c5f052584b61def00","author":{"_account_id":7952,"name":"Euan Harris","email":"euan.harris@citrix.com","username":"euanh"},"date":"2013-07-11 14:26:47.000000000","message":"Patch Set 2:\n\nAnnoyingly, dom0\u0027s kpartx is too old to support the -s flag.   Work around it with a wait loop.","accounts_in_message":[],"_revision_number":2},{"id":"2d64db9efaf1c9d4ba051cae63917f195ceaab7c","author":{"_account_id":7952,"name":"Euan Harris","email":"euan.harris@citrix.com","username":"euanh"},"date":"2013-07-11 14:51:08.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"0296bbc6bd15edc42e262b95fce8451fe517b165","author":{"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"},"date":"2013-07-11 15:12:53.000000000","message":"Patch Set 3: Looks good to me, but someone else must approve\n\nThanks for the timeout!","accounts_in_message":[],"_revision_number":3},{"id":"3fa980c4f34600649fe3e7be9d0061307fc1b3bb","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-07-11 15:35:07.000000000","message":"Patch Set 3: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/36461/3/check/gate-tempest-devstack-vm-full/31178 : SUCCESS in 43m 28s\n- http://logs.openstack.org/36461/3/check/gate-tempest-devstack-vm-postgres-full/30871 : SUCCESS in 43m 51s\n- http://logs.openstack.org/36461/3/check/gate-tempest-devstack-vm-neutron/2477 : SUCCESS in 17m 31s\n- http://logs.openstack.org/36461/3/check/gate-grenade-devstack-vm/17249 : FAILURE in 26m 39s (non-voting)\n","accounts_in_message":[],"_revision_number":3},{"id":"0f9515d4c62a80f1e1d908cb7a1d568cca4de0cb","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-07-12 14:58:21.000000000","message":"Patch Set 3: I would prefer that you didn\u0027t merge this\n\n(1 inline comment)\n\n","accounts_in_message":[],"_revision_number":3},{"id":"57f84d47d57f02930175dad2ec700c6d1f799562","author":{"_account_id":7952,"name":"Euan Harris","email":"euan.harris@citrix.com","username":"euanh"},"date":"2013-07-12 15:11:34.000000000","message":"Patch Set 3: (1 inline comment)\n\n@Dean: Replied inline.","accounts_in_message":[],"_revision_number":3},{"id":"cb387c517c5ecbe4ca112dd3ab6174b8cebb450c","author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"date":"2013-07-18 07:46:55.000000000","message":"Patch Set 3: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":3},{"id":"4c8fa076329923fd1265b7f9ff528ee4be5e2bdd","author":{"_account_id":7952,"name":"Euan Harris","email":"euan.harris@citrix.com","username":"euanh"},"date":"2013-07-18 08:23:03.000000000","message":"Patch Set 3:\n\n@Dean:\nCould you please reconsider your review?   We can\u0027t use the timeout pattern in Dom0 because we don\u0027t have the \u0027timeout\u0027 command.","accounts_in_message":[],"_revision_number":3},{"id":"82d704e10e956c34d9987d28d98d5dbb5e91ab88","author":{"_account_id":9,"name":"LaunchpadSync","username":"launchpadsync"},"date":"2013-07-26 06:03:06.000000000","message":"Patch Set 3:\n\ncode review expired after 1 week of no activity after a negative review, it can be restored using the `Restore Change` button under the Patch Set on the web interface","accounts_in_message":[],"_revision_number":3},{"id":"fa28908aeba5f8b088ca77446e1f8bc350a59b5a","author":{"_account_id":9,"name":"LaunchpadSync","username":"launchpadsync"},"date":"2013-07-26 06:03:06.000000000","message":"Patch Set 3: Abandoned\n\ncode review expired after 1 week of no activity after a negative review, it can be restored using the `Restore Change` button under the Patch Set on the web interface","accounts_in_message":[],"_revision_number":3},{"id":"f212624f616c5d817a3fa7fe2c62d01844858713","author":{"_account_id":7952,"name":"Euan Harris","email":"euan.harris@citrix.com","username":"euanh"},"date":"2013-07-26 08:11:55.000000000","message":"Patch Set 3: Restored\n\nTimed out.   Dean, please respond.","accounts_in_message":[],"_revision_number":3},{"id":"4bfdbbb4d9f149675cbae44d24aaf5039b2009b9","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-07-31 13:50:40.000000000","message":"Patch Set 3: Looks good to me (core reviewer); Approved\n\nApologies for missing this earlier...","accounts_in_message":[],"_revision_number":3},{"id":"c13c237367b39c4570945c815e614cace251f9d9","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-07-31 13:50:41.000000000","message":"Patch Set 3: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":3},{"id":"fdf6255a34a0eae3bd8320daab84862970d6aa11","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-07-31 14:48:21.000000000","message":"Patch Set 3: Verified\n\nBuild succeeded.\n\n- http://logs.openstack.org/61/36461/3/gate/gate-tempest-devstack-vm-full/37815 : SUCCESS in 45m 29s\n- http://logs.openstack.org/61/36461/3/gate/gate-tempest-devstack-vm-postgres-full/37490 : SUCCESS in 57m 37s\n- http://logs.openstack.org/61/36461/3/gate/gate-tempest-devstack-vm-neutron/9967 : FAILURE in 10m 37s (non-voting)\n- http://logs.openstack.org/61/36461/3/gate/gate-grenade-devstack-vm/23140 : SUCCESS in 22m 55s\n","accounts_in_message":[],"_revision_number":3},{"id":"37ebfe855afe95946a606a11cec2c6665c038a76","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-07-31 14:48:21.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"a3ec804ad90fd13944c1b6bd5e55e85e6ecdb62c","revisions":{"055da205f6dde77f3024fd1ec5cec91af1456190":{"kind":"REWORK","_number":1,"created":"2013-07-10 13:07:27.000000000","uploader":{"_account_id":7952,"name":"Euan Harris","email":"euan.harris@citrix.com","username":"euanh"},"ref":"refs/changes/61/36461/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/61/36461/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/61/36461/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/61/36461/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/61/36461/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/61/36461/1"}}},"commit":{"parents":[{"commit":"314ae87a911b457590cd268a6d50872c957de343","subject":"Merge \"xenapi: Add qemu-utils as a cinder dependency\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/314ae87a911b457590cd268a6d50872c957de343"}]}],"author":{"name":"Euan Harris","email":"euan.harris@citrix.com","date":"2013-07-04 15:25:33.000000000","tz":60},"committer":{"name":"Euan Harris","email":"euan.harris@citrix.com","date":"2013-07-10 13:07:08.000000000","tz":60},"subject":"manage-vdi: Wait until devices have been created before mounting","message":"manage-vdi: Wait until devices have been created before mounting\n\nBy default kpartx runs in asynchronous mode, in which it returns before\nthe partition devices have been created.   This may cause the subsequent\nmount call to fail, because the device it is trying to mount does not\nyet exist.   Make kpartx block until the devices have been created.\n\nChange-Id: I823a8eac4f3a2ef313d06e21da0f38ed46d7386a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/055da205f6dde77f3024fd1ec5cec91af1456190"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/055da205f6dde77f3024fd1ec5cec91af1456190"}]},"branch":"refs/heads/master"},"20dc0046c1397b020e25525aa61fb22fdfdde230":{"kind":"REWORK","_number":2,"created":"2013-07-11 14:26:21.000000000","uploader":{"_account_id":7952,"name":"Euan Harris","email":"euan.harris@citrix.com","username":"euanh"},"ref":"refs/changes/61/36461/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/61/36461/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/61/36461/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/61/36461/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/61/36461/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/61/36461/2"}}},"commit":{"parents":[{"commit":"314ae87a911b457590cd268a6d50872c957de343","subject":"Merge \"xenapi: Add qemu-utils as a cinder dependency\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/314ae87a911b457590cd268a6d50872c957de343"}]}],"author":{"name":"Euan Harris","email":"euan.harris@citrix.com","date":"2013-07-04 15:25:33.000000000","tz":60},"committer":{"name":"Euan Harris","email":"euan.harris@citrix.com","date":"2013-07-11 14:25:43.000000000","tz":60},"subject":"manage-vdi: Wait until devices have been created before mounting","message":"manage-vdi: Wait until devices have been created before mounting\n\nkpartx creates partition devices asynchronously - it may exit before\nthe devices have been created.  This may cause a subsequent mount call\nto fail, because the device it is trying to mount does not yet exist.\nkpartx now has a -s option which makes it wait for the devices to be\ncreated before returning, but the version XenServer\u0027s dom0 doesn\u0027t have\nthis option.   Work around this by retrying the mount.\n\nWhen dom0\u0027s kpartx is updated, we will be able to use the -s option.\n\nChange-Id: I823a8eac4f3a2ef313d06e21da0f38ed46d7386a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/20dc0046c1397b020e25525aa61fb22fdfdde230"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/20dc0046c1397b020e25525aa61fb22fdfdde230"}]},"branch":"refs/heads/master"},"a3ec804ad90fd13944c1b6bd5e55e85e6ecdb62c":{"kind":"REWORK","_number":3,"created":"2013-07-11 14:51:08.000000000","uploader":{"_account_id":7952,"name":"Euan Harris","email":"euan.harris@citrix.com","username":"euanh"},"ref":"refs/changes/61/36461/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/61/36461/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/61/36461/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/61/36461/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/61/36461/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/61/36461/3"}}},"commit":{"parents":[{"commit":"314ae87a911b457590cd268a6d50872c957de343","subject":"Merge \"xenapi: Add qemu-utils as a cinder dependency\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/314ae87a911b457590cd268a6d50872c957de343"}]}],"author":{"name":"Euan Harris","email":"euan.harris@citrix.com","date":"2013-07-04 15:25:33.000000000","tz":60},"committer":{"name":"Euan Harris","email":"euan.harris@citrix.com","date":"2013-07-11 14:50:32.000000000","tz":60},"subject":"manage-vdi: Wait until devices have been created before mounting","message":"manage-vdi: Wait until devices have been created before mounting\n\nkpartx creates partition devices asynchronously - it may exit before\nthe devices have been created.  This may cause a subsequent mount call\nto fail, because the device it is trying to mount does not yet exist.\nkpartx now has a -s option which makes it wait for the devices to be\ncreated before returning, but the version XenServer\u0027s dom0 doesn\u0027t have\nthis option.   Work around this by retrying the mount.\n\nWhen dom0\u0027s kpartx is updated, we will be able to use the -s option.\n\nChange-Id: I823a8eac4f3a2ef313d06e21da0f38ed46d7386a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/a3ec804ad90fd13944c1b6bd5e55e85e6ecdb62c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/a3ec804ad90fd13944c1b6bd5e55e85e6ecdb62c"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}