Onboard clients only if they are on same version as provider - 2

- Operator major and minor version of client and provider should match for onboarding process to begin - After this commit older clients will fail to get onboarded Signed-off-by: Leela Venkaiah G <[email protected]>
red-hat-storage · Dec 7, 2023 · d2dc5c4 · d2dc5c4
1 parent a0a6503
commit d2dc5c4
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)