openbmc / technical-oversight-forum

6 stars 1 forks source link

Unresponsive maintainers report. #27

Closed williamspatrick closed 1 year ago

williamspatrick commented 1 year ago

Background

Per https://github.com/openbmc/docs/blob/master/process/subproject-maintainership.md:

Maintainers are considered unresponsive after failing to address contributions for a minimum period of 1 month.

The following trivial commits were submitted on April 12th and have not received feedback, nor has a follow up ping on May 2nd been acknowledged:

Questions

bradbishop commented 1 year ago

FWIW I merged openbmc/phosphor-misc - https://gerrit.openbmc.org/c/openbmc/phosphor-misc/+/62392 just now. I don't check my repositories very often. I need to set up some better mail filtering...

amboar commented 1 year ago

Merged

62378: meson: remove deprecated get_pkgconfig_variable | https://gerrit.openbmc.org/c/openbmc/telemetry/+/62378 62392: meson: remove deprecated get_pkgconfig_variable | https://gerrit.openbmc.org/c/openbmc/phosphor-misc/+/62392

Unreviewed

62387: meson: remove deprecated get_pkgconfig_variable | https://gerrit.openbmc.org/c/openbmc/service-config-manager/+/62387

How does the TOF record unresponsiveness and/or follow up with affected maintainers?

I didn't really spell that out, but I figured we just email a warning to the set of maintainers and CC openbmc@lists.ozlabs.org to make sure the communication is public.

williamspatrick commented 1 year ago

Additional commits which have had over a month elapse:

amboar commented 1 year ago

The patches for each of openbmc/dbus-sensors, openbmc/telemetry, and openbmc/phosphor-misc have all been merged at this point in time. I've held off on sending notifications of unresponsiveness for those projects.

For the remaining projects listed, the notifications are now on the list:

williamspatrick commented 1 year ago

https://gerrit.openbmc.org/c/openbmc/ipmi-fru-parser/+/63228 is still not resolved. @amboar

williamspatrick commented 1 year ago

https://gerrit.openbmc.org/c/openbmc/ipmi-fru-parser/+/63228 is still not resolved. @amboar

The lack of maintainer attention on ipmi-fru-parser is also affecting getting more useful changes into the repository (beyond simply reformatting so that CI passes):

amboar commented 1 year ago

@williamspatrick I've addressed both of the ipmi-fru-parser patches mentioned above in the second notification as follows:

https://lore.kernel.org/openbmc/d47818d4-4167-4508-bec0-db6eee93edb0@app.fastmail.com/

williamspatrick commented 1 year ago

Closing now. All reports are resolved.