Skip to content

Commit e6dd450

Browse files
committed
fix(router): fix router common prefix app publishing
using k8s scheduler apps with names which share a common prefix, e.g. myapp, myapp1, myapp2, failed to get published to the router component. This fix enforces dir etcd listing of discovered apps. Without the fix, when apps share a common prefix, multiple non existent entries are returned causing confd failures.
1 parent 27eab71 commit e6dd450

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

router/rootfs/etc/confd/templates/nginx.conf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -409,7 +409,7 @@ http {
409409

410410
#start k8s apps
411411
{{ range $k8namespace := lsdir "/registry/services/specs/" }}
412-
{{ $k8appdir := printf "/registry/services/specs/%s" $k8namespace}}{{ range $kapp := ls $k8appdir }}
412+
{{ $k8appdir := printf "/registry/services/specs/%s/" $k8namespace}}{{ range $kapp := ls $k8appdir }}
413413
{{ $k8appPath := printf "/registry/services/specs/%s/%s" $k8namespace $kapp}}{{ $k8Svc := json (getv $k8appPath) }}
414414
{{ $upstreams := printf "/registry/services/specs/%s/%s" $k8namespace $kapp}}
415415
upstream {{ if $k8Svc.metadata.labels.name }}{{ $k8Svc.metadata.labels.name }}{{ else }}{{ $k8Svc.metadata.name }}{{ end }} {

0 commit comments

Comments
 (0)