xref: /openbmc/bmcweb/REDFISH_CHECKLIST.md (revision c3805d43)
1When making changes to the Redfish tree, please follow the checklist below.
2
3If any of these are incomplete when pushing a patch to gerrit, either mark the
4patch WIP (Work in progress), or explicitly include the items from the checklist
5that have not been completed.
6
71. Verify the documentation in Redfish.md includes all properties and schemas
8   that have been added or are used in the commit.
9
102. Document any client-facing changes in behavior in the commit message. Note,
11   that this should include behavior changes that may not affect your system.
12
133. Verify that any additional properties exist in the CSDL schema in
14   static/redfish/v1/odata. DO NOT rely solely on the JSON schema files, as they
15   are less restrictive than odata.
16
174. Document in the commit message what these property additions are used for.
18   Verify that the @odata.type on the schema on which you've added functionality
19   has a version that includes the parameters you've added
20
215. Verify that testing was performed per TESTING.md and it is documented in my
22   commit message. Redfish-Service-Validator is the MINIMUM required for any
23   Redfish change. Most changes require more tests than simply service
24   validator. If testing was performed on a previous commit in the series, but
25   not the present commit, note that explicitly with "Tested on prior commit".
26
276. Ensure that you have written unit tests for any behavior that does not have
28   external dependencies (DBus).
29
307. Read COMMON_ERRORS.md, and verify that to the best of your knowledge no
31   common errors are present in the patchset.
32
338. Verify that all DBus usages match phosphor-dbus-interfaces, or include a link
34   to the gerrit review for phosphor-dbus-interfaces in the commit message.
35