-
Notifications
You must be signed in to change notification settings - Fork 494
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
V3 breaks by somehow assuming the use of OIDC #792
Comments
If you aren't intending to use OIDC then what are you attempting to use? The set of inputs you have here indicate you want to use OIDC https://github.com/aws-actions/configure-aws-credentials#using-this-action, this hasn't changed from v2, are you trying to use credentials that exist in your runner already? |
I'm Assuming Role using existing credentials. It worked for me in v2, so I was surprised when the v3 update broke this. Was the usage of the workflow for my use case using the wrong parameters? |
The way v2 used existing credentials was inconsistent depending on certain ways the workflow was setup - I honestly didn't know the setup you have was valid in v2 as it was undocumented, and since I've taken over I don't think I've seen anyone have a setup like this that worked. So, I didn't consider this case for v3 when it was necessary to rewrite the logic. I'd like v3 to be fully backwards compatible, so I'll see if I can smoothly fit this use case back in tomorrow. Until then, you could continue using v2, or you can set the |
I'm having the same issue. My scenario is self-hosted runners in EKS with IRSA. I have no need for the Github OIDC endpoint. My configuration worked great in v2 but now breaks in v3.
edit: |
** Note ** |
Describe the bug
The action fails and somehow assumes the use of OIDC, even though it wasn't being used / configured.
Expected Behavior
No breakage when updating to V3
Current Behavior
Action fails with the following error:
Reproduction Steps
Possible Solution
No response
Additional Information/Context
No response
The text was updated successfully, but these errors were encountered: