Skip to content

Conversation

@atharrva01
Copy link

Summary

This PR adds support for skipBackgroundRequests to MutatingPolicy and
GeneratingPolicy so they behave the same way as ClusterPolicy.

What I changed

  • Added an optional skipBackgroundRequests field to the evaluation
    configuration for both policy types
  • Added helper methods to resolve the effective value safely, defaulting
    to true
  • Regenerated CRDs, Helm charts, and API docs to include the new field

Why this change

Right now, skipBackgroundRequests is available for ClusterPolicy, but not for
MutatingPolicy or GeneratingPolicy. This makes it hard to use features like
mutateExisting or generateExisting with background requests in a consistent
way.

With this change, users can explicitly set skipBackgroundRequests: false when
they want background controller requests to be processed, while keeping the
existing default behavior unchanged.

Backward compatibility

This is fully backward compatible. Policies that don’t specify the field will
continue to skip background requests by default.

Related issue

@atharrva01
Copy link
Author

atharrva01 commented Jan 26, 2026

hi @eddycharly @JimBugwadia I’ve added skipBackgroundRequests support for MutatingPolicy and GeneratingPolicy to bring them in line with ClusterPolicy. Would appreciate a quick look when you get a chance, happy to make any adjustments.

…tingPolicy

Signed-off-by: atharrva01 <atharvaborade568@gmail.com>
@atharrva01 atharrva01 force-pushed the feat/skip-background-requests branch from 3fff0da to b232cfd Compare January 26, 2026 11:46
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