[DPE-8473] Handle invalid database name (databases) (#1190)

* Update lib to handle new invalid database name and to handle storage ownership and permissions

Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>

* Update lib

Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>

* Update lib

Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>

* Update lib

Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>

* Update lib

Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>

* Update lib

Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>

* Update lib

Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>

---------

Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>

Author: marceloneppel

poetry.lock

@@ -64,6 +64,7 @@
     REWIND_USER,
     SYSTEM_USERS,
     USER,
+    Substrates,
 )
 from single_kernel_postgresql.utils.postgresql import (
     ACCESS_GROUP_IDENTITY,
@@ -79,7 +80,6 @@
     PostgreSQLGrantDatabasePrivilegesToUserError,
     PostgreSQLListUsersError,
     PostgreSQLUpdateUserPasswordError,
-    Substrates,
 )
 from tenacity import RetryError, Retrying, retry, stop_after_attempt, stop_after_delay, wait_fixed
 

src/charm.py

@@ -17,6 +17,7 @@
     ACCESS_GROUP_RELATION,
     ACCESS_GROUPS,
     INVALID_DATABASE_NAME_BLOCKING_MESSAGE,
+    INVALID_DATABASE_NAMES,
     INVALID_EXTRA_USER_ROLE_BLOCKING_MESSAGE,
     PostgreSQLBaseError,
     PostgreSQLCreateDatabaseError,
@@ -417,7 +418,7 @@ def check_for_invalid_database_name(self, relation_id: int) -> bool:
             for data in relation.data.values():
                 database = data.get("database")
                 if database is not None and (
-                    len(database) > 49 or database in ["postgres", "template0", "template1"]
+                    len(database) > 49 or database in INVALID_DATABASE_NAMES
                 ):
                     return True
         return False

src/relations/postgresql_provider.py

@@ -66,6 +66,7 @@ def test_database(juju: jubilant.Juju, predefined_roles) -> None:
     """Check that an invalid database name makes the database charm block."""
     del predefined_roles[""]
     invalid_database_names = [
+        "databases",
         "postgres",
         "template0",
         "template1",