Add default 15 second timeout to pebble exec (#394)

Fixes #393

Author: carlcsaposs-canonical

src/container.py

@@ -228,14 +228,11 @@ def set_mysql_router_rest_api_password(
         self.create_router_rest_api_credentials_file()
 
         if not password:
-            users_credentials = self._run_command(
-                [
-                    self._mysql_router_password_command,
-                    "list",
-                    str(self.rest_api_credentials_file),
-                ],
-                timeout=30,
-            )
+            users_credentials = self._run_command([
+                self._mysql_router_password_command,
+                "list",
+                str(self.rest_api_credentials_file),
+            ])
             if user not in users_credentials:
                 return
 
@@ -248,5 +245,4 @@ def set_mysql_router_rest_api_password(
                 user,
             ],
             input=password,
-            timeout=30,
         )

src/rock.py

@@ -235,7 +235,7 @@ def _run_command(
         self,
         command: typing.List[str],
         *,
-        timeout: typing.Optional[int],
+        timeout: typing.Optional[int] = 15,
         input: str = None,  # noqa: A002 Match subprocess.run()
     ) -> str:
         try:

src/workload.py

@@ -266,7 +266,7 @@ def _bootstrap_router(self, *, event, tls: bool) -> None:
 
         command = self._get_bootstrap_command(event=event, connection_info=self._connection_info)
         try:
-            self._container.run_mysql_router(command, timeout=30)
+            self._container.run_mysql_router(command)
         except container.CalledProcessError as e:
             # Original exception contains password
             # Re-raising would log the password to Juju's debug log