Fix locking contention for #1697 (#1712)

* Pull updating configs out of context * Update CHANGELOG.md
AgnostiqHQ · Jun 27, 2023 · 9a8e955 · 9a8e955
1 parent 7e7bf51
commit 9a8e955
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -16,6 +16,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 - File transfer strategy for GCP storage
 - Add CLI status for zombie, stopped process.
+- Fix for double locking file in configurations.
 
 ### Docs
 

diff --git a/covalent/_shared_files/config.py b/covalent/_shared_files/config.py
@@ -113,12 +113,12 @@ def update_nested_dict(old_dict, new_dict, override_existing: bool = True):
             fcntl.lockf(f, fcntl.LOCK_EX)
             file_config = toml.load(f)
 
-            update_nested_dict(self.config_data, file_config)
-            if new_entries:
-                update_nested_dict(self.config_data, new_entries, override_existing)
+        update_nested_dict(self.config_data, file_config)
+        if new_entries:
+            update_nested_dict(self.config_data, new_entries, override_existing)
 
-            # Writing it back to the file
-            self.write_config()
+        # Writing it back to the file
+        self.write_config()
 
     def read_config(self) -> None:
         """