Feature: Allow weight tag in Consul

[composer22]

"registrator" of paths into consul should also allow weight parameter to be passed so that traffic can be balanced to env needs. Some services may be running on nodes with less resources. Right now, rebalancing is distributed evenly.

Enhancement example:

docker run -d
-e "SERVICE_TAGS=urlprefix-/hello-world:0" 
--name hello_world_1  \

results in...

/hello-world
  /hello_world_1   100%

adding another specific weighted service...

docker run -d
-e "SERVICE_TAGS=urlprefix-/hello-world:20" 
--name hello_world_2  \

would result in...

/hello-world
  /hello_world_1   80%
  /hello_world_2   20%

[magiconair]

I like the idea but would prefer a different syntax. Especially with the idea of the tcp proxy I'm thinking about using a URL for this:

p-[scheme]://[host][:port][/path][?opt=val[&opt=val]...]

Examples:

p-tcp://host:port
p-http://host:port/hello
p-https://host/hello
p-ws:///hello
p-/hello (equivalent to p-http:///hello)
p-/hello?weight=0.8
...

urlprefix-host/path then translates to p-http://host/path and p-https://host/path

[composer22]

@magiconair I like it.

Let your imagination fly!

[tiago-msilva]

Hi guys,

Any news on this matter?
I'm really interested in getting weights on TCP proxy mode.

Thanks!

[nanoz]

Maybe we can help developing this feature ? If so, what would be the syntax ?

/path weight=xxx ?

[magiconair]

@nanoz and @tiago-msilva I thought that this would be more work but the route add command already supports a weight option which does the right thing. The only thing missing was to expose this via the urlprefix- tag. You can now add a weight=.x option to specify a weight on a target, e.g. urlprefix-:3306 weight=0.2 proto=tcp

Sorry, @nanoz I did not want to take the opportunity for contributing away from you but was done pretty much when I looked at the the place which I wanted to point out to you.

To illustrate this I've added a new detail option for the log.routes.format which you can use to dump the routing table in a different format.

./fabio -log.routes.format detail
2017/06/19 11:51:14 [INFO] Updated config to
+-- host=:3306
|   +-- path=
|       |-- addr=127.0.0.1:5001 weight 0.20 slots 2000/10000
|       +-- addr=127.0.0.1:5000 weight 0.80 slots 8000/10000
+-- host=:3307
    +-- path=
        +-- addr=127.0.0.1:5002 weight 1.00 slots 1/1

Would you mind testing this? This is on the issue-42-weigh-targets branch.

[tiago-msilva]

@magiconair
First of all thank you for looking into this matter.

I don't know a thing about go.
I tried to build this with:
git clone ...
git checkout issue-42-weigh-targets
cd fabio
go build .
and I got this:
./main.go:399: t.Dump undefined (type route.Table has no field or method Dump)
Does this help out?

[magiconair]

try this:

mkdir ~/go
go get -u github.com/fabiolb/fabio
cd ~/go/src/github.com/fabiolb/fabio
git checkout issue-42-weigh-targets
go install
ls ~/go/bin/fabio

[magiconair]

= go1.8.3