)]}'
{"id":"openstack%2Fdevstack~34640","triplet_id":"openstack%2Fdevstack~master~I09a5e061a9665c5310383f9f9eb281bfdc8e416d","project":"openstack/devstack","branch":"master","topic":"rhel-lvm-autoextend","hashtags":[],"change_id":"I09a5e061a9665c5310383f9f9eb281bfdc8e416d","subject":"Update LVM autoextend config for RHEL6","status":"MERGED","created":"2013-06-26 23:14:09.000000000","updated":"2013-06-28 13:21:07.000000000","submitted":"2013-06-27 21:28:38.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"53de585b14e0434a3696456629806737796f04ec","_number":34640,"virtual_id_number":34640,"owner":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":0,"_account_id":6473,"name":"Sunil Thaha","email":"sthaha@redhat.com","username":"sthaha"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":2,"date":"2013-06-27 21:28:38.000000000","post_submit":true,"_account_id":3,"name":"Jenkins","username":"jenkins"}],"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":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"recommended":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"all":[{"value":1,"date":"2013-06-27 15:01:25.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},{"value":2,"date":"2013-06-27 14:50:44.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":1,"date":"2013-06-27 15:12:38.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":6473,"name":"Sunil Thaha","email":"sthaha@redhat.com","username":"sthaha"},{"value":2,"date":"2013-06-27 20:34:18.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"}],"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":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"all":[{"value":0,"date":"2013-06-27 12:59:50.000000000","_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},{"value":0,"date":"2013-06-27 12:58:31.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"date":"2013-06-27 12:58:44.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":0,"date":"2013-06-27 12:58:53.000000000","_account_id":6473,"name":"Sunil Thaha","email":"sthaha@redhat.com","username":"sthaha"},{"value":1,"date":"2013-06-27 20:34:18.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":0,"_account_id":6473,"name":"Sunil Thaha","email":"sthaha@redhat.com","username":"sthaha"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"}],"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":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"_account_id":6473,"name":"Sunil Thaha","email":"sthaha@redhat.com","username":"sthaha"},{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-06-27 12:58:31.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 12:58:44.000000000","updated_by":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"reviewer":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"state":"REVIEWER"},{"updated":"2013-06-27 12:58:53.000000000","updated_by":{"_account_id":6473,"name":"Sunil Thaha","email":"sthaha@redhat.com","username":"sthaha"},"reviewer":{"_account_id":6473,"name":"Sunil Thaha","email":"sthaha@redhat.com","username":"sthaha"},"state":"REVIEWER"},{"updated":"2013-06-27 12:59:50.000000000","updated_by":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"reviewer":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"state":"REVIEWER"},{"updated":"2013-06-27 20:34:18.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 21:28:38.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"6b76e432db69fafda625187b0a503049d88be890","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-06-27 00:02:04.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/34640/1/check/gate-tempest-devstack-vm-full/26609 : SUCCESS in 46m 15s\n- http://logs.openstack.org/34640/1/check/gate-tempest-devstack-vm-postgres-full/26315 : SUCCESS in 47m 07s\n- http://logs.openstack.org/34640/1/check/gate-tempest-devstack-vm-quantum/34273 : SUCCESS in 21m 02s\n- http://logs.openstack.org/34640/1/check/gate-grenade-devstack-vm/13154 : FAILURE in 33m 17s (non-voting)\n","accounts_in_message":[],"_revision_number":1},{"id":"07d7752cfbc11c686f8ebab3db281a6b5ed1dfd3","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-06-27 14:50:44.000000000","message":"Patch Set 1: Looks good to me (core reviewer)\n\nseems fine. I\u0027m going to assume once RHEL7 gets out we can gut all these RHEL6 work arounds? I\u0027m concerned how much of devstack is needed to change for those RHEL6.\n\nI\u0027m +2 based on the assumption that after RHEL7 launch, we purge the RHEL6 bits.","accounts_in_message":[],"_revision_number":1},{"id":"cb8d370bcfecba61167a9c5306111fdaf6f82296","author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"date":"2013-06-27 15:01:25.000000000","message":"Patch Set 1: Looks good to me, but someone else must approve\n\nSean, this is actually a workaround for a bug in Cinder: https://bugs.launchpad.net/cinder/+bug/1191812\n\nIn RHEL the autoextend feature of LVM snaps is disabled but it could well be disabled also on other systems too\n\nBtw, devstack is _ready_ for use on RHEL/CentOS ... was one of the tasks I worked on, have a pull request open to update the website: https://github.com/cloudbuilders/devstack/pull/252","accounts_in_message":[],"_revision_number":1},{"id":"97418d37d5c51645031e9f8209f7eeb49fa0b331","author":{"_account_id":6473,"name":"Sunil Thaha","email":"sthaha@redhat.com","username":"sthaha"},"date":"2013-06-27 15:12:38.000000000","message":"Patch Set 1: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":1},{"id":"8440bffff159dd96ab786aee91142124bfb60cc7","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-06-27 20:34:18.000000000","message":"Patch Set 1: Looks good to me (core reviewer); Approved\n\nWe\u0027ll put this in but I think it needs a bit more investigation:\n* my raring laptop has the exact same config as RHEL6 in lvm.com re snapshot_autoextend_*, precise does not have them at all.\n* as long as we keep testing for specific releases like this things will likely break in the future.  For platforms that aren\u0027t ubuntu or fedora the folks adding these checks should plan on doing the verify/update if necessary for new releases","accounts_in_message":[],"_revision_number":1},{"id":"3487b3a7c716fff985c7ca8384783bf46b0b08d9","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-06-27 20:34:19.000000000","message":"Patch Set 1: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":1},{"id":"39a827df9b9ddf6d620f1fad3028541e1b4894a0","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-06-27 21:28:38.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":1},{"id":"2f7743e173ff76c4a31e8f8512888ce76c98b5ef","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-06-27 21:28:38.000000000","message":"Patch Set 1: Verified\n\nBuild succeeded.\n\n- http://logs.openstack.org/34640/1/gate/gate-tempest-devstack-vm-full/26907 : SUCCESS in 54m 17s\n- http://logs.openstack.org/34640/1/gate/gate-tempest-devstack-vm-postgres-full/26613 : SUCCESS in 44m 14s\n- http://logs.openstack.org/34640/1/gate/gate-tempest-devstack-vm-quantum/34609 : SUCCESS in 22m 10s\n- http://logs.openstack.org/34640/1/gate/gate-grenade-devstack-vm/13413 : FAILURE in 18m 03s (non-voting)\n","accounts_in_message":[],"_revision_number":1},{"id":"f9e5924fb932431ba7761501af189099fe3b18f8","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2013-06-28 00:56:19.000000000","message":"Patch Set 1:\n\n\u003e my raring laptop has the exact same config as RHEL6 in lvm.com re snapshot_autoextend_*, precise does not have them at all.\n\nDo you run tempest on these?  Specifically tempest.api.volume.test_volumes_get:VolumesGetTestXML.test_volume_create_get_delete_as_clone?","accounts_in_message":[],"_revision_number":1},{"id":"0ef51d5b2d72bde9ec24f13dbd6ddb83c1d00367","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-06-28 13:21:07.000000000","message":"Patch Set 1:\n\nIan, I was just wondering if the same behaviour has been seen on raring or if there was some other difference that mitigated the issue there.","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"0db171378dc97fea47f7d536ab92ab363e544127","revisions":{"0db171378dc97fea47f7d536ab92ab363e544127":{"kind":"REWORK","_number":1,"created":"2013-06-26 23:14:09.000000000","uploader":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"ref":"refs/changes/40/34640/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/40/34640/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/40/34640/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/40/34640/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/40/34640/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/40/34640/1"}}},"commit":{"parents":[{"commit":"b3e8ea847feaf63fe4a46c2a9a417f8a3f760938","subject":"Merge \"xenapi: separate disk for cinder volumes\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b3e8ea847feaf63fe4a46c2a9a417f8a3f760938"}]}],"author":{"name":"Ian Wienand","email":"iwienand@redhat.com","date":"2013-06-26 12:31:48.000000000","tz":600},"committer":{"name":"Ian Wienand","email":"iwienand@redhat.com","date":"2013-06-26 23:13:48.000000000","tz":600},"subject":"Update LVM autoextend config for RHEL6","message":"Update LVM autoextend config for RHEL6\n\nCinder clones are slightly larger due to some extra metadata.  RHEL6\nwill not allow auto-extending of LV\u0027s without this, leading to clones\ngiving hard-to-track disk I/O errors.\n\nSee https://bugzilla.redhat.com/show_bug.cgi?id\u003d975052\n\nChange-Id: I09a5e061a9665c5310383f9f9eb281bfdc8e416d\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/0db171378dc97fea47f7d536ab92ab363e544127"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/0db171378dc97fea47f7d536ab92ab363e544127"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"b3e8ea847feaf63fe4a46c2a9a417f8a3f760938","is_merged_in_target_branch":true}],"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}