)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"437b3612ea757c42ef0d6a1feeada9ef89ee6397","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9c06fcc0_0a2bd8cc","updated":"2021-11-19 16:56:42.000000000","message":"I\u0027d like to see more reviews on this before +2.. maybe we could just add `*args, **kwargs` to the abstract methods. \nJust for the sake of completeness the related classes that use this are:\n- https://opendev.org/openstack/cinder/src/branch/master/cinder/image/accelerators/gzip.py#L44\n- https://opendev.org/openstack/cinder/src/branch/master/cinder/image/accelerators/qat.py#L45\n","commit_id":"c1222fc00879e90e37efaa22268521ddf1d65b9c"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"a7ea2fa5bf524b6083efbbdf89d719adb89b96e1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"53b7be28_2013fe10","updated":"2021-11-19 18:12:00.000000000","message":"Thanks for digging up the references, Sofia.  I think we do want the method signatures in the abstract base class to match what\u0027s actually being used by the subclasses.  Also, since the abstract class is being used to define the interface, I don\u0027t think we want to allow arbitrary args and kwargs there.  So I\u0027m OK with this change.","commit_id":"c1222fc00879e90e37efaa22268521ddf1d65b9c"}]}
