search

gliderlabs / registrator

Service registry bridge for Docker with pluggable adapters
http://gliderlabs.com/registrator
MIT License
4.66k stars 912 forks source link

Convert from check.Script to check.Args for Consul API changes #669

Closed pmundt closed 4 years ago

pmundt commented 5 years ago

In upstream Consul (as of May 2018), check.Script has been removed in favour of check.Args, which now results in the following build error:

consul/consul.go:108:8: check.Script undefined (type api.AgentServiceCheck has no field or method Script) consul/consul.go:110:8: check.Script undefined (type api.AgentServiceCheck has no field or method Script) consul/consul.go:121:10: check.Script undefined (type *api.AgentServiceCheck has no field or method Script)

Update to use the new convention.

nicofuccella commented 5 years ago

+1