fix: use the same spinner for bento and model apis (#5161)

Signed-off-by: Frost Ming <[email protected]>

Author: frostming

src/bentoml/_internal/cloud/__init__.py

@@ -4,6 +4,7 @@
 
 from bentoml._internal.cloud.client import RestApiClient
 
+from .base import Spinner
 from .bento import BentoAPI
 from .config import DEFAULT_ENDPOINT
 from .config import CloudClientConfig
@@ -50,8 +51,9 @@ def __init__(
             endpoint = ctx.endpoint
 
         client = RestApiClient(endpoint, api_key, timeout)
-        bento = BentoAPI(client)
-        model = ModelAPI(client)
+        spinner = Spinner()
+        bento = BentoAPI(client, spinner=spinner)
+        model = ModelAPI(client, spinner=spinner)
         deployment = DeploymentAPI(client)
         secret = SecretAPI(client)
 

src/bentoml/_internal/cloud/base.py

@@ -119,6 +119,7 @@ def __init__(self, console: Console | None = None) -> None:
         )
         self._spinner_task_id: t.Optional[TaskID] = None
         self._live = Live(self, console=self.console)
+        self._start_count = 0
 
     @contextmanager
     def spin(self, text: str) -> t.Generator[TaskID, None, None]:
@@ -150,10 +151,14 @@ def __rich_console__(
 
     def start(self) -> None:
         """Start live updating."""
+        self._start_count += 1
         self._live.start()
 
     def stop(self) -> None:
         """Stop live updating."""
+        self._start_count -= 1
+        if self._start_count > 0:
+            return
         if self._spinner_task_id is not None:
             self._spinner_progress.remove_task(self._spinner_task_id)
             self._spinner_task_id = None