Skip to content

Commit

Permalink
fix: Use Correct OTLP Port Number in Exporters
Browse files Browse the repository at this point in the history
  • Loading branch information
simek-m committed Dec 18, 2024
1 parent 8a8542e commit 7126bef
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 7 deletions.
6 changes: 3 additions & 3 deletions extension/solarwindsextension/internal/config.go
Original file line number Diff line number Diff line change
Expand Up @@ -136,9 +136,9 @@ func (cfg *Config) EndpointUrl() (string, error) {
// dataCenterToURLMapping maps a data center ID to
// to its corresponding OTLP endpoint URL.
var dataCenterToURLMapping = map[string]string{
"na-01": "otel.collector.na-01.cloud.solarwinds.com:4317",
"na-02": "otel.collector.na-02.cloud.solarwinds.com:4317",
"eu-01": "otel.collector.eu-01.cloud.solarwinds.com:4317",
"na-01": "otel.collector.na-01.cloud.solarwinds.com:443",
"na-02": "otel.collector.na-02.cloud.solarwinds.com:443",
"eu-01": "otel.collector.eu-01.cloud.solarwinds.com:443",
}

// lookupDataCenterURL returns the OTLP endpoint URL
Expand Down
8 changes: 4 additions & 4 deletions extension/solarwindsextension/internal/datacell_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -31,10 +31,10 @@ func TestConfigValidateDataCenters(t *testing.T) {
}

tests := []test{
{dataCenter: "na-01", url: "otel.collector.na-01.cloud.solarwinds.com:4317", ok: true},
{dataCenter: "na-02", url: "otel.collector.na-02.cloud.solarwinds.com:4317", ok: true},
{dataCenter: "eu-01", url: "otel.collector.eu-01.cloud.solarwinds.com:4317", ok: true},
{dataCenter: "NA-01", url: "otel.collector.na-01.cloud.solarwinds.com:4317", ok: true},
{dataCenter: "na-01", url: "otel.collector.na-01.cloud.solarwinds.com:443", ok: true},
{dataCenter: "na-02", url: "otel.collector.na-02.cloud.solarwinds.com:443", ok: true},
{dataCenter: "eu-01", url: "otel.collector.eu-01.cloud.solarwinds.com:443", ok: true},
{dataCenter: "NA-01", url: "otel.collector.na-01.cloud.solarwinds.com:443", ok: true},
{dataCenter: "apj-01", url: "", ok: false},
}

Expand Down

0 comments on commit 7126bef

Please sign in to comment.