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

S3Source does not extend any Chef classes, so config is not an inherited attribute #667

Merged
merged 1 commit into from
Mar 25, 2021

Conversation

signe
Copy link
Contributor

@signe signe commented Mar 8, 2021

S3Source does not extend any Chef classes, so config is not an inherited attribute and as such needs to be passed in to fetch() explicitly

Description

All calls to S3Source.fetch were failing because none of the AWS access information was available using config attribute. (Modified in 2cc491d)

Also fixes rspec tests for S3Source which have been broken and untested for... a long time. At least since 2cc491d, but it appears much longer.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Chore (non-breaking change that does not add functionality or fix an issue)

Checklist:

  • I have read the CONTRIBUTING document.
  • I have run the pre-merge tests locally and they pass.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • All commits have been signed-off for the Developer Certificate of Origin.

…ted attribute and as such needs to be passed in to fetch() explicitly

Signed-off-by: Kat Rossiter <[email protected]>
@tas50 tas50 merged commit dd68fc6 into chef:master Mar 25, 2021
@tas50
Copy link
Contributor

tas50 commented Mar 25, 2021

Thanks @signe

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

Successfully merging this pull request may close these issues.

2 participants