fix: Support importing multiple Bentos with shared models (#5543)

Fixes #5538

Signed-off-by: Frost Ming <me@frostming.com>

Author: frostming

src/bentoml/_internal/bento/bento.py

@@ -27,6 +27,7 @@
 from ..configuration import BENTOML_VERSION
 from ..configuration.containers import BentoMLContainer
 from ..models import ModelStore
+from ..models import copy_model
 from ..runner import Runner
 from ..store import Store
 from ..store import StoreItem
@@ -587,7 +588,16 @@ def save(
             if self._model_store is not None:
                 # Move models to the global model store, if any
                 for model in self._model_store.list():
-                    model.save(model_store)
+                    copied = copy_model(
+                        model.tag,
+                        src_model_store=self._model_store,
+                        target_model_store=model_store,
+                    )
+                    if not copied:
+                        logger.warning(
+                            "Model %s already exists in the global model store, skipping copy.",
+                            model.tag,
+                        )
 
             def ignore_models(src_path: str, names: list[str]) -> t.Iterable[str]:
                 if Path(src_path) == self._path and "models" in names:

src/bentoml/_internal/models/model.py

@@ -700,18 +700,20 @@ def copy_model(
     *,
     src_model_store: ModelStore,
     target_model_store: ModelStore,
-):
+) -> bool:
     """copy a model from src model store to target modelstore, and do nothing if the
-    model tag already exist in target model store
+    model tag already exist in target model store.
+    Returns True if the model is copied, False if the model already exists in target model store.
     """
     try:
         target_model_store.get(model_tag)  # if model tag already found in target
-        return
+        return False
     except NotFound:
         pass
 
     model = src_model_store.get(model_tag)
     model.save(target_model_store)
+    return True
 
 
 def _ModelInfo_dumper(dumper: yaml.Dumper, info: ModelInfo) -> yaml.Node: