Skip to content

Commit

Permalink
remove collectd/df monitor
Browse files Browse the repository at this point in the history
  • Loading branch information
atoulme committed Dec 7, 2023
1 parent 8e1fd98 commit 5f44aff
Show file tree
Hide file tree
Showing 8 changed files with 1 addition and 342 deletions.
1 change: 0 additions & 1 deletion internal/signalfx-agent/pkg/core/modules_linux.go
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@ import (
_ "github.com/signalfx/signalfx-agent/pkg/monitors/collectd/cpu"
_ "github.com/signalfx/signalfx-agent/pkg/monitors/collectd/cpufreq"
_ "github.com/signalfx/signalfx-agent/pkg/monitors/collectd/custom"
_ "github.com/signalfx/signalfx-agent/pkg/monitors/collectd/df"
_ "github.com/signalfx/signalfx-agent/pkg/monitors/collectd/disk"
_ "github.com/signalfx/signalfx-agent/pkg/monitors/collectd/genericjmx"
_ "github.com/signalfx/signalfx-agent/pkg/monitors/collectd/hadoopjmx"
Expand Down
93 changes: 0 additions & 93 deletions internal/signalfx-agent/pkg/monitors/collectd/df/df.go

This file was deleted.

25 changes: 0 additions & 25 deletions internal/signalfx-agent/pkg/monitors/collectd/df/df.tmpl

This file was deleted.

78 changes: 0 additions & 78 deletions internal/signalfx-agent/pkg/monitors/collectd/df/genmetadata.go

This file was deleted.

81 changes: 0 additions & 81 deletions internal/signalfx-agent/pkg/monitors/collectd/df/metadata.yaml

This file was deleted.

41 changes: 0 additions & 41 deletions internal/signalfx-agent/pkg/monitors/collectd/df/template.go

This file was deleted.

22 changes: 0 additions & 22 deletions internal/signalfx-agent/pkg/monitors/filesystems/metadata.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,28 +13,6 @@ monitors:
- type: filesystems
hostFSPath: /hostfs
```
## Migrating from collectd/df
The `collectd/df` monitor is being deprecated in favor of the `filesystems`
monitor. While the `collectd/df` monitor will still be available in
5.0, it is recommended that you switch to the `filesystems` monitor soon
after upgrading. There are a few incompatibilities to be aware of between
the two monitors:
- `collectd/df` used a dimension called `plugin_instance` to identify the
mount point or device of the filesystem. This dimension is completely
removed in the `filesystems` monitor and replaced by the `mountpoint`
and `device` dimensions. You no longer have to select between the two
(the `reportByDevice` option on `collectd/df`) as both are always
reported.
- The mountpoints in the `plugin_instance` dimension of `collectd/df`
were reported with `-` instead of the more conventional `/` separated
path segments. The `filesystems` monitor always reports mountpoints in
the `mountpoint` dimension and uses the conventional `/` separator.
- The `collectd/df` plugin set a dimension `plugin: df` on all datapoints,
but `filesystems` has no such comparable dimension.
metrics:
df_complex.free:
description: Free disk space in bytes
Expand Down
2 changes: 1 addition & 1 deletion pkg/receiver/smartagentreceiver/config.go
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ var (
errDimensionClientValue = fmt.Errorf("dimensionClients must be an array of compatible exporter names")
nonWindowsMonitors = map[string]bool{
"collectd/activemq": true, "collectd/apache": true, "collectd/cassandra": true, "collectd/chrony": true,
"collectd/cpu": true, "collectd/cpufreq": true, "collectd/custom": true, "collectd/df": true, "collectd/disk": true,
"collectd/cpu": true, "collectd/cpufreq": true, "collectd/custom": true, "collectd/disk": true,
"collectd/genericjmx": true, "collectd/hadoopjmx": true, "collectd/kafka": true, "collectd/kafka_consumer": true,
"collectd/kafka_producer": true, "collectd/load": true, "collectd/memcached": true, "collectd/memory": true,
"collectd/mysql": true, "collectd/netinterface": true, "collectd/nginx": true, "collectd/php-fpm": true,
Expand Down

0 comments on commit 5f44aff

Please sign in to comment.