Continued:

- update instance when error has happened (ops, missed this)
- also output warning when fields list is empty

diff --git a/fba/commands.py b/fba/commands.py
index 45bfbd751e56e23334b890f69974e143ed0e9b19..87150617a036603a02699bbdac2af7df9bd4c21a 100644 (file)
--- a/fba/commands.py
+++ b/fba/commands.py
@@ -1075,6 +1075,7 @@ ORDER BY total_peers DESC, last_response_time ASC, last_updated ASC"
         except network.exceptions as exception:
             logger.warning("Exception '%s' during fetching instances (fetch_instances) from domain='%s'", type(exception), domain)
             instances.set_last_error(domain, exception)
+            instances.update(domain)
 
     logger.debug("Success - EXIT!")
     return 0

diff --git a/fba/models/instances.py b/fba/models/instances.py
index 788f4030c5133ae58c2edffc68f9a4dcff8ef888..ff3c0fcbffc6ae00c2193eddd1daab0805ed6444 100644 (file)
--- a/fba/models/instances.py
+++ b/fba/models/instances.py
@@ -140,7 +140,7 @@ def update(domain: str):
             sql_string += f" {key} = ?,"
 
     logger.debug("sql_string(%d)='%s'", len(sql_string), sql_string)
-    if sql_string == "":
+    if sql_string == "" or len(fields) == 0:
         raise ValueError(f"No fields have been set, but function invoked, domain='{domain}'")
 
     # Set last_updated to current timestamp