Skip to content

Change ClusterId to a string, use as opaque config reference. #32

@smawson

Description

@smawson

In the proxy config we currently have a ClusterId which is a complex type, and is expected to match exactly in multiple places in the config. This seems onerous, and instead we should have a simple string that is used for matching between multiple places in the config, and leave the tags part to the UpstreamCluster definition.

For example:

message UpstreamCluster {
string cluster_id = 1;
repeated string tags = 2;

string health_check_endpoint = 3;
...
}

message WeightedCluster {
// Must match a cluster_id from an UpstreamCluster definition.
string dst_cluster = 1;
uint32 weight = 2;
}

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions