Skip to content

feat/actionable sequencer#154

Open
fernandezcuesta wants to merge 7 commits into
crossplane-contrib:mainfrom
fernandezcuesta:feat/actionable-sequencer
Open

feat/actionable sequencer#154
fernandezcuesta wants to merge 7 commits into
crossplane-contrib:mainfrom
fernandezcuesta:feat/actionable-sequencer

Conversation

@fernandezcuesta
Copy link
Copy Markdown
Contributor

@fernandezcuesta fernandezcuesta commented Jun 2, 2026

Description of your changes

Fixes #69

It adds a CEL condition field for each rule, so we can opt-in to enable/disable that rule.
In order to avoid flaky conditions, once a condition is met and a Usage gets created, we'll keep that even if the condition flips.

Also adds a deletion only mode, for rules where we want to ensure deletion ordering but don't care that much about create (we rely on eventual consistency).

I have:

Signed-off-by: Jesús Fernández <7312236+fernandezcuesta@users.noreply.github.com>
Signed-off-by: Jesús Fernández <7312236+fernandezcuesta@users.noreply.github.com>
@fernandezcuesta fernandezcuesta marked this pull request as draft June 2, 2026 16:30
Signed-off-by: Jesús Fernández <7312236+fernandezcuesta@users.noreply.github.com>
Signed-off-by: Jesús Fernández <7312236+fernandezcuesta@users.noreply.github.com>
Signed-off-by: Jesús Fernández <7312236+fernandezcuesta@users.noreply.github.com>
Signed-off-by: Jesús Fernández <7312236+fernandezcuesta@users.noreply.github.com>
@fernandezcuesta
Copy link
Copy Markdown
Contributor Author

Just did an e2e test with a composition and the fork and it works as expected.

Signed-off-by: Jesús Fernández <7312236+fernandezcuesta@users.noreply.github.com>
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.

Sequence matching no desired resources triggers false positivre

1 participant