Skip to content

Commit

Permalink
sfxexporter: add disk utilization translations (#782)
Browse files Browse the repository at this point in the history
  • Loading branch information
pmcollins committed Aug 24, 2020
1 parent ed2f213 commit 285092c
Show file tree
Hide file tree
Showing 2 changed files with 31 additions and 1 deletion.
13 changes: 12 additions & 1 deletion exporter/signalfxexporter/factory_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -172,7 +172,7 @@ func TestCreateMetricsExporterWithDefaultTranslaitonRules(t *testing.T) {

// Validate that default translation rules are loaded
// Expected values has to be updated once default config changed
assert.Equal(t, 33, len(config.TranslationRules))
assert.Equal(t, 37, len(config.TranslationRules))
assert.Equal(t, translation.ActionRenameDimensionKeys, config.TranslationRules[0].Action)
assert.Equal(t, 32, len(config.TranslationRules[0].Mapping))
}
Expand Down Expand Up @@ -654,4 +654,15 @@ func TestDefaultDiskTranslations(t *testing.T) {
require.True(t, ok)
require.Equal(t, 1, len(utPts))
require.Equal(t, 3, len(utPts[0].Dimensions))

du, ok := m["disk.utilization"]
require.True(t, ok)
require.Equal(t, 4, len(du[0].Dimensions))
// cheap test for pct conversion
require.True(t, *du[0].Value.DoubleValue > 1)

dsu, ok := m["disk.summary_utilization"]
require.True(t, ok)
require.Equal(t, 3, len(dsu[0].Dimensions))
require.True(t, *dsu[0].Value.DoubleValue > 1)
}
19 changes: 19 additions & 0 deletions exporter/signalfxexporter/translation/constants.go
Original file line number Diff line number Diff line change
Expand Up @@ -258,6 +258,25 @@ translation_rules:
without_dimensions:
- device
# disk utilization
- action: calculate_new_metric
metric_name: disk.utilization
operand1_metric: df_complex.used
operand2_metric: disk.total
operator: /
- action: multiply_float
scale_factors_float:
disk.utilization: 100
- action: calculate_new_metric
metric_name: disk.summary_utilization
operand1_metric: df_complex.used_total
operand2_metric: disk.summary_total
operator: /
- action: multiply_float
scale_factors_float:
disk.summary_utilization: 100
# convert disk I/O metrics
- action: rename_dimension_keys
metric_names:
Expand Down

0 comments on commit 285092c

Please sign in to comment.