Skip to content

feat: add receiver for redfish #43279

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 31 commits into
base: main
Choose a base branch
from
Open

feat: add receiver for redfish #43279

wants to merge 31 commits into from

Conversation

steven-freed
Copy link

@steven-freed steven-freed commented Oct 8, 2025 •
edited
Loading

Description

This PR is to add a redfish receiver. Redfish is an open standard API that is commonly implemented on enterprise grade servers with BMCs (baseboard management controller) used for both in-band and out-of-band server management. This receiver provides a way for engineers to monitor their servers and server components such as fans and temperature sensors.

Link to tracking issue #33724

Testing

Testing has been done on HPE iLO machines and logic similar to the Telegraf redfish input receiver has been used (which I have contributed to in the past).

@steven-freed
docs: add README.md
f34eda7 
Signed-off-by: Steve Freed <[email protected]>
@steven-freed
chore: add Makefile
3bbc421 
Signed-off-by: Steve Freed <[email protected]>
@steven-freed
feat: add metadata metrics and their docs
7332fe2 
Signed-off-by: Steve Freed <[email protected]>
@steven-freed
feat: add generated metric and config code
b8a748d 
Signed-off-by: Steve Freed <[email protected]>
@steven-freed
feat: add generated component and package go tests
848c528 
Signed-off-by: Steve Freed <[email protected]>
@steven-freed
chore: add go.mod and go.sum
479717d 
Signed-off-by: Steve Freed <[email protected]>
@steven-freed
feat: add receiver config and factory go
0a79d9e 
Signed-off-by: Steve Freed <[email protected]>
@steven-freed
chore: add redfishreceiver to versions.yaml
a9df032 
Signed-off-by: Steve Freed <[email protected]>
@steven-freed
chore: updated README.md
de88aa6 
Signed-off-by: Steve Freed <[email protected]>
@steven-freed
fix: add license header comments to go files
e55212a 
Signed-off-by: Steve Freed <[email protected]>
@steven-freed
feat: add resource type
4552680 
Signed-off-by: Steve Freed <[email protected]>
@steven-freed
chore: add github issue templates
d6608ed 
Signed-off-by: Steve Freed <[email protected]>
@linux-foundation-easycla Linux Foundation: EasyCLA
Copy link

linux-foundation-easycla bot commented Oct 8, 2025 •
edited
Loading

CLA Signed

The committers listed above are authorized under a signed CLA.

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Oct 8, 2025

Welcome, contributor! Thank you for your contribution to opentelemetry-collector-contrib.

Important reminders:

A maintainer will review your pull request soon. Thank you for helping make OpenTelemetry better!

@github-actions github-actions bot added the first-time contributor PRs made by new contributors label Oct 8, 2025
@steven-freed steven-freed marked this pull request as ready for review October 8, 2025 15:41
@steven-freed steven-freed requested a review from a team as a code owner October 8, 2025 15:41
@steven-freed steven-freed requested a review from crobert-1 October 8, 2025 15:41
@steven-freed
Copy link
Author

steven-freed commented Oct 8, 2025 •
edited
Loading

@crobert-1 There is currently not a code owner for this receiver as it is brand new so I put myself for the time being. I am not currently a member of opentelemetry. I don't have a ton of dev time to devote to this long term but I'd be happy to occasionally check in maybe every other week or so. If that is good enough, then I can be a code owner. Please advise.

@crobert-1 crobert-1 requested a review from atoulme October 8, 2025 16:26
@songy23 songy23 added the Accepted Component New component has been sponsored label Oct 8, 2025
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Oct 8, 2025

Welcome, contributor! Thank you for your contribution to opentelemetry-collector-contrib.

Important reminders:

A maintainer will review your pull request soon. Thank you for helping make OpenTelemetry better!

@github-actions github-actions bot added the cmd/otelcontribcol otelcontribcol command label Oct 8, 2025
@atoulme
Copy link
Contributor

atoulme commented Oct 9, 2025 •
edited
Loading

2025/10/08 22:09:01 codeowners are not members: steven-freed

add yourself under cmd/githubgen/allowlist

atoulme
atoulme reviewed Oct 9, 2025
View reviewed changes
@steven-freed
Copy link
Author

Hoping this build succeeds. I do see that the main branch build is in a failed state. However, when I run all the first PR make's from the contrib docs all make's run without errors except for this one make gengithub.

@atoulme
Copy link
Contributor

atoulme commented Oct 10, 2025

please run:

./.github/workflows/scripts/check-collector-module-version.sh

on your box.

Please also run:

make chlog-new

Finally run:

make tidylist

that will help some.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@atoulme atoulme atoulme left review comments

@crobert-1 crobert-1 Awaiting requested review from crobert-1 crobert-1 is a code owner automatically assigned from open-telemetry/collector-contrib-approvers

At least 1 approving review is required to merge this pull request.

Assignees

@evan-bradley evan-bradley

Labels

Accepted Component New component has been sponsored cmd/otelcontribcol otelcontribcol command first-time contributor PRs made by new contributors

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@steven-freed @atoulme @songy23 @evan-bradley