Skip to content

fix(go.d/powerstore): remove extra_details field#22291

Merged
ilyam8 merged 1 commit into
netdata:masterfrom
ilyam8:fix-dell-extra-hw
Apr 27, 2026
Merged

fix(go.d/powerstore): remove extra_details field#22291
ilyam8 merged 1 commit into
netdata:masterfrom
ilyam8:fix-dell-extra-hw

Conversation

@ilyam8

@ilyam8 ilyam8 commented Apr 27, 2026

Copy link
Copy Markdown
Member
Summary

Fixes

Job check failed: error decoding /hardware response: json: cannot unmarshal object into Go struct field Hardware.extra_details of type string

This field is not used.

Test Plan
Additional Information
For users: How does this change affect me?

Summary by cubic

Removed the unused extra_details field from the Hardware struct in go.d/powerstore to fix JSON decoding errors on /hardware responses. This prevents job check failures when the API returns an object for extra_details.

Written for commit eb7caaa. Summary will update on new commits. Review in cubic

@github-actions github-actions Bot added area/collectors Everything related to data collection collectors/go.d area/go labels Apr 27, 2026
@sonarqubecloud

Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai Bot 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.

No issues found across 1 file

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

@ilyam8 ilyam8 merged commit c4e0fb8 into netdata:master Apr 27, 2026
154 of 155 checks passed
stelfrag pushed a commit to stelfrag/netdata that referenced this pull request Apr 27, 2026
@stelfrag stelfrag mentioned this pull request Apr 27, 2026
Ferroin pushed a commit that referenced this pull request Apr 27, 2026
@ilyam8 ilyam8 deleted the fix-dell-extra-hw branch June 3, 2026 07:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/collectors Everything related to data collection area/go collectors/go.d

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants