Skip to content

Conversation

@benoit-nexthop
Copy link
Contributor

@benoit-nexthop benoit-nexthop commented Dec 18, 2025

Pre-submission checklist

  • I've ran the linters locally and fixed lint errors related to the files I modified in this PR. You can install the linters by running pip install -r requirements-dev.txt && pre-commit install
  • pre-commit run

Summary

This adds just the command fboss2 config history.

Note: this change is part of a series, the previous one is #758, the next one is #760.

Test Plan

Unit tests.

Sample usage

[admin@fboss101 benoit]$ ./fboss2-dev config history
 Revision  Owner  Commit Time         
------------------------------------------
 r1        root   2025-11-05 13:26:58 
 r2        root   2025-11-05 13:27:35 
 r3        root   2025-11-05 13:28:13 
 r4        admin  2025-11-05 14:37:51 
 r5        admin  2025-11-05 14:38:31

@meta-cla meta-cla bot added the CLA Signed label Dec 18, 2025
@benoit-nexthop benoit-nexthop marked this pull request as ready for review December 18, 2025 00:37
@benoit-nexthop benoit-nexthop force-pushed the fboss2-cli-prototype_part07 branch 2 times, most recently from 0e8e5eb to 249c7cf Compare December 23, 2025 21:39
@benoit-nexthop benoit-nexthop force-pushed the fboss2-cli-prototype_part07 branch 5 times, most recently from a965dbb to 0fe0152 Compare December 30, 2025 15:53
@benoit-nexthop benoit-nexthop force-pushed the fboss2-cli-prototype_part07 branch 3 times, most recently from b57e4d2 to 2fe0271 Compare January 7, 2026 06:29
@benoit-nexthop benoit-nexthop force-pushed the fboss2-cli-prototype_part07 branch 4 times, most recently from f1b4246 to 575d211 Compare January 14, 2026 21:12
meta-codesync bot pushed a commit that referenced this pull request Jan 15, 2026
Summary:
**Pre-submission checklist**
- [x] I've ran the linters locally and fixed lint errors related to the files I modified in this PR. You can install the linters by running `pip install -r requirements-dev.txt && pre-commit install`
- [x] `pre-commit run`

This adds just the command `fboss2-dev config rollback`.

Note: this change is part of a series, the previous one is #757, the next one is #759.

Pull Request resolved: #758

Test Plan:
Unit tests.

## Sample usage

In this state:
```
[admin@fboss101 benoit]$ ./fboss2-dev config history
 Revision  Owner  Commit Time
------------------------------------------
 r1        root   2025-11-05 13:26:58
 r2        root   2025-11-05 13:27:35
 r3        root   2025-11-05 13:28:13
 r4        admin  2025-11-05 14:37:51
 r5        admin  2025-11-05 14:38:31
```
One can do `fboss2 config rollback r3` or just `fboss2 config rollback` to just go back to whatever was the last revision. Doing so creates a new revision (`r6` in this example) that is identical to the one we rolled back to.

Reviewed By: shiva-menta

Differential Revision: D90739629

Pulled By: joseph5wu

fbshipit-source-id: 5acef8e3a63a0bee5116526fb5baf47225a79c03
@benoit-nexthop benoit-nexthop force-pushed the fboss2-cli-prototype_part07 branch from 575d211 to 14fdc36 Compare January 15, 2026 21:35
@meta-codesync
Copy link

meta-codesync bot commented Jan 15, 2026

@joseph5wu has imported this pull request. If you are a Meta employee, you can view this in D90801711.

@meta-codesync
Copy link

meta-codesync bot commented Jan 16, 2026

@joseph5wu merged this pull request in 6f38394.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants