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

Btrfs #1123

Merged
merged 3 commits into from
Sep 23, 2014
Merged

Btrfs #1123

merged 3 commits into from
Sep 23, 2014

Conversation

remh
Copy link
Contributor

@remh remh commented Sep 16, 2014

It mimics the behavior of the btrfs fi df command.

@sirupsen
Copy link

Nice! I'll try to test this in production tomorrow.

@sirupsen
Copy link

\cc @graemej @burke

@sirupsen
Copy link

Another metric needed here is the subvolume count. This is important to monitor as well.

sudo btrfs subvolume list /u | wc -l is what we have in our script currently, but I'm sure there are more metrics in there that needs to be measured.

@remh
Copy link
Contributor Author

remh commented Sep 18, 2014

The problem with these metrics is that it requires root access to perform these operations.

Do you know any workaround to get those without being root?

I'll be happy to implement it if so.

@sirupsen
Copy link

Yeah, there's no good way to get it then. Could you make it optional from the config, then you can add privileges to the datadog user with sudo to run that command?

@remh
Copy link
Contributor Author

remh commented Sep 23, 2014

@sirupsen I created that gist https://gist.github.com/remh/4e9287f444b955dbbd1c that contains the subvolume count.

You'll need to give sudo rights to the dd-agent user.

I don't think we're going to add that in the default check for now as we prefer to avoid asking our users to give sudo rights to the dd-agent but feel free to add this check in your custom checks!

remh added a commit that referenced this pull request Sep 23, 2014
@remh remh merged commit 6b72241 into master Sep 23, 2014
@remh remh deleted the btrfs branch September 29, 2014 21:19
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