commit ffa488f926f544ad38875d869b30a4b91fe806e7
author Anupama B R <anupama.b.r1@ibm.com> Thu May 22 07:34:00 2025 -0500
committer SunnySrivastava <sunnsr25@in.ibm.com> Mon Jun 02 14:17:58 2025 +0000
tree 2e677c312167d955f2249c385ff2f260c5db1340
parent 24691d2d4acfbdc96896593629b13b46e096d4b6

Update CollectionStatus on FRU deletion

This commit updates the CollectionStatus of the FRU to its default
value “NotStarted”, when resetDataUnderPIM API is triggered.

deleteFRUVPD method internally calls resetDataUnderPIM API, hence the
default CollectionStatus will be reflected for all the sub FRUs after
completion of the deletion.

Output:
```
root@p10bmc:~# busctl call  com.ibm.VPD.Manager /com/ibm/VPD/Manager com.ibm.VPD.Manager deleteFRUVPD o /xyz/openbmc_project/inventory/system/chassis/motherboard/dimm2

root@p10bmc:~# busctl get-property  xyz.openbmc_project.Inventory.Manager /xyz/openbmc_project/inventory/system/chassis/motherboard/dimm2 com.ibm.VPD.Collection  CollectionStatus
s "com.ibm.VPD.Collection.Status.NotStarted"

root@p10bmc:~# busctl get-property  xyz.openbmc_project.Inventory.Manager /xyz/openbmc_project/inventory/system/chassis/motherboard/dimm2/unit0 com.ibm.VPD.Collection  CollectionStatus
s "com.ibm.VPD.Collection.Status.NotStarted"

root@p10bmc:~# busctl get-property  xyz.openbmc_project.Inventory.Manager /xyz/openbmc_project/inventory/system/chassis/motherboard/dimm2/unit1 com.ibm.VPD.Collection  CollectionStatus
s "com.ibm.VPD.Collection.Status.NotStarted"

root@p10bmc:~# busctl get-property  xyz.openbmc_project.Inventory.Manager /xyz/openbmc_project/inventory/system/chassis/motherboard/dimm2/unit2 com.ibm.VPD.Collection  CollectionStatus
s "com.ibm.VPD.Collection.Status.NotStarted"

root@p10bmc:~# busctl get-property  xyz.openbmc_project.Inventory.Manager /xyz/openbmc_project/inventory/system/chassis/motherboard/dimm2/unit3 com.ibm.VPD.Collection  CollectionStatus
s "com.ibm.VPD.Collection.Status.NotStarted"
```

Change-Id: I8d329754553de78fb8463493322c49a63b16120e
Signed-off-by: Anupama B R <anupama.b.r1@ibm.com>

vpd-manager/include/utility/vpd_specific_utility.hpp

diff --git a/vpd-manager/include/utility/vpd_specific_utility.hpp b/vpd-manager/include/utility/vpd_specific_utility.hpp
index 8493730..ced7ef1 100644
--- a/vpd-manager/include/utility/vpd_specific_utility.hpp
+++ b/vpd-manager/include/utility/vpd_specific_utility.hpp
@@ -583,7 +583,7 @@
 
         const std::vector<std::string>& l_vpdRelatedInterfaces{
             constants::operationalStatusInf, constants::inventoryItemInf,
-            constants::assetInf};
+            constants::assetInf, constants::vpdCollectionInterface};
 
         for (const auto& [l_service, l_interfaceList] : l_getObjectMap)
         {
@@ -621,8 +621,18 @@
                         else if (std::holds_alternative<std::string>(
                                      l_propertyValue))
                         {
-                            l_propertyMap.emplace(l_propertyName,
-                                                  std::string{});
+                            if (l_propertyName.compare("CollectionStatus") ==
+                                constants::STR_CMP_SUCCESS)
+                            {
+                                l_propertyMap.emplace(
+                                    l_propertyName,
+                                    constants::vpdCollectionNotStarted);
+                            }
+                            else
+                            {
+                                l_propertyMap.emplace(l_propertyName,
+                                                      std::string{});
+                            }
                         }
                         else if (std::holds_alternative<bool>(l_propertyValue))
                         {