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

Change rollback behavior #2005

Closed
decoupca opened this issue Sep 6, 2023 · 0 comments
Closed

Change rollback behavior #2005

decoupca opened this issue Sep 6, 2023 · 0 comments

Comments

@decoupca
Copy link
Contributor

decoupca commented Sep 6, 2023

Based on this discussion.

Currently, calling rollback() on NXOS does nothing unless napalm is aware of making changes. However, in some cases it makes sense to call rollback() in a new napalm session.

Proposed behavior is that rollback() will become agnostic to any changes napalm is aware of, and will have the net effect of reverting to the most recent state. For platforms with a native rollback feature, this will revert to the most recent snapshot. Platforms that don't, such as IOS, will attempt to revert to the rollback state that napalm created in a previous session. If such a rollback state does not exist, an exception will raise.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants