Skip to content
This repository has been archived by the owner on Sep 17, 2019. It is now read-only.

Better exceptions #218

Open
mirceaulinic opened this issue Mar 6, 2017 · 1 comment
Open

Better exceptions #218

mirceaulinic opened this issue Mar 6, 2017 · 1 comment

Comments

@mirceaulinic
Copy link
Member

We will need to create few more exceptions, to avoid as much as possible propagating exceptions from the underlying libraries.

Some examples:

  • LockError
  • UnlockError
  • CommitError
  • ConnectAuthError
  • ConnectTimeoutError
@mirceaulinic
Copy link
Member Author

Solved under #248, but leaving this open for future ideas.

bewing added a commit to bewing/napalm-base that referenced this issue Jun 22, 2017
* 'develop' of github.com:bewing/napalm-base: (53 commits)
  Documentation: Fix example mac addresses to be proper form
  Fix mac address format
  Version 0.24.1
  Test when as number is int
  Fix the as_number helper
  Adding note about is_up and positive uptime
  Add reference example for get_bgp_neighbors
  Add numeric compare for validation (napalm-automation#243)
  Reword docstring as per David's suggestions
  Prepare release 0.24.0
  Defien more exceptions as per napalm-automation#218
  New exception: ConnectionClosedException
  Fixing unneeded import
  Removing some obsolete PY3 transition code
  Fixing test name error
  Remove unneeded print statement
  Updating unit tests for new helpers; updating PY3 support to include fortinet
  Working on helpers
  napalm-base release 0.23.3
  Adding more tests; fix pylama
  ...
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant