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

Providers::AWS calls nonexistent Cheffish::MergedConfig#delete #57

Closed
justindossey opened this issue Sep 19, 2014 · 0 comments
Closed
Labels
Type: Bug Doesn't work as expected.
Milestone

Comments

@justindossey
Copy link
Contributor

in Providers::AWS.get_aws_credentials, the driver_options :aws_config_file and :aws_csv_file are loaded using:

265           if driver_options[:aws_config_file]
266             aws_credentials.load_ini(driver_options.delete(:aws_config_file))
267           elsif driver_options[:aws_csv_file]
268             aws_credentials.load_csv(driver_options.delete(:aws_csv_file))
269           else
270             aws_credentials.load_default
271           end

In this context, driver_options is an instance of Cheffish::MergedConfig, which does not support a delete method. Therefore, :aws_config_file and :aws_csv_file do not work.

@jkeiser jkeiser added this to the 1.0 milestone Nov 6, 2014
@jkeiser jkeiser removed the ready label Jan 26, 2015
@tas50 tas50 added Type: Bug Doesn't work as expected. and removed Bug labels Jul 31, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Doesn't work as expected.
Development

No branches or pull requests

3 participants