Skip to content

[Nexthop][fboss2-dev] Document the difference between set and config operations - #1621

Closed
benoit-nexthop wants to merge 1 commit into
meta:mainfrom
nexthop-ai:config-vs-set-docs
Closed

benoit-nexthop wants to merge 1 commit into
meta:mainfrom
nexthop-ai:config-vs-set-docs

Conversation

@benoit-nexthop

@benoit-nexthop benoit-nexthop commented Sep 10, 2026 •

Copy link
Copy Markdown
Contributor

Summary

This adds some additional text to fboss_cli.md detailing the differences between the fboss2/fboss2-dev set and config commands, and what one should expect from each:

  • set is a runtime-only write via a thrift client to the running agent; nothing is persisted.
  • config edits the configuration file through a staged session and is persisted on commit.
  • A config commit or reload (or a warm/cold boot) re-applies config and reverts any set.
  • set changes are invisible to config session diff / config history.

Test Plan

Docs-only change; no test additions or changes. Confirmed with visual inspection of the generated output using docusaurus.

Co-Authored-By: Steve Ulrich sulrich@nexthop.ai

@benoit-nexthop
benoit-nexthop requested a review from a team as a code owner September 10, 2026 07:23
@meta-cla meta-cla Bot added the CLA Signed label Sep 10, 2026
@benoit-nexthop

Copy link
Copy Markdown
Contributor Author

cc @sulrich-nexthop

@benoit-nexthop

Copy link
Copy Markdown
Contributor Author

/autorebase

Update fboss_cli.md with an explanation of the differences between
`set` and `config` and what one should expect from each command.

# Summary

This adds some additional text to `fboss_cli.md` detailing the
differences between the fboss2(-dev) `set` and `config` commands.

# Test Plan

No test additions / changes. Confirmed with visual inspection of the
generated output using docusaurus.

Co-Authored-By: steve ulrich <sulrich@nexthop.ai>
@meta-codesync

meta-codesync Bot commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

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

@meta-codesync meta-codesync Bot closed this in d4bab34 Oct 5, 2026
@meta-codesync meta-codesync Bot added the Merged label Oct 5, 2026
@meta-codesync

meta-codesync Bot commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

@AarjunC merged this pull request in d4bab34.

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.

1 participant