fix: make Run not to fail if getK8sClientFunc returns an err which is appropriate only for 8.17.x

pkoutsovasilis · pkoutsovasilis · commit e04c85f2dfbe · 2025-02-14T20:36:37.000+02:00
diff --git a/internal/pkg/composable/providers/kubernetessecrets/kubernetes_secrets.go b/internal/pkg/composable/providers/kubernetessecrets/kubernetes_secrets.go
@@ -107,7 +107,7 @@ func (p *contextProviderK8SSecrets) Run(ctx context.Context, comm corecomp.Conte
 		// signal that the provider has initialized
 		close(p.running)
 		p.logger.Debug(k8sSecretsProviderName, " provider skipped, unable to connect: ", err.Error())
-		return err
+		return nil
 	}
 	p.clientMtx.Lock()
 	p.client = client

Original file line number	Diff line number	Diff line change
`@@ -107,7 +107,7 @@ func (p *contextProviderK8SSecrets) Run(ctx context.Context, comm corecomp.Conte`
`107`	`107`	`// signal that the provider has initialized`
`108`	`108`	`close(p.running)`
`109`	`109`	`p.logger.Debug(k8sSecretsProviderName, " provider skipped, unable to connect: ", err.Error())`
`110`		`- return err`
	`110`	`+ return nil`
`111`	`111`	`}`
`112`	`112`	`p.clientMtx.Lock()`
`113`	`113`	`p.client = client`