)]}'
{"id":"openstack%2Fcinderlib~694524","triplet_id":"openstack%2Fcinderlib~master~I7eade434b3d86e72678769f8ec929f66967d74e3","project":"openstack/cinderlib","branch":"master","topic":"bug/1852629","hashtags":[],"change_id":"I7eade434b3d86e72678769f8ec929f66967d74e3","subject":"Fix LVM extend volume","status":"MERGED","created":"2019-11-15 12:16:47.000000000","updated":"2019-12-03 19:57:53.000000000","submitted":"2019-12-03 19:56:05.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"694524-1575402966146-73dee3f6","meta_rev_id":"03a17c656e0103d7303079f93ca2cc29fce5aac9","_number":694524,"virtual_id_number":694524,"owner":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2019-12-03 19:56:05.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"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":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"all":[{"value":2,"date":"2019-12-03 16:00:12.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":2,"date":"2019-11-28 04:13:21.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"all":[{"value":1,"date":"2019-12-03 16:00:12.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"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":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2019-11-28 04:13:21.000000000","updated_by":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"reviewer":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"state":"REVIEWER"},{"updated":"2019-12-03 16:00:12.000000000","updated_by":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"reviewer":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"state":"REVIEWER"},{"updated":"2019-12-03 19:56:05.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"}],"messages":[{"id":"779ba43ec7217c9c2a9d5e14795644f16cf6d502","author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"date":"2019-11-15 12:16:47.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"983326ff08ccbc460fccaa016515a0889a24d575","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-11-15 12:44:57.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/ba163954daff4a3e9349560a1717be15 : SUCCESS in 8m 06s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/c321a7d20436497691b1963de5c365ac : SUCCESS in 5m 50s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/fba5083797ef4e0ea398f669a53f359f : SUCCESS in 5m 08s\n- cinderlib-tox-py27 https://zuul.opendev.org/t/openstack/build/b08b2b34a91348bb890e761543fdabb7 : SUCCESS in 6m 03s\n- cinderlib-tox-py36 https://zuul.opendev.org/t/openstack/build/28e486f090824f5b91ae77d2ad7ed5c2 : SUCCESS in 7m 24s\n- cinderlib-tox-py37 https://zuul.opendev.org/t/openstack/build/2cf675bc181b4114b63052be895cca3e : SUCCESS in 7m 30s\n- cinderlib-lvm-functional https://zuul.opendev.org/t/openstack/build/50c09b93077e44f4829f808bec98b6b3 : FAILURE in 8m 51s\n- cinderlib-ceph-functional https://zuul.opendev.org/t/openstack/build/7d91fdf81edb496c8d1d40a706c28e2a : FAILURE in 9m 40s","accounts_in_message":[],"_revision_number":1},{"id":"70360b9b97fe9aac552440f64fe03fd3be0d57d4","author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"date":"2019-11-18 18:39:52.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"3454020efa44680d41fe26494e40f7e2de5716ea","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-11-18 20:11:12.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/d23e091c52904cf6856a6e9d99050a32 : SUCCESS in 6m 21s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/3a394c4decf143efb56d03fc3fdd7e97 : SUCCESS in 5m 53s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/252fe4affd634bb79449e7da0fbf8a7a : SUCCESS in 5m 04s\n- cinderlib-tox-py27 https://zuul.opendev.org/t/openstack/build/ef500844e8374f178d2ecc6e0daf591d : SUCCESS in 5m 29s\n- cinderlib-tox-py36 https://zuul.opendev.org/t/openstack/build/e5d40a0b5e104b57a52dbe582fa8fe05 : SUCCESS in 6m 30s\n- cinderlib-tox-py37 https://zuul.opendev.org/t/openstack/build/64efef9102b94e1b8bcd56b2697f0007 : SUCCESS in 6m 38s\n- cinderlib-lvm-functional https://zuul.opendev.org/t/openstack/build/0f7b8386289247cc92c99a76b6cf959f : SUCCESS in 8m 10s\n- cinderlib-ceph-functional https://zuul.opendev.org/t/openstack/build/173481de2faf4dff9c5c10987569bd5d : SUCCESS in 10m 13s","accounts_in_message":[],"_revision_number":2},{"id":"abe49a1999bd870c70c1bbb8d3d52d8b2715a0cf","author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"date":"2019-11-28 04:13:21.000000000","message":"Patch Set 2: Code-Review+2\n\nLooks like a correct fix.","accounts_in_message":[],"_revision_number":2},{"id":"58c91a8ef7fa49db76e069ef9a59be07750ae377","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2019-12-03 16:00:12.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1\n\nLooks good to me.","accounts_in_message":[],"_revision_number":2},{"id":"a31588a844e8288399af11b2c0f9580dec091566","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-12-03 16:00:37.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":2},{"id":"3aac99a5f821cd488978eaa1149ed3c972b1dc60","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-12-03 19:56:05.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/182570e029724e3586120ef94deb17d8 : SUCCESS in 7m 23s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/fe5012fac9b84b75a9ea6a94ea199397 : SUCCESS in 5m 23s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/7094638ebc7548c791ae51eda010fd99 : SUCCESS in 5m 23s\n- cinderlib-tox-py27 https://zuul.opendev.org/t/openstack/build/f6144905eb4b47c58521c6f3427f02f3 : SUCCESS in 6m 17s\n- cinderlib-tox-py36 https://zuul.opendev.org/t/openstack/build/60c1673d17144e3e8560ebe01d771de9 : SUCCESS in 8m 04s\n- cinderlib-tox-py37 https://zuul.opendev.org/t/openstack/build/9b7100a0637b4765ae284963f9e28f9a : SUCCESS in 7m 06s\n- cinderlib-lvm-functional https://zuul.opendev.org/t/openstack/build/39c7085b0e05455d98060e2b550458fb : SUCCESS in 10m 35s\n- cinderlib-ceph-functional https://zuul.opendev.org/t/openstack/build/27994ae756da48cf9b356b7b4d0eaffc : SUCCESS in 13m 10s","accounts_in_message":[],"_revision_number":2},{"id":"abae5c5608d4f1b42bb52556fa14a2dea830832e","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-12-03 19:56:06.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":2},{"id":"089b6dd92156cfe6b9c72fd34530336a0934d9a4","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-12-03 19:57:53.000000000","message":"Patch Set 2:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/dd14414d0865422f8f9a581cd5a9b604 : SUCCESS in 1m 20s\n- promote-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/b5cc8522a8724f36a440c429817ef02f : SUCCESS in 50s","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"b7f27bb9c40e5937661872f645291ea432886851","revisions":{"2125eb9c14b011bcd9f3c3e10d30dd01a888bb3f":{"kind":"REWORK","_number":1,"created":"2019-11-15 12:16:47.000000000","uploader":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"ref":"refs/changes/24/694524/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cinderlib","ref":"refs/changes/24/694524/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cinderlib refs/changes/24/694524/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cinderlib refs/changes/24/694524/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cinderlib refs/changes/24/694524/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cinderlib refs/changes/24/694524/1"}}},"commit":{"parents":[{"commit":"7f61d743378f015cdf02507aef3c8ec73eac784e","subject":"Merge \"Fake unused packages\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinderlib/commit/7f61d743378f015cdf02507aef3c8ec73eac784e"}]}],"author":{"name":"Gorka Eguileor","email":"geguileo@redhat.com","date":"2019-11-13 10:09:41.000000000","tz":60},"committer":{"name":"Gorka Eguileor","email":"geguileo@redhat.com","date":"2019-11-15 12:15:38.000000000","tz":60},"subject":"Fix LVM extend volume","message":"Fix LVM extend volume\n\nWhen we try to extend an attached LVM volume after retrieving the volume\nmetadata from the persistent storage we get a TargetUpdateFailed\nexception cause by exception AttributeError: \u0027DB\u0027 object has no\nattribute \u0027volume_attachment_get_all_by_volume_id\u0027.\n\nThis is caused by the LVM driver checking volume_attachment which is not\npopulated, which triggers a lazy loading on the OVO which calls a method\nin the DB layer that is not available in our fake DB instance.\n\nThis patch resolves this by replacing the get_all_by_volume_id method in\nthe VolumeAttachmentList OVO.\n\nClose-Bug: #1852629\nChange-Id: I7eade434b3d86e72678769f8ec929f66967d74e3\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinderlib/commit/2125eb9c14b011bcd9f3c3e10d30dd01a888bb3f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinderlib/commit/2125eb9c14b011bcd9f3c3e10d30dd01a888bb3f"}]},"branch":"refs/heads/master"},"b7f27bb9c40e5937661872f645291ea432886851":{"kind":"REWORK","_number":2,"created":"2019-11-18 18:39:52.000000000","uploader":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"ref":"refs/changes/24/694524/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cinderlib","ref":"refs/changes/24/694524/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cinderlib refs/changes/24/694524/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cinderlib refs/changes/24/694524/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cinderlib refs/changes/24/694524/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cinderlib refs/changes/24/694524/2"}}},"commit":{"parents":[{"commit":"7f61d743378f015cdf02507aef3c8ec73eac784e","subject":"Merge \"Fake unused packages\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinderlib/commit/7f61d743378f015cdf02507aef3c8ec73eac784e"}]}],"author":{"name":"Gorka Eguileor","email":"geguileo@redhat.com","date":"2019-11-13 10:09:41.000000000","tz":60},"committer":{"name":"Gorka Eguileor","email":"geguileo@redhat.com","date":"2019-11-18 18:39:48.000000000","tz":60},"subject":"Fix LVM extend volume","message":"Fix LVM extend volume\n\nWhen we try to extend an attached LVM volume after retrieving the volume\nmetadata from the persistent storage we get a TargetUpdateFailed\nexception cause by exception AttributeError: \u0027DB\u0027 object has no\nattribute \u0027volume_attachment_get_all_by_volume_id\u0027.\n\nThis is caused by the LVM driver checking volume_attachment which is not\npopulated, which triggers a lazy loading on the OVO which calls a method\nin the DB layer that is not available in our fake DB instance.\n\nThis patch resolves this by replacing the get_all_by_volume_id method in\nthe VolumeAttachmentList OVO.\n\nClose-Bug: #1852629\nChange-Id: I7eade434b3d86e72678769f8ec929f66967d74e3\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinderlib/commit/b7f27bb9c40e5937661872f645291ea432886851"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinderlib/commit/b7f27bb9c40e5937661872f645291ea432886851"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}