Skip to content

[raft/scd] Extract delete opintent - #1654

Merged
mickmis merged 1 commit into
interuss:masterfrom
Orbitalize:scd_extract_delete_opintent
Sep 2, 2026
Merged

[raft/scd] Extract delete opintent#1654
mickmis merged 1 commit into
interuss:masterfrom
Orbitalize:scd_extract_delete_opintent

Conversation

@MariemBaccari

@MariemBaccari MariemBaccari commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Chained PR: #1627 -> #1642 -> #1643 -> #1644 -> #1645 -> #1646 -> #1649 -> #1650 -> #1651 -> #1653 -> #1654 -> #1656 -> #1657 -> #1655 -> #1666 -> #1667 -> #1668 -> #1669

Extract delete opintent business logic.

@MariemBaccari
MariemBaccari marked this pull request as ready for review August 20, 2026 09:26
@MariemBaccari
MariemBaccari force-pushed the scd_extract_delete_opintent branch from be0e4cf to 44bfcb3 Compare August 20, 2026 09:29
@barroco barroco added the dss-raft Relating to the application-layer consensus implemenation based on raft label Aug 20, 2026
@MariemBaccari
MariemBaccari force-pushed the scd_extract_delete_opintent branch 2 times, most recently from 30cec77 to 25270fa Compare August 20, 2026 15:29
@MariemBaccari
MariemBaccari force-pushed the scd_extract_delete_opintent branch from 25270fa to 6613cf6 Compare August 28, 2026 10:09

@mickmis mickmis left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(6613cf6#r3895361616) LGTM

Comment thread pkg/scd/actions/operational_intents.go Outdated
@MariemBaccari
MariemBaccari force-pushed the scd_extract_delete_opintent branch 2 times, most recently from 3d46207 to a404ca1 Compare September 2, 2026 08:24
@MariemBaccari
MariemBaccari force-pushed the scd_extract_delete_opintent branch from a404ca1 to 52cd0b1 Compare September 2, 2026 11:38
@mickmis
mickmis merged commit 06ca2da into interuss:master Sep 2, 2026
18 checks passed
@mickmis
mickmis deleted the scd_extract_delete_opintent branch September 2, 2026 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dss-raft Relating to the application-layer consensus implemenation based on raft

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants