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
Lcg/fix network listeners #261
Conversation
need to require 'sigar' early so that it blows up during the definition rather than execution with a LoadError (is this a bug in ohai core? this feels like a hack). also drop hpux and aix specificity, not sure why that got added.
i don't think this platform makes any sense, suspect we need to remove it everywhere
back out the require 'sigar' hack
- setup a class heirarchy of exceptions so i can easily rethrow any internal ohai error - catch all the non-internal errors from running the plugin and just warn on those - convert the argumenterror raises to ohai invalidplugin errors
class DependencyCycle < Exception; end | ||
class DependencyNotFound < Exception; end | ||
class AttributeSyntaxError < Exception; end | ||
class Error < Exception; end |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe we should change this to inherit from StandardError
? The difference being that just rescue
won't catch Exception
, you have to rescue Exception
explicitly. I don't think any of these exceptions represent bad enough situations that we want to force downstream users into using rescue Exception
.
@danielsdeleo yeah, agreed, fixed that |
LGTM, though travis found a test failure. |
@danielsdeleo annoying to track down, but fixed |
Lcg/fix network listeners
No description provided.