Skip to content

Commit 6c34080

Browse files
committed
legacy wildcard mode in sotw and delta packages
Signed-off-by: Zhiyan Foo <[email protected]>
1 parent 42a731f commit 6c34080

File tree

3 files changed

+36
-10
lines changed

3 files changed

+36
-10
lines changed

pkg/server/config/config.go

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -49,22 +49,12 @@ func (o Opts) IsLegacyWildcardActive(typeURL string) bool {
4949
// this allows for easy inference as to which opt applies to what implementation.
5050
type XDSOption func(*Opts)
5151

52-
// DeactivateLegacyWildcard deactivates legacy wildcard mode for all resource types.
53-
// In legacy wildcard mode, empty requests to a stream, are treated as wildcard requests as long
54-
// as there is no request made with resources or explicit wildcard requests on the same stream.
55-
// When deactivated, empty requests are treated as a request with no subscriptions to any resource.
56-
// This is recommended for when you are using the go-control-plane to serve grpc-xds clients.
57-
// These clients never want to treat an empty request as a wildcard subscription.
5852
func DeactivateLegacyWildcard() XDSOption {
5953
return func(o *Opts) {
6054
o.legacyWildcardDeactivated = true
6155
}
6256
}
6357

64-
// DeactivateLegacyWildcardForTypes deactivates legacy wildcard mode for specific resource types.
65-
// In legacy wildcard mode, empty requests to a stream, are treated as wildcard requests as long
66-
// as there is no request made with resources or explicit wildcard requests on the same stream.
67-
// When deactivated, empty requests are treated as a request with no subscriptions to any resource.
6858
func DeactivateLegacyWildcardForTypes(types []string) XDSOption {
6959
return func(o *Opts) {
7060
typeMap := make(map[string]struct{}, len(types))

pkg/server/delta/v3/server.go

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,24 @@ func WithLogger(logger log.Logger) config.XDSOption {
5757
}
5858
}
5959

60+
// DeactivateLegacyWildcard deactivates legacy wildcard mode for all resource types.
61+
// In legacy wildcard mode, empty requests to a stream, are treated as wildcard requests as long
62+
// as there is no request made with resources or explicit wildcard requests on the same stream.
63+
// When deactivated, empty requests are treated as a request with no subscriptions to any resource.
64+
// This is recommended for when you are using the go-control-plane to serve grpc-xds clients.
65+
// These clients never want to treat an empty request as a wildcard subscription.
66+
func DeactivateLegacyWildcard() config.XDSOption {
67+
return config.DeactivateLegacyWildcard()
68+
}
69+
70+
// DeactivateLegacyWildcardForTypes deactivates legacy wildcard mode for specific resource types.
71+
// In legacy wildcard mode, empty requests to a stream, are treated as wildcard requests as long
72+
// as there is no request made with resources or explicit wildcard requests on the same stream.
73+
// When deactivated, empty requests are treated as a request with no subscriptions to any resource.
74+
func DeactivateLegacyWildcardForTypes(types []string) config.XDSOption {
75+
return config.DeactivateLegacyWildcardForTypes(types)
76+
}
77+
6078
// NewServer creates a delta xDS specific server which utilizes a ConfigWatcher and delta Callbacks.
6179
func NewServer(ctx context.Context, config cache.ConfigWatcher, callbacks Callbacks, opts ...config.XDSOption) Server {
6280
s := &server{

pkg/server/sotw/v3/server.go

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -74,6 +74,24 @@ func WithLogger(logger log.Logger) config.XDSOption {
7474
}
7575
}
7676

77+
// DeactivateLegacyWildcard deactivates legacy wildcard mode for all resource types.
78+
// In legacy wildcard mode, empty requests to a stream, are treated as wildcard requests as long
79+
// as there is no request made with resources or explicit wildcard requests on the same stream.
80+
// When deactivated, empty requests are treated as a request with no subscriptions to any resource.
81+
// This is recommended for when you are using the go-control-plane to serve grpc-xds clients.
82+
// These clients never want to treat an empty request as a wildcard subscription.
83+
func DeactivateLegacyWildcard() config.XDSOption {
84+
return config.DeactivateLegacyWildcard()
85+
}
86+
87+
// DeactivateLegacyWildcardForTypes deactivates legacy wildcard mode for specific resource types.
88+
// In legacy wildcard mode, empty requests to a stream, are treated as wildcard requests as long
89+
// as there is no request made with resources or explicit wildcard requests on the same stream.
90+
// When deactivated, empty requests are treated as a request with no subscriptions to any resource.
91+
func DeactivateLegacyWildcardForTypes(types []string) config.XDSOption {
92+
return config.DeactivateLegacyWildcardForTypes(types)
93+
}
94+
7795
type server struct {
7896
cache cache.ConfigWatcher
7997
callbacks Callbacks

0 commit comments

Comments
 (0)