)]}'
{".zuul.yaml":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"fe6f3b9e1181ac45a72440068afc38b6a1f1dcf4","unresolved":true,"context_lines":[{"line_number":166,"context_line":"        test-config:"},{"line_number":167,"context_line":"          $TEMPEST_CONFIG:"},{"line_number":168,"context_line":"            volume-feature-enabled:"},{"line_number":169,"context_line":"              volume_revert: True"},{"line_number":170,"context_line":"      devstack_services:"},{"line_number":171,"context_line":"        c-bak: true"},{"line_number":172,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":10,"id":"12be28d8_a1a21b7d","line":169,"updated":"2022-06-17 15:08:11.000000000","message":"should there also be\n  backup: true\n\nhere?","commit_id":"807010948852edab04d0a2e7bbaf4e117449c749"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"e1624222bf4b765055c8065b7d4fd06d97eefc0c","unresolved":false,"context_lines":[{"line_number":166,"context_line":"        test-config:"},{"line_number":167,"context_line":"          $TEMPEST_CONFIG:"},{"line_number":168,"context_line":"            volume-feature-enabled:"},{"line_number":169,"context_line":"              volume_revert: True"},{"line_number":170,"context_line":"      devstack_services:"},{"line_number":171,"context_line":"        c-bak: true"},{"line_number":172,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":10,"id":"19320c3f_1f41f82a","line":169,"in_reply_to":"12be28d8_a1a21b7d","updated":"2022-06-28 09:24:30.000000000","message":"That\u0027s true by default.","commit_id":"807010948852edab04d0a2e7bbaf4e117449c749"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"6592e17d28c73b4b670ae5c7ff1a2d8e64e1a549","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"38fe8e2f_764d8373","updated":"2022-05-17 09:48:02.000000000","message":"Minor query.","commit_id":"02d9ae425dd5c100810b5ed7d2e2244d1cd7a722"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"372c2144824a3b843b75baefb2dac1a1c57895cb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"b3592305_7b6c80d1","updated":"2022-05-17 20:15:46.000000000","message":"recheck","commit_id":"9ca6560047247b0c9ae538dd3185f31625098b4c"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"e620e66b23cac0d67f00c71b223e5efc4711ea12","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"c881a032_dbd42cce","updated":"2022-06-14 20:41:32.000000000","message":"Dependency has merged into devstack.  Going to do a recheck because all the logs are gone.","commit_id":"b59061d1d1ddca7c46f69220bef3a72b84f737fc"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"b93792af3bd970df715fedc13becde5ccdf2ae20","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"3af17817_4fc823fc","updated":"2022-05-30 09:42:00.000000000","message":"My minor query is resolved. Zuul also passed. So +1.","commit_id":"b59061d1d1ddca7c46f69220bef3a72b84f737fc"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"8456eb54cfefc5b5b1e42845fca0bbf29a2098b4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"df31018b_027c2894","updated":"2022-05-27 15:19:15.000000000","message":"recheck","commit_id":"b59061d1d1ddca7c46f69220bef3a72b84f737fc"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"69f841ab808a95e66e1667848b0cac0e3970de64","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"e4e27d89_d539386c","updated":"2022-06-14 20:41:43.000000000","message":"recheck","commit_id":"b59061d1d1ddca7c46f69220bef3a72b84f737fc"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"fe6f3b9e1181ac45a72440068afc38b6a1f1dcf4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"03e2deea_f1354d84","updated":"2022-06-17 15:08:11.000000000","message":"Can you point me to where I can see the tests running?  I looked at cinder-tempest-plugin-cbak-ceph and the first entry for TestVolumeBackupRestore invokes tearDownClass because of a skip-test being raised [0].  I think maybe you need to enable backup in tempest.conf to run any of the scenario tests?[1].  Looks like we don\u0027t set that for the job in .zuul.yaml.  Or am I looking in the wrong place?\n\n\n[0] https://zuul.opendev.org/t/openstack/build/869e0e4a723d422b9e4477f1dbed0611/log/controller/logs/tempest_log.txt#59642\n[1] https://zuul.opendev.org/t/openstack/build/869e0e4a723d422b9e4477f1dbed0611/log/controller/logs/tempest_conf.txt#98","commit_id":"807010948852edab04d0a2e7bbaf4e117449c749"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"e1624222bf4b765055c8065b7d4fd06d97eefc0c","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":10,"id":"fedcb242_44176bec","in_reply_to":"03e2deea_f1354d84","updated":"2022-06-28 09:24:30.000000000","message":"The test requires multiple cinder backend, and I guess that\u0027s the problem.","commit_id":"807010948852edab04d0a2e7bbaf4e117449c749"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"04ee1a55c5b5be7666d47795a2a6ad7f580667c4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"ef5e1881_214e865a","in_reply_to":"fedcb242_44176bec","updated":"2023-02-20 14:56:31.000000000","message":"https://review.opendev.org/c/openstack/cinder/+/784815/","commit_id":"807010948852edab04d0a2e7bbaf4e117449c749"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"33ebee336bb33257602b435693ab6e5674baff5b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"d0f815dc_f11c84d2","updated":"2022-10-25 13:46:43.000000000","message":"cinder-tempest-plugin-basic-wallaby is FAILURE  because of conflict of version, it\u0027s not related to this AFAIK. \n\n\n```\nThe conflict is caused by:\n    keystonemiddleware 9.2.0 depends on oslo.utils\u003e\u003d3.33.0\n    The user requested (constraint) oslo-utils\u003d\u003d\u003d4.8.2\n\nTo fix this you could try to:\n1. loosen the range of package versions you\u0027ve specified\n2. remove package versions to allow pip attempt to solve the dependency conflict\n``` \n","commit_id":"a3c40f0fce74dc8c5dde2b95639ee35e0e9bce18"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"c4f03d06a76053113a417fab1fd71361b0630a23","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"383965b8_2046fdc9","updated":"2022-10-25 13:46:57.000000000","message":"recheck cinder-tempest-plugin-basic-wallaby","commit_id":"a3c40f0fce74dc8c5dde2b95639ee35e0e9bce18"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"93ad011b7fa435a36f3d9ecc4ffaedfa210ca4ae","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"5908aaf4_c88debfc","updated":"2022-12-01 15:52:38.000000000","message":"recheck cinder-tempest-plugin-lvm-tgt-barbican\n\n\nThe failure is not related to this patch. \n{2} setUpClass (cinder_tempest_plugin.scenario.test_volume_backup_restore.TestVolumeBackupRestore) ... SKIPPED: Cinder multi-backend feature disabled","commit_id":"5449931fa15f6cf1acc8147ccf714b0c4308e0b8"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"3fe776618647b8cb29d3e3b4eec29117ac7d830b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"fd2d4a63_85aa1cdf","updated":"2022-12-06 12:46:32.000000000","message":"cinder-tempest-plugin-lvm-tgt-barbican Timed out 😞\n\nhttps://lists.openstack.org/pipermail/openstack-discuss/2022-December/031394.html","commit_id":"13fe88af69168f8e478feaf865719fa6e8550727"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"3ec0315786bbb87626178385c46ae5e9f11519e7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"9c4229ec_32c6b56f","updated":"2022-12-06 09:33:28.000000000","message":"recheck","commit_id":"13fe88af69168f8e478feaf865719fa6e8550727"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"8a4d908581ae9b656d1763225f682a3f60912478","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"1bc8a19a_f071d31a","updated":"2022-12-06 15:12:44.000000000","message":"recheck cinder-tempest-plugin-lvm-tgt-barbican","commit_id":"13fe88af69168f8e478feaf865719fa6e8550727"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"41184b7e9ca59b2cfbc4466d9d31882415a0c80c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"b60e9a43_8ac81fd7","updated":"2023-01-23 14:07:49.000000000","message":"recheck","commit_id":"0b91ae4155b0bbcb9ce9f124ee8e27e53ccfd105"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"0eeee4e68cdd84ff2bcc297ceaccc8691d784d3d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":18,"id":"583e531c_2da9772a","updated":"2023-02-16 14:39:06.000000000","message":"recheck\n\npep8 was fixed and tgt is now skipped","commit_id":"d845d60303c964641a9c383a1cbfadb251d5a598"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"53d0f5905a2c8bd419e8c2d59703d92c5578ab1f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"6581d0db_1317aead","updated":"2023-02-20 23:11:47.000000000","message":"\u003e  cinder-tempest-plugin-basic-wallaby https://zuul.opendev.org/t/openstack/build/10d2b97dba4c4976afba2724fed348ee : FAILURE in 31m 42s \n```\nERROR: Could not find a version that satisfies the requirement tempest\u003e\u003d31.0.0 (from cinder-tempest-plugin)\nERROR: No matching distribution found for tempest\u003e\u003d31.0.0\n```","commit_id":"2e5f67ebd83c526862b8b7292f16195799b9e6dd"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"e99f9283ba9c5d9843824cf99e7fb713af0f13d9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"524164b5_90826d1e","updated":"2023-02-20 17:58:52.000000000","message":"The tests are not running, not sure why Zuul is failling\n```\n{1} setUpClass (cinder_tempest_plugin.scenario.test_volume_backup_restore.TestVolumeBackupRestore) ... SKIPPED: Cinder multi-backend feature disabled\n```","commit_id":"2e5f67ebd83c526862b8b7292f16195799b9e6dd"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"036e6a8656c7304794dbd0187491edcb23966af8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"25a99fed_1e3a2e06","updated":"2023-02-20 17:58:59.000000000","message":"recheck","commit_id":"2e5f67ebd83c526862b8b7292f16195799b9e6dd"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"91a7c83e54cbb1371130e26d1c6fc6ebe3c32b67","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"45aa473f_be127733","updated":"2023-02-21 10:25:12.000000000","message":"recheck","commit_id":"2e5f67ebd83c526862b8b7292f16195799b9e6dd"}],"cinder_tempest_plugin/scenario/test_volume_backup_restore.py":[{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"47b765d7edaa305ae7f6dee9c501047bd929dd16","unresolved":true,"context_lines":[{"line_number":54,"context_line":"            raise cls.skipException(\"Requires at least two different \""},{"line_number":55,"context_line":"                                    \"backend names\")"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    def _boot_instance_from_volume(self, vol_id, keypair, security_group):"},{"line_number":58,"context_line":"        key_name \u003d keypair[\u0027name\u0027]"},{"line_number":59,"context_line":"        security_groups \u003d [{\u0027name\u0027: security_group[\u0027name\u0027]}]"},{"line_number":60,"context_line":"        block_device_mapping \u003d [{\u0027device_name\u0027: \u0027vda\u0027, \u0027volume_id\u0027: vol_id,"}],"source_content_type":"text/x-python","patch_set":2,"id":"7e6da540_2b41ef11","line":57,"updated":"2022-03-29 04:46:34.000000000","message":"You dont need to re-define it , there is a function in tempest.\n\ninstance_1st \u003d self.boot_instance_from_resource(\n            source_id\u003dvolume_origin[\u0027id\u0027],\n            source_type\u003d\u0027volume\u0027,\n            keypair\u003dkeypair,\n            security_group\u003dsecurity_group)","commit_id":"f5920f6d06984fc4398e55b8157d195bcb586f13"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"a69104e51fc58d314e8db9c82d447a9f613c1f46","unresolved":false,"context_lines":[{"line_number":54,"context_line":"            raise cls.skipException(\"Requires at least two different \""},{"line_number":55,"context_line":"                                    \"backend names\")"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    def _boot_instance_from_volume(self, vol_id, keypair, security_group):"},{"line_number":58,"context_line":"        key_name \u003d keypair[\u0027name\u0027]"},{"line_number":59,"context_line":"        security_groups \u003d [{\u0027name\u0027: security_group[\u0027name\u0027]}]"},{"line_number":60,"context_line":"        block_device_mapping \u003d [{\u0027device_name\u0027: \u0027vda\u0027, \u0027volume_id\u0027: vol_id,"}],"source_content_type":"text/x-python","patch_set":2,"id":"92a86dc7_cca90341","line":57,"in_reply_to":"7e6da540_2b41ef11","updated":"2022-03-29 15:32:41.000000000","message":"Done","commit_id":"f5920f6d06984fc4398e55b8157d195bcb586f13"},{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"47b765d7edaa305ae7f6dee9c501047bd929dd16","unresolved":true,"context_lines":[{"line_number":77,"context_line":"        return ({\u0027name\u0027: source_body[\u0027name\u0027], \u0027host\u0027: backend_source},"},{"line_number":78,"context_line":"                {\u0027name\u0027: dest_body[\u0027name\u0027], \u0027host\u0027: backend_dest})"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"    @ddt.data(True, False)"},{"line_number":81,"context_line":"    @decorators.idempotent_id(\u00272ce5e55c-4085-43c1-98c6-582525334ad7\u0027)"},{"line_number":82,"context_line":"    @decorators.attr(type\u003d\u0027slow\u0027)"},{"line_number":83,"context_line":"    @utils.services(\u0027compute\u0027, \u0027volume\u0027, \u0027image\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"972d60aa_83f84f70","line":80,"updated":"2022-03-29 04:46:34.000000000","message":"Not sure about ddt usage here, i did not see it in tempest ...\nI dont know how other decorator handle the func attributes","commit_id":"f5920f6d06984fc4398e55b8157d195bcb586f13"},{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"65f93ee9be3006adad8e203278586c479881d78c","unresolved":true,"context_lines":[{"line_number":77,"context_line":"        return ({\u0027name\u0027: source_body[\u0027name\u0027], \u0027host\u0027: backend_source},"},{"line_number":78,"context_line":"                {\u0027name\u0027: dest_body[\u0027name\u0027], \u0027host\u0027: backend_dest})"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"    @ddt.data(True, False)"},{"line_number":81,"context_line":"    @decorators.idempotent_id(\u00272ce5e55c-4085-43c1-98c6-582525334ad7\u0027)"},{"line_number":82,"context_line":"    @decorators.attr(type\u003d\u0027slow\u0027)"},{"line_number":83,"context_line":"    @utils.services(\u0027compute\u0027, \u0027volume\u0027, \u0027image\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"e4e35530_d7ea34bc","line":80,"in_reply_to":"972d60aa_83f84f70","updated":"2022-03-29 05:25:28.000000000","message":"I suspect that the DDT will create two testcases with the same uuid (i did not check it)","commit_id":"f5920f6d06984fc4398e55b8157d195bcb586f13"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"a69104e51fc58d314e8db9c82d447a9f613c1f46","unresolved":false,"context_lines":[{"line_number":77,"context_line":"        return ({\u0027name\u0027: source_body[\u0027name\u0027], \u0027host\u0027: backend_source},"},{"line_number":78,"context_line":"                {\u0027name\u0027: dest_body[\u0027name\u0027], \u0027host\u0027: backend_dest})"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"    @ddt.data(True, False)"},{"line_number":81,"context_line":"    @decorators.idempotent_id(\u00272ce5e55c-4085-43c1-98c6-582525334ad7\u0027)"},{"line_number":82,"context_line":"    @decorators.attr(type\u003d\u0027slow\u0027)"},{"line_number":83,"context_line":"    @utils.services(\u0027compute\u0027, \u0027volume\u0027, \u0027image\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"0a613384_a86df454","line":80,"in_reply_to":"e4e35530_d7ea34bc","updated":"2022-03-29 15:32:41.000000000","message":"Done","commit_id":"f5920f6d06984fc4398e55b8157d195bcb586f13"},{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"2087c00d26dfdeb68c64756fb70601f42496589e","unresolved":true,"context_lines":[{"line_number":120,"context_line":"    @utils.services(\u0027compute\u0027, \u0027volume\u0027, \u0027image\u0027)"},{"line_number":121,"context_line":"    def test_volume_backup_restore_multibackend(self):"},{"line_number":122,"context_line":"        \"\"\"Test backup restore with multibackend.\"\"\""},{"line_number":123,"context_line":"        source_type, dest_type \u003d self._create_volume_types()"},{"line_number":124,"context_line":"        self._volume_backup_restore(source_type\u003dsource_type,"},{"line_number":125,"context_line":"                                    dest_type\u003ddest_type)"},{"line_number":126,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"98546161_fbde0880","line":123,"updated":"2022-03-29 15:43:20.000000000","message":"Good , it will be easy to extend the tests for encrypted type when needed...","commit_id":"30ad6fcd34ef93a7fe6a6ff6ccc80bf91bf84c79"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"145814f6edb7e3f16260daffd23464f415e2bb88","unresolved":false,"context_lines":[{"line_number":120,"context_line":"    @utils.services(\u0027compute\u0027, \u0027volume\u0027, \u0027image\u0027)"},{"line_number":121,"context_line":"    def test_volume_backup_restore_multibackend(self):"},{"line_number":122,"context_line":"        \"\"\"Test backup restore with multibackend.\"\"\""},{"line_number":123,"context_line":"        source_type, dest_type \u003d self._create_volume_types()"},{"line_number":124,"context_line":"        self._volume_backup_restore(source_type\u003dsource_type,"},{"line_number":125,"context_line":"                                    dest_type\u003ddest_type)"},{"line_number":126,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"1645015e_d815c866","line":123,"in_reply_to":"98546161_fbde0880","updated":"2022-03-29 21:45:00.000000000","message":"Ack","commit_id":"30ad6fcd34ef93a7fe6a6ff6ccc80bf91bf84c79"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"eb6ace381c0ef2d8f2b3bba8a4aa0a77c8e01ccf","unresolved":true,"context_lines":[{"line_number":130,"context_line":"    def test_volume_backup_restore_multibackend_reverse(self):"},{"line_number":131,"context_line":"        \"\"\"Test backup restore with multibackend.\"\"\""},{"line_number":132,"context_line":"        source_type, dest_type \u003d self._create_volume_types()"},{"line_number":133,"context_line":"        self._volume_backup_restore(source_type\u003ddest_type,"},{"line_number":134,"context_line":"                                    dest_type\u003dsource_type)"}],"source_content_type":"text/x-python","patch_set":3,"id":"23acba0a_70971085","line":134,"range":{"start_line":133,"start_character":36,"end_line":134,"end_character":57},"updated":"2022-03-29 19:24:21.000000000","message":"This is pretty confusing, it would be better to reverse them in line 132 instead of in the params on this method call.","commit_id":"30ad6fcd34ef93a7fe6a6ff6ccc80bf91bf84c79"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"145814f6edb7e3f16260daffd23464f415e2bb88","unresolved":false,"context_lines":[{"line_number":130,"context_line":"    def test_volume_backup_restore_multibackend_reverse(self):"},{"line_number":131,"context_line":"        \"\"\"Test backup restore with multibackend.\"\"\""},{"line_number":132,"context_line":"        source_type, dest_type \u003d self._create_volume_types()"},{"line_number":133,"context_line":"        self._volume_backup_restore(source_type\u003ddest_type,"},{"line_number":134,"context_line":"                                    dest_type\u003dsource_type)"}],"source_content_type":"text/x-python","patch_set":3,"id":"a274e76f_735e6d24","line":134,"range":{"start_line":133,"start_character":36,"end_line":134,"end_character":57},"in_reply_to":"23acba0a_70971085","updated":"2022-03-29 21:45:00.000000000","message":"Ack","commit_id":"30ad6fcd34ef93a7fe6a6ff6ccc80bf91bf84c79"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"48306ee25d007cf706bbc42e43de15fa67bc8d8a","unresolved":false,"context_lines":[{"line_number":130,"context_line":"    def test_volume_backup_restore_multibackend_reverse(self):"},{"line_number":131,"context_line":"        \"\"\"Test backup restore with multibackend.\"\"\""},{"line_number":132,"context_line":"        source_type, dest_type \u003d self._create_volume_types()"},{"line_number":133,"context_line":"        self._volume_backup_restore(source_type\u003ddest_type,"},{"line_number":134,"context_line":"                                    dest_type\u003dsource_type)"}],"source_content_type":"text/x-python","patch_set":3,"id":"0a172c1d_08becd6a","line":134,"range":{"start_line":133,"start_character":36,"end_line":134,"end_character":57},"in_reply_to":"a274e76f_735e6d24","updated":"2022-04-27 16:14:18.000000000","message":"For the record, I was going to say the opposite: in the current version of the patch I didn\u0027t immediately see the difference between the two tests, and I was expecting to see the difference in this method.\nBut not a big problem. It seems the other requests have been addressed.","commit_id":"30ad6fcd34ef93a7fe6a6ff6ccc80bf91bf84c79"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"6592e17d28c73b4b670ae5c7ff1a2d8e64e1a549","unresolved":true,"context_lines":[{"line_number":128,"context_line":"    @utils.services(\u0027compute\u0027, \u0027volume\u0027, \u0027image\u0027)"},{"line_number":129,"context_line":"    @decorators.idempotent_id(\u0027ce959459-a063-431c-bb50-c4357742588f\u0027)"},{"line_number":130,"context_line":"    def test_volume_backup_restore_multibackend_reverse(self):"},{"line_number":131,"context_line":"        \"\"\"Test backup restore with multibackend.\"\"\""},{"line_number":132,"context_line":"        dest_type, source_type \u003d self._create_volume_types()"},{"line_number":133,"context_line":"        self._volume_backup_restore(source_type\u003dsource_type,"},{"line_number":134,"context_line":"                                    dest_type\u003ddest_type)"}],"source_content_type":"text/x-python","patch_set":6,"id":"fadafe51_c1fda809","line":131,"updated":"2022-05-17 09:48:02.000000000","message":"nit: this line looks same as line 122.\ncan we add word \"reverse\" after the word \"multibackend\" ?","commit_id":"02d9ae425dd5c100810b5ed7d2e2244d1cd7a722"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"ef14c2db7e550e9c0e4bde1b1d081df7e0c968ee","unresolved":false,"context_lines":[{"line_number":128,"context_line":"    @utils.services(\u0027compute\u0027, \u0027volume\u0027, \u0027image\u0027)"},{"line_number":129,"context_line":"    @decorators.idempotent_id(\u0027ce959459-a063-431c-bb50-c4357742588f\u0027)"},{"line_number":130,"context_line":"    def test_volume_backup_restore_multibackend_reverse(self):"},{"line_number":131,"context_line":"        \"\"\"Test backup restore with multibackend.\"\"\""},{"line_number":132,"context_line":"        dest_type, source_type \u003d self._create_volume_types()"},{"line_number":133,"context_line":"        self._volume_backup_restore(source_type\u003dsource_type,"},{"line_number":134,"context_line":"                                    dest_type\u003ddest_type)"}],"source_content_type":"text/x-python","patch_set":6,"id":"ef67bb80_680c4d31","line":131,"in_reply_to":"fadafe51_c1fda809","updated":"2022-05-17 14:14:44.000000000","message":"Sure!","commit_id":"02d9ae425dd5c100810b5ed7d2e2244d1cd7a722"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"1b4db4a66160a22b4beddbf88e896768224bb6fc","unresolved":true,"context_lines":[{"line_number":43,"context_line":"    def skip_checks(cls):"},{"line_number":44,"context_line":"        super(TestVolumeBackupRestore, cls).skip_checks()"},{"line_number":45,"context_line":"        if not CONF.volume_feature_enabled.backup:"},{"line_number":46,"context_line":"            raise cls.skipException(\u0027Backup is not enable.\u0027)"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"        if not CONF.volume_feature_enabled.multi_backend:"},{"line_number":49,"context_line":"            raise cls.skipException(\"Cinder multi-backend feature disabled\")"}],"source_content_type":"text/x-python","patch_set":12,"id":"784eb609_d306deb6","line":46,"range":{"start_line":46,"start_character":51,"end_line":46,"end_character":57},"updated":"2022-12-01 13:27:32.000000000","message":"typo: \"enabled\"","commit_id":"3e8bc14e6f698d798599ba62a6ff55667bdc6dd3"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"36d1da64b647fa84901fd2918bf4224e360d6470","unresolved":false,"context_lines":[{"line_number":43,"context_line":"    def skip_checks(cls):"},{"line_number":44,"context_line":"        super(TestVolumeBackupRestore, cls).skip_checks()"},{"line_number":45,"context_line":"        if not CONF.volume_feature_enabled.backup:"},{"line_number":46,"context_line":"            raise cls.skipException(\u0027Backup is not enable.\u0027)"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"        if not CONF.volume_feature_enabled.multi_backend:"},{"line_number":49,"context_line":"            raise cls.skipException(\"Cinder multi-backend feature disabled\")"}],"source_content_type":"text/x-python","patch_set":12,"id":"a767f258_811f585c","line":46,"range":{"start_line":46,"start_character":51,"end_line":46,"end_character":57},"in_reply_to":"784eb609_d306deb6","updated":"2022-12-01 13:41:45.000000000","message":"Done","commit_id":"3e8bc14e6f698d798599ba62a6ff55667bdc6dd3"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"1b4db4a66160a22b4beddbf88e896768224bb6fc","unresolved":true,"context_lines":[{"line_number":52,"context_line":"            raise cls.skipException(\"Requires at least two different \""},{"line_number":53,"context_line":"                                    \"backend names\")"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"    def _create_volume_types(self):"},{"line_number":56,"context_line":"        backend_names \u003d CONF.volume.backend_names"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"        backend_source \u003d backend_names[0]"}],"source_content_type":"text/x-python","patch_set":12,"id":"367343d7_8d2bb10a","line":55,"range":{"start_line":55,"start_character":8,"end_line":55,"end_character":28},"updated":"2022-12-01 13:27:32.000000000","message":"This method doesn\u0027t appear to create volume types -- would be good to rename it to match what it does.","commit_id":"3e8bc14e6f698d798599ba62a6ff55667bdc6dd3"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"36d1da64b647fa84901fd2918bf4224e360d6470","unresolved":false,"context_lines":[{"line_number":52,"context_line":"            raise cls.skipException(\"Requires at least two different \""},{"line_number":53,"context_line":"                                    \"backend names\")"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"    def _create_volume_types(self):"},{"line_number":56,"context_line":"        backend_names \u003d CONF.volume.backend_names"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"        backend_source \u003d backend_names[0]"}],"source_content_type":"text/x-python","patch_set":12,"id":"5a349efd_2464a9fb","line":55,"range":{"start_line":55,"start_character":8,"end_line":55,"end_character":28},"in_reply_to":"367343d7_8d2bb10a","updated":"2022-12-01 13:41:45.000000000","message":"Done","commit_id":"3e8bc14e6f698d798599ba62a6ff55667bdc6dd3"}],"requirements.txt":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"b9bbd5ad99a747d080274a5f78a29e0734b95ed2","unresolved":true,"context_lines":[{"line_number":5,"context_line":"pbr!\u003d2.1.0,\u003e\u003d2.0.0 # Apache-2.0"},{"line_number":6,"context_line":"oslo.config\u003e\u003d5.1.0 # Apache-2.0"},{"line_number":7,"context_line":"oslo.serialization!\u003d2.19.1,\u003e\u003d2.18.0 # Apache-2.0"},{"line_number":8,"context_line":"tempest\u003e\u003d27.0.0 # Apache-2.0"}],"source_content_type":"text/plain","patch_set":10,"id":"b72ae954_4cbaaf8d","line":8,"range":{"start_line":8,"start_character":9,"end_line":8,"end_character":15},"updated":"2022-06-29 16:55:06.000000000","message":"Should bump to 31.0.0 to include https://review.opendev.org/c/openstack/tempest/+/843542","commit_id":"807010948852edab04d0a2e7bbaf4e117449c749"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"04ee1a55c5b5be7666d47795a2a6ad7f580667c4","unresolved":false,"context_lines":[{"line_number":5,"context_line":"pbr!\u003d2.1.0,\u003e\u003d2.0.0 # Apache-2.0"},{"line_number":6,"context_line":"oslo.config\u003e\u003d5.1.0 # Apache-2.0"},{"line_number":7,"context_line":"oslo.serialization!\u003d2.19.1,\u003e\u003d2.18.0 # Apache-2.0"},{"line_number":8,"context_line":"tempest\u003e\u003d27.0.0 # Apache-2.0"}],"source_content_type":"text/plain","patch_set":10,"id":"5aaf302d_c7d87c02","line":8,"range":{"start_line":8,"start_character":9,"end_line":8,"end_character":15},"in_reply_to":"b72ae954_4cbaaf8d","updated":"2023-02-20 14:56:31.000000000","message":"Done","commit_id":"807010948852edab04d0a2e7bbaf4e117449c749"}]}