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

add get_config() #65

Closed
dbarrosop opened this issue Sep 9, 2016 · 4 comments
Closed

add get_config() #65

dbarrosop opened this issue Sep 9, 2016 · 4 comments
Milestone

Comments

@dbarrosop
Copy link
Member

dbarrosop commented Sep 9, 2016

Original ticket napalm-automation/napalm#291 created by @grizz

Add a function to get the full config as a text blob

Would obviously be platform specific, but would be handy for using NAPALM for development and device changes.

@dbarrosop
Copy link
Member Author

The description of the method can be found here:

https://github.com/napalm-automation/napalm-base/pull/69/files

Thoughts?

@dbarrosop
Copy link
Member Author

@grizz let me know if this proposal looks good to you.

@grizz
Copy link

grizz commented Sep 12, 2016

Looks good, thanks for stubbing it in. I wonder if it would be better to pass a flag to determine which configs to get since it can take a while on some platforms?

@mirceaulinic
Copy link
Member

This has been implemented on all drivers, except IBM - which we consider deprecated.
So this is fixed IMO.

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

3 participants