Skip to content

Commit f4aadf7

Browse files
Use K8S depedent env vars instead of custom ExpandNamespace
1 parent 3a55d11 commit f4aadf7

File tree

11 files changed

+22
-35
lines changed

11 files changed

+22
-35
lines changed

k8s/base/driver.yaml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -32,17 +32,17 @@ spec:
3232
fieldPath: metadata.namespace
3333
- name: SIGNADOT_BASELINE_NAME
3434
value: "driver"
35-
# service dependencies (<NAMESPACE> get replaced at runtime by the app)
35+
# service dependencies
3636
- name: SIGNADOT_ROUTESERVER
3737
value: routeserver.signadot.svc:7777
3838
- name: ROUTE_ADDR
39-
value: route.<NAMESPACE>:8083
39+
value: route.$(SIGNADOT_BASELINE_NAMESPACE):8083
4040
- name: REDIS_ADDR
41-
value: redis.<NAMESPACE>:6379
41+
value: redis.$(SIGNADOT_BASELINE_NAMESPACE):6379
4242
- name: KAFKA_BROKER_ADDR
43-
value: kafka-headless.<NAMESPACE>:9092
43+
value: kafka-headless.$(SIGNADOT_BASELINE_NAMESPACE):9092
4444
- name: OTEL_EXPORTER_OTLP_ENDPOINT
45-
value: http://jaeger.<NAMESPACE>:4318
45+
value: http://jaeger.$(SIGNADOT_BASELINE_NAMESPACE):4318
4646
image: signadot/hotrod:latest
4747
imagePullPolicy: Always
4848
readinessProbe:

k8s/base/frontend.yaml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -32,15 +32,15 @@ spec:
3232
fieldPath: metadata.namespace
3333
- name: SIGNADOT_BASELINE_NAME
3434
value: "frontend"
35-
# service dependencies (<NAMESPACE> get replaced at runtime by the app)
35+
# service dependencies
3636
- name: LOCATION_ADDR
37-
value: location.<NAMESPACE>:8081
37+
value: location.$(SIGNADOT_BASELINE_NAMESPACE):8081
3838
- name: REDIS_ADDR
39-
value: redis.<NAMESPACE>:6379
39+
value: redis.$(SIGNADOT_BASELINE_NAMESPACE):6379
4040
- name: KAFKA_BROKER_ADDR
41-
value: kafka-headless.<NAMESPACE>:9092
41+
value: kafka-headless.$(SIGNADOT_BASELINE_NAMESPACE):9092
4242
- name: OTEL_EXPORTER_OTLP_ENDPOINT
43-
value: http://jaeger.<NAMESPACE>:4318
43+
value: http://jaeger.$(SIGNADOT_BASELINE_NAMESPACE):4318
4444
image: signadot/hotrod:latest
4545
imagePullPolicy: Always
4646
ports:

k8s/base/location.yaml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -31,15 +31,15 @@ spec:
3131
fieldPath: metadata.namespace
3232
- name: SIGNADOT_BASELINE_NAME
3333
value: location
34-
# service dependencies (<NAMESPACE> get replaced at runtime by the app)
34+
# service dependencies
3535
- name: MYSQL_ADDR
36-
value: mysql.<NAMESPACE>:3306
36+
value: mysql.$(SIGNADOT_BASELINE_NAMESPACE):3306
3737
- name: MYSQL_PASS
3838
value: abc
3939
- name: REDIS_ADDR
40-
value: redis.<NAMESPACE>:6379
40+
value: redis.$(SIGNADOT_BASELINE_NAMESPACE):6379
4141
- name: OTEL_EXPORTER_OTLP_ENDPOINT
42-
value: http://jaeger.<NAMESPACE>:4318
42+
value: http://jaeger.$(SIGNADOT_BASELINE_NAMESPACE):4318
4343
image: signadot/hotrod:latest
4444
imagePullPolicy: Always
4545
ports:

k8s/base/route.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,9 +33,9 @@ spec:
3333
value: "route"
3434
# service dependencies
3535
- name: REDIS_ADDR
36-
value: redis.<NAMESPACE>:6379
36+
value: redis.$(SIGNADOT_BASELINE_NAMESPACE):6379
3737
- name: OTEL_EXPORTER_OTLP_ENDPOINT
38-
value: http://jaeger.<NAMESPACE>:4318
38+
value: http://jaeger.$(SIGNADOT_BASELINE_NAMESPACE):4318
3939
image: signadot/hotrod:latest
4040
imagePullPolicy: Always
4141
ports:

pkg/config/kafka.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
package config
22

33
func GetKafkaBrokerAddr() string {
4-
return ExpandNamespace(EnvDefault("KAFKA_BROKER_ADDR", "kafka-headless:9092"))
4+
return EnvDefault("KAFKA_BROKER_ADDR", "kafka-headless:9092")
55
}
66

77
func GetKafkaBrokers() []string {

pkg/config/location.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,5 +5,5 @@ func GetLocationBindPort() string {
55
}
66

77
func GetLocationAddr() string {
8-
return ExpandNamespace(EnvDefault("LOCATION_ADDR", "location:8081"))
8+
return EnvDefault("LOCATION_ADDR", "location:8081")
99
}

pkg/config/mysql.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ import (
66
)
77

88
func GetMySQLAddr() string {
9-
return ExpandNamespace(EnvDefault("MYSQL_ADDR", "mysql:3306"))
9+
return EnvDefault("MYSQL_ADDR", "mysql:3306")
1010
}
1111

1212
func GetMySQLUser() string {

pkg/config/namespace.go

Lines changed: 0 additions & 13 deletions
This file was deleted.

pkg/config/otel.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,5 +13,5 @@ func InitOtelExporter() {
1313
if addr == "" {
1414
return
1515
}
16-
os.Setenv("OTEL_EXPORTER_OTLP_ENDPOINT", ExpandNamespace(addr))
16+
os.Setenv("OTEL_EXPORTER_OTLP_ENDPOINT", addr)
1717
}

pkg/config/redis.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
package config
22

33
func GetRedisAddr() string {
4-
return ExpandNamespace(EnvDefault("REDIS_ADDR", "redis:6379"))
4+
return EnvDefault("REDIS_ADDR", "redis:6379")
55
}
66

77
func GetRedisPassword() string {

0 commit comments

Comments
 (0)