pylxd/models: Adapt StorageVolumes to use old API versions

Signed-off-by: hamistao <[email protected]>

pylxd/models/storage_pool.py

@@ -701,6 +701,10 @@ def get(cls, volume, name):
         # This field is normally empty so derive it from its volume.
         snapshot.content_type = volume.content_type
 
+        # When dealing with older API version the response parsing can be messy, so we set those values to None.
+        if not isinstance(snapshot.created_at, str):
+            snapshot.created_at = None
+
         # Snapshot names are namespaced in LXD, as volume-name/snapshot-name.
         # We hide that implementation detail.
         snapshot.name = snapshot.name.split("/")[-1]
@@ -737,6 +741,12 @@ def parse_response_item(snapshot_json):
                 # We hide that implementation detail.
                 snapshot_object.name = snapshot_object.name.split("/")[-1]
 
+                # When dealing with older API versions, the response parsing can be messy, so we manually set those values to None.
+                if not isinstance(snapshot_object.created_at, str):
+                    snapshot_object.created_at = None
+                if snapshot_json["expires_at"] in (None, "0001-01-01T00:00:00Z"):
+                    snapshot_object.expires_at = None
+
                 return snapshot_object
 
             response = volume.api.snapshots.get(params={"recursion": 1})