Merge pull request #180 from eth-cscs/Fix-162-transferDirectives-serialization

Fix 162 transfer directives serialization

Author: theely

CHANGELOG.md

@@ -14,6 +14,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Changed
 
 ### Fixed
+- ***⚠️ API Breaking*** Fix transfer directives serialization, now properties names are properly camelcased (see issue: #162).
 
 ## [2.4.2]
 

f7t-api-config.local-env.yaml

@@ -51,6 +51,8 @@ clusters:
         timeout: 20
       filesystems:
         timeout: 20
+      ssh:
+        timeout: 5
   datatransfer_jobs_directives:
     - "#SBATCH --constraint=mc"
     - "#SBATCH --nodes=1"
@@ -85,6 +87,8 @@ clusters:
         timeout: 5
       filesystems:
         timeout: 5
+      scheduler:
+        timeout: 20
   datatransfer_jobs_directives:
     - "#SBATCH --nodes=1"
     - "#SBATCH --time=0-00:15:00"

src/lib/datatransfers/datatransfer_base.py

@@ -70,13 +70,6 @@ class DataTransferOperation(CamelModel):
         discriminator="transfer_method",
     )
 
-    class Config:
-        json_encoders = {
-            DataTransferDirective: lambda d: {
-                k: v for k, v in d.__dict__.items() if v is not None
-            }
-        }
-
 
 class JobHelper:
     job_param = None