search

cloudfoundry / diego-release

BOSH Release for Diego
Apache License 2.0
201 stars 212 forks source link

Implement per-route options in route-emitter #940

Closed maxmoehl closed 1 week ago

maxmoehl commented 3 months ago

Proposed Change

In RFC0027 Generic Per-Route Features we decided to implement a new field for routes which can contain additional, custom configuration for a route. Together with this new feature we also want to implement the first property making use of it: customisable load balancing algorithms.

This new field must be implemented in route-emitter to announce this field via nats to gorouter. This depends on support in BBS which is tracked in #939. If possible, route-emitter should not consider the contents of the per-route options. It can expect that the content is JSON encoded and can be emitted as part of the message as-is.

Acceptance criteria

Related links

a18e commented 1 week ago

@maxmoehl both https://github.com/cloudfoundry/routing-info/pull/6 and https://github.com/cloudfoundry/route-emitter/pull/47 have been merged. That should cover the AC for this issue. I think we can resolve this issue and move on to CC.

maxmoehl commented 1 week ago

Thanks, closing!