replication-manager icon indicating copy to clipboard operation
replication-manager copied to clipboard

does the parameter "registry-servers" support remote consul servers now?

Open tangweichun opened this issue 5 years ago • 2 comments

does the parameter "registry-servers" support remote consul servers now? for example,i have a consul cluster:192.168.56.10,192.168.56.11,192.168.56.12, can i set registry-servers='192.168.56.10,192.168.56.11,192.168.56.12'?

tangweichun avatar Sep 12 '18 06:09 tangweichun

Hello,

Did you already tried it ? the code looks to pass in consul client option all the hosts separated per ","

Thanks /svar

   var opt registry.Options
if cluster.Conf.RegistryConsul == false || cluster.IsActive() == false {
	return nil
}
opt.Addrs = strings.Split(cluster.Conf.RegistryHosts, ",")

scrambledb avatar Sep 12 '18 07:09 scrambledb

Yes,

Indeed changing this would mean embedding consul agent. We welcome contribution or sponsoring.

But anyway i'll keep this feature in mind for the roadmap.

scrambledb avatar Sep 12 '18 08:09 scrambledb