-
Notifications
You must be signed in to change notification settings - Fork 660
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
[BUG] Keyring errors when intialising flyte remote client #4354
Comments
#take |
this is the weird part of the stack trace
if there's no 401 received, we shouldn't be trying to instantiate a |
hmmm I guess we always call upgrade_channel_to_authenticated which will always call get_authenticator which will always initialize the pkce authenticator if that's the selected option. can we delay this somehow please? can we make it so that only when there's an error, do we create the authenticator? or something like that. |
@Tom-Newton can you help me test this one? |
Describe the bug
Get a keyring error when initialising a simple flyte remote client. It seems to only happen on some systems and the exact error varies slightly. I've seen
InitError: Failed to create the collection: Prompt dismissed..
KeyringLocked: Failed to unlock the collection!
Expected behavior
I'm not trying to use any authentication so I would expect it to ignore any exceptions about the system keyring. Alternatively we could add
AuthType.NONE
so that all keyvault related code can be avoided.Additional context to reproduce
On my system I can reproduce with:
This is SSHing from one linux system to another linux system. Both are used with a Desktop setup so they probably have the GNOME keyring installed.
I tried using all the different Auth types but they all failed for one reason or another (this is largely expected because we do not have any authentication configured on our flyte deployment).
Full stack trace:
stack_trace.txt
Screenshots
No response
Are you sure this issue hasn't been raised already?
Have you read the Code of Conduct?
The text was updated successfully, but these errors were encountered: