-
Notifications
You must be signed in to change notification settings - Fork 80
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Setting AuthenticationMode
to ApiAndConfigMap
breaks
#1435
Comments
If I try to add a new resource I am getting:
|
Thank you for sending a repro! I'm running this to see if I can get the same issue. Indeed do. While we are chasing this down, have you found an acceptable workaround? I am noticing that this configuration setting is deprecated.
Tangentially, our team is also working on a 3.x.x release that will come with a migration guide to get off these deprecated entries. |
I can reproduce. There is something I found in digging further. The gRPC Construct response indicates a Resource reference to an AWS AcessEntry resource.
That seems consistent with the code: https://github.com/pulumi/pulumi-eks/blob/master/nodejs/eks/authenticationMode.ts#L183
However the schema indicates a type:
This was introduced in #1171 I think this cannot work properly as language runtime support does not guarantee that type references can be substituted into normal types for all languages. |
@flostadler do you think this could be fixed if the provider implementation changed to repackage aws.eks.AccessEntry resource into a plain object corresponding to the #/types/eks:index:AccessEntry" type-spec so that it does not project as a resource reference on the wire? |
@t0yv0 yeah we should change the provider implementation to send @avinassh you could try omitting the |
What happened?
If you set
AuthenticationMode
inClusterArgs
, the resources are setup correctly, but at the end it fails with following error:The error happens if you run
pulumi up
again or add any resources to the stack, thus blocking any further operationsExample
a reproducer code:
Output of
pulumi about
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).
The text was updated successfully, but these errors were encountered: