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

Work around EC2 bug #241

Closed
wants to merge 1 commit into from
Closed

Work around EC2 bug #241

wants to merge 1 commit into from

Conversation

maf23
Copy link

@maf23 maf23 commented Dec 14, 2013

Added a rescue clause since ec2 may return a 404 for a resource it has
said it can provide. This is an ec2 bug which I have reported to them,
but while we wait for them to fix it this patch is needed since it
gets chef working again.

Without this patch ohai returns no ec2 data at all when one of the
resources return 404.

says it can provide. This is an ec2 bug which I have reported to them,
but while we wait for them to fix it this patch is usable since it
gets chef working again.

Without this patch ohai returns no ec2 data at all when one of the
resources return 404.
@sersut
Copy link

sersut commented May 21, 2014

As a rule of thumb we don't want to have rescue clauses without exception type unless we have to @maf23. Also I believe #332 will be a solution to your problem.

Please reopen if it doesn't work for you and we can investigate.

@sersut sersut closed this May 21, 2014
@chef chef locked and limited conversation to collaborators Nov 16, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants