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.

Sign up for GitHub

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

Jump to bottom

Fix for Aws MissingCredentialsError #634

Merged
merged 1 commit into from
Jan 15, 2020
Merged

Fix for Aws MissingCredentialsError #634

merged 1 commit into from
Jan 15, 2020

Conversation

kapilchouhan99
Copy link

Signed-off-by: Kapil Chouhan kapil.chouhan@msystechnologies.com

Description

  • In the plugin_setup! we are calling connection_protocol_ec2 method, so at this time it initialize Aws::EC2::Client before reading aws credentials file, so aws_access_key_id and aws_secret_access_key values are not available at this time, so I have fixed it.
  • Ensured chef-style on the code changes made

Issues Resolved

Fixes: #632

Check List

Fix for Aws MissingCredentialsError
c04a0a7 
Signed-off-by: Kapil Chouhan <kapil.chouhan@msystechnologies.com>
@kapilchouhan99 kapilchouhan99 requested a review from tas50 January 15, 2020 11:27
@tas50 tas50 merged commit 09a8166 into master Jan 15, 2020
@chef-expeditor chef-expeditor bot deleted the Kapil/MSYS-1234_Aws_Errors_MissingCredentialsError_unable_to_sign_request_without_credentials_set branch January 15, 2020 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tas50 tas50 Awaiting requested review from tas50

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Another ERROR: Aws::Errors::MissingCredentialsError: unable to sign request without credentials set
2 participants
@kapilchouhan99 @tas50