)]}'
{"id":"openstack%2Fdevstack~36457","triplet_id":"openstack%2Fdevstack~master~I0a66cd03f000f19c8ff63a8ae4231c00fda88380","project":"openstack/devstack","branch":"master","topic":"xenserver-core-paths","hashtags":[],"change_id":"I0a66cd03f000f19c8ff63a8ae4231c00fda88380","subject":"devstack: Add paths specific to XenServer Core","status":"MERGED","created":"2013-07-10 13:01:11.000000000","updated":"2013-07-10 20:30:39.000000000","submitted":"2013-07-10 20:30:39.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"43112a731f1fed3e485fec3a030c93d2edf0bc77","_number":36457,"virtual_id_number":36457,"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-10 20:30:39.000000000","post_submit":true,"_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-10 14:43:47.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,"date":"2013-07-10 14:49:45.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-07-10 17:00:39.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":1,"date":"2013-07-10 16:19:25.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-10 13:11:03.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,"date":"2013-07-10 13:05:55.000000000","_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-10 17:00:39.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,"date":"2013-07-10 13:14:40.000000000","_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:11:03.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-10 13:14: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-07-10 14:49:45.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 16:19:25.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-10 17:00:39.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-10 20:30:39.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"71dc0a4f324d7fcfea8190ed74b10a3eeb889d3f","author":{"_account_id":7952,"name":"Euan Harris","email":"euan.harris@citrix.com","username":"euanh"},"date":"2013-07-10 13:05:55.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"aa33d61833577477d5e815b9aeae737d8b4c8442","author":{"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"},"date":"2013-07-10 13:11:03.000000000","message":"Patch Set 2: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":2},{"id":"d96b269eb619003a257462e3fddb7f0460f64ff4","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-07-10 14:00:40.000000000","message":"Patch Set 2: 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/36457/2/check/gate-tempest-devstack-vm-full/30316 : SUCCESS in 54m 23s\n- http://logs.openstack.org/36457/2/check/gate-tempest-devstack-vm-postgres-full/30020 : SUCCESS in 47m 56s\n- http://logs.openstack.org/36457/2/check/gate-tempest-devstack-vm-neutron/1531 : FAILURE in 35m 51s\n- http://logs.openstack.org/36457/2/check/gate-grenade-devstack-vm/16466 : FAILURE in 26m 23s (non-voting)\n","accounts_in_message":[],"_revision_number":2},{"id":"559665bf7db68bf31c5d97f9c716fad844e6de08","author":{"_account_id":7952,"name":"Euan Harris","email":"euan.harris@citrix.com","username":"euanh"},"date":"2013-07-10 14:43:47.000000000","message":"Patch Set 2:\n\nRecheck bug 1194026","accounts_in_message":[],"_revision_number":2},{"id":"813371f7ec6cfb8f95f6175cbc80efeb0a7a6450","author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"date":"2013-07-10 14:49:45.000000000","message":"Patch Set 2: (1 inline comment)\n\nSee comments inline","accounts_in_message":[],"_revision_number":2},{"id":"578bfbd0c9dc59a8b3519121e028804c00d89861","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-07-10 14:50:10.000000000","message":"Patch Set 2:\n\nI\u0027m fine with the code, does there need to be anything in tools/xen/README.md regarding XenSever vs XenServer Core?","accounts_in_message":[],"_revision_number":2},{"id":"a7cd93fe750669abbbca8f60734f5dc2b7923bf4","author":{"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"},"date":"2013-07-10 14:53:26.000000000","message":"Patch Set 2:\n\nWe\u0027d rather do more testing and commit any more necessary fixes before updating the README to include references to xenserver-core.\n\nThese two changes are known to be needed, but as we work through the tempest tests we might find more that are needed as well.","accounts_in_message":[],"_revision_number":2},{"id":"b620824a388fd0198808db873b749cebb236c943","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-07-10 15:47:01.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/36457/2/check/gate-tempest-devstack-vm-full/30361 : SUCCESS in 44m 29s\n- http://logs.openstack.org/36457/2/check/gate-tempest-devstack-vm-postgres-full/30065 : SUCCESS in 57m 01s\n- http://logs.openstack.org/36457/2/check/gate-tempest-devstack-vm-neutron/1584 : SUCCESS in 19m 35s\n- http://logs.openstack.org/36457/2/check/gate-grenade-devstack-vm/16506 : FAILURE in 22m 36s (non-voting)\n","accounts_in_message":[],"_revision_number":2},{"id":"2d869f6b0f43ce369812ad84f39f00933592064b","author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"date":"2013-07-10 16:19:25.000000000","message":"Patch Set 2: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":2},{"id":"954eb7c4529c3e356153dc38e2468608bba6347d","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-07-10 17:00:39.000000000","message":"Patch Set 2: Looks good to me (core reviewer); Approved\n\nBob, sounds good, thanks","accounts_in_message":[],"_revision_number":2},{"id":"b78377cb80a8dd49a098d757800218610043e480","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-07-10 19:30:18.000000000","message":"Patch Set 2: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"d5dab31fa967f035c973c4b4ca2d92eaa973a41e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-07-10 20:30:39.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":2},{"id":"ab16f6a24427941238621a44156defb1932c74c3","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-07-10 20:30:39.000000000","message":"Patch Set 2: Verified\n\nBuild succeeded.\n\n- http://logs.openstack.org/36457/2/gate/gate-tempest-devstack-vm-full/30464 : SUCCESS in 57m 48s\n- http://logs.openstack.org/36457/2/gate/gate-tempest-devstack-vm-postgres-full/30168 : SUCCESS in 48m 52s\n- http://logs.openstack.org/36457/2/gate/gate-tempest-devstack-vm-neutron/1700 : SUCCESS in 21m 16s\n- http://logs.openstack.org/36457/2/gate/gate-grenade-devstack-vm/16603 : FAILURE in 25m 53s (non-voting)\n","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"12229a77a525ea92f037fb2bdfd54681de6bde6a","revisions":{"8cad85e1b7d86ad53423ba89bc53a5576bc3408f":{"kind":"REWORK","_number":1,"created":"2013-07-10 13:01:11.000000000","uploader":{"_account_id":7952,"name":"Euan Harris","email":"euan.harris@citrix.com","username":"euanh"},"ref":"refs/changes/57/36457/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/57/36457/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/57/36457/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/57/36457/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/57/36457/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/57/36457/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-03 16:51:01.000000000","tz":60},"committer":{"name":"Euan Harris","email":"euan.harris@citrix.com","date":"2013-07-10 13:00:51.000000000","tz":60},"subject":"devstack: Add paths specific to XenServer Core","message":"devstack: Add paths specific to XenServer Core\n\n * XenServer Core keeps its Xapi plugins in a different path to XenServer,\n   in order to comply with the Filesystem Hierarchy Standard.\n\n * XenServer Core mounts devices under /dev/xen/blktap-2 in dom0,\n   whereas XenServer mounts them under /dev/sr.\n\nChange-Id: I0a66cd03f000f19c8ff63a8ae4231c00fda88380\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/8cad85e1b7d86ad53423ba89bc53a5576bc3408f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/8cad85e1b7d86ad53423ba89bc53a5576bc3408f"}]},"branch":"refs/heads/master"},"12229a77a525ea92f037fb2bdfd54681de6bde6a":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2013-07-10 13:05:55.000000000","uploader":{"_account_id":7952,"name":"Euan Harris","email":"euan.harris@citrix.com","username":"euanh"},"ref":"refs/changes/57/36457/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/57/36457/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/57/36457/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/57/36457/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/57/36457/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/57/36457/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-03 16:51:01.000000000","tz":60},"committer":{"name":"Euan Harris","email":"euan.harris@citrix.com","date":"2013-07-10 13:05:22.000000000","tz":60},"subject":"devstack: Add paths specific to XenServer Core","message":"devstack: Add paths specific to XenServer Core\n\n * XenServer Core keeps its Xapi plugins in a different path to XenServer,\n   in order to comply with the Filesystem Hierarchy Standard.\n\n * XenServer Core mounts devices under /dev/xen/blktap-2 in dom0,\n   whereas XenServer mounts them under /dev/sm.\n\nChange-Id: I0a66cd03f000f19c8ff63a8ae4231c00fda88380\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/12229a77a525ea92f037fb2bdfd54681de6bde6a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/12229a77a525ea92f037fb2bdfd54681de6bde6a"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}