Fix REST and schema registry shutdown

Shutdown handler for aiohttp Application was set only for
KarapaceAll. On some environments where REST application and
schema registry pplication is run separately, e.g. containers,
the shutdown times out and container manager sends kill signal.

Fix is to move the shutdown handler registration from KarapaceAll
to KarapaceBase.

karapace/karapace.py

@@ -39,6 +39,11 @@ def __init__(self, config: dict) -> None:
         self.app.on_startup.append(self.create_http_client)
         self.master_lock = asyncio.Lock()
         self.log.info("Karapace initialized")
+        self.app.on_shutdown.append(self.close_by_app)
+
+    async def close_by_app(self, app):
+        # pylint: disable=unused-argument
+        await self.close()
 
     def _create_producer(self) -> KafkaProducer:
         while True:

karapace/karapace_all.py

@@ -15,11 +15,6 @@ class KarapaceAll(KafkaRest, KarapaceSchemaRegistry):
     def __init__(self, config: dict) -> None:
         super().__init__(config=config)
         self.log = logging.getLogger("KarapaceAll")
-        self.app.on_shutdown.append(self.close_by_app)
-
-    async def close_by_app(self, app):
-        # pylint: disable=unused-argument
-        await self.close()
 
 
 def main() -> int: