Merge pull request #119 from Cray-HPE/casmcms-8962-csm-1.4

CASMCMS-8962: Properly handle response from DB.get_all
Cray-HPE · Apr 5, 2024 · 0182d06 · 0182d06
2 parents b9331fb + 375754a
commit 0182d06
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -6,6 +6,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [1.12.7] - 4/5/2024
+### Fixed
+- Fix bug in `patch_v2_components_dict` to properly handle response from `DB.get_all()`
+
 ## [1.12.6] - 4/5/2024
 ### Fixed
 - Fix bug in `patch_v2_components_dict` that causes it to only patch a single component.

diff --git a/src/server/cray/cfs/api/controllers/components.py b/src/server/cray/cfs/api/controllers/components.py
@@ -222,7 +222,7 @@ def patch_v2_components_dict(data):
     else:
         # TODO: On large scale systems, this response may be too large
         # and require paging to be implemented
-        components = DB.get_all()
+        components = [ (component_data["id"], component_data) for component_data in DB.get_all() ]
 
     response = []
     patch = data.get("patch", {})