Merge pull request #237 from stacklok/fix_local_backup

fix: deal with weaviate multiple network issues

Author: yrobla

README.md

@@ -123,6 +123,19 @@ source venv/bin/activate  # On Windows: venv\Scripts\activate
 pip install -e ".[dev]"
 ```
 
+### Running locally with several network interfaces
+
+By default weaviate is picking the default route as the ip for the cluster nodes. It may cause
+some issues when dealing with multiple interfaces. To make it work, localhost needs to be the
+default route:
+
+```bash
+sudo route delete default
+sudo route add default 127.0.0.1
+sudo route add -net 0.0.0.0/1 <public_ip_gateway>
+sudo route add -net 128.0.0.0/1 <public_ip_gateway>
+```
+
 ### Testing
 ```bash
 pytest

scripts/import_packages.py

@@ -3,6 +3,7 @@
 import os
 import shutil
 
+
 import weaviate
 from weaviate.classes.config import DataType, Property
 from weaviate.embedded import EmbeddedOptions
@@ -53,11 +54,14 @@ def take_backup(self):
             shutil.rmtree(backup_path)
 
         #  take a backup of the data
-        self.client.backup.create(
-            backup_id=os.getenv("BACKUP_TARGET_ID", "backup"),
-            backend="filesystem",
-            wait_for_completion=True,
-        )
+        try:
+            self.client.backup.create(
+                backup_id=os.getenv("BACKUP_TARGET_ID", "backup"),
+                backend="filesystem",
+                wait_for_completion=True,
+            )
+        except Exception as e:
+            print(f"Failed to take backup: {e}")
 
     def setup_schema(self):
         if not self.client.collections.exists("Package"):