)]}'
{"id":"openstack%2Fdevstack~30227","triplet_id":"openstack%2Fdevstack~master~I4f01dd65a4e5dd0dff33ea7c0024fb75fe83834b","project":"openstack/devstack","branch":"master","topic":"clean_lvm","hashtags":[],"change_id":"I4f01dd65a4e5dd0dff33ea7c0024fb75fe83834b","subject":"On unstack, cleanup LVM setup for Cinder","status":"MERGED","created":"2013-05-23 08:31:29.000000000","updated":"2013-06-19 11:08:10.000000000","submitted":"2013-06-19 11:08:09.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":5,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"ac9d5c001d9f0631344409ac86057466fde40767","_number":30227,"virtual_id_number":30227,"owner":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":2976,"name":"Adalberto Medeiros","email":"adalbas@gmail.com","username":"adalbas"},{"value":2,"date":"2013-06-19 11:08:09.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},{"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"},"recommended":{"_account_id":2976,"name":"Adalberto Medeiros","email":"adalbas@gmail.com","username":"adalbas"},"all":[{"value":1,"date":"2013-06-06 14:16:59.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":2976,"name":"Adalberto Medeiros","email":"adalbas@gmail.com","username":"adalbas"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"date":"2013-06-04 09:48:36.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},{"value":2,"date":"2013-06-18 16:20:01.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":2,"date":"2013-06-19 10:16:39.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":"","value":1,"default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"all":[{"value":0,"_account_id":2976,"name":"Adalberto Medeiros","email":"adalbas@gmail.com","username":"adalbas"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},{"value":0,"date":"2013-06-18 16:20:01.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":1,"date":"2013-06-19 10:16:39.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":2976,"name":"Adalberto Medeiros","email":"adalbas@gmail.com","username":"adalbas"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},{"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":2976,"name":"Adalberto Medeiros","email":"adalbas@gmail.com","username":"adalbas"},{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-06-06 14:16:59.000000000","updated_by":{"_account_id":2976,"name":"Adalberto Medeiros","email":"adalbas@gmail.com","username":"adalbas"},"reviewer":{"_account_id":2976,"name":"Adalberto Medeiros","email":"adalbas@gmail.com","username":"adalbas"},"state":"REVIEWER"},{"updated":"2013-06-18 16:20:01.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-19 10:16:39.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-19 11:08:09.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"e0b3e1f51dd9881dc0cad6f955c113b08bfcc4b9","author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"date":"2013-05-23 08:36:07.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"4106d01f42da2a2089588b789da30ac0b9fe8144","author":{"_account_id":5494,"name":"Trivial Rebase","email":"trivial-rebase@review.openstack.org","username":"trivial-rebase"},"date":"2013-05-23 08:36:13.000000000","message":"Patch Set 2:\n\nNew patchset patch-id matches previous patchset, but whitespace content has changed.","accounts_in_message":[],"_revision_number":2},{"id":"9bc7a0bfa68c244bfd368cd06da6759dc6b41d28","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-05-23 09:21:59.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/30227/2/check/gate-tempest-devstack-vm-full/17958 : SUCCESS in 43m 56s\n- http://logs.openstack.org/30227/2/check/gate-tempest-devstack-vm-postgres-full/17668 : FAILURE in 41m 04s (non-voting)\n- http://logs.openstack.org/30227/2/check/gate-tempest-devstack-vm-quantum/24684 : FAILURE in 45m 38s\n- http://logs.openstack.org/30227/2/check/gate-grenade-devstack-vm/5455 : FAILURE in 9m 47s (non-voting)\n","accounts_in_message":[],"_revision_number":2},{"id":"b0bc49a1c5a310dc7447c7e6aecab997743dd6ed","author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"date":"2013-05-23 12:11:49.000000000","message":"Patch Set 2:\n\nrecheck no bug","accounts_in_message":[],"_revision_number":2},{"id":"efe3d1f8bc64f0516c37b358cc368ea8d90f61c0","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-05-23 12:57:12.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/30227/2/check/gate-tempest-devstack-vm-full/17989 : SUCCESS in 43m 45s\n- http://logs.openstack.org/30227/2/check/gate-tempest-devstack-vm-postgres-full/17699 : FAILURE in 45m 11s (non-voting)\n- http://logs.openstack.org/30227/2/check/gate-tempest-devstack-vm-quantum/24723 : SUCCESS in 21m 59s\n- http://logs.openstack.org/30227/2/check/gate-grenade-devstack-vm/5483 : FAILURE in 10m 00s (non-voting)\n","accounts_in_message":[],"_revision_number":2},{"id":"273f6b9caf89e596b859922cd3e363891d8c0672","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-05-24 09:58:37.000000000","message":"Patch Set 2: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":2},{"id":"542b6416715a707a256234bb40770c24de7e3124","author":{"_account_id":2976,"name":"Adalberto Medeiros","email":"adalbas@gmail.com","username":"adalbas"},"date":"2013-05-24 12:28:17.000000000","message":"Patch Set 2: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":2},{"id":"ee238d2b0c1c67e89eefd4c3a513a98d682131ed","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-05-24 15:50:53.000000000","message":"Patch Set 2: Do not merge\n\n(1 inline comment)\n\nI\u0027d put the cleanup code in _clean_volume_group() as it also needs to run on the second vg (VOLUME_GROUP2).\n\nAlso, moving the setting of VOLUME_BACKING_FILE and VOLUME_BACKING_FILE2 out of create_cinder_volume_group() and to the top next to VOLUME_GROUP would make the names available everywhere","accounts_in_message":[],"_revision_number":2},{"id":"a752d393c387b60e3f4cc3373d73b7f1c6ba38e5","author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"date":"2013-05-24 16:35:58.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"04ea1de8f221f06bec73d237b47f93ba1f9afe3e","author":{"_account_id":2976,"name":"Adalberto Medeiros","email":"adalbas@gmail.com","username":"adalbas"},"date":"2013-05-24 17:17:21.000000000","message":"Patch Set 3: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":3},{"id":"b1f923f383e405c6cc5b8fcfce0403eeb3167204","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-05-24 17:27:20.000000000","message":"Patch Set 3: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/30227/3/check/gate-tempest-devstack-vm-full/18370 : SUCCESS in 48m 28s\n- http://logs.openstack.org/30227/3/check/gate-tempest-devstack-vm-postgres-full/18080 : FAILURE in 51m 08s (non-voting)\n- http://logs.openstack.org/30227/3/check/gate-tempest-devstack-vm-quantum/25143 : SUCCESS in 20m 43s\n- http://logs.openstack.org/30227/3/check/gate-grenade-devstack-vm/5825 : FAILURE in 17m 12s (non-voting)\n","accounts_in_message":[],"_revision_number":3},{"id":"0cf5e48b83a49fb22b473554ad1fc6cd637e2a50","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-05-24 17:41:07.000000000","message":"Patch Set 3:\n\nThe point of moving the name setting was to reuse it in the cleanup code.\n\nWhen will _clean_lvm_backing_file() be called apart from _clean_lvm_group()?  I don\u0027t see the need for two functions here.","accounts_in_message":[],"_revision_number":3},{"id":"1ca03fadf326fc9d8f1155f6252d53fb2838c7df","author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"date":"2013-05-24 21:08:23.000000000","message":"Patch Set 3:\n\nI didn\u0027t want to rename the function \"_clean_volume_group\". And I couldn\u0027t put the lvm cleaning code inside this function because the function would have done more than just cleaning the volume group.\n\nPlus I prefer to have small functions (max 10 lines) so having 2 functions seemed to be the better option.\n\nAnd you never know, maybe one day one of the 2 functions will be called individually :)","accounts_in_message":[],"_revision_number":3},{"id":"597e2b6090deeb266f5c53b465d3a151bce842d5","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-05-29 15:34:16.000000000","message":"Patch Set 3: (4 inline comments)\n\n[I thought I\u0027d posted this earlier...]\n\nDon\u0027t worry about requirements that do not exist yet, keep things simple, and in this case I think one function is simpler than two since they are closely related.","accounts_in_message":[],"_revision_number":3},{"id":"c4a3c0e7cf8f4b468549376218acc3426a837c9d","author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"date":"2013-06-04 09:22:21.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"7469807b8d2eb94fd532cbd362f26ed77f6b1ae9","author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"date":"2013-06-04 09:48:36.000000000","message":"Patch Set 4:\n\nI think vgremove/pvremove is not mandatory. I have done a lot of stack/unstack without it and my lvm is still in good shape. I believe each time you issue an lv* command the list of physical volume and volume group is rescanned and as the backing device is gone, LVM \"understands\" and doesnt complain.\n\nThere\u0027s still is \"losetup -d\" in clean.sh because the initial logic in clean.sh is to delete the loop device unconditionnaly whereas in lib/cinder, the loop device is deleted only if there is no LV left.","accounts_in_message":[],"_revision_number":4},{"id":"a44ef18bbef00bb951a656ffe6bd41f4be48d1e5","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-06-04 10:16:28.000000000","message":"Patch Set 4: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/30227/4/check/gate-tempest-devstack-vm-full/20292 : SUCCESS in 45m 11s\n- http://logs.openstack.org/30227/4/check/gate-tempest-devstack-vm-postgres-full/19999 : FAILURE in 53m 49s (non-voting)\n- http://logs.openstack.org/30227/4/check/gate-tempest-devstack-vm-quantum/27314 : SUCCESS in 20m 53s\n- http://logs.openstack.org/30227/4/check/gate-grenade-devstack-vm/7657 : SUCCESS in 14m 38s (non-voting)\n","accounts_in_message":[],"_revision_number":4},{"id":"0d3a4446f61af399a3ab0127053fb8b4797d3eac","author":{"_account_id":2976,"name":"Adalberto Medeiros","email":"adalbas@gmail.com","username":"adalbas"},"date":"2013-06-06 14:16:59.000000000","message":"Patch Set 4: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":4},{"id":"39647e34cd13e879a550a699ae6a2579fc7fa872","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-06-18 16:20:01.000000000","message":"Patch Set 4: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":4},{"id":"fb02d4950ca8a8954d2d8f8e040c3e644920afd5","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-06-19 10:16:39.000000000","message":"Patch Set 4: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":4},{"id":"c04d29bebb0c5db001f98f7db9529f9ce8c3f000","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-06-19 10:16:40.000000000","message":"Patch Set 4: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":4},{"id":"bfd8414bdc9e2593577048577be86ba37fea20ae","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-06-19 11:08:09.000000000","message":"Patch Set 4: Verified\n\nBuild succeeded.\n\n- http://logs.openstack.org/30227/4/gate/gate-tempest-devstack-vm-full/24551 : SUCCESS in 49m 37s\n- http://logs.openstack.org/30227/4/gate/gate-tempest-devstack-vm-postgres-full/24257 : SUCCESS in 40m 14s\n- http://logs.openstack.org/30227/4/gate/gate-tempest-devstack-vm-quantum/32020 : SUCCESS in 16m 38s\n- http://logs.openstack.org/30227/4/gate/gate-grenade-devstack-vm/11398 : FAILURE in 38m 28s (non-voting)\n","accounts_in_message":[],"_revision_number":4},{"id":"027dc62c2a1df1cc2bb8b35f5f196dece1158495","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-06-19 11:08:10.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":4}],"current_revision_number":4,"current_revision":"51c90b858d5106a6a3fe57aa417e1fb8faea414d","revisions":{"4b515fed93c408c11b3137615573171ad5cb35be":{"kind":"REWORK","_number":1,"created":"2013-05-23 08:31:29.000000000","uploader":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"ref":"refs/changes/27/30227/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/27/30227/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/27/30227/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/27/30227/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/27/30227/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/27/30227/1"}}},"commit":{"parents":[{"commit":"7640350bb5b343fae0932c86b7e536bb5d5e8db1","subject":"Merge \"Avoid installing python-nose for RHEL6.\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/7640350bb5b343fae0932c86b7e536bb5d5e8db1"}]}],"author":{"name":"JordanP","email":"jordan.pittier@gmail.com","date":"2013-05-23 08:27:51.000000000","tz":120},"committer":{"name":"JordanP","email":"jordan.pittier@gmail.com","date":"2013-05-23 08:27:51.000000000","tz":120},"subject":"On unstack, cleanup LVM setup for Cinder","message":"On unstack, cleanup LVM setup for Cinder\n\nChange-Id: I4f01dd65a4e5dd0dff33ea7c0024fb75fe83834b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/4b515fed93c408c11b3137615573171ad5cb35be"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/4b515fed93c408c11b3137615573171ad5cb35be"}]},"branch":"refs/heads/master"},"b79dffd7ba342871fa1ac06374a9f236b5338b43":{"kind":"REWORK","_number":2,"created":"2013-05-23 08:36:07.000000000","uploader":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"ref":"refs/changes/27/30227/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/27/30227/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/27/30227/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/27/30227/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/27/30227/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/27/30227/2"}}},"commit":{"parents":[{"commit":"7640350bb5b343fae0932c86b7e536bb5d5e8db1","subject":"Merge \"Avoid installing python-nose for RHEL6.\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/7640350bb5b343fae0932c86b7e536bb5d5e8db1"}]}],"author":{"name":"JordanP","email":"jordan.pittier@gmail.com","date":"2013-05-23 08:27:51.000000000","tz":120},"committer":{"name":"JordanP","email":"jordan.pittier@gmail.com","date":"2013-05-23 08:35:48.000000000","tz":120},"subject":"On unstack, cleanup LVM setup for Cinder","message":"On unstack, cleanup LVM setup for Cinder\n\nChange-Id: I4f01dd65a4e5dd0dff33ea7c0024fb75fe83834b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b79dffd7ba342871fa1ac06374a9f236b5338b43"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b79dffd7ba342871fa1ac06374a9f236b5338b43"}]},"branch":"refs/heads/master"},"e3a7025a680471cdb67c2ce37a2972379876a68f":{"kind":"REWORK","_number":3,"created":"2013-05-24 16:35:58.000000000","uploader":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"ref":"refs/changes/27/30227/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/27/30227/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/27/30227/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/27/30227/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/27/30227/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/27/30227/3"}}},"commit":{"parents":[{"commit":"edbea45888cea7f94bfbb8c94e0de3c47443ac4d","subject":"Merge \"xenapi - cleanup network configuration\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/edbea45888cea7f94bfbb8c94e0de3c47443ac4d"}]}],"author":{"name":"JordanP","email":"jordan.pittier@gmail.com","date":"2013-05-23 08:27:51.000000000","tz":120},"committer":{"name":"JordanP","email":"jordan.pittier@gmail.com","date":"2013-05-24 16:35:47.000000000","tz":120},"subject":"On unstack, cleanup LVM setup for Cinder","message":"On unstack, cleanup LVM setup for Cinder\n\nChange-Id: I4f01dd65a4e5dd0dff33ea7c0024fb75fe83834b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/e3a7025a680471cdb67c2ce37a2972379876a68f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/e3a7025a680471cdb67c2ce37a2972379876a68f"}]},"branch":"refs/heads/master"},"51c90b858d5106a6a3fe57aa417e1fb8faea414d":{"kind":"REWORK","_number":4,"created":"2013-06-04 09:22:21.000000000","uploader":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"ref":"refs/changes/27/30227/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/27/30227/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/27/30227/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/27/30227/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/27/30227/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/27/30227/4"}}},"commit":{"parents":[{"commit":"cc8f47dca545466efea971edf3ffd99f5bdcaee0","subject":"Merge \"xenapi: kernel_cmdline moved out from template\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/cc8f47dca545466efea971edf3ffd99f5bdcaee0"}]}],"author":{"name":"JordanP","email":"jordan.pittier@gmail.com","date":"2013-05-23 08:27:51.000000000","tz":120},"committer":{"name":"JordanP","email":"jordan.pittier@gmail.com","date":"2013-06-04 09:18:24.000000000","tz":120},"subject":"On unstack, cleanup LVM setup for Cinder","message":"On unstack, cleanup LVM setup for Cinder\n\nChange-Id: I4f01dd65a4e5dd0dff33ea7c0024fb75fe83834b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/51c90b858d5106a6a3fe57aa417e1fb8faea414d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/51c90b858d5106a6a3fe57aa417e1fb8faea414d"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}