Skip to content

Commit

Permalink
fix stage failures
Browse files Browse the repository at this point in the history
  • Loading branch information
zpzhuSplunk committed Mar 22, 2024
1 parent 2418764 commit f8de4f9
Show file tree
Hide file tree
Showing 12 changed files with 28 additions and 9 deletions.
3 changes: 3 additions & 0 deletions cmd/configschema/go.mod
Expand Up @@ -488,6 +488,7 @@ require (
github.com/open-telemetry/opentelemetry-collector-contrib/exporter/opencensusexporter v0.96.0 // indirect
github.com/open-telemetry/opentelemetry-collector-contrib/exporter/opensearchexporter v0.96.0 // indirect
github.com/open-telemetry/opentelemetry-collector-contrib/exporter/prometheusexporter v0.96.0 // indirect
github.com/open-telemetry/opentelemetry-collector-contrib/extension/ackextension v0.96.0 // indirect
github.com/open-telemetry/opentelemetry-collector-contrib/extension/asapauthextension v0.96.0 // indirect
github.com/open-telemetry/opentelemetry-collector-contrib/extension/awsproxy v0.96.0 // indirect
github.com/open-telemetry/opentelemetry-collector-contrib/extension/basicauthextension v0.96.0 // indirect
Expand Down Expand Up @@ -1155,3 +1156,5 @@ replace github.com/open-telemetry/opentelemetry-collector-contrib/extension/stor
replace github.com/open-telemetry/opentelemetry-collector-contrib/extension/encoding/otlpencodingextension => ../../extension/encoding/otlpencodingextension

replace github.com/open-telemetry/opentelemetry-collector-contrib/extension/encoding => ../../extension/encoding

replace github.com/open-telemetry/opentelemetry-collector-contrib/extension/ackextension => ../../extension/ackextension
1 change: 1 addition & 0 deletions cmd/otelcontribcol/builder-config.yaml
Expand Up @@ -444,6 +444,7 @@ replaces:
- github.com/open-telemetry/opentelemetry-collector-contrib/extension/opampextension => ../../extension/opampextension
- github.com/open-telemetry/opentelemetry-collector-contrib/extension/solarwindsapmsettingsextension => ../../extension/solarwindsapmsettingsextension
- github.com/open-telemetry/opentelemetry-collector-contrib/extension/sumologicextension => ../../extension/sumologicextension
- github.com/open-telemetry/opentelemetry-collector-contrib/extension/ackextension => ../../extension/ackextension
- github.com/open-telemetry/opentelemetry-collector-contrib/receiver/namedpipereceiver => ../../receiver/namedpipereceiver
- github.com/open-telemetry/opentelemetry-collector-contrib/internal/sqlquery => ../../internal/sqlquery

3 changes: 3 additions & 0 deletions cmd/otelcontribcol/go.mod
Expand Up @@ -530,6 +530,7 @@ require (
github.com/nginxinc/nginx-prometheus-exporter v0.11.0 // indirect
github.com/oklog/ulid/v2 v2.1.0 // indirect
github.com/open-telemetry/opamp-go v0.12.0 // indirect
github.com/open-telemetry/opentelemetry-collector-contrib/extension/ackextension v0.96.0 // indirect
github.com/open-telemetry/opentelemetry-collector-contrib/extension/encoding v0.96.0 // indirect
github.com/open-telemetry/opentelemetry-collector-contrib/extension/observer v0.96.0 // indirect
github.com/open-telemetry/opentelemetry-collector-contrib/internal/aws/awsutil v0.96.0 // indirect
Expand Down Expand Up @@ -1193,6 +1194,8 @@ replace github.com/open-telemetry/opentelemetry-collector-contrib/extension/sola

replace github.com/open-telemetry/opentelemetry-collector-contrib/extension/sumologicextension => ../../extension/sumologicextension

replace github.com/open-telemetry/opentelemetry-collector-contrib/extension/ackextension => ../../extension/ackextension

replace github.com/open-telemetry/opentelemetry-collector-contrib/receiver/namedpipereceiver => ../../receiver/namedpipereceiver

replace github.com/open-telemetry/opentelemetry-collector-contrib/internal/sqlquery => ../../internal/sqlquery
3 changes: 3 additions & 0 deletions cmd/oteltestbedcol/go.mod
Expand Up @@ -166,6 +166,7 @@ require (
github.com/mostynb/go-grpc-compression v1.2.2 // indirect
github.com/munnerz/goautoneg v0.0.0-20191010083416-a7dc8b61c822 // indirect
github.com/mwitkow/go-conntrack v0.0.0-20190716064945-2f068394615f // indirect
github.com/open-telemetry/opentelemetry-collector-contrib/extension/ackextension v0.96.0 // indirect
github.com/open-telemetry/opentelemetry-collector-contrib/internal/common v0.96.0 // indirect
github.com/open-telemetry/opentelemetry-collector-contrib/internal/coreinternal v0.96.0 // indirect
github.com/open-telemetry/opentelemetry-collector-contrib/internal/filter v0.96.0 // indirect
Expand Down Expand Up @@ -393,3 +394,5 @@ replace github.com/open-telemetry/opentelemetry-collector-contrib/pkg/pdatautil
replace github.com/openshift/api v3.9.0+incompatible => github.com/openshift/api v0.0.0-20180801171038-322a19404e37

replace github.com/outcaste-io/ristretto v0.2.0 => github.com/outcaste-io/ristretto v0.2.1

replace github.com/open-telemetry/opentelemetry-collector-contrib/extension/ackextension => ../../extension/ackextension
2 changes: 2 additions & 0 deletions go.mod
Expand Up @@ -1157,3 +1157,5 @@ replace github.com/open-telemetry/opentelemetry-collector-contrib/internal/sqlqu
replace github.com/open-telemetry/opentelemetry-collector-contrib/extension/encoding => ./extension/encoding

replace github.com/open-telemetry/opentelemetry-collector-contrib/extension/encoding/otlpencodingextension => ./extension/encoding/otlpencodingextension

replace github.com/open-telemetry/opentelemetry-collector-contrib/extension/ackextension => ./extension/ackextension
2 changes: 0 additions & 2 deletions go.sum

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion internal/splunk/common.go
Expand Up @@ -23,7 +23,7 @@ const (
DefaultSeverityTextLabel = "otel.log.severity.text"
DefaultSeverityNumberLabel = "otel.log.severity.number"
HECTokenHeader = "Splunk"
HttpSplunkChannelHeader = "X-Splunk-Request-Channel"
HTTPSplunkChannelHeader = "X-Splunk-Request-Channel"

HecTokenLabel = "com.splunk.hec.access_token" // #nosec
// HecEventMetricType is the type of HEC event. Set to metric, as per https://docs.splunk.com/Documentation/Splunk/8.0.3/Metrics/GetMetricsInOther.
Expand Down
3 changes: 3 additions & 0 deletions receiver/splunkhecreceiver/go.mod
Expand Up @@ -42,6 +42,7 @@ require (
github.com/golang/protobuf v1.5.3 // indirect
github.com/golang/snappy v0.0.4 // indirect
github.com/hashicorp/go-version v1.6.0 // indirect
github.com/hashicorp/golang-lru/v2 v2.0.7 // indirect
github.com/klauspost/compress v1.17.7 // indirect
github.com/knadh/koanf/maps v0.1.1 // indirect
github.com/knadh/koanf/providers/confmap v0.1.0 // indirect
Expand Down Expand Up @@ -109,3 +110,5 @@ retract (
replace github.com/open-telemetry/opentelemetry-collector-contrib/internal/sharedcomponent => ../../internal/sharedcomponent

replace github.com/open-telemetry/opentelemetry-collector-contrib/pkg/golden => ../../pkg/golden

replace github.com/open-telemetry/opentelemetry-collector-contrib/extension/ackextension => ../../extension/ackextension
5 changes: 2 additions & 3 deletions receiver/splunkhecreceiver/go.sum

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

7 changes: 4 additions & 3 deletions receiver/splunkhecreceiver/receiver.go
Expand Up @@ -27,6 +27,7 @@ import (
"go.uber.org/zap"

"github.com/google/uuid"

"github.com/open-telemetry/opentelemetry-collector-contrib/extension/ackextension"

"github.com/open-telemetry/opentelemetry-collector-contrib/internal/splunk"
Expand Down Expand Up @@ -291,7 +292,7 @@ func (r *splunkReceiver) handleAck(resp http.ResponseWriter, req *http.Request)
}

// check that the channel exists
partitionID := req.Header.Get(splunk.HttpSplunkChannelHeader)
partitionID := req.Header.Get(splunk.HTTPSplunkChannelHeader)
if len(partitionID) <= 0 {
r.failRequest(ctx, resp, http.StatusBadRequest, requiredDataChannelHeader, 0, nil)
return
Expand Down Expand Up @@ -335,7 +336,7 @@ func (r *splunkReceiver) handleRawReq(resp http.ResponseWriter, req *http.Reques
var partitionID string
if r.ackExt != nil {
// check channel header exists
partitionID = req.Header.Get(splunk.HttpSplunkChannelHeader)
partitionID = req.Header.Get(splunk.HTTPSplunkChannelHeader)
if len(partitionID) <= 0 {
r.failRequest(ctx, resp, http.StatusBadRequest, requiredDataChannelHeader, 0, nil)
return
Expand Down Expand Up @@ -437,7 +438,7 @@ func (r *splunkReceiver) handleReq(resp http.ResponseWriter, req *http.Request)
var partitionID string
if r.ackExt != nil {
// check channel header exists
partitionID = req.Header.Get(splunk.HttpSplunkChannelHeader)
partitionID = req.Header.Get(splunk.HTTPSplunkChannelHeader)
if len(partitionID) <= 0 {
r.failRequest(ctx, resp, http.StatusBadRequest, requiredDataChannelHeader, 0, nil)
return
Expand Down
4 changes: 4 additions & 0 deletions testbed/go.mod
Expand Up @@ -143,6 +143,7 @@ require (
github.com/hashicorp/go-rootcerts v1.0.2 // indirect
github.com/hashicorp/go-version v1.6.0 // indirect
github.com/hashicorp/golang-lru v1.0.2 // indirect
github.com/hashicorp/golang-lru/v2 v2.0.7 // indirect
github.com/hashicorp/nomad/api v0.0.0-20230721134942-515895c7690c // indirect
github.com/hashicorp/serf v0.10.1 // indirect
github.com/hetznercloud/hcloud-go/v2 v2.4.0 // indirect
Expand Down Expand Up @@ -177,6 +178,7 @@ require (
github.com/mostynb/go-grpc-compression v1.2.2 // indirect
github.com/munnerz/goautoneg v0.0.0-20191010083416-a7dc8b61c822 // indirect
github.com/mwitkow/go-conntrack v0.0.0-20190716064945-2f068394615f // indirect
github.com/open-telemetry/opentelemetry-collector-contrib/extension/ackextension v0.96.0 // indirect
github.com/open-telemetry/opentelemetry-collector-contrib/internal/sharedcomponent v0.96.0 // indirect
github.com/open-telemetry/opentelemetry-collector-contrib/pkg/batchperresourceattr v0.96.0 // indirect
github.com/open-telemetry/opentelemetry-collector-contrib/pkg/experimentalmetricmetadata v0.96.0 // indirect
Expand Down Expand Up @@ -362,3 +364,5 @@ replace github.com/open-telemetry/opentelemetry-collector-contrib/pkg/pdatatest
replace github.com/open-telemetry/opentelemetry-collector-contrib/pkg/pdatautil => ../pkg/pdatautil

replace github.com/open-telemetry/opentelemetry-collector-contrib/pkg/golden => ../pkg/golden

replace github.com/open-telemetry/opentelemetry-collector-contrib/extension/ackextension => ../extension/ackextension
2 changes: 2 additions & 0 deletions testbed/go.sum

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit f8de4f9

Please sign in to comment.