)]}'
{"id":"openstack%2Fcinderlib~870092","triplet_id":"openstack%2Fcinderlib~master~I74ce945291697bc1fcc63a35d60ef71494fdaf19","project":"openstack/cinderlib","branch":"master","topic":"fix-dbms","attention_set":{},"removed_from_attention_set":{"9535":{"account":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"last_update":"2023-01-20 05:56:04.000000000","reason":"Change was submitted"}},"hashtags":[],"change_id":"I74ce945291697bc1fcc63a35d60ef71494fdaf19","subject":"Fix DBMS storage plugin","status":"MERGED","created":"2023-01-13 13:40:56.000000000","updated":"2023-01-20 05:57:10.000000000","submitted":"2023-01-20 05:56:04.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":4,"unresolved_comment_count":2,"has_review_started":true,"submission_id":"870092-fix-dbms","meta_rev_id":"8fac012aeaa37f4ed192c518558951558ec54864","_number":870092,"virtual_id_number":870092,"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":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2023-01-20 05:56:04.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"}],"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":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"all":[{"value":2,"date":"2023-01-20 04:03:17.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"]},{"value":2,"date":"2023-01-19 16:31:55.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"}],"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":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"all":[{"value":1,"date":"2023-01-20 04:03:17.000000000","permitted_voting_range":{"min":1,"max":1},"_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"]},{"value":0,"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"approved":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"all":[{"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"]},{"value":2,"date":"2023-01-19 16:31:55.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"}],"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":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2023-01-13 15:48:54.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"},{"updated":"2023-01-19 09:48:05.000000000","updated_by":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"reviewer":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"state":"REVIEWER"},{"updated":"2023-01-20 04:03:17.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"}],"messages":[{"id":"c17a8202a8c14badcfe4c0d36395c72b4bdbfc57","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"date":"2023-01-13 13:40:56.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"6a56631b46bfb3c9211f3c1bab01ee70aca4cc7e","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2023-01-13 15:48:54.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\nhttps://zuul.opendev.org/t/openstack/buildset/ea0e42f7dcf5490db5e61329b38cd2ef\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/084ae20c7b284d2584e0f2a991967c82 : FAILURE in 4m 30s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/c4882adf94774ea0a3c22be2241eab89 : SUCCESS in 3m 41s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/7d6fff19c09245909fbf97fb77fb2d15 : FAILURE in 2m 26s\n- cinderlib-tox-py38 https://zuul.opendev.org/t/openstack/build/ce7aa7d1e1fe4ee6beab0e5c7c18f20a : FAILURE in 3m 34s\n- cinderlib-tox-py310 https://zuul.opendev.org/t/openstack/build/65aea487cb3c4999a549b04cf03c6067 : FAILURE in 2m 35s\n- cinderlib-tox-py311 https://zuul.opendev.org/t/openstack/build/5cad5b6b0de6481598bb8d9b5ece46e8 : FAILURE in 3m 56s (non-voting)\n- cinderlib-lvm-functional https://zuul.opendev.org/t/openstack/build/8bb82a3cd069455488216a249a5409ad : FAILURE in 5m 11s\n- cinderlib-ceph-functional https://zuul.opendev.org/t/openstack/build/dbdb2bdfb0304a9ca03a75a807486ab7 : FAILURE in 5m 45s\n- os-brick-src-tempest-lvm-lio-barbican https://zuul.opendev.org/t/openstack/build/ad709d1aec3d4112b7b733ea8462c2e7 : SUCCESS in 1h 58m 13s","accounts_in_message":[],"_revision_number":1},{"id":"43d2e582b2415bf6868d8094c62ba70ef9218c49","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"date":"2023-01-16 18:32:10.000000000","message":"Uploaded patch set 2: Patch Set 1 was rebased.","accounts_in_message":[],"_revision_number":2},{"id":"c342852ce6f6613e298f3fc8d3d2609e79328210","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2023-01-16 20:44:30.000000000","message":"Patch Set 2: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\nhttps://zuul.opendev.org/t/openstack/buildset/e26ff5de558045d297d5774e97b5e95a\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/e772db64a53c482daee5397669b95e27 : SUCCESS in 5m 04s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/11fb802c3ec94186a2b4dd9d0818999f : SUCCESS in 3m 38s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/2b3695e44db344269845355a22f9d4e7 : SUCCESS in 2m 45s\n- cinderlib-tox-py38 https://zuul.opendev.org/t/openstack/build/e0e697445ffd4cefa6e9ff18ce5c7ab9 : FAILURE in 2m 38s\n- cinderlib-tox-py310 https://zuul.opendev.org/t/openstack/build/439500bce227497d84a918673ed23b5b : FAILURE in 2m 37s\n- cinderlib-tox-py311 https://zuul.opendev.org/t/openstack/build/34061c22520044bfb60b945b366ae713 : FAILURE in 2m 44s (non-voting)\n- cinderlib-lvm-functional https://zuul.opendev.org/t/openstack/build/5e37df7aa72e4d3b91d558bf6a2f0c81 : FAILURE in 4m 43s\n- cinderlib-ceph-functional https://zuul.opendev.org/t/openstack/build/aeb6492c80284eec848474eab367e306 : FAILURE in 5m 37s\n- os-brick-src-tempest-lvm-lio-barbican https://zuul.opendev.org/t/openstack/build/1c2f274213924610a6673f53a3d0b7d6 : SUCCESS in 2h 04m 02s","accounts_in_message":[],"_revision_number":2},{"id":"49a22681bee86e980538685b991e901c59f391c3","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"date":"2023-01-17 11:01:25.000000000","message":"Uploaded patch set 3: Patch Set 2 was rebased.","accounts_in_message":[],"_revision_number":3},{"id":"589ebcdbfb666c0a2ff61879c285a2a36d51b502","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2023-01-17 13:37:17.000000000","message":"Patch Set 3: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\nhttps://zuul.opendev.org/t/openstack/buildset/0dfddc50237b4328b41ce624a20b003e\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/63dbe6359d7347878ac54052ea1d6833 : SUCCESS in 4m 44s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/d7c63a773eb24663854656561b7a62be : SUCCESS in 3m 50s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/165b2c93fdb74a0cb17af36f92c2c22f : SUCCESS in 3m 37s\n- cinderlib-tox-py38 https://zuul.opendev.org/t/openstack/build/192b7e3b48094cf5af17a0bc4255f3ed : FAILURE in 5m 52s\n- cinderlib-tox-py310 https://zuul.opendev.org/t/openstack/build/a6ca66d77d644c019f5e2679c4019090 : FAILURE in 6m 05s\n- cinderlib-tox-py311 https://zuul.opendev.org/t/openstack/build/c322de3a195b4385a4fca53952659e7d : FAILURE in 6m 53s (non-voting)\n- cinderlib-lvm-functional https://zuul.opendev.org/t/openstack/build/6da07754104f4edf9a2d941005555d5d : FAILURE in 4m 55s\n- cinderlib-ceph-functional https://zuul.opendev.org/t/openstack/build/d5892c0060494e43ad6b6972680de682 : FAILURE in 5m 02s\n- os-brick-src-tempest-lvm-lio-barbican https://zuul.opendev.org/t/openstack/build/061f2564ada5440cbb573f4ff7631829 : SUCCESS in 1h 24m 51s","accounts_in_message":[],"_revision_number":3},{"id":"1589a7aa846ffab9e862b67d78f37b161f1f7723","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"date":"2023-01-18 10:00:01.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"dbfeee1dabafb9b4abaab16b2449ffeaf8f33381","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2023-01-18 10:25:41.000000000","message":"Patch Set 4: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\nhttps://zuul.opendev.org/t/openstack/buildset/a0bd1fb7b09f49f79d0ad4fa61413a6f\n\n- cinderlib-tox-py38 https://zuul.opendev.org/t/openstack/build/8197b88f91944b76b2f10db1bb595bc1 : FAILURE in 7m 19s\n- cinderlib-lvm-functional https://zuul.opendev.org/t/openstack/build/3336d6665f504fceac66c7d9ad5b6abf : FAILURE in 4m 54s","accounts_in_message":[],"_revision_number":4},{"id":"1fde15fa5b3f9a4299e2622e446b0986030df922","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"date":"2023-01-18 11:37:33.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"dd6fd07d9e5de26a76c452d50a0c0614d178bc0c","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2023-01-18 12:15:08.000000000","message":"Patch Set 5: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/778ce96caa8b4c0a901bd712d76346db\n\n- test-release-openstack https://zuul.opendev.org/t/openstack/build/a2b2352d35cc4d9bb2e89f6aad674b3f : SUCCESS in 2m 04s\n- cinderlib-tox-py38 https://zuul.opendev.org/t/openstack/build/725cf1d795dd4f33b015123969cc89f6 : SUCCESS in 8m 09s\n- cinderlib-lvm-functional https://zuul.opendev.org/t/openstack/build/2cb50da693cd4275a03fd128875657f4 : SUCCESS in 14m 20s","accounts_in_message":[],"_revision_number":5},{"id":"7a1ab037dd6c1085bad28026ed4733741ad93754","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"date":"2023-01-18 14:14:34.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"c752e8da3b4edbf8e84c3cfe6e82346565c54f1c","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2023-01-18 16:43:37.000000000","message":"Patch Set 6: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/e74bbfd9b50d4d4b8fa8eec31070498c\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/54d1dc993c78400f93602ce29c3e1593 : SUCCESS in 7m 27s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/d67719887a464a72b97d86902dc395b7 : SUCCESS in 3m 39s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/c8fc2922dc1e434d9526d82ea9c2571e : SUCCESS in 3m 41s\n- cinderlib-tox-py38 https://zuul.opendev.org/t/openstack/build/8ee47a963a9940dca15635066700c9bd : SUCCESS in 6m 21s\n- cinderlib-tox-py310 https://zuul.opendev.org/t/openstack/build/3cab4a62f8f94e8e9294986cb492dcdc : SUCCESS in 9m 03s\n- cinderlib-tox-py311 https://zuul.opendev.org/t/openstack/build/7add647c1be64ec98dd31f4b47864f7d : SUCCESS in 10m 29s (non-voting)\n- cinderlib-lvm-functional https://zuul.opendev.org/t/openstack/build/2814344b6e29466486097ddb2f4a3eae : SUCCESS in 14m 21s\n- cinderlib-ceph-functional https://zuul.opendev.org/t/openstack/build/249eea2670db40e59602914f7ed1efe0 : SUCCESS in 13m 42s\n- os-brick-src-tempest-lvm-lio-barbican https://zuul.opendev.org/t/openstack/build/727def3815e147388dc2d66d785023c8 : SUCCESS in 2h 15m 35s","accounts_in_message":[],"_revision_number":6},{"id":"ab55d3224052065fed941806aa69ae304369205b","author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"date":"2023-01-19 09:48:05.000000000","message":"Patch Set 6: Code-Review+2 Review-Priority+2\n\n(3 comments)","accounts_in_message":[],"_revision_number":6},{"id":"c8202316bcb0ff7cd50a3801d38679f0634b4460","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"date":"2023-01-19 16:31:55.000000000","message":"Uploaded patch set 7: Patch Set 6 was rebased.","accounts_in_message":[],"_revision_number":7},{"id":"8aec104bdb3559e396f4ff16e86c78e8b8bf7bb4","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2023-01-19 18:18:42.000000000","message":"Patch Set 7: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/c6d02238c03f4c53be3605bd22282857\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/9bfe3b7bfc4743cd8ac62da391a8bd39 : SUCCESS in 7m 28s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/9705a6c0ec654e46955304e5119fe1fb : SUCCESS in 3m 42s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/e39687a38b3c4f879628df47577d2abd : SUCCESS in 3m 30s\n- cinderlib-tox-py38 https://zuul.opendev.org/t/openstack/build/7b60e6e426e44ad985bfc945b0c62adc : SUCCESS in 5m 24s\n- cinderlib-tox-py310 https://zuul.opendev.org/t/openstack/build/f31f7c2bf1df4668bbae7fbf87f14986 : SUCCESS in 5m 58s\n- cinderlib-tox-py311 https://zuul.opendev.org/t/openstack/build/491185cb8b3a487586ba1286c5fac717 : SUCCESS in 7m 12s (non-voting)\n- cinderlib-lvm-functional https://zuul.opendev.org/t/openstack/build/dc372d60a2324d59864f233835e749b7 : SUCCESS in 14m 44s\n- cinderlib-ceph-functional https://zuul.opendev.org/t/openstack/build/78acc8fc86b647ed90509dcf0144a3e9 : SUCCESS in 10m 13s\n- os-brick-src-tempest-lvm-lio-barbican https://zuul.opendev.org/t/openstack/build/445e46c0a08247d4beca58ee2a8d5752 : SUCCESS in 1h 41m 20s","accounts_in_message":[],"_revision_number":7},{"id":"8188b246d1161128cf79b5a2ccd16ac7856824a5","author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"date":"2023-01-20 04:03:17.000000000","message":"Patch Set 7: Code-Review+2 Workflow+1\n\n(1 comment)","accounts_in_message":[],"_revision_number":7},{"id":"7ef25e0b6d68ab2a4565fd8dd0400c1e3d177b1a","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2023-01-20 04:03:50.000000000","message":"Patch Set 7: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":7},{"id":"1231abca489f7f9bf66e2af306bfd2a476b385ab","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2023-01-20 05:56:04.000000000","message":"Patch Set 7: Verified+2\n\nBuild succeeded (gate pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/66cd7851da7c4dfaacc25c7d0f4b26e3\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/bdb9ede5ed1f458ab9c9b4c10ae086f3 : SUCCESS in 7m 43s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/442998b302594e4dac52bab236d5838d : SUCCESS in 3m 28s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/5f497dbd7c5b4b09ab94020494014312 : SUCCESS in 3m 49s\n- cinderlib-tox-py38 https://zuul.opendev.org/t/openstack/build/faac9f0b20a04fb399f899cc60674eb0 : SUCCESS in 11m 28s\n- cinderlib-tox-py310 https://zuul.opendev.org/t/openstack/build/01aa2b26bc2743fe9d884a5ab65a7935 : SUCCESS in 9m 31s\n- cinderlib-lvm-functional https://zuul.opendev.org/t/openstack/build/c20c30290cde4c558ad5e668d014630e : SUCCESS in 12m 54s\n- cinderlib-ceph-functional https://zuul.opendev.org/t/openstack/build/7802153fc6f04491980ba832e3dc6b01 : SUCCESS in 9m 02s\n- os-brick-src-tempest-lvm-lio-barbican https://zuul.opendev.org/t/openstack/build/bbe3af5cf2be443cae6499a8f6dcd46b : SUCCESS in 1h 47m 24s","accounts_in_message":[],"_revision_number":7},{"id":"216f3d6a48ca9912acf6d5ec30fcc3c1c6300635","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2023-01-20 05:56:04.000000000","message":"Change has been successfully merged","accounts_in_message":[],"_revision_number":7},{"id":"8fac012aeaa37f4ed192c518558951558ec54864","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2023-01-20 05:57:10.000000000","message":"Patch Set 7:\n\nBuild succeeded (promote pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/2119e75d866b4096a1a293b32f741d79\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/938f28eebff749e799c6fd4a38fad849 : SUCCESS in 42s\n- promote-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/93b815a526f54831a93e86cb1ade8719 : SUCCESS in 39s","accounts_in_message":[],"_revision_number":7}],"current_revision_number":7,"current_revision":"a123ff5f82938828367baaf04c70ffd917cff844","revisions":{"1d80cf5167996e09e2995dfd1d902fdc64a8cbb7":{"kind":"REWORK","_number":1,"created":"2023-01-13 13:40:56.000000000","uploader":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"ref":"refs/changes/92/870092/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cinderlib","ref":"refs/changes/92/870092/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cinderlib refs/changes/92/870092/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cinderlib refs/changes/92/870092/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cinderlib refs/changes/92/870092/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cinderlib refs/changes/92/870092/1"}}},"commit":{"parents":[{"commit":"2d1246d41b1d324c12e930882d524434ded62199","subject":"Merge \"Open cinderlib for 2023.1 (Antelope) development\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinderlib/commit/2d1246d41b1d324c12e930882d524434ded62199"}]}],"author":{"name":"Gorka Eguileor","email":"geguileo@redhat.com","date":"2023-01-13 11:47:45.000000000","tz":60},"committer":{"name":"Gorka Eguileor","email":"geguileo@redhat.com","date":"2023-01-13 12:26:00.000000000","tz":60},"subject":"Fix DBMS storage plugin","message":"Fix DBMS storage plugin\n\nIn previous patches (I469b4b9dd9966be8477f3cefbd6284bdaf17ca17 and\nI4981ad558195895341c91fa37803713c8c0ab684) we assumed that unit tests\nwere failing because they needed to be updated, but that\u0027s only half of\nthe issue.  The other half is that the DBMS plugin needs to be updated\nas well.\n\nThis misunderstanding lead to a broken DBMS system that unit tests don\u0027t\ndetect.\n\nThis patch fixes the DBMS system and reverts changes to unit tests that\nthose 2 patches introduced and are not necessary as well as removed the\nFIXME comments that are now fixed.\n\nHere\u0027s a brief description of why some of the issues were happening:\n\n- Locked database issues were happening when a unit test from\n  TestBasePersistence run before a DBMS unit test. It was caused by the\n  replacement of the main_context_manager when it had not been used.  No\n  longer a problem since we no longer replace it and we only replace its\n  transaction factory when it has been used.\n\n- \"ValueError: Backend named fake_backend already exists with a\n  different configuration\" error was caused because in TestDBPersistence\n  the tearDown method only called the parent\u0027s tearDown after the\n  cleanup of the DB, so if the cleanup of the DB failed then the global\n  existing backend would remain and a following test would have trouble\n  with it.  Calling it first now to fix this.\n\n- Splitting the running of unit tests in tox.ini was necessary because\n  methods in Cinder DB API were always using the context manager\n  transaction factory from the first DBMS Test Case class.  So some DBMS\n  plugin methods would end up writing data to the previous database\n  while others would write to the new database.  Now that we replace the\n  transaction factory everything is going to the same DB.\n\nChange-Id: I74ce945291697bc1fcc63a35d60ef71494fdaf19\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinderlib/commit/1d80cf5167996e09e2995dfd1d902fdc64a8cbb7"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinderlib/commit/1d80cf5167996e09e2995dfd1d902fdc64a8cbb7"}]},"branch":"refs/heads/master"},"bafb1d0d515992664782d3ab12caeb19eaa246e7":{"kind":"TRIVIAL_REBASE","_number":2,"created":"2023-01-16 18:32:10.000000000","uploader":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"ref":"refs/changes/92/870092/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cinderlib","ref":"refs/changes/92/870092/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cinderlib refs/changes/92/870092/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cinderlib refs/changes/92/870092/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cinderlib refs/changes/92/870092/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cinderlib refs/changes/92/870092/2"}}},"commit":{"parents":[{"commit":"5afaa74d81d168fa7129c02903fca3ab4eb959d7","subject":"Fix tox.ini","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinderlib/commit/5afaa74d81d168fa7129c02903fca3ab4eb959d7"}]}],"author":{"name":"Gorka Eguileor","email":"geguileo@redhat.com","date":"2023-01-13 11:47:45.000000000","tz":60},"committer":{"name":"Gorka Eguileor","email":"geguileo@redhat.com","date":"2023-01-16 18:31:53.000000000","tz":60},"subject":"Fix DBMS storage plugin","message":"Fix DBMS storage plugin\n\nIn previous patches (I469b4b9dd9966be8477f3cefbd6284bdaf17ca17 and\nI4981ad558195895341c91fa37803713c8c0ab684) we assumed that unit tests\nwere failing because they needed to be updated, but that\u0027s only half of\nthe issue.  The other half is that the DBMS plugin needs to be updated\nas well.\n\nThis misunderstanding lead to a broken DBMS system that unit tests don\u0027t\ndetect.\n\nThis patch fixes the DBMS system and reverts changes to unit tests that\nthose 2 patches introduced and are not necessary as well as removed the\nFIXME comments that are now fixed.\n\nHere\u0027s a brief description of why some of the issues were happening:\n\n- Locked database issues were happening when a unit test from\n  TestBasePersistence run before a DBMS unit test. It was caused by the\n  replacement of the main_context_manager when it had not been used.  No\n  longer a problem since we no longer replace it and we only replace its\n  transaction factory when it has been used.\n\n- \"ValueError: Backend named fake_backend already exists with a\n  different configuration\" error was caused because in TestDBPersistence\n  the tearDown method only called the parent\u0027s tearDown after the\n  cleanup of the DB, so if the cleanup of the DB failed then the global\n  existing backend would remain and a following test would have trouble\n  with it.  Calling it first now to fix this.\n\n- Splitting the running of unit tests in tox.ini was necessary because\n  methods in Cinder DB API were always using the context manager\n  transaction factory from the first DBMS Test Case class.  So some DBMS\n  plugin methods would end up writing data to the previous database\n  while others would write to the new database.  Now that we replace the\n  transaction factory everything is going to the same DB.\n\nChange-Id: I74ce945291697bc1fcc63a35d60ef71494fdaf19\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinderlib/commit/bafb1d0d515992664782d3ab12caeb19eaa246e7"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinderlib/commit/bafb1d0d515992664782d3ab12caeb19eaa246e7"}]},"branch":"refs/heads/master"},"5bea8f678c3b32fe1e3054accc9d5f9eb1058661":{"kind":"TRIVIAL_REBASE","_number":3,"created":"2023-01-17 11:01:25.000000000","uploader":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"ref":"refs/changes/92/870092/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cinderlib","ref":"refs/changes/92/870092/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cinderlib refs/changes/92/870092/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cinderlib refs/changes/92/870092/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cinderlib refs/changes/92/870092/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cinderlib refs/changes/92/870092/3"}}},"commit":{"parents":[{"commit":"bcb757b06208de18cb45ab4b950714f334905bd7","subject":"Fix tox.ini","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinderlib/commit/bcb757b06208de18cb45ab4b950714f334905bd7"}]}],"author":{"name":"Gorka Eguileor","email":"geguileo@redhat.com","date":"2023-01-13 11:47:45.000000000","tz":60},"committer":{"name":"Gorka Eguileor","email":"geguileo@redhat.com","date":"2023-01-17 11:01:16.000000000","tz":60},"subject":"Fix DBMS storage plugin","message":"Fix DBMS storage plugin\n\nIn previous patches (I469b4b9dd9966be8477f3cefbd6284bdaf17ca17 and\nI4981ad558195895341c91fa37803713c8c0ab684) we assumed that unit tests\nwere failing because they needed to be updated, but that\u0027s only half of\nthe issue.  The other half is that the DBMS plugin needs to be updated\nas well.\n\nThis misunderstanding lead to a broken DBMS system that unit tests don\u0027t\ndetect.\n\nThis patch fixes the DBMS system and reverts changes to unit tests that\nthose 2 patches introduced and are not necessary as well as removed the\nFIXME comments that are now fixed.\n\nHere\u0027s a brief description of why some of the issues were happening:\n\n- Locked database issues were happening when a unit test from\n  TestBasePersistence run before a DBMS unit test. It was caused by the\n  replacement of the main_context_manager when it had not been used.  No\n  longer a problem since we no longer replace it and we only replace its\n  transaction factory when it has been used.\n\n- \"ValueError: Backend named fake_backend already exists with a\n  different configuration\" error was caused because in TestDBPersistence\n  the tearDown method only called the parent\u0027s tearDown after the\n  cleanup of the DB, so if the cleanup of the DB failed then the global\n  existing backend would remain and a following test would have trouble\n  with it.  Calling it first now to fix this.\n\n- Splitting the running of unit tests in tox.ini was necessary because\n  methods in Cinder DB API were always using the context manager\n  transaction factory from the first DBMS Test Case class.  So some DBMS\n  plugin methods would end up writing data to the previous database\n  while others would write to the new database.  Now that we replace the\n  transaction factory everything is going to the same DB.\n\nChange-Id: I74ce945291697bc1fcc63a35d60ef71494fdaf19\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinderlib/commit/5bea8f678c3b32fe1e3054accc9d5f9eb1058661"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinderlib/commit/5bea8f678c3b32fe1e3054accc9d5f9eb1058661"}]},"branch":"refs/heads/master"},"cd767bb9afa30d5200e5e545130f9f14adfa1a83":{"kind":"REWORK","_number":4,"created":"2023-01-18 10:00:01.000000000","uploader":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"ref":"refs/changes/92/870092/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cinderlib","ref":"refs/changes/92/870092/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cinderlib refs/changes/92/870092/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cinderlib refs/changes/92/870092/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cinderlib refs/changes/92/870092/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cinderlib refs/changes/92/870092/4"}}},"commit":{"parents":[{"commit":"2d1246d41b1d324c12e930882d524434ded62199","subject":"Merge \"Open cinderlib for 2023.1 (Antelope) development\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinderlib/commit/2d1246d41b1d324c12e930882d524434ded62199"}]}],"author":{"name":"Gorka Eguileor","email":"geguileo@redhat.com","date":"2023-01-16 18:29:55.000000000","tz":60},"committer":{"name":"Gorka Eguileor","email":"geguileo@redhat.com","date":"2023-01-18 09:57:57.000000000","tz":60},"subject":"WIP: Figure out what\u0027s wrong in the gate","message":"WIP: Figure out what\u0027s wrong in the gate\n\nCinderlib gates seem to be broken and cannot reproduce locally\n\nTrying to figure out what\u0027s going on.\n\nRelated patches:\n  https://review.opendev.org/c/openstack/cinderlib/+/870668\n  https://review.opendev.org/c/openstack/cinderlib/+/870092/3\n\nChange-Id: I74ce945291697bc1fcc63a35d60ef71494fdaf19\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinderlib/commit/cd767bb9afa30d5200e5e545130f9f14adfa1a83"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinderlib/commit/cd767bb9afa30d5200e5e545130f9f14adfa1a83"}]},"branch":"refs/heads/master"},"a737849ddc3492cbae10d489f178d498ed2064b0":{"kind":"REWORK","_number":5,"created":"2023-01-18 11:37:33.000000000","uploader":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"ref":"refs/changes/92/870092/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cinderlib","ref":"refs/changes/92/870092/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cinderlib refs/changes/92/870092/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cinderlib refs/changes/92/870092/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cinderlib refs/changes/92/870092/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cinderlib refs/changes/92/870092/5"}}},"commit":{"parents":[{"commit":"2d1246d41b1d324c12e930882d524434ded62199","subject":"Merge \"Open cinderlib for 2023.1 (Antelope) development\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinderlib/commit/2d1246d41b1d324c12e930882d524434ded62199"}]}],"author":{"name":"Gorka Eguileor","email":"geguileo@redhat.com","date":"2023-01-16 18:29:55.000000000","tz":60},"committer":{"name":"Gorka Eguileor","email":"geguileo@redhat.com","date":"2023-01-18 11:37:24.000000000","tz":60},"subject":"WIP: Figure out what\u0027s wrong in the gate","message":"WIP: Figure out what\u0027s wrong in the gate\n\nCinderlib gates seem to be broken and cannot reproduce locally\n\nTrying to figure out what\u0027s going on.\n\nRelated patches:\n  https://review.opendev.org/c/openstack/cinderlib/+/870668\n  https://review.opendev.org/c/openstack/cinderlib/+/870092/3\n\nChange-Id: I74ce945291697bc1fcc63a35d60ef71494fdaf19\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinderlib/commit/a737849ddc3492cbae10d489f178d498ed2064b0"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinderlib/commit/a737849ddc3492cbae10d489f178d498ed2064b0"}]},"branch":"refs/heads/master"},"5ded2caeb53fe8ce7c84ccac68b4a16d5788c66a":{"kind":"REWORK","_number":6,"created":"2023-01-18 14:14:34.000000000","uploader":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"ref":"refs/changes/92/870092/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cinderlib","ref":"refs/changes/92/870092/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cinderlib refs/changes/92/870092/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cinderlib refs/changes/92/870092/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cinderlib refs/changes/92/870092/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cinderlib refs/changes/92/870092/6"}}},"commit":{"parents":[{"commit":"f56d5ab91406b5dcee64940e2337050bc8362eab","subject":"Fix tox.ini","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinderlib/commit/f56d5ab91406b5dcee64940e2337050bc8362eab"}]}],"author":{"name":"Gorka Eguileor","email":"geguileo@redhat.com","date":"2023-01-13 11:47:45.000000000","tz":60},"committer":{"name":"Gorka Eguileor","email":"geguileo@redhat.com","date":"2023-01-18 14:13:59.000000000","tz":60},"subject":"Fix DBMS storage plugin","message":"Fix DBMS storage plugin\n\nIn previous patches (I469b4b9dd9966be8477f3cefbd6284bdaf17ca17 and\nI4981ad558195895341c91fa37803713c8c0ab684) we assumed that unit tests\nwere failing because they needed to be updated, but that\u0027s only half of\nthe issue.  The other half is that the DBMS plugin needs to be updated\nas well.\n\nThis misunderstanding lead to a broken DBMS system that unit tests don\u0027t\ndetect.\n\nThis patch fixes the DBMS system and reverts changes to unit tests that\nthose 2 patches introduced and are not necessary as well as removed the\nFIXME comments that are now fixed.\n\nHere\u0027s a brief description of why some of the issues were happening:\n\n- Locked database issues were happening when a unit test from\n  TestBasePersistence run before a DBMS unit test. It was caused by the\n  replacement of the main_context_manager when it had not been used.  No\n  longer a problem since we no longer replace it and we only replace its\n  transaction factory when it has been used.\n\n- \"ValueError: Backend named fake_backend already exists with a\n  different configuration\" error was caused because in TestDBPersistence\n  the tearDown method only called the parent\u0027s tearDown after the\n  cleanup of the DB, so if the cleanup of the DB failed then the global\n  existing backend would remain and a following test would have trouble\n  with it.  Calling it first now to fix this.\n\n- Splitting the running of unit tests in tox.ini was necessary because\n  methods in Cinder DB API were always using the context manager\n  transaction factory from the first DBMS Test Case class.  So some DBMS\n  plugin methods would end up writing data to the previous database\n  while others would write to the new database.  Now that we replace the\n  transaction factory everything is going to the same DB.\n\nChange-Id: I74ce945291697bc1fcc63a35d60ef71494fdaf19\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinderlib/commit/5ded2caeb53fe8ce7c84ccac68b4a16d5788c66a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinderlib/commit/5ded2caeb53fe8ce7c84ccac68b4a16d5788c66a"}]},"branch":"refs/heads/master"},"a123ff5f82938828367baaf04c70ffd917cff844":{"kind":"TRIVIAL_REBASE","_number":7,"created":"2023-01-19 16:31:55.000000000","uploader":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"ref":"refs/changes/92/870092/7","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cinderlib","ref":"refs/changes/92/870092/7","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cinderlib refs/changes/92/870092/7 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cinderlib refs/changes/92/870092/7 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cinderlib refs/changes/92/870092/7 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cinderlib refs/changes/92/870092/7"}}},"commit":{"parents":[{"commit":"436bc6d74bec023a00b07f0f2d4df252fda470d7","subject":"Fix tox.ini","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinderlib/commit/436bc6d74bec023a00b07f0f2d4df252fda470d7"}]}],"author":{"name":"Gorka Eguileor","email":"geguileo@redhat.com","date":"2023-01-13 11:47:45.000000000","tz":60},"committer":{"name":"Gorka Eguileor","email":"geguileo@redhat.com","date":"2023-01-19 16:31:13.000000000","tz":60},"subject":"Fix DBMS storage plugin","message":"Fix DBMS storage plugin\n\nIn previous patches (I469b4b9dd9966be8477f3cefbd6284bdaf17ca17 and\nI4981ad558195895341c91fa37803713c8c0ab684) we assumed that unit tests\nwere failing because they needed to be updated, but that\u0027s only half of\nthe issue.  The other half is that the DBMS plugin needs to be updated\nas well.\n\nThis misunderstanding lead to a broken DBMS system that unit tests don\u0027t\ndetect.\n\nThis patch fixes the DBMS system and reverts changes to unit tests that\nthose 2 patches introduced and are not necessary as well as removed the\nFIXME comments that are now fixed.\n\nHere\u0027s a brief description of why some of the issues were happening:\n\n- Locked database issues were happening when a unit test from\n  TestBasePersistence run before a DBMS unit test. It was caused by the\n  replacement of the main_context_manager when it had not been used.  No\n  longer a problem since we no longer replace it and we only replace its\n  transaction factory when it has been used.\n\n- \"ValueError: Backend named fake_backend already exists with a\n  different configuration\" error was caused because in TestDBPersistence\n  the tearDown method only called the parent\u0027s tearDown after the\n  cleanup of the DB, so if the cleanup of the DB failed then the global\n  existing backend would remain and a following test would have trouble\n  with it.  Calling it first now to fix this.\n\n- Splitting the running of unit tests in tox.ini was necessary because\n  methods in Cinder DB API were always using the context manager\n  transaction factory from the first DBMS Test Case class.  So some DBMS\n  plugin methods would end up writing data to the previous database\n  while others would write to the new database.  Now that we replace the\n  transaction factory everything is going to the same DB.\n\nChange-Id: I74ce945291697bc1fcc63a35d60ef71494fdaf19\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinderlib/commit/a123ff5f82938828367baaf04c70ffd917cff844"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinderlib/commit/a123ff5f82938828367baaf04c70ffd917cff844"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[{"rule_name":"gerrit~DefaultSubmitRule","status":"CLOSED","labels":[{"label":"Verified","status":"OK","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"OK","applied_by":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"}},{"label":"Workflow","status":"OK","applied_by":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"}},{"label":"Review-Priority","status":"MAY","applied_by":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"}}]}],"submit_requirements":[]}
