commit f797c6a6abe0917662ab6d742273b2831863eea4
author Patrick Williams <patrick@stwcx.xyz> Fri Jul 16 12:29:59 2021 -0500
committer Patrick Williams <patrick@stwcx.xyz> Tue Jul 20 18:18:16 2021 +0000
tree a3b86ca784c14d091b0c15269e30027939f2e8f3
parent 4017f2a8032a5a7265a58c23400226301d89d18e

treewide: inventory: switch ChassisType to future enum value

Change Ie1a7c389edb6b7a048836a49283ceb62de51bba5 will be transitioning
the 'Type' property in Inventory.Item.Chassis to an enumeration.
In order to avoid crashing in PIM we need these default values in the
starting YAML to be fully-qualified enumeration names that will match
the values which will start in Ie1a7c.

This code is safe to merge as is without any PDI or PIM changes.  Prior
to the PDI change, we must make a change to PIM that allows conversion
automatic from string->enum, leveraging library interfaces available in
sdbusplus.  These will be submitted independently.

I checked the codebase for usage of this string.  It appears that the
value is currently, effectively, write-only.  There is code in bmcweb
that fills in the equivalent Redfish value but currently just hard-codes
the string 'RackMount'.

Tested: Booted Witherspoon in a QEMU model with this change and proposed
changes to sdbusplus + PIM.  PIM no longer coredumps with the PDI change
and yields an expected persistence file:

```
$ pwd
/var/lib/phosphor-inventory-manager/xyz/openbmc_project/inventory/system/chassis
$ cat xyz.openbmc_project.Inventory.Item.Chassis
{
    "value0": {
        "cereal_class_version": 2,
        "Type": 3
    }
}
```

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Icaf9447f31ccdd945cdf74b3e017682e4aed686f