Skip to content

Refresh codebase - #12

Merged
MurrayGroves merged 12 commits into
mainfrom
refresh
Oct 22, 2026
Merged

MurrayGroves merged 12 commits into
mainfrom
refresh

Conversation

@MurrayGroves

@MurrayGroves MurrayGroves commented Oct 15, 2026 •

Copy link
Copy Markdown
  • Bump dependencies
  • Move logging to slog as apex/log is un-maintained and has an unusual logging format
  • Update Dockerfile to build from local source instead of cloning from Github
  • Gofish now provides missing metrics as nil pointers instead of zero. Updated to handle this behaviour
  • Patched incorrect calculation of fan percentage from RPM

@dougszumski dougszumski 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 @MurrayGroves - great start :D

Comment thread collector/chassis_collector.go Outdated
Comment thread collector/chassis_collector.go Outdated
Comment thread collector/system_collector.go
Comment thread collector/chassis_collector.go Outdated
Comment thread collector/chassis_collector.go Outdated

@dougszumski dougszumski 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 for updating @MurrayGroves

@MurrayGroves
MurrayGroves marked this pull request as ready for review October 22, 2026 14:35
@MurrayGroves
MurrayGroves merged commit 54184bd into main Oct 22, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants