)]}'
{"id":"openstack%2Fcinder~56091","triplet_id":"openstack%2Fcinder~master~I8a2b8ce972cb0a585555dd92c78dd8bbade1a9b2","project":"openstack/cinder","branch":"master","topic":"bugfix/lvm-thin-volume-create","hashtags":[],"change_id":"I8a2b8ce972cb0a585555dd92c78dd8bbade1a9b2","subject":"LVM: report thin pool capacity in a reasonable way","status":"ABANDONED","created":"2013-11-12 20:49:12.000000000","updated":"2013-12-17 06:03:04.000000000","total_comment_count":4,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"c82a68a47451bb4fb19e5bc761b506ad9f106c1b","_number":56091,"virtual_id_number":56091,"owner":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},{"_account_id":8574,"name":"Qin Zhao","username":"zhaoqin","inactive":true},{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":1,"date":"2013-12-05 20:58:37.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":1,"default_value":0,"optional":true},"Code-Review":{"rejected":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"all":[{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":8574,"name":"Qin Zhao","username":"zhaoqin","inactive":true},{"value":-2,"date":"2013-12-09 20:47:01.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"}],"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":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},{"_account_id":8574,"name":"Qin Zhao","username":"zhaoqin","inactive":true},{"value":0,"date":"2013-12-09 20:47:01.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":2},"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},{"_account_id":8574,"name":"Qin Zhao","username":"zhaoqin","inactive":true},{"value":0,"permitted_voting_range":{"min":-1,"max":2},"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"value":0,"permitted_voting_range":{"min":-1,"max":2},"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"}],"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":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},{"_account_id":8574,"name":"Qin Zhao","username":"zhaoqin","inactive":true},{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-11-13 15:40:06.000000000","updated_by":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"reviewer":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2013-11-13 15:45:54.000000000","updated_by":{"_account_id":8574,"name":"Qin Zhao","username":"zhaoqin","inactive":true},"reviewer":{"_account_id":8574,"name":"Qin Zhao","username":"zhaoqin","inactive":true},"state":"REVIEWER"},{"updated":"2013-11-18 14:51:55.000000000","updated_by":{"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},"reviewer":{"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},"state":"REVIEWER"},{"updated":"2013-12-03 17:43:15.000000000","updated_by":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"reviewer":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"state":"REVIEWER"},{"updated":"2013-12-03 21:18:35.000000000","updated_by":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"reviewer":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"state":"REVIEWER"},{"updated":"2013-12-05 20:58:37.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2013-12-09 20:47:01.000000000","updated_by":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"reviewer":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"state":"REVIEWER"}],"messages":[{"id":"31d406940e4a4c5fa5bc1ab61609bffd6099501c","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2013-11-12 21:12:49.000000000","message":"Patch Set 1: Works for me\n\nSmokeStack Results (patch set 1):\n- Libvirt (Fedora 19) https://smokestack.openstack.org/?go\u003d/jobs/156469 : SUCCESS \n- Libvirt (Centos 6.4) https://smokestack.openstack.org/?go\u003d/jobs/156470 : SUCCESS \n","accounts_in_message":[],"_revision_number":1},{"id":"ce29e2887259e3a905e2e1b379ef347a0ba0927c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-11-12 21:27:27.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- gate-cinder-pep8 http://logs.openstack.org/91/56091/1/check/gate-cinder-pep8/086b921 : SUCCESS in 3m 39s\n- gate-cinder-docs http://docs-draft.openstack.org/91/56091/1/check/gate-cinder-docs/6533cc2/doc/build/html/ : SUCCESS in 5m 59s\n- gate-cinder-python26 http://logs.openstack.org/91/56091/1/check/gate-cinder-python26/7bb12fa : SUCCESS in 5m 41s\n- gate-cinder-python27 http://logs.openstack.org/91/56091/1/check/gate-cinder-python27/364c27b : SUCCESS in 4m 30s\n- gate-cinder-pylint http://logs.openstack.org/91/56091/1/check/gate-cinder-pylint/a15b6f1 : SUCCESS in 5m 42s (non-voting)\n- check-tempest-devstack-vm-full http://logs.openstack.org/91/56091/1/check/check-tempest-devstack-vm-full/3db138a : SUCCESS in 31m 53s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/91/56091/1/check/check-tempest-devstack-vm-postgres-full/0bd889f : SUCCESS in 31m 34s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/91/56091/1/check/check-tempest-devstack-vm-neutron/173ae82 : SUCCESS in 30m 03s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/91/56091/1/check/gate-tempest-devstack-vm-large-ops/3815643 : SUCCESS in 14m 06s\n- gate-tempest-devstack-vm-neutron-large-ops http://logs.openstack.org/91/56091/1/check/gate-tempest-devstack-vm-neutron-large-ops/96ace0c : FAILURE in 20m 12s (non-voting)\n- check-grenade-devstack-vm http://logs.openstack.org/91/56091/1/check/check-grenade-devstack-vm/66a0c83 : SUCCESS in 38m 08s\n","accounts_in_message":[],"_revision_number":1},{"id":"8c0ec7e36f46def7941258dc4ae526e689f6c478","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2013-11-12 23:24:48.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\n(1 inline comment)\n\nOther than a typo in the commit message I think this looks good.","accounts_in_message":[],"_revision_number":1},{"id":"ed763c7cee5fce41f1be778b9382cd5a9bf01d54","author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"date":"2013-11-13 14:02:32.000000000","message":"Patch Set 1: (1 inline comment)\n\nSince Jay was already picking on it, I\u0027ll note that the commit msg text should also wrap at 72 chars.  (Hard to read in \"git log\" like this.)","accounts_in_message":[],"_revision_number":1},{"id":"93b4972b32c9c87439bddf8de169ebaf5b48454c","author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"date":"2013-11-13 15:21:40.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"2f5b6db8151d081c9134062bc4365cb394f941a7","author":{"_account_id":5494,"name":"Trivial Rebase","email":"trivial-rebase@review.openstack.org","username":"trivial-rebase"},"date":"2013-11-13 15:21:49.000000000","message":"Patch Set 2:\n\nNew patchset patch-id matches previous patchset, but commit message has changed.","accounts_in_message":[],"_revision_number":2},{"id":"84876c7ee81abd89d827119ce394418b0e80fbc0","author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"date":"2013-11-13 15:23:03.000000000","message":"Patch Set 2:\n\nAhh, thanks for the feedback.  Vim configuration updated, this will not happen again.","accounts_in_message":[],"_revision_number":2},{"id":"214591238239e8092e440a4aa84148618185493b","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2013-11-13 15:40:06.000000000","message":"Patch Set 2: Works for me\n\nSmokeStack Results (patch set 2):\n- Libvirt (Fedora 19) https://smokestack.openstack.org/?go\u003d/jobs/156837 : SUCCESS \n- Libvirt (Centos 6.4) https://smokestack.openstack.org/?go\u003d/jobs/156838 : SUCCESS \n","accounts_in_message":[],"_revision_number":2},{"id":"9b8290ec32cd35edd65b22fafc7770538643aaa6","author":{"_account_id":8574,"name":"Qin Zhao","username":"zhaoqin","inactive":true},"date":"2013-11-13 15:45:54.000000000","message":"Patch Set 2: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":2},{"id":"98eaaaee80a4e72b40479c33295f09c1e97faa46","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-11-13 16:22:04.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/GerritJenkinsGit#Test_Failures\n\n- gate-cinder-pep8 http://logs.openstack.org/91/56091/2/check/gate-cinder-pep8/d0eb3c2 : SUCCESS in 2m 23s\n- gate-cinder-docs http://docs-draft.openstack.org/91/56091/2/check/gate-cinder-docs/1cacc55/doc/build/html/ : SUCCESS in 5m 12s\n- gate-cinder-python26 http://logs.openstack.org/91/56091/2/check/gate-cinder-python26/7f179a9 : SUCCESS in 6m 27s\n- gate-cinder-python27 http://logs.openstack.org/91/56091/2/check/gate-cinder-python27/b81742c : SUCCESS in 4m 57s\n- gate-cinder-pylint http://logs.openstack.org/91/56091/2/check/gate-cinder-pylint/214efbc : SUCCESS in 4m 02s (non-voting)\n- check-tempest-devstack-vm-full http://logs.openstack.org/91/56091/2/check/check-tempest-devstack-vm-full/c357cca : SUCCESS in 29m 48s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/91/56091/2/check/check-tempest-devstack-vm-postgres-full/60298ea : SUCCESS in 34m 31s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/91/56091/2/check/check-tempest-devstack-vm-neutron/bdadaf7 : SUCCESS in 34m 50s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/91/56091/2/check/gate-tempest-devstack-vm-large-ops/9a52572 : SUCCESS in 14m 05s\n- gate-tempest-devstack-vm-neutron-large-ops http://logs.openstack.org/91/56091/2/check/gate-tempest-devstack-vm-neutron-large-ops/c413436 : FAILURE in 20m 52s (non-voting)\n- check-grenade-devstack-vm http://logs.openstack.org/91/56091/2/check/check-grenade-devstack-vm/5c88025 : FAILURE in 1h 00m 15s\n","accounts_in_message":[],"_revision_number":2},{"id":"f2f3fdfcc723ad554a5267359c78460d69aefe21","author":{"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},"date":"2013-11-14 16:49:12.000000000","message":"Patch Set 2: I would prefer that you didn\u0027t merge this\n\nPlease add a unit test, and add \"LVM\" to the commit message title.  Thanks!","accounts_in_message":[],"_revision_number":2},{"id":"d6fa3b525c335422cccfcafe768f72c38be817f9","author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"date":"2013-11-15 22:18:01.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"25909c20ab52567f984023a7c7d3beab76fb204f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-11-15 23:16:43.000000000","message":"Patch Set 3: Doesn\u0027t seem to work\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n- gate-cinder-pep8 http://logs.openstack.org/91/56091/3/check/gate-cinder-pep8/ed1d281 : SUCCESS in 2m 26s\n- gate-cinder-docs http://docs-draft.openstack.org/91/56091/3/check/gate-cinder-docs/9b864a9/doc/build/html/ : SUCCESS in 5m 37s\n- gate-cinder-python26 http://logs.openstack.org/91/56091/3/check/gate-cinder-python26/d047b2d : FAILURE in 3m 52s\n- gate-cinder-python27 http://logs.openstack.org/91/56091/3/check/gate-cinder-python27/cd54592 : FAILURE in 2m 17s\n- gate-cinder-pylint http://logs.openstack.org/91/56091/3/check/gate-cinder-pylint/ff51a2a : SUCCESS in 4m 45s (non-voting)\n- check-tempest-devstack-vm-full http://logs.openstack.org/91/56091/3/check/check-tempest-devstack-vm-full/37c478c : SUCCESS in 44m 14s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/91/56091/3/check/check-tempest-devstack-vm-postgres-full/5c9e197 : SUCCESS in 58m 34s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/91/56091/3/check/check-tempest-devstack-vm-neutron/16e9808 : SUCCESS in 34m 07s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/91/56091/3/check/gate-tempest-devstack-vm-large-ops/73903df : SUCCESS in 14m 27s\n- gate-tempest-devstack-vm-neutron-large-ops http://logs.openstack.org/91/56091/3/check/gate-tempest-devstack-vm-neutron-large-ops/4fb63bc : FAILURE in 20m 01s (non-voting)\n- check-grenade-devstack-vm http://logs.openstack.org/91/56091/3/check/check-grenade-devstack-vm/fb651ad : SUCCESS in 46m 55s\n","accounts_in_message":[],"_revision_number":3},{"id":"cd009c112e0ee7babaf16d1ef9c53fafe346a08d","author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"date":"2013-11-16 17:08:21.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"792fe3a95d0cfdda08bf7aacd4f79a358c891fd2","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-11-16 17:41:06.000000000","message":"Patch Set 4: Works for me\n\nBuild succeeded.\n\n- gate-cinder-pep8 http://logs.openstack.org/91/56091/4/check/gate-cinder-pep8/a776204 : SUCCESS in 2m 52s\n- gate-cinder-docs http://docs-draft.openstack.org/91/56091/4/check/gate-cinder-docs/4393bbb/doc/build/html/ : SUCCESS in 5m 05s\n- gate-cinder-python26 http://logs.openstack.org/91/56091/4/check/gate-cinder-python26/564b402 : SUCCESS in 6m 39s\n- gate-cinder-python27 http://logs.openstack.org/91/56091/4/check/gate-cinder-python27/ae3fcc1 : SUCCESS in 4m 41s\n- gate-cinder-pylint http://logs.openstack.org/91/56091/4/check/gate-cinder-pylint/565c4da : SUCCESS in 4m 02s (non-voting)\n- check-tempest-devstack-vm-full http://logs.openstack.org/91/56091/4/check/check-tempest-devstack-vm-full/1d2f90f : SUCCESS in 32m 14s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/91/56091/4/check/check-tempest-devstack-vm-postgres-full/084d3d9 : SUCCESS in 32m 38s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/91/56091/4/check/check-tempest-devstack-vm-neutron/ce71b7c : SUCCESS in 27m 46s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/91/56091/4/check/gate-tempest-devstack-vm-large-ops/20ddce2 : SUCCESS in 14m 47s\n- gate-tempest-devstack-vm-neutron-large-ops http://logs.openstack.org/91/56091/4/check/gate-tempest-devstack-vm-neutron-large-ops/1e600f9 : SUCCESS in 15m 51s (non-voting)\n- check-grenade-devstack-vm http://logs.openstack.org/91/56091/4/check/check-grenade-devstack-vm/0e9a0c8 : SUCCESS in 32m 37s\n","accounts_in_message":[],"_revision_number":4},{"id":"d279da4b63112df96c90e53b9e0212cd76569355","author":{"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},"date":"2013-11-18 14:51:55.000000000","message":"Patch Set 4: I would prefer that you didn\u0027t merge this\n\nPlease add \"LVM\" to the commit message title","accounts_in_message":[],"_revision_number":4},{"id":"231bddb988e55c063ba73ad100ec7314d5bce34f","author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"date":"2013-11-18 16:57:21.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"f9c277a6576e353bb5df7b2c3ee18995b557ff53","author":{"_account_id":5494,"name":"Trivial Rebase","email":"trivial-rebase@review.openstack.org","username":"trivial-rebase"},"date":"2013-11-18 16:57:29.000000000","message":"Patch Set 5:\n\nNew patchset patch-id matches previous patchset, but commit message has changed.","accounts_in_message":[],"_revision_number":5},{"id":"9415ba305447c349c4a2037a001d1955b4e73bc3","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2013-11-18 17:01:57.000000000","message":"Patch Set 5: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":5},{"id":"56de84d8004f14efa8d65b9e2818d72df34656e5","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-11-18 18:26:50.000000000","message":"Patch Set 5: Doesn\u0027t seem to work\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n- gate-cinder-pep8 http://logs.openstack.org/91/56091/5/check/gate-cinder-pep8/d05fece : SUCCESS in 1m 50s\n- gate-cinder-docs http://docs-draft.openstack.org/91/56091/5/check/gate-cinder-docs/6f5e4bb/doc/build/html/ : SUCCESS in 6m 28s\n- gate-cinder-python26 http://logs.openstack.org/91/56091/5/check/gate-cinder-python26/a741b46 : SUCCESS in 6m 03s\n- gate-cinder-python27 http://logs.openstack.org/91/56091/5/check/gate-cinder-python27/1b1f9ec : SUCCESS in 4m 41s\n- gate-cinder-pylint http://logs.openstack.org/91/56091/5/check/gate-cinder-pylint/ee06f4b : SUCCESS in 4m 58s (non-voting)\n- check-tempest-devstack-vm-full http://logs.openstack.org/91/56091/5/check/check-tempest-devstack-vm-full/7e94cd1 : SUCCESS in 34m 36s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/91/56091/5/check/check-tempest-devstack-vm-postgres-full/8c98568 : SUCCESS in 39m 38s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/91/56091/5/check/check-tempest-devstack-vm-neutron/d8cf7ba : SUCCESS in 26m 21s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/91/56091/5/check/gate-tempest-devstack-vm-large-ops/4956408 : SUCCESS in 14m 05s\n- gate-tempest-devstack-vm-neutron-large-ops http://logs.openstack.org/91/56091/5/check/gate-tempest-devstack-vm-neutron-large-ops/b191c17 : SUCCESS in 16m 40s (non-voting)\n- check-grenade-devstack-vm http://logs.openstack.org/91/56091/5/check/check-grenade-devstack-vm/20d4d22 : FAILURE in 28m 17s\n","accounts_in_message":[],"_revision_number":5},{"id":"10a52b89a084510a9f8460db1325baf3bad08d19","author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"date":"2013-11-19 16:54:29.000000000","message":"Patch Set 5: I would prefer that you didn\u0027t merge this\n\n(1 inline comment)\n\nThis patch essentially enables infinite over-commit for ThinLVM.  This is better than what we have today (and I think makes sense to go in since it actually works).\n\nBut, we should probably a) note this in the commit message and b) make a small Icehouse blueprint about adding an overcommit_ratio option to ThinLVM that defaults to 1, with related code to report free space accordingly, in a follow-up patch.","accounts_in_message":[],"_revision_number":5},{"id":"40241d7fd930e72092a2e58e587583c9a3aeae1b","author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"date":"2013-11-19 17:25:48.000000000","message":"Patch Set 5:\n\nRe a, that was my intent with the second paragraph of the commit message.  What additional information do you think would be useful?","accounts_in_message":[],"_revision_number":5},{"id":"33e90ca60d5ce1ffb36920aecd0cc124599b8900","author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"date":"2013-11-19 17:30:05.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"78d8213505efe0e1ec34192cebdec969b2f651d3","author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"date":"2013-11-19 17:30:14.000000000","message":"Patch Set 6: I would prefer that you didn\u0027t merge this\n\nAutomatically re-added by Gerrit trivial rebase detection script.","accounts_in_message":[],"_revision_number":6},{"id":"557c8e57bc5c928ce8d87f0fdcc0831020fb0022","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2013-11-19 17:30:14.000000000","message":"Patch Set 6: Looks good to me, but someone else must approve\n\nAutomatically re-added by Gerrit trivial rebase detection script.","accounts_in_message":[],"_revision_number":6},{"id":"9cb1658ca1c9ced4c7324ab91ba482f7ac79480a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-11-19 18:50:41.000000000","message":"Patch Set 6: Doesn\u0027t seem to work\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n- gate-cinder-pep8 http://logs.openstack.org/91/56091/6/check/gate-cinder-pep8/05efc33 : FAILURE in 2m 35s\n- gate-cinder-docs http://docs-draft.openstack.org/91/56091/6/check/gate-cinder-docs/d5f7a71/doc/build/html/ : SUCCESS in 5m 11s\n- gate-cinder-python26 http://logs.openstack.org/91/56091/6/check/gate-cinder-python26/9a7678b : SUCCESS in 6m 43s\n- gate-cinder-python27 http://logs.openstack.org/91/56091/6/check/gate-cinder-python27/cb88674 : SUCCESS in 5m 12s\n- gate-cinder-pylint http://logs.openstack.org/91/56091/6/check/gate-cinder-pylint/ff1bfb2 : SUCCESS in 4m 51s (non-voting)\n- check-tempest-devstack-vm-full http://logs.openstack.org/91/56091/6/check/check-tempest-devstack-vm-full/2ae6224 : FAILURE in 52m 59s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/91/56091/6/check/check-tempest-devstack-vm-postgres-full/1b9f7f6 : SUCCESS in 51m 15s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/91/56091/6/check/check-tempest-devstack-vm-neutron/93f92e3 : FAILURE in 37m 45s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/91/56091/6/check/gate-tempest-devstack-vm-large-ops/04ae593 : SUCCESS in 18m 25s\n- gate-tempest-devstack-vm-neutron-large-ops http://logs.openstack.org/91/56091/6/check/gate-tempest-devstack-vm-neutron-large-ops/d7e357c : SUCCESS in 16m 46s (non-voting)\n- check-grenade-devstack-vm http://logs.openstack.org/91/56091/6/check/check-grenade-devstack-vm/de91fec : SUCCESS in 42m 35s\n","accounts_in_message":[],"_revision_number":6},{"id":"2e4bee7a2875754dc603c16c2a7c1ef6b59f1cc6","author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"date":"2013-11-19 20:38:40.000000000","message":"Uploaded patch set 7.","accounts_in_message":[],"_revision_number":7},{"id":"818f237ff19004b4b48c05c180e5c1d3ebe0a151","author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"date":"2013-11-19 20:38:48.000000000","message":"Patch Set 7: I would prefer that you didn\u0027t merge this\n\nAutomatically re-added by Gerrit trivial rebase detection script.","accounts_in_message":[],"_revision_number":7},{"id":"f1d2ddd05cc83ad9cacbc76077581c6a656be4ae","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2013-11-19 20:38:48.000000000","message":"Patch Set 7: Looks good to me, but someone else must approve\n\nAutomatically re-added by Gerrit trivial rebase detection script.","accounts_in_message":[],"_revision_number":7},{"id":"9b3724bd00c66b8c9fc7f084c9de31f5bc3d9738","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-11-19 22:25:42.000000000","message":"Patch Set 7: Works for me\n\nBuild succeeded.\n\n- gate-cinder-pep8 http://logs.openstack.org/91/56091/7/check/gate-cinder-pep8/e8aa9cf : SUCCESS in 2m 05s\n- gate-cinder-docs http://docs-draft.openstack.org/91/56091/7/check/gate-cinder-docs/860acba/doc/build/html/ : SUCCESS in 5m 05s\n- gate-cinder-python26 http://logs.openstack.org/91/56091/7/check/gate-cinder-python26/04bbbad : SUCCESS in 6m 42s\n- gate-cinder-python27 http://logs.openstack.org/91/56091/7/check/gate-cinder-python27/d4e9941 : SUCCESS in 5m 20s\n- gate-cinder-pylint http://logs.openstack.org/91/56091/7/check/gate-cinder-pylint/ba50f1d : SUCCESS in 4m 19s (non-voting)\n- check-tempest-devstack-vm-full http://logs.openstack.org/91/56091/7/check/check-tempest-devstack-vm-full/297b303 : SUCCESS in 54m 56s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/91/56091/7/check/check-tempest-devstack-vm-postgres-full/3b3e270 : SUCCESS in 47m 49s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/91/56091/7/check/check-tempest-devstack-vm-neutron/a45ea5a : SUCCESS in 36m 39s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/91/56091/7/check/gate-tempest-devstack-vm-large-ops/56f35b3 : SUCCESS in 19m 19s\n- gate-tempest-devstack-vm-neutron-large-ops http://logs.openstack.org/91/56091/7/check/gate-tempest-devstack-vm-neutron-large-ops/6bb1e61 : SUCCESS in 19m 45s (non-voting)\n- check-grenade-devstack-vm http://logs.openstack.org/91/56091/7/check/check-grenade-devstack-vm/d6be4bf : SUCCESS in 37m 52s\n","accounts_in_message":[],"_revision_number":7},{"id":"b1b266586ff61143c99d5cde04c2c320052c1bae","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2013-11-20 03:06:35.000000000","message":"Patch Set 7: I would prefer that you didn\u0027t merge this\n\nI like Eric\u0027s comment about updating the commit message to explain what this enables and the plan going forward to improve.","accounts_in_message":[],"_revision_number":7},{"id":"0cfb5018dbe20fb6d6bed82fe2523a2e5cf0322b","author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"date":"2013-11-21 23:12:04.000000000","message":"Uploaded patch set 8.","accounts_in_message":[],"_revision_number":8},{"id":"836003eb4ce8af867209e9dc66980c7bfcf2e715","author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"date":"2013-11-21 23:12:12.000000000","message":"Patch Set 8: I would prefer that you didn\u0027t merge this\n\nAutomatically re-added by Gerrit trivial rebase detection script.","accounts_in_message":[],"_revision_number":8},{"id":"444bbcff5a1249a1ee24de2687cd39565a36dc3c","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2013-11-21 23:12:12.000000000","message":"Patch Set 8: I would prefer that you didn\u0027t merge this\n\nAutomatically re-added by Gerrit trivial rebase detection script.","accounts_in_message":[],"_revision_number":8},{"id":"add0c9f3b9aecdf0200ce72d1a9244bdccd228a1","author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"date":"2013-11-21 23:27:50.000000000","message":"Patch Set 8: Do not merge\n\n(1 inline comment)\n\n","accounts_in_message":[],"_revision_number":8},{"id":"946d2c71d3a4e4661a42221da9cb50c27774e7cf","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-11-22 00:14:04.000000000","message":"Patch Set 8: Works for me\n\nBuild succeeded.\n\n- gate-cinder-pep8 http://logs.openstack.org/91/56091/8/check/gate-cinder-pep8/043b92c : SUCCESS in 2m 03s\n- gate-cinder-docs http://docs-draft.openstack.org/91/56091/8/check/gate-cinder-docs/dfa675b/doc/build/html/ : SUCCESS in 4m 42s\n- gate-cinder-python26 http://logs.openstack.org/91/56091/8/check/gate-cinder-python26/908beb2 : SUCCESS in 5m 34s\n- gate-cinder-python27 http://logs.openstack.org/91/56091/8/check/gate-cinder-python27/c68ca51 : SUCCESS in 3m 40s\n- gate-cinder-pylint http://logs.openstack.org/91/56091/8/check/gate-cinder-pylint/62ce419 : SUCCESS in 5m 07s (non-voting)\n- check-tempest-devstack-vm-full http://logs.openstack.org/91/56091/8/check/check-tempest-devstack-vm-full/85bdc57 : SUCCESS in 32m 28s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/91/56091/8/check/check-tempest-devstack-vm-postgres-full/627be08 : SUCCESS in 31m 48s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/91/56091/8/check/check-tempest-devstack-vm-neutron/e5a1070 : SUCCESS in 31m 38s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/91/56091/8/check/gate-tempest-devstack-vm-large-ops/004b5f8 : SUCCESS in 14m 19s\n- gate-tempest-devstack-vm-neutron-large-ops http://logs.openstack.org/91/56091/8/check/gate-tempest-devstack-vm-neutron-large-ops/1c30089 : SUCCESS in 17m 22s (non-voting)\n- check-grenade-devstack-vm http://logs.openstack.org/91/56091/8/check/check-grenade-devstack-vm/0bbe895 : SUCCESS in 34m 28s\n","accounts_in_message":[],"_revision_number":8},{"id":"5df33ab177b7d8d5558094bccbb8a549255e3895","author":{"_account_id":9,"name":"LaunchpadSync","username":"launchpadsync"},"date":"2013-11-29 06:03:04.000000000","message":"Patch Set 8:\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":8},{"id":"fc64a5138884c486981ca6548f4395bf12a12053","author":{"_account_id":9,"name":"LaunchpadSync","username":"launchpadsync"},"date":"2013-11-29 06:03:04.000000000","message":"Patch Set 8: 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":8},{"id":"0f4baef6286d9a5e2ecd381a3b88296029bf968c","author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"date":"2013-12-03 17:42:52.000000000","message":"Patch Set 8: Restored","accounts_in_message":[],"_revision_number":8},{"id":"0f0fe72498cb73589361ffd73cfcb3735731e211","author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"date":"2013-12-03 17:43:07.000000000","message":"Uploaded patch set 9.","accounts_in_message":[],"_revision_number":9},{"id":"799151f3707a065294f2813c0840f69870404554","author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"date":"2013-12-03 17:43:15.000000000","message":"Patch Set 9:\n\nAutomatically re-added by Gerrit trivial rebase detection script.","accounts_in_message":[],"_revision_number":9},{"id":"24ba10eca6bdc2ae735dcdfe752a460fcd208501","author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"date":"2013-12-03 17:43:15.000000000","message":"Patch Set 9: I would prefer that you didn\u0027t merge this\n\nAutomatically re-added by Gerrit trivial rebase detection script.","accounts_in_message":[],"_revision_number":9},{"id":"2f8e2f6100514fb2128790871ab0131d7a604b86","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2013-12-03 17:43:15.000000000","message":"Patch Set 9: I would prefer that you didn\u0027t merge this\n\nAutomatically re-added by Gerrit trivial rebase detection script.","accounts_in_message":[],"_revision_number":9},{"id":"1029e395a65a2ca4af404600b3421b375b361e66","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-12-03 18:42:35.000000000","message":"Patch Set 9: Works for me\n\nBuild succeeded.\n\n- gate-cinder-pep8 http://logs.openstack.org/91/56091/9/check/gate-cinder-pep8/78ed484 : SUCCESS in 2m 13s\n- gate-cinder-docs http://docs-draft.openstack.org/91/56091/9/check/gate-cinder-docs/129c764/doc/build/html/ : SUCCESS in 5m 40s\n- gate-cinder-python26 http://logs.openstack.org/91/56091/9/check/gate-cinder-python26/5f468a8 : SUCCESS in 5m 29s\n- gate-cinder-python27 http://logs.openstack.org/91/56091/9/check/gate-cinder-python27/664c6c0 : SUCCESS in 4m 33s\n- gate-cinder-pylint http://logs.openstack.org/91/56091/9/check/gate-cinder-pylint/4cb50f7 : SUCCESS in 4m 17s (non-voting)\n- check-tempest-dsvm-full http://logs.openstack.org/91/56091/9/check/check-tempest-dsvm-full/1110a8f : SUCCESS in 52m 53s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/91/56091/9/check/check-tempest-dsvm-postgres-full/05ff771 : SUCCESS in 53m 35s\n- check-tempest-dsvm-neutron http://logs.openstack.org/91/56091/9/check/check-tempest-dsvm-neutron/7bf5508 : SUCCESS in 32m 48s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/91/56091/9/check/gate-tempest-dsvm-large-ops/d4c8758 : SUCCESS in 14m 28s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/91/56091/9/check/gate-tempest-dsvm-neutron-large-ops/945d375 : SUCCESS in 18m 07s\n- check-grenade-dsvm http://logs.openstack.org/91/56091/9/check/check-grenade-dsvm/671404b : SUCCESS in 32m 34s\n","accounts_in_message":[],"_revision_number":9},{"id":"14a416091a550b2d8dea9d498bae0f3d71b5eb20","author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"date":"2013-12-03 21:18:35.000000000","message":"Patch Set 9: No score\n\n","accounts_in_message":[],"_revision_number":9},{"id":"3145588a98378263f2ae3509a0fe0061827d25b6","author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"date":"2013-12-03 21:19:43.000000000","message":"Patch Set 9:\n\nThink we should talk tomorrow about the choice between your work here and:\nhttps://review.openstack.org/#/c/59698/","accounts_in_message":[],"_revision_number":9},{"id":"121c87b6974d840ed1347499405ca8af0a870f3b","author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"date":"2013-12-05 20:20:48.000000000","message":"Uploaded patch set 10.","accounts_in_message":[],"_revision_number":10},{"id":"9d3df35a7836916196497b42dfe1a21886d9cc9a","author":{"_account_id":5494,"name":"Trivial Rebase","email":"trivial-rebase@review.openstack.org","username":"trivial-rebase"},"date":"2013-12-05 20:20:55.000000000","message":"Patch Set 10:\n\nNew patchset patch-id matches previous patchset, but commit message has changed.","accounts_in_message":[],"_revision_number":10},{"id":"f7b487edbd9d8f9e00ee4f8d3f1684b79f9787a2","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-12-05 20:58:37.000000000","message":"Patch Set 10: Works for me\n\nBuild succeeded.\n\n- gate-cinder-pep8 http://logs.openstack.org/91/56091/10/check/gate-cinder-pep8/34ddd74 : SUCCESS in 2m 15s\n- gate-cinder-docs http://docs-draft.openstack.org/91/56091/10/check/gate-cinder-docs/7a8bc0a/doc/build/html/ : SUCCESS in 5m 23s\n- gate-cinder-python26 http://logs.openstack.org/91/56091/10/check/gate-cinder-python26/028be19 : SUCCESS in 5m 29s\n- gate-cinder-python27 http://logs.openstack.org/91/56091/10/check/gate-cinder-python27/cd97e81 : SUCCESS in 5m 17s\n- gate-cinder-pylint http://logs.openstack.org/91/56091/10/check/gate-cinder-pylint/dc5c14f : SUCCESS in 5m 11s (non-voting)\n- check-tempest-dsvm-full http://logs.openstack.org/91/56091/10/check/check-tempest-dsvm-full/14371f0 : SUCCESS in 32m 49s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/91/56091/10/check/check-tempest-dsvm-postgres-full/361dc6a : SUCCESS in 37m 42s\n- check-tempest-dsvm-neutron http://logs.openstack.org/91/56091/10/check/check-tempest-dsvm-neutron/7fec725 : SUCCESS in 28m 22s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/91/56091/10/check/gate-tempest-dsvm-large-ops/c59cc96 : SUCCESS in 17m 45s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/91/56091/10/check/gate-tempest-dsvm-neutron-large-ops/4fb8cbe : SUCCESS in 13m 13s\n- check-grenade-dsvm http://logs.openstack.org/91/56091/10/check/check-grenade-dsvm/9284af9 : SUCCESS in 26m 32s\n","accounts_in_message":[],"_revision_number":10},{"id":"5c73cf2a4c895264dc2514f695ad1faf0e190162","author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"date":"2013-12-09 20:47:01.000000000","message":"Patch Set 10: Do not merge\n\nCurrently obsolete due to https://review.openstack.org/#/c/59698/ , but work can continue here on further efforts.","accounts_in_message":[],"_revision_number":10},{"id":"e898abe513a8f926fa01c6137561745704bd22d1","author":{"_account_id":9,"name":"LaunchpadSync","username":"launchpadsync"},"date":"2013-12-17 06:03:04.000000000","message":"Patch Set 10:\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":10},{"id":"2ecbe3d7c7e5f8720b207fca5905473b4dbb708b","author":{"_account_id":9,"name":"LaunchpadSync","username":"launchpadsync"},"date":"2013-12-17 06:03:04.000000000","message":"Patch Set 10: 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":10}],"current_revision_number":10,"current_revision":"15a446f1ea26669f57f01eeb65bb4010c77cd434","revisions":{"69641c305d1b7a47bac9db71d308f66db2f5e89e":{"kind":"REWORK","_number":1,"created":"2013-11-12 20:49:12.000000000","uploader":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"ref":"refs/changes/91/56091/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cinder","ref":"refs/changes/91/56091/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cinder refs/changes/91/56091/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cinder refs/changes/91/56091/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cinder refs/changes/91/56091/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cinder refs/changes/91/56091/1"}}},"commit":{"parents":[{"commit":"b1a9a7ff25f384bb6821990a1ff0e68ce23d060e","subject":"Create thin lvm pools of correct size","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/b1a9a7ff25f384bb6821990a1ff0e68ce23d060e"}]}],"author":{"name":"Jon Bernard","email":"jobernar@redhat.com","date":"2013-11-12 20:39:48.000000000","tz":-300},"committer":{"name":"Jon Bernard","email":"jobernar@redhat.com","date":"2013-11-12 20:39:48.000000000","tz":-300},"subject":"Report thin volume capacity correctly","message":"Report thin volume capacity correctly\n\nThin LVM volumes require a different capacity calculation.  New volumes are\ncreated from the thin pool (not the volume group like normal LVM volumes), so\nthe size of the thin pool much be considered instead of the containing volume\ngroup.  In addition, thin volumes can be overcommitted so capacity does not\nnecessarily decrease when a new volume is created.\n\nThis patch gets us to a working thin LVM volume implementation by always\nreporting total and free capacity as the size of the thin pool.  There are\ncertainly alternative implementations, but this serves as a starting point for\nfurther discussion.\n\nChange-Id: I8a2b8ce972cb0a585555dd92c78dd8bbade1a9b2\nCloses-Bug: #1249782\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/69641c305d1b7a47bac9db71d308f66db2f5e89e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/69641c305d1b7a47bac9db71d308f66db2f5e89e"}]},"branch":"refs/heads/master"},"bb2110f2d8e939ef32301b75b3ceb3063996d758":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2013-11-13 15:21:40.000000000","uploader":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"ref":"refs/changes/91/56091/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cinder","ref":"refs/changes/91/56091/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cinder refs/changes/91/56091/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cinder refs/changes/91/56091/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cinder refs/changes/91/56091/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cinder refs/changes/91/56091/2"}}},"commit":{"parents":[{"commit":"b1a9a7ff25f384bb6821990a1ff0e68ce23d060e","subject":"Create thin lvm pools of correct size","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/b1a9a7ff25f384bb6821990a1ff0e68ce23d060e"}]}],"author":{"name":"Jon Bernard","email":"jobernar@redhat.com","date":"2013-11-12 20:39:48.000000000","tz":-300},"committer":{"name":"Jon Bernard","email":"jobernar@redhat.com","date":"2013-11-13 15:18:57.000000000","tz":-300},"subject":"Report thin volume capacity correctly","message":"Report thin volume capacity correctly\n\nThin LVM volumes require a different capacity calculation.  New volumes\nare created from the thin pool (not the volume group like normal LVM\nvolumes), so the size of the thin pool must be considered instead of the\ncontaining volume group.  In addition, thin volumes can be overcommitted\nso capacity does not necessarily decrease when a new volume is created.\n\nThis patch gets us to a working thin LVM volume implementation by always\nreporting total and free capacity as the size of the thin pool.  There\nare certainly alternative implementations, but this serves as a starting\npoint for further discussion.\n\nChange-Id: I8a2b8ce972cb0a585555dd92c78dd8bbade1a9b2\nCloses-Bug: #1249782\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/bb2110f2d8e939ef32301b75b3ceb3063996d758"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/bb2110f2d8e939ef32301b75b3ceb3063996d758"}]},"branch":"refs/heads/master"},"eabcfcd22bd12be87e6c8bec400e18f4fda7b838":{"kind":"REWORK","_number":3,"created":"2013-11-15 22:18:01.000000000","uploader":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"ref":"refs/changes/91/56091/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cinder","ref":"refs/changes/91/56091/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cinder refs/changes/91/56091/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cinder refs/changes/91/56091/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cinder refs/changes/91/56091/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cinder refs/changes/91/56091/3"}}},"commit":{"parents":[{"commit":"b1a9a7ff25f384bb6821990a1ff0e68ce23d060e","subject":"Create thin lvm pools of correct size","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/b1a9a7ff25f384bb6821990a1ff0e68ce23d060e"}]}],"author":{"name":"Jon Bernard","email":"jobernar@redhat.com","date":"2013-11-12 20:39:48.000000000","tz":-300},"committer":{"name":"Jon Bernard","email":"jobernar@redhat.com","date":"2013-11-15 22:16:34.000000000","tz":-300},"subject":"Report thin volume capacity correctly","message":"Report thin volume capacity correctly\n\nThin LVM volumes require a different capacity calculation.  New volumes\nare created from the thin pool (not the volume group like normal LVM\nvolumes), so the size of the thin pool must be considered instead of the\ncontaining volume group.  In addition, thin volumes can be overcommitted\nso capacity does not necessarily decrease when a new volume is created.\n\nThis patch gets us to a working thin LVM volume implementation by always\nreporting total and free capacity as the size of the thin pool.  There\nare certainly alternative implementations, but this serves as a starting\npoint for further discussion.\n\nChange-Id: I8a2b8ce972cb0a585555dd92c78dd8bbade1a9b2\nCloses-Bug: #1249782\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/eabcfcd22bd12be87e6c8bec400e18f4fda7b838"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/eabcfcd22bd12be87e6c8bec400e18f4fda7b838"}]},"branch":"refs/heads/master"},"ddee53ab3d85111b9b086f335da3c9a91d1737ec":{"kind":"TRIVIAL_REBASE","_number":4,"created":"2013-11-16 17:08:21.000000000","uploader":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"ref":"refs/changes/91/56091/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cinder","ref":"refs/changes/91/56091/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cinder refs/changes/91/56091/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cinder refs/changes/91/56091/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cinder refs/changes/91/56091/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cinder refs/changes/91/56091/4"}}},"commit":{"parents":[{"commit":"e5c89ea7c5cd95b139fd19dd9de06a02c5f98cfd","subject":"Create thin lvm pools of correct size","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/e5c89ea7c5cd95b139fd19dd9de06a02c5f98cfd"}]}],"author":{"name":"Jon Bernard","email":"jobernar@redhat.com","date":"2013-11-12 20:39:48.000000000","tz":-300},"committer":{"name":"Jon Bernard","email":"jobernar@redhat.com","date":"2013-11-16 17:06:19.000000000","tz":-300},"subject":"Report thin volume capacity correctly","message":"Report thin volume capacity correctly\n\nThin LVM volumes require a different capacity calculation.  New volumes\nare created from the thin pool (not the volume group like normal LVM\nvolumes), so the size of the thin pool must be considered instead of the\ncontaining volume group.  In addition, thin volumes can be overcommitted\nso capacity does not necessarily decrease when a new volume is created.\n\nThis patch gets us to a working thin LVM volume implementation by always\nreporting total and free capacity as the size of the thin pool.  There\nare certainly alternative implementations, but this serves as a starting\npoint for further discussion.\n\nChange-Id: I8a2b8ce972cb0a585555dd92c78dd8bbade1a9b2\nCloses-Bug: #1249782\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/ddee53ab3d85111b9b086f335da3c9a91d1737ec"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/ddee53ab3d85111b9b086f335da3c9a91d1737ec"}]},"branch":"refs/heads/master"},"1204e6cb83de7f8e4aaa9eb710b2c76f26f568a5":{"kind":"NO_CODE_CHANGE","_number":5,"created":"2013-11-18 16:57:21.000000000","uploader":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"ref":"refs/changes/91/56091/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cinder","ref":"refs/changes/91/56091/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cinder refs/changes/91/56091/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cinder refs/changes/91/56091/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cinder refs/changes/91/56091/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cinder refs/changes/91/56091/5"}}},"commit":{"parents":[{"commit":"e5c89ea7c5cd95b139fd19dd9de06a02c5f98cfd","subject":"Create thin lvm pools of correct size","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/e5c89ea7c5cd95b139fd19dd9de06a02c5f98cfd"}]}],"author":{"name":"Jon Bernard","email":"jobernar@redhat.com","date":"2013-11-12 20:39:48.000000000","tz":-300},"committer":{"name":"Jon Bernard","email":"jobernar@redhat.com","date":"2013-11-18 16:56:32.000000000","tz":-300},"subject":"Report thin LVM volume capacity correctly","message":"Report thin LVM volume capacity correctly\n\nThin LVM volumes require a different capacity calculation.  New volumes\nare created from the thin pool (not the volume group like normal LVM\nvolumes), so the size of the thin pool must be considered instead of the\ncontaining volume group.  In addition, thin volumes can be overcommitted\nso capacity does not necessarily decrease when a new volume is created.\n\nThis patch gets us to a working thin LVM volume implementation by always\nreporting total and free capacity as the size of the thin pool.  There\nare certainly alternative implementations, but this serves as a starting\npoint for further discussion.\n\nChange-Id: I8a2b8ce972cb0a585555dd92c78dd8bbade1a9b2\nCloses-Bug: #1249782\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/1204e6cb83de7f8e4aaa9eb710b2c76f26f568a5"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/1204e6cb83de7f8e4aaa9eb710b2c76f26f568a5"}]},"branch":"refs/heads/master"},"cb6f71169da78b123cfbf4d7172a94bf4115a68f":{"kind":"TRIVIAL_REBASE","_number":6,"created":"2013-11-19 17:30:05.000000000","uploader":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"ref":"refs/changes/91/56091/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cinder","ref":"refs/changes/91/56091/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cinder refs/changes/91/56091/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cinder refs/changes/91/56091/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cinder refs/changes/91/56091/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cinder refs/changes/91/56091/6"}}},"commit":{"parents":[{"commit":"7b6b33f7211b67f5dc8f5079f8fd507973f23547","subject":"Create thin lvm pools of correct size","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/7b6b33f7211b67f5dc8f5079f8fd507973f23547"}]}],"author":{"name":"Jon Bernard","email":"jobernar@redhat.com","date":"2013-11-12 20:39:48.000000000","tz":-300},"committer":{"name":"Jon Bernard","email":"jobernar@redhat.com","date":"2013-11-19 17:20:23.000000000","tz":-300},"subject":"Report thin LVM volume capacity correctly","message":"Report thin LVM volume capacity correctly\n\nThin LVM volumes require a different capacity calculation.  New volumes\nare created from the thin pool (not the volume group like normal LVM\nvolumes), so the size of the thin pool must be considered instead of the\ncontaining volume group.  In addition, thin volumes can be overcommitted\nso capacity does not necessarily decrease when a new volume is created.\n\nThis patch gets us to a working thin LVM volume implementation by always\nreporting total and free capacity as the size of the thin pool.  There\nare certainly alternative implementations, but this serves as a starting\npoint for further discussion.\n\nChange-Id: I8a2b8ce972cb0a585555dd92c78dd8bbade1a9b2\nCloses-Bug: #1249782\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/cb6f71169da78b123cfbf4d7172a94bf4115a68f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/cb6f71169da78b123cfbf4d7172a94bf4115a68f"}]},"branch":"refs/heads/master"},"5828cb0bcb3bd83caff79f344febf0c63ca1f536":{"kind":"TRIVIAL_REBASE","_number":7,"created":"2013-11-19 20:38:40.000000000","uploader":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"ref":"refs/changes/91/56091/7","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cinder","ref":"refs/changes/91/56091/7","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cinder refs/changes/91/56091/7 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cinder refs/changes/91/56091/7 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cinder refs/changes/91/56091/7 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cinder refs/changes/91/56091/7"}}},"commit":{"parents":[{"commit":"5f7e5c8995f17eb62f81d44d307668e202193590","subject":"Create thin lvm pools of correct size","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/5f7e5c8995f17eb62f81d44d307668e202193590"}]}],"author":{"name":"Jon Bernard","email":"jobernar@redhat.com","date":"2013-11-12 20:39:48.000000000","tz":-300},"committer":{"name":"Jon Bernard","email":"jobernar@redhat.com","date":"2013-11-19 20:36:21.000000000","tz":-300},"subject":"Report thin LVM volume capacity correctly","message":"Report thin LVM volume capacity correctly\n\nThin LVM volumes require a different capacity calculation.  New volumes\nare created from the thin pool (not the volume group like normal LVM\nvolumes), so the size of the thin pool must be considered instead of the\ncontaining volume group.  In addition, thin volumes can be overcommitted\nso capacity does not necessarily decrease when a new volume is created.\n\nThis patch gets us to a working thin LVM volume implementation by always\nreporting total and free capacity as the size of the thin pool.  There\nare certainly alternative implementations, but this serves as a starting\npoint for further discussion.\n\nChange-Id: I8a2b8ce972cb0a585555dd92c78dd8bbade1a9b2\nCloses-Bug: #1249782\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/5828cb0bcb3bd83caff79f344febf0c63ca1f536"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/5828cb0bcb3bd83caff79f344febf0c63ca1f536"}]},"branch":"refs/heads/master"},"86fea4f3eb96f8bb00d706e762155f42774a4322":{"kind":"TRIVIAL_REBASE","_number":8,"created":"2013-11-21 23:12:04.000000000","uploader":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"ref":"refs/changes/91/56091/8","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cinder","ref":"refs/changes/91/56091/8","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cinder refs/changes/91/56091/8 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cinder refs/changes/91/56091/8 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cinder refs/changes/91/56091/8 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cinder refs/changes/91/56091/8"}}},"commit":{"parents":[{"commit":"24ec99da20f14eef1eed66bd0bed4f8ae0e6b50a","subject":"Create thin lvm pools of correct size","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/24ec99da20f14eef1eed66bd0bed4f8ae0e6b50a"}]}],"author":{"name":"Jon Bernard","email":"jobernar@redhat.com","date":"2013-11-12 20:39:48.000000000","tz":-300},"committer":{"name":"Jon Bernard","email":"jobernar@redhat.com","date":"2013-11-21 23:11:20.000000000","tz":-300},"subject":"Report thin LVM volume capacity correctly","message":"Report thin LVM volume capacity correctly\n\nThin LVM volumes require a different capacity calculation.  New volumes\nare created from the thin pool (not the volume group like normal LVM\nvolumes), so the size of the thin pool must be considered instead of the\ncontaining volume group.  In addition, thin volumes can be overcommitted\nso capacity does not necessarily decrease when a new volume is created.\n\nThis patch gets us to a working thin LVM volume implementation by always\nreporting total and free capacity as the size of the thin pool.  There\nare certainly alternative implementations, but this serves as a starting\npoint for further discussion.\n\nChange-Id: I8a2b8ce972cb0a585555dd92c78dd8bbade1a9b2\nCloses-Bug: #1249782\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/86fea4f3eb96f8bb00d706e762155f42774a4322"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/86fea4f3eb96f8bb00d706e762155f42774a4322"}]},"branch":"refs/heads/master"},"01ee4485adb52f25b4b418a2cc460013a4f3e32c":{"kind":"TRIVIAL_REBASE","_number":9,"created":"2013-12-03 17:43:07.000000000","uploader":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"ref":"refs/changes/91/56091/9","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cinder","ref":"refs/changes/91/56091/9","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cinder refs/changes/91/56091/9 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cinder refs/changes/91/56091/9 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cinder refs/changes/91/56091/9 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cinder refs/changes/91/56091/9"}}},"commit":{"parents":[{"commit":"8d051ca97601eeb0d74198bb398c0c63ae1f48c9","subject":"LVM: Create thin pools of adequate size","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/8d051ca97601eeb0d74198bb398c0c63ae1f48c9"}]}],"author":{"name":"Jon Bernard","email":"jobernar@redhat.com","date":"2013-11-12 20:39:48.000000000","tz":-300},"committer":{"name":"Jon Bernard","email":"jobernar@redhat.com","date":"2013-12-03 17:41:32.000000000","tz":-300},"subject":"Report thin LVM volume capacity correctly","message":"Report thin LVM volume capacity correctly\n\nThin LVM volumes require a different capacity calculation.  New volumes\nare created from the thin pool (not the volume group like normal LVM\nvolumes), so the size of the thin pool must be considered instead of the\ncontaining volume group.  In addition, thin volumes can be overcommitted\nso capacity does not necessarily decrease when a new volume is created.\n\nThis patch gets us to a working thin LVM volume implementation by always\nreporting total and free capacity as the size of the thin pool.  There\nare certainly alternative implementations, but this serves as a starting\npoint for further discussion.\n\nChange-Id: I8a2b8ce972cb0a585555dd92c78dd8bbade1a9b2\nCloses-Bug: #1249782\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/01ee4485adb52f25b4b418a2cc460013a4f3e32c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/01ee4485adb52f25b4b418a2cc460013a4f3e32c"}]},"branch":"refs/heads/master"},"15a446f1ea26669f57f01eeb65bb4010c77cd434":{"kind":"TRIVIAL_REBASE_WITH_MESSAGE_UPDATE","_number":10,"created":"2013-12-05 20:20:48.000000000","uploader":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"ref":"refs/changes/91/56091/10","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cinder","ref":"refs/changes/91/56091/10","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cinder refs/changes/91/56091/10 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cinder refs/changes/91/56091/10 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cinder refs/changes/91/56091/10 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cinder refs/changes/91/56091/10"}}},"commit":{"parents":[{"commit":"d72914f739b1467ad849dd47fddd321965fed928","subject":"LVM: Create thin pools of adequate size","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/d72914f739b1467ad849dd47fddd321965fed928"}]}],"author":{"name":"Jon Bernard","email":"jobernar@redhat.com","date":"2013-11-12 20:39:48.000000000","tz":-300},"committer":{"name":"Jon Bernard","email":"jobernar@redhat.com","date":"2013-12-05 19:44:39.000000000","tz":-300},"subject":"LVM: report thin pool capacity in a reasonable way","message":"LVM: report thin pool capacity in a reasonable way\n\nNow that thin pools are created of adequate size, capacity reporting\nmust be updated so that users can successfully create new volumes.  The\nexisting code reports the size and remaining capacity of the volume\ngroup.  When \u0027thin\u0027 is enabled, the thin pool takes up nearly all of the\nvolume group and available capacity appears to be 0 - preventing any new\nvolumes from being created.\n\nThis patch changes thin LVM capacity reporting to always report\navailable capacity as the total size of the thin pool.  For users, this\nmeans a couple of things:\n\n  1. No single volume can exceed the size of the thin pool.\n  2. An infinite number of volumes can be created, all backed by the\n     same thin pool.\n\nThis allows infinite over-provisioning of thin LVM.  It is left to the\nuser to monitor the actual free space remaining in the pool and make use\nof thin volumes accordingly.\n\nAdditional improvements are being considered, so expect updates to this\nlogic in the near future.\n\nChange-Id: I8a2b8ce972cb0a585555dd92c78dd8bbade1a9b2\nCloses-Bug: #1249782\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/15a446f1ea26669f57f01eeb65bb4010c77cd434"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/15a446f1ea26669f57f01eeb65bb4010c77cd434"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}