Skip to content

Comments

fix_lb_port_forward#2

Open
ls-avanier wants to merge 1 commit intoPeriscopeData:masterfrom
ls-avanier:master
Open

fix_lb_port_forward#2
ls-avanier wants to merge 1 commit intoPeriscopeData:masterfrom
ls-avanier:master

Conversation

@ls-avanier
Copy link

This PR suggests a fix to an error we're getting where a worker fail the following way :
{"timestamp":"1475780538.338568926","source":"worker","message":"worker.beacon.restarting","log_level":2,"data":{"error":"failed to dial: failed to connect to TSA:%!(EXTRA *net.OpError=dial tcp: too many colons in address XXX.XXX.XXX.XXX:8080:2222)","session":"3"}}

It seems the port is hardcoded in the upstream Dockerfile and the exported $(CONCOURSE_WEB_SERVICE_PORT) variable doesn't override that value as expected.

Currently, loading those files in Kubernetes v1.4.0 doesn't work. This fix allows the ssh calls from the workers to traverse the service loadBalancer to reach the TSA.

More information can be found in the issue I opened in my fork.

@ClayShentrup
Copy link
Contributor

@ls-avanier Thanks for reporting this! We'll look into it soon. /cc @scopej

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants