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

Dynamic inventory: Error handling and debugging must be improved #28

Open
jbuchhammer opened this issue May 20, 2019 · 0 comments
Open

Comments

@jbuchhammer
Copy link

Description

As it showed up in issues #26 and #27 error handling in profitbricks_inventory.py is rudimentary. Only a few parts are covered by try/except clauses and debugging information on the context of the error is missing completely. As a user I have to manually edit the inventory script to find the ressource where the error occurred and get more details.

Expected Behaviour

An error handling must be implemented that

  • provides detailed information on the context of the error (e.g. affected resource)
  • catches errors early
  • automatically tries to recover from errors if possible, e.g. wait and retry if API calls return a PBRateLimitExceededError or skip resources that are not suitable (as in Dynamic inventory fails if a server has no boot device #27).

In addition, a logging with configurable log level would be helpful.

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

No branches or pull requests

1 participant