Skip to content

Add feature gate to disable commit status event - #1755

Merged
matheuscscp merged 1 commit into
mainfrom
feat-disable-commit-status-event
Sep 21, 2026
Merged

matheuscscp merged 1 commit into
mainfrom
feat-disable-commit-status-event

Conversation

@matheuscscp

@matheuscscp matheuscscp commented Sep 21, 2026

Copy link
Copy Markdown
Member

Adds the opt-in DisableCommitStatusEvent feature gate (default false), which disables the reconciliation success event that notification-controller uses to update the Git commit status.

This gives users affected by the success event noise an opt-out without losing error events. The gate state is read once at startup and propagated to the reconciler as a boolean field.

Closes: #1742

Add the opt-in DisableCommitStatusEvent feature gate, which disables the
reconciliation success event that notification-controller uses to update
the Git commit status. This gives users affected by the success event
noise an opt-out without losing error events.

The gate state is read once at startup and propagated to the reconciler
as a boolean field.

Signed-off-by: Matheus Pimenta <matheuscscp@gmail.com>
Assisted-by: opencode/deepseek-v4.1-flash
@matheuscscp matheuscscp added backport:release/v1.9.x To be backported to release/v1.9.x enhancement New feature or request labels Sep 21, 2026

@stefanprodan stefanprodan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@matheuscscp
matheuscscp merged commit eb3c30a into main Sep 21, 2026
9 checks passed
@matheuscscp
matheuscscp deleted the feat-disable-commit-status-event branch September 21, 2026 14:41
@fluxcdbot

Copy link
Copy Markdown
Member

Successfully created backport PR for release/v1.9.x:

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

Labels

backport:release/v1.9.x To be backported to release/v1.9.x enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

too many "Reconciliation finished in" events raised.

3 participants