Skip to content

Adding revert_paths to TransactionIntent#107

Open
steiler wants to merge 3 commits intomainfrom
deviationClear
Open

Adding revert_paths to TransactionIntent#107
steiler wants to merge 3 commits intomainfrom
deviationClear

Conversation

@steiler
Copy link
Contributor

@steiler steiler commented Feb 24, 2026

This PR adds support for reverting specific paths in transaction intents by introducing a new revert_paths field to the TransactionIntent message.

Changes

  • data.proto: Added revert_paths field (field number 10) to TransactionIntent message

    • Type: repeated schema.Path
    • Purpose: Stores paths that are meant to be reverted during transaction processing
  • sdcpb/path_additions.go: Added IsParentPathOf helper method

    • Checks if a path is a parent of another path
    • Compares origin, target, and root-based status
    • Handles special case for last element without keys (matches on name only)
  • sdcpb/data.pb.go: Auto-generated protobuf code updates

  • sdcpb/paths.go: Auto-generated code updates

Use Case

The revert_paths field allows transaction intents to explicitly specify which paths should be reverted, providing control over configuration reversion as part of a transaction.

@steiler steiler requested a review from a team as a code owner February 24, 2026 12:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant