Validator: Warning: Oem.OpenBmc.Certificates not defined in schema OemManager

[gtmills]

From the validator:
WARNING - Oem.OpenBmc.Certificates not defined in schema OemManager (check version, spelling and casing)

https://github.com/openbmc/bmcweb/blob/master/redfish-core/lib/managers.hpp#L1627

  "Oem": {
    "@odata.context": "/redfish/v1/$metadata#OemManager.Oem",
    "@odata.id": "/redfish/v1/Managers/bmc#/Oem",
    "@odata.type": "#OemManager.Oem",
    "OpenBmc": {
      "@odata.context": "/redfish/v1/$metadata#OemManager.OpenBmc",
      "@odata.id": "/redfish/v1/Managers/bmc#/Oem/OpenBmc",
      "@odata.type": "#OemManager.OpenBmc",
      "Certificates": {
        "@odata.id": "/redfish/v1/Managers/bmc/Truststore/Certificates"
      }

But don't see Certificates defined in the OemManager schema.
https://github.com/openbmc/bmcweb/blob/master/static/redfish/v1/schema/OemManager_v1.xml
https://github.com/openbmc/bmcweb/blob/master/static/redfish/v1/JsonSchemas/OemManager/index.json

[feistjj]

cfcd5f6 introduced this issue, @devenrao can you please fix this issue?

+        oemOpenbmc["Certificates"] = {
+            {"@odata.id", "/redfish/v1/Managers/bmc/Truststore/Certificates"}};

[devenrao]

cfcd5f6 introduced this issue, @devenrao can you please fix this issue?

+        oemOpenbmc["Certificates"] = {
+            {"@odata.id", "/redfish/v1/Managers/bmc/Truststore/Certificates"}};

sure will take a look

[edtanous]

Can this be closed?

[gtmills]

This is still an issue.
at Managers/bmc I see

 "Oem": {
    "@odata.context": "/redfish/v1/$metadata#OemManager.Oem",
    "@odata.id": "/redfish/v1/Managers/bmc#/Oem",
    "@odata.type": "#OemManager.Oem",
    "OpenBmc": {
      "@odata.context": "/redfish/v1/$metadata#OemManager.OpenBmc",
      "@odata.id": "/redfish/v1/Managers/bmc#/Oem/OpenBmc",
      "@odata.type": "#OemManager.OpenBmc",
      "Certificates": {
        "@odata.id": "/redfish/v1/Managers/bmc/Truststore/Certificates"
      }
    }
  },

but nothing in https://github.com/openbmc/bmcweb/blob/master/static/redfish/v1/schema/OemManager_v1.xml for certificates. @devenrao Can you have a look?

[matthewfischer]

This has since been renamed to OpenBMCManaged by commit fc1cdd1

I am not sure if the issue still exists or not.

[gtmills]

Still a problem

bmcweb/Redfish.md

Line 699 in daadfb2

### /redfish/v1/Managers/bmc/Truststore/Certificates/

But /redfish/v1/Managers/$managerId/Truststore/Certificates isn't an allowed location
https://redfish.dmtf.org/schemas/v1/Certificate_v1.xml

https://gerrit.openbmc.org/c/openbmc/bmcweb/+/61958 was an attempt to fix this