containous / traefik-extra-service-fabric

Traefik extra: Service Fabric Provider
Apache License 2.0
12 stars 14 forks source link

fix: backend name for stateful service. #27

Closed ldez closed 6 years ago

ldez commented 6 years ago

fix: backend name for stateful service.

Closes #22

{"Interval":30000000000},"RespondingTimeouts":null,"ForwardingTimeouts":null,"Web":null,"Docker":null,"File":null,"Marathon":null,"Consul":null,"ConsulCatalog":null,"Etcd":null,"Zookeeper":null,"Boltdb":null,"Kubernetes":null,"Mesos":null,"Eureka":null,"ECS":null,"Rancher":null,"DynamoDB":null,"ServiceFabric":{"Watch":false,"Filename":"","Constraints":null,"Trace":false,"DebugLogGeneratedTemplate":false,"ClusterManagementURL":"https://servicefabric-prod02.klwines.com:19080","APIVersion":"3.0","RefreshSeconds":0,"TLS":{"CA":"","CAOptional":false,"Cert":"certs/servicefabric.crt","Key":"certs/servicefabric.key","InsecureSkipVerify":true}},"Rest":null,"API":{"EntryPoint":"traefik","Dashboard":true,"Debug":true,"CurrentConfigurations":null,"Statistics":null},"Metrics":null,"Ping":null}"
time="2018-04-13T01:42:15Z" level=info msg="Preparing server traefik &{Network: Address::8080 TLS:<nil> Redirect:<nil> Auth:<nil> WhitelistSourceRange:[] Compress:false ProxyProtocol:<nil> ForwardedHeaders:0xc042601080} with readTimeout=0s writeTimeout=0s idleTimeout=3m0s"
time="2018-04-13T01:42:15Z" level=info msg="Preparing server http &{Network: Address::80 TLS:<nil> Redirect:<nil> Auth:<nil> WhitelistSourceRange:[] Compress:false ProxyProtocol:<nil> ForwardedHeaders:0xc0426010a0} with readTimeout=0s writeTimeout=0s idleTimeout=3m0s"
time="2018-04-13T01:42:15Z" level=info msg="Starting server on :8080"
time="2018-04-13T01:42:15Z" level=info msg="Starting server on :80"
time="2018-04-13T01:42:15Z" level=info msg="Starting provider *servicefabric.Provider {"Watch":false,"Filename":"","Constraints":null,"Trace":false,"DebugLogGeneratedTemplate":false,"ClusterManagementURL":"https://servicefabric-prod02.xxxxxxxx.com:19080","APIVersion":"3.0","RefreshSeconds":0,"TLS":{"CA":"","CAOptional":false,"Cert":"certs/servicefabric.crt","Key":"certs/servicefabric.key","InsecureSkipVerify":true}}"
time="2018-04-13T01:42:25Z" level=info msg="Checking service fabric config"
time="2018-04-13T01:42:25Z" level=error msg="Provider connection error: template: :59:53: executing "" at <$service.Name>: wrong type for value; expected servicefabric.ServiceItemExtended; got string; retrying in 502.60397ms"
time="2018-04-13T01:42:36Z" level=info msg="Checking service fabric config"
time="2018-04-13T01:42:36Z" level=error msg="Provider connection error: template: :59:53: executing "" at <$service.Name>: wrong type for value; expected servicefabric.ServiceItemExtended; got string; retrying in 1.08877612s"
time="2018-04-13T01:42:47Z" level=info msg="Checking service fabric config"
time="2018-04-13T01:42:47Z" level=error msg="Provider connection error: template: :59:53: executing "" at <$service.Name>: wrong type for value; expected servicefabric.ServiceItemExtended; got string; retrying in 308.484166ms"
time="2018-04-13T01:42:58Z" level=info msg="Checking service fabric config"
time="2018-04-13T01:42:58Z" level=error msg="Provider connection error: template: :59:53: executing "" at <$service.Name>: wrong type for value; expected servicefabric.ServiceItemExtended; got string; retrying in 503.160157ms" 
lawrencegripper commented 6 years ago

This looks good to me, sorry for the bug - my bad :(

ldez commented 6 years ago

@lawrencegripper could you validate with the review system

lawrencegripper commented 6 years ago

Would you like me to pull this down and run a test against a live cluster with a stateful service before we merge?