PEL: Registry: Make "Action Flags" optional
The Action Flags bits in the PEL need to be in agreement with other PEL
fields, such as if the severity is informational then the 'report' flag
cannot be on. In fact, for most logs all of these flags can be deduced
from the other PEL fields.
So, to avoid the case of the action flags in the registry not matching
the other PEL fields, make them optional and the BMC code will set them
appropriately. And when they are specified, the BMC code will still run
through the same code to ensure everything is in agreement.
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: I5bea83d01846d7a2d3307666e6d0758a99229e3f
diff --git a/extensions/openpower-pels/registry/README.md b/extensions/openpower-pels/registry/README.md
index dae6bbc..a0e6f49 100644
--- a/extensions/openpower-pels/registry/README.md
+++ b/extensions/openpower-pels/registry/README.md
@@ -80,7 +80,8 @@
### Event Type
This field is part of the PEL User Header section, and is used to specify
the event type, as defined by the PEL spec. It is optional and defaults to
-"not applicable".
+"not applicable" for non-informational logs, and "misc_information_only" for
+informational ones.
```
"EventType": "na"
@@ -90,6 +91,13 @@
This field is part of the PEL User Header section, and is used to specify the
PEL action flags, as defined by the PEL spec. It is an array of enumerations.
+The action flags can usually be deduced from other PEL fields, such as the
+severity or if there are any callouts. As such, this is an optional field and
+if not supplied the code will fill them in based on those fields.
+
+In fact, even if supplied here, the code may still modify them to ensure they
+are correct.
+
```
"ActionFlags": ["service_action", "report", "call_home"]
```
diff --git a/extensions/openpower-pels/registry/schema/schema.json b/extensions/openpower-pels/registry/schema/schema.json
index 3e94cd4..10fdff9 100644
--- a/extensions/openpower-pels/registry/schema/schema.json
+++ b/extensions/openpower-pels/registry/schema/schema.json
@@ -52,8 +52,7 @@
"ComponentID": {"$ref": "#/definitions/componentID" }
},
- "required": ["Name", "SRC", "Subsystem", "Documentation",
- "ActionFlags"],
+ "required": ["Name", "SRC", "Subsystem", "Documentation"],
"additionalProperties": false
}
},