Merge pull request #2305 from leelavg/5072-onboard

Onboard clients only if they are on same version as provider
red-hat-storage · Dec 6, 2023 · 19293d0 · 19293d0
2 parents df7a769 + 33bc29a
commit 19293d0
Show file tree

Hide file tree

Showing 4 changed files with 191 additions and 158 deletions.
diff --git a/services/provider/client/client.go b/services/provider/client/client.go
@@ -55,14 +55,15 @@ func (cc *OCSProviderClient) Close() {
 
 // OnboardConsumer to validate the consumer and create StorageConsumer
 // resource on the StorageProvider cluster
-func (cc *OCSProviderClient) OnboardConsumer(ctx context.Context, ticket, name string) (*pb.OnboardConsumerResponse, error) {
+func (cc *OCSProviderClient) OnboardConsumer(ctx context.Context, ticket, name, operatorVersion string) (*pb.OnboardConsumerResponse, error) {
 	if cc.Client == nil || cc.clientConn == nil {
 		return nil, fmt.Errorf("provider client is closed")
 	}
 
 	req := &pb.OnboardConsumerRequest{
-		OnboardingTicket: ticket,
-		ConsumerName:     name,
+		OnboardingTicket:      ticket,
+		ConsumerName:          name,
+		ClientOperatorVersion: operatorVersion,
 	}
 
 	apiCtx, cancel := context.WithTimeout(ctx, cc.timeout)