Skip to content
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

validate_credentials needs further consideration #31

Open
venkatBala opened this issue Dec 15, 2022 · 0 comments
Open

validate_credentials needs further consideration #31

venkatBala opened this issue Dec 15, 2022 · 0 comments
Assignees

Comments

@venkatBala
Copy link
Contributor

venkatBala commented Dec 15, 2022

  • Validating credentials when an instance of AWSExecutor is created results in a few problems when interacting with covalent self hosted
  • If the AWS_PROFILE env variable is set, the client side configuration file gets rendered with profile=<AWS_PROFILE> value. When a user then dispatches a workflow to self-hosted the executor will get populated with the profile information parsed from the configuration file which eventually will cause an error on the self-hosted side as there will be no values for the profile.
  • A fix would be to not run credential validation on executor instantiation and rather let boto3 session calls fail downstream if the credentials are expired
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants