This is a place to store API specifications and schemas for the Host Inventory. Currently it is used only for System Profile.
This is an initial draft and any technical details may change over time. Published early to allow work and discussion on the actual System Profile shape.
System Profile specification is written in YAML containing an OpenAPI 3.0 definition schema. All entities are under $def root key, internal references ($ref) use relative referencing (#/$def/entity).
Document version is reflected in the filename (v1.yaml) and under $version root key. $id is a file name used in the Host Inventory.
When contributing a new field to the system_profile schema, please ensure you complete the following steps:
- Add the new field
- Annotate the field
- Add an example of the value(s) you expect to receive using the
example
keyword. For string fields, provide at least 2 unique examples. - Add a description of the field. If the field should support
range
orwildcard
operations when queried against, note that here.
- Add an example of the value(s) you expect to receive using the
- Add filtering flags
- If the field should support wildcard operations in filtering, add
x-wildcard: true
. Defaults tofalse
otherwise.
- If the field should support wildcard operations in filtering, add
- Validate the field
- The field should have the strictest possible validation rules applied to it.
- Add positive and negative test examples
- Add examples of valid/invalid values in
tests/utils/valids
andtests/utils/invalids
respectively.
- Add examples of valid/invalid values in
- Mirror the changes in HBI
- Open another PR against
insights-host-inventory
and reflect the schema changes there.
- Open another PR against
-
Once a pull request is opened to update a pre-existing field and all the requested changes are resolved, someone from RedHatInsights/host-based-inventory-comitters team should open a merge request with App-SRE to query the database for potential impact to the HBI reporters.
-
The repository maintainers will analyze the data returned from the App-SRE job, produce a report similar to to the following, and post it in the pull request thread.
Reporters Result Count ingress Pass 10 ingress Fail 2 puptoo Pass 37 puptoo Fail 42 -
If the report seems satisfactory and there are no more concerns, a repository maintainer will merge the PR.
-
In the event that the report is unsatisfactory (e.g. it shows a high number of failures from one or multiple reporters), the PR owner must coordinate with the repository maintainers and the stakeholders from the reporter(s) that show failures. Together, they must decide whether to change the PR or to update the reporter(s).
-
If the pull request only adds new fields there is no need to generate a report since the database will not contain values to compare against for the new fields. A repository maintainer will help in getting consensus with the schema Stakeholders before merging the PR.
-
Once the PR is merged, the HBI team member who merged it can open another PR in the HBI repository to reflect those changes.
- RHSM - Kevin Howell
- Yupana - Jaylin Zhou, Xiangce Liu
- Puptoo - Jaylin Zhou, Xiangce Liu
- BU - Jerome Marc