Skip to content

Commit

Permalink
.gitignore entry for macOs; removed unsupported honeycomb metric type…
Browse files Browse the repository at this point in the history
…s; added validation for raw metrics - forbidden
  • Loading branch information
bdw-nobl9 committed Jan 24, 2025
1 parent 6fc6aed commit a74b722
Show file tree
Hide file tree
Showing 5 changed files with 4 additions and 825 deletions.
3 changes: 3 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,9 @@
*.so
*.dylib

# macOs specific files
.DS_Store

# Test binary, built with `go test -c`
*.test

Expand Down
3 changes: 0 additions & 3 deletions internal/manifest/v1alpha/examples/slo.go
Original file line number Diff line number Diff line change
Expand Up @@ -120,9 +120,6 @@ var customMetricExamples = map[v1alpha.DataSourceType]map[metricVariant][]metric
},
v1alpha.Honeycomb: {
metricVariantSingleQueryGoodRatio: []metricSubVariant{},
metricVariantGoodRatio: []metricSubVariant{},
metricVariantBadRatio: []metricSubVariant{},
metricVariantThreshold: []metricSubVariant{},
},
v1alpha.Splunk: {
metricVariantSingleQueryGoodRatio: []metricSubVariant{},
Expand Down
21 changes: 0 additions & 21 deletions internal/manifest/v1alpha/examples/slo_variants.go
Original file line number Diff line number Diff line change
Expand Up @@ -987,27 +987,6 @@ func (s sloExample) generateMetricVariant(slo v1alphaSLO.SLO) v1alphaSLO.SLO {
}
case v1alpha.Honeycomb:
switch s.MetricVariant {
case metricVariantThreshold:
return setThresholdMetric(slo, newMetricSpec(v1alphaSLO.HoneycombMetric{
Calculation: "AVG",
Attribute: "requestsLatency",
}))
case metricVariantGoodRatio:
return setGoodOverTotalMetric(slo, newMetricSpec(v1alphaSLO.HoneycombMetric{
Calculation: "SUM",
Attribute: "counterGood",
}), newMetricSpec(v1alphaSLO.HoneycombMetric{
Calculation: "SUM",
Attribute: "counterTotal",
}))
case metricVariantBadRatio:
return setBadOverTotalMetric(slo, newMetricSpec(v1alphaSLO.HoneycombMetric{
Calculation: "SUM",
Attribute: "counterBad",
}), newMetricSpec(v1alphaSLO.HoneycombMetric{
Calculation: "SUM",
Attribute: "counterTotal",
}))
case metricVariantSingleQueryGoodRatio:
return setSingleQueryGoodOverTotalMetric(slo, newMetricSpec(v1alphaSLO.HoneycombMetric{
Attribute: "dc.sli.some-service-availability",
Expand Down
Loading

0 comments on commit a74b722

Please sign in to comment.