)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"cbbb5b1a480708af891936b750e7b1b8cdd90a7d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"0f70375d_8ca7905f","updated":"2022-01-28 13:06:28.000000000","message":"recheck","commit_id":"5eb69ae87f04d7c7d8f7cbe7effa5afd8843c59d"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"fcf842c1a5751f46763a4852d9c218647cbd7df8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9d2ba217_167b2d17","updated":"2022-01-27 18:20:17.000000000","message":"recheck","commit_id":"5eb69ae87f04d7c7d8f7cbe7effa5afd8843c59d"}],"cinder_tempest_plugin/scenario/manager.py":[{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"cb5c3c95cfa16244b5c542cd96c9e9787601f7ea","unresolved":true,"context_lines":[{"line_number":214,"context_line":"            pass"},{"line_number":215,"context_line":""},{"line_number":216,"context_line":""},{"line_number":217,"context_line":"class EncryptionScenarioTest(manager.EncryptionScenarioTest):"},{"line_number":218,"context_line":"    \"\"\"Base class for encryption scenario tests\"\"\""},{"line_number":219,"context_line":""},{"line_number":220,"context_line":"    def create_encrypted_volume_from_image(self, encryption_provider,"}],"source_content_type":"text/x-python","patch_set":2,"id":"7cb5ef14_fba6ee5b","line":217,"updated":"2022-03-18 15:04:23.000000000","message":"If we do this, don\u0027t we lose all methods defined inside \ncinder_tempest_plugin.scenario.manager.ScenarioTest?","commit_id":"5eb69ae87f04d7c7d8f7cbe7effa5afd8843c59d"},{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"b8c8326a388e6853d9261b58bd8375ec0c04ddcc","unresolved":true,"context_lines":[{"line_number":214,"context_line":"            pass"},{"line_number":215,"context_line":""},{"line_number":216,"context_line":""},{"line_number":217,"context_line":"class EncryptionScenarioTest(manager.EncryptionScenarioTest):"},{"line_number":218,"context_line":"    \"\"\"Base class for encryption scenario tests\"\"\""},{"line_number":219,"context_line":""},{"line_number":220,"context_line":"    def create_encrypted_volume_from_image(self, encryption_provider,"}],"source_content_type":"text/x-python","patch_set":2,"id":"3ca4dd48_24778f78","line":217,"in_reply_to":"7cb5ef14_fba6ee5b","updated":"2022-03-18 17:01:17.000000000","message":"you can always inherit from both  ScenarioTest and or EncryptionScenarioTest\ni personally prfer to inherits from a single parent and only when no other option add multiple inheritence.\n\ncinder_tempest_plugin manager inherits from tempest scenario , a middle level is the classed here. the testcase class should inherit from cinder_tempest manager and not from tempest directly.","commit_id":"5eb69ae87f04d7c7d8f7cbe7effa5afd8843c59d"}]}