Skip to content

test(inputs.redfish): Extend coverage in preparation of lib change#19163

Merged
skartikey merged 7 commits into
influxdata:masterfrom
inhinias:rf_gf_tests_adaption
Jul 20, 2026
Merged

test(inputs.redfish): Extend coverage in preparation of lib change#19163
skartikey merged 7 commits into
influxdata:masterfrom
inhinias:rf_gf_tests_adaption

Conversation

@inhinias

Copy link
Copy Markdown
Contributor

Summary

Since the plan is to move to the gofish lib we extend the tests to work with them in a later pr.
Because gofish expects to move from the top of the api to lower nodes, every testcase requires
/redfish/v1/ & /redfish/v1/Systems/ to be present.

Also gofish does no trigger an invalid Response with a trailing "{". Therefore its moved to the start of the data.

Datacenter is supposed to be under the Postal address: tags["datacenter"] = chassis.Location.PostalAddress.DataCenter but its nowhere to be found in the standard. See page 35.

It should be considered if the "room" tag should be changed too since PostalAddress is deprecated.
This pr removes just the datacenter tag from the tests but not the actual program. That test would fail and is therefor skipped. Please inform me on how to handle this case for the "moving to gofish" pr.

Also i already added the testdata for the subsystems that will be added as a feature later.

Checklist

Related issues

Related to: #18963

@telegraf-tiger telegraf-tiger Bot added fix pr to fix corresponding bug plugin/input 1. Request for new input plugins 2. Issues/PRs that are related to input plugins labels Jun 26, 2026
@inhinias inhinias closed this Jun 26, 2026
@inhinias inhinias reopened this Jun 26, 2026
@inhinias
inhinias marked this pull request as ready for review June 26, 2026 12:53

@srebhan srebhan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @inhinias! Please try to not change existing tests! Find my comments in the code...

Comment thread plugins/inputs/redfish/testdata/dell/dell_thermalinvalid.json Outdated
Comment thread plugins/inputs/redfish/testdata/hp/hp_powerinvalid.json
Comment thread plugins/inputs/redfish/testdata/hp/hp_systemsinvalid.json
Comment thread plugins/inputs/redfish/testdata/hp/hp_thermalinvalid.json
Comment thread plugins/inputs/redfish/redfish_test.go Outdated
Comment thread plugins/inputs/redfish/redfish_test.go Outdated
Comment thread plugins/inputs/redfish/redfish_test.go
@srebhan srebhan changed the title fix(inputs.redfish): Extend testcases for gofish lib change test(inputs.redfish): Extend coverage in preparation of lib change Jul 1, 2026
@srebhan srebhan self-assigned this Jul 1, 2026
@telegraf-tiger telegraf-tiger Bot added the test label Jul 1, 2026
@srebhan srebhan removed the fix pr to fix corresponding bug label Jul 1, 2026
@srebhan

srebhan commented Jul 2, 2026

Copy link
Copy Markdown
Member

@inhinias I marked the resolved items as "resolved". However, there are three issues left...

@inhinias

inhinias commented Jul 6, 2026

Copy link
Copy Markdown
Contributor Author

@srebhan SystemID, Datacenter Tags and test skip reverted.

@telegraf-tiger

telegraf-tiger Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

@srebhan srebhan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome! Thanks @inhinias!

@srebhan srebhan added the ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review. label Jul 15, 2026
@srebhan srebhan assigned skartikey and unassigned srebhan Jul 15, 2026

@skartikey skartikey left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@inhinias Thanks for the contribution!

@skartikey
skartikey merged commit d5a8049 into influxdata:master Jul 20, 2026
27 checks passed
srebhan pushed a commit that referenced this pull request Jul 20, 2026
…19163)

Co-authored-by: Saphira Mathis <saphira.mathis@ur.ch>
(cherry picked from commit d5a8049)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

plugin/input 1. Request for new input plugins 2. Issues/PRs that are related to input plugins ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review. test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants