Skip to content

Commit 9e91a55

Browse files
committed
refactoring
1 parent 458d199 commit 9e91a55

2 files changed

Lines changed: 8 additions & 9 deletions

File tree

src/java/frameworks/open_telemetry_javaagent.go

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -85,11 +85,8 @@ func (o *OpenTelemetryJavaagentFramework) Finalize() error {
8585
vcapServices, _ := GetVCAPServices()
8686

8787
// Try to find service by various patterns
88-
var service *VCAPService
89-
if vcapServices.HasService("otel-collector") {
90-
service = vcapServices.GetService("otel-collector")
91-
}
92-
if service == nil && vcapServices.HasService("opentelemetry") {
88+
service := vcapServices.GetService("otel-collector")
89+
if service == nil {
9390
service = vcapServices.GetService("opentelemetry")
9491
}
9592
if service == nil {

src/java/frameworks/splunk_otel_java_agent.go

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -62,9 +62,8 @@ func (s *SplunkOtelJavaAgentFramework) Detect() (string, error) {
6262
if vcapServices.HasService("splunk") ||
6363
vcapServices.HasService("splunk-otel") ||
6464
vcapServices.HasTag("splunk") ||
65-
vcapServices.HasTag("otel") ||
6665
vcapServices.HasServiceByNamePattern("splunk") ||
67-
vcapServices.HasServiceByNamePattern("otel") {
66+
vcapServices.HasServiceByNamePattern("splunk-o11y") {
6867
s.context.Log.Info("Splunk OTEL service detected!")
6968
return "Splunk OTEL", nil
7069
}
@@ -185,7 +184,7 @@ func (s *SplunkOtelJavaAgentFramework) getCredentials() SplunkCredentials {
185184

186185
// Find the first matching Splunk service, preferring explicit labels over name pattern matches
187186
var service *common.VCAPService
188-
for _, label := range []string{"splunk", "splunk-otel"} {
187+
for _, label := range []string{"splunk", "splunk-otel", "splunk-o11y"} {
189188
if vcapServices.HasService(label) {
190189
service = vcapServices.GetService(label)
191190
break
@@ -195,7 +194,10 @@ func (s *SplunkOtelJavaAgentFramework) getCredentials() SplunkCredentials {
195194
service = vcapServices.GetServiceByNamePattern("splunk")
196195
}
197196
if service == nil {
198-
service = vcapServices.GetServiceByNamePattern("otel")
197+
service = vcapServices.GetServiceByNamePattern("splunk-otel")
198+
}
199+
if service == nil {
200+
service = vcapServices.GetServiceByNamePattern("splunk-o11y")
199201
}
200202
if service == nil {
201203
return creds

0 commit comments

Comments
 (0)