Skip to content
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.

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

Fix filesystem collector defaults on darwin / Mac OS #819

Merged
merged 2 commits into from
Aug 12, 2021

Conversation

eamonryan
Copy link
Member

PR Description

This adds darwin to the switch statement for the node_exporter default config, making it so that filesystem metrics are actually collected.

Which issue(s) this PR fixes

Fixed #818

Notes to the Reviewer

PR Checklist

  • CHANGELOG updated
  • [N/A] Documentation added
  • Tests updated

@CLAassistant
Copy link

CLAassistant commented Aug 12, 2021

CLA assistant check
All committers have signed the CLA.

@eamonryan eamonryan changed the title Fix filesystem collector darwin Fix filesystem collector defaults on darwin / Mac OS Aug 12, 2021
Copy link
Member

@rfratto rfratto left a comment

Choose a reason for hiding this comment

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

Thanks! Once you sign the CLA I can merge :)

@rfratto rfratto merged commit f4cb115 into main Aug 12, 2021
@rfratto rfratto deleted the fix-filesystem-collector-darwin branch August 12, 2021 00:40
@mattdurham mattdurham mentioned this pull request Sep 7, 2021
3 tasks
mattdurham pushed a commit that referenced this pull request Nov 11, 2021
* Fix darwin filesystem collector defaults

* Changelog update
@github-actions github-actions bot added the frozen-due-to-age Locked due to a period of inactivity. Please open new issues or PRs if more discussion is needed. label Apr 11, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
frozen-due-to-age Locked due to a period of inactivity. Please open new issues or PRs if more discussion is needed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

node_exporter not initializing filesystem collector for darwin
3 participants