Merge pull request #3504 from eduser25/fix-sds

Do not send SDS on full config updates
openservicemesh · Jun 3, 2021 · 4c390a5 · 4c390a5
2 parents bf42d5c + c32f3c5
commit 4c390a5
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/pkg/envoy/ads/stream.go b/pkg/envoy/ads/stream.go
@@ -120,7 +120,8 @@ func (s *Server) StreamAggregatedResources(server xds_discovery.AggregatedDiscov
 			}
 
 			// Queue a full configuration update
-			<-s.workqueues.AddJob(newJob(envoy.XDSResponseOrder, nil))
+			// Do not send SDS, let envoy figure out what certs does it want.
+			<-s.workqueues.AddJob(newJob([]envoy.TypeURI{envoy.TypeCDS, envoy.TypeEDS, envoy.TypeLDS, envoy.TypeRDS}, nil))
 
 		case certUpdateMsg := <-certAnnouncement:
 			cert := certUpdateMsg.(events.PubSubMessage).NewObj.(certificate.Certificater)