Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🤖 Sync from open-cluster-management-io/config-policy-controller: #118 #465

Merged
merged 1 commit into from
Apr 6, 2023

Commits on Apr 6, 2023

  1. Use patches for adding and removing finalizers

    Using update can have unintended consequences such as setting empty
    values of fields that were nil on the server. This happens for
    the evaluationInterval field where if it wasn't set, the update adds an
    empty map, which triggers the spec-sync to remove the nil map. This
    unnecessary spec-sync update causes the status update to fail later on
    in the ConfigurationPolicy processing.
    
    Additionally, when a finalizer can't be added or removed, the evaluation
    stops and would be retried on the next run. This is a safer approach.
    
    Signed-off-by: mprahl <mprahl@users.noreply.github.com>
    (cherry picked from commit bdd6450)
    mprahl authored and Magic Mirror committed Apr 6, 2023
    Configuration menu
    Copy the full SHA
    2c00e9a View commit details
    Browse the repository at this point in the history