deprecation warnings

Author: pcrespov

packages/postgres-database/src/simcore_postgres_database/aiopg_errors.py

@@ -21,6 +21,8 @@
     SEE https://www.postgresql.org/docs/current/errcodes-appendix.html
 """
 
+import warnings
+
 # NOTE: psycopg2.errors are created dynamically
 # pylint: disable=no-name-in-module
 from psycopg2 import (
@@ -46,6 +48,19 @@
 
 assert issubclass(UniqueViolation, IntegrityError)  # nosec
 
+
+warnings.warn(
+    (
+        "DEPRECATED: The aiopg DBAPI exceptions in this module are no longer used. "
+        "Please use exceptions from the `sqlalchemy.exc` module instead. "
+        "See https://docs.sqlalchemy.org/en/21/core/exceptions.html for details. "
+        "This change is part of the migration to SQLAlchemy async support with asyncpg. "
+        "See migration issue: https://github.com/ITISFoundation/osparc-simcore/issues/4529"
+    ),
+    DeprecationWarning,
+    stacklevel=2,
+)
+
 __all__: tuple[str, ...] = (
     "CheckViolation",
     "DatabaseError",

services/web/server/src/simcore_service_webserver/db/_aiopg.py

@@ -5,6 +5,7 @@
 """
 
 import logging
+import warnings
 from collections.abc import AsyncIterator
 from typing import Any, cast
 
@@ -29,6 +30,16 @@
 
 _logger = logging.getLogger(__name__)
 
+warnings.warn(
+    (
+        "simcore_service_webserver.db._aiopg is deprecated and will be removed in a future release. "
+        "Please use simcore_service_webserver.db._asyncpg instead. "
+        "See migration details: https://github.com/ITISFoundation/osparc-simcore/issues/4529"
+    ),
+    DeprecationWarning,
+    stacklevel=2,
+)
+
 
 @retry(**PostgresRetryPolicyUponInitialization(_logger).kwargs)
 async def _ensure_pg_ready(

services/web/server/src/simcore_service_webserver/db/plugin.py

@@ -35,7 +35,7 @@
 )
 def setup_db(app: web.Application):
 
-    # ensures keys exist
+    # ensures keys exist DEPRECATED
     app[APP_AIOPG_ENGINE_KEY] = None
     assert get_database_engine_legacy(app) is None  # nosec