Skip to content

Conversation

afritzler
Copy link
Member

Proposed Changes

As the client interface has changed with kube 1.26 the GetOptions has now to be passed in the GET method.

Otherwise you might end up with the following error:

cannot use s (variable of type *memorystore.Store) as client.Client value in argument to utils.GetSecretSelector: *memorystore.Store does not implement client.Client (wrong type for method Get)
                have Get(_ context.Context, objectKey types.NamespacedName, obj client.Object) error
                want Get(ctx context.Context, key types.NamespacedName, obj client.Object, opts ...client.GetOption) error (typecheck)

@afritzler afritzler requested a review from a team as a code owner January 16, 2023 12:23
@afritzler afritzler requested a review from adracus January 16, 2023 12:23
@afritzler afritzler added the enhancement New feature or request label Jan 16, 2023
As the client interface has changed with kube 1.26 the `GetOptions` has
now to be passed in the GET method.

Otherwise you might end up with the following error:
```
cannot use s (variable of type *memorystore.Store) as client.Client value in argument to utils.GetSecretSelector: *memorystore.Store does not implement client.Client (wrong type for method Get)
                have Get(_ context.Context, objectKey types.NamespacedName, obj client.Object) error
                want Get(ctx context.Context, key types.NamespacedName, obj client.Object, opts ...client.GetOption) error (typecheck)
```
@afritzler afritzler force-pushed the enh/client-get-options branch from 9b42b19 to 647d62e Compare January 16, 2023 13:10
@github-actions github-actions bot added size/S and removed size/XS labels Jan 16, 2023
@adracus adracus merged commit 7f5baf1 into main Jan 16, 2023
@adracus adracus deleted the enh/client-get-options branch January 16, 2023 13:56
@hardikdr hardikdr added the area/iaas Issues related to IronCore IaaS development. label Jun 26, 2025
@hardikdr hardikdr added this to Roadmap Jun 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/iaas Issues related to IronCore IaaS development. enhancement New feature or request size/S
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

3 participants