1ceeba4f9SEd Tanous# Below lists the current bmcweb maintainers. bmcweb is used in a number of 2ceeba4f9SEd Tanous# different contexts, and is one of the few nearly-universally used core 3ceeba4f9SEd Tanous# components in OpenBMC. As such, given the severe consequences of mistakes 4ceeba4f9SEd Tanous# made within the codebase, maintainers on this list are expected to: 5ceeba4f9SEd Tanous# - Have a solid understanding of the bmcweb core code, and how it's used. 6ceeba4f9SEd Tanous# 7ceeba4f9SEd Tanous# - Have access to at least one upstream platform to test relevant patchsets. 8ceeba4f9SEd Tanous# 9ceeba4f9SEd Tanous# - Help to manage the orderly merging of patchsets onto master through review. 10ceeba4f9SEd Tanous# It is expected that bmcweb maintainers participate on a majority of code 11ceeba4f9SEd Tanous# reviews, and although maintainers may work with each other to segment the 12ceeba4f9SEd Tanous# responsibilities into sub-parts the codebase, it is expected that maintainers 13ceeba4f9SEd Tanous# should be capable of reviewing all code in all modules if the need arises. 14ceeba4f9SEd Tanous# 15ceeba4f9SEd Tanous# - Provide help in testing and triage of cross-platform issues that arise as a 16ceeba4f9SEd Tanous# result of merging new features. 17ceeba4f9SEd Tanous# 18ceeba4f9SEd Tanous# - Have an in-depth understanding of the Redfish standard, its constraints in 19ceeba4f9SEd Tanous# how it interacts with OpenBMC, and how the bmcweb implementation compares to 20ceeba4f9SEd Tanous# other Redfish instances and how changes effect compatibility with other 21ceeba4f9SEd Tanous# Redfish services compatibility. 22ceeba4f9SEd Tanous# 23ceeba4f9SEd Tanous# - Be capable of, and have a track record of posing questions, clarifications, 24ceeba4f9SEd Tanous# and specification changes to [DMTF](https://www.dmtf.org/standards/redfish) 25ceeba4f9SEd Tanous# for resources implemented within the Redfish standard. bmcweb maintainers 26ceeba4f9SEd Tanous# regularly attend the Redfish specification meetings to have a handle on 27ceeba4f9SEd Tanous# "intent" behind Redfish APIs. In many cases, the role of the maintainer 28ceeba4f9SEd Tanous# requires knowing when a Redfish resource is underspecified, and direct people 29ceeba4f9SEd Tanous# to the standard before their changes can be accepted. 30ceeba4f9SEd Tanous# 31ceeba4f9SEd Tanous# - Have an understanding of, and track record of executing the various test 32ceeba4f9SEd Tanous# harnesses that bmcweb needs to pass, listed in CLIENTS.md, and at least a 33ceeba4f9SEd Tanous# rudimentary understanding of their requirements, and limitations. 34ceeba4f9SEd Tanous# 35ceeba4f9SEd Tanous# - Have an understanding of DBus and the specific implementations of sdbusplus 36ceeba4f9SEd Tanous# APIs that bmcweb uses, and their limitations in versioning, consistency, and 37ceeba4f9SEd Tanous# general implementation completeness. 38ceeba4f9SEd Tanous# 39ceeba4f9SEd Tanous# - Join and answer questions of the #bmcweb-and-redfish channel within 40ceeba4f9SEd Tanous# discord. 41ceeba4f9SEd Tanous# 42ceeba4f9SEd Tanous# - Join and answer architecture queries posed to the mailing list concerning 43ceeba4f9SEd Tanous# bmcweb. 44ceeba4f9SEd Tanous# 45ceeba4f9SEd Tanous# - Be capable of responding to CVE queries forwarded from the 46ceeba4f9SEd Tanous# [openbmc-security-response-team] 47ceeba4f9SEd Tanous# (https://github.com/openbmc/docs/blob/master/security/obmc-security-response-team.md). 48ceeba4f9SEd Tanous# Considering that in most implementations of the OpenBMC security model, 49ceeba4f9SEd Tanous# bmcweb is the primary attacker/client facing application on the network, it 50ceeba4f9SEd Tanous# is expected that a number of potential CVEs will be posted, for which bmcweb 51ceeba4f9SEd Tanous# forms a component of the alleged attack. Maintainers should be prepared to 52ceeba4f9SEd Tanous# respond to such requests in the timeframe required by the CVE process, and 53ceeba4f9SEd Tanous# ideally should have a track record of doing it in the past. 54ceeba4f9SEd Tanous# 55ceeba4f9SEd Tanous# - Understand the webui variants (webui-vue and phosphor-webui) that bmcweb 56ceeba4f9SEd Tanous# can optionally host, its use cases, and how they differ from "normal" client 57ceeba4f9SEd Tanous# based use cases, as well as an understanding of hosting web services in 58ceeba4f9SEd Tanous# general. 59ceeba4f9SEd Tanous# 60ceeba4f9SEd Tanous# If you believe you meet the qualifications for the above, please open a 61ceeba4f9SEd Tanous# patchset, adding your name to the list below, documenting some evidence of 62ceeba4f9SEd Tanous# the above requirements being met, and the existing maintainers will happily 63ceeba4f9SEd Tanous# add you to the list. 64ceeba4f9SEd Tanous 65eaa96da3SEd Tanousowners: 66eaa96da3SEd Tanous- ed@tanous.net 67eaa96da3SEd Tanous- gmills@linux.vnet.ibm.com 686afb06d9SGunnar Mills 69ceeba4f9SEd Tanous 70ceeba4f9SEd Tanous# The below specifies a list of reviewers and interested parties that should be 71ceeba4f9SEd Tanous# included on code reviews to stay informed of progress. 72ceeba4f9SEd Tanous 736afb06d9SGunnar Millsreviewers: 746afb06d9SGunnar Mills- krzysztof.grobelny@intel.com 7501358549SNan Zhou- nanzhoumails@gmail.com 76b0d3a856SEd Tanous 77b0d3a856SEd Tanousmatchers: 78b0d3a856SEd Tanous# unit tests 79b0d3a856SEd Tanous- suffix: _test.cpp 80b0d3a856SEd Tanous owners: 81df0c4ee0SNan Zhou - nanzhoumails@gmail.com 82b0d3a856SEd Tanous 83b0d3a856SEd Tanous# Redfish schemas 84b4d7181aSNan Zhou- exact: redfish-core/include/query.hpp 85df0c4ee0SNan Zhou owners: 86df0c4ee0SNan Zhou - nanzhoumails@gmail.com 87b4d7181aSNan Zhou- exact: redfish-core/include/utils/query_param.hpp 88df0c4ee0SNan Zhou owners: 89df0c4ee0SNan Zhou - nanzhoumails@gmail.com 90b0d3a856SEd Tanous- exact: redfish-core/lib/certificate_service.hpp 91df0c4ee0SNan Zhou owners: 92df0c4ee0SNan Zhou - nanzhoumails@gmail.com 93*3daf7270SNan Zhou- exact: redfish-core/lib/log_services.hpp 94df0c4ee0SNan Zhou owners: 95df0c4ee0SNan Zhou - nanzhoumails@gmail.com 96b0d3a856SEd Tanous- exact: redfish-core/lib/memory.hpp 97df0c4ee0SNan Zhou owners: 98df0c4ee0SNan Zhou - nanzhoumails@gmail.com 99b0d3a856SEd Tanous- exact: redfish-core/lib/sensors.hpp 100df0c4ee0SNan Zhou owners: 101df0c4ee0SNan Zhou - nanzhoumails@gmail.com 102df0c4ee0SNan Zhou - krzysztof.grobelny@intel.com 103b0d3a856SEd Tanous- exact: redfish-core/lib/event_service.hpp 104df0c4ee0SNan Zhou owners: 105df0c4ee0SNan Zhou - krzysztof.grobelny@intel.com 106b0d3a856SEd Tanous- exact: redfish-core/lib/power.hpp 107df0c4ee0SNan Zhou owners: 108df0c4ee0SNan Zhou - krzysztof.grobelny@intel.com 109b0d3a856SEd Tanous- exact: redfish-core/lib/thermal.hpp 110df0c4ee0SNan Zhou owners: 111df0c4ee0SNan Zhou - krzysztof.grobelny@intel.com 112b0d3a856SEd Tanous- exact: redfish-core/lib/virtual_media.hpp 113df0c4ee0SNan Zhou owners: 114df0c4ee0SNan Zhou - krzysztof.grobelny@intel.com 115b0d3a856SEd Tanous 116