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

Add support for approving InstallPlans with multiple CSVs #260

Merged

Commits on Jun 6, 2024

  1. Add a label and annotation for subscriptions managed by OperatorPolicy

    The operatorpolicy.policy.open-cluster-management.io/managed label is
    set to an empty value on the Subscription. The
    operatorpolicy.policy.open-cluster-management.io/managed annotation
    is set to <policy name>.<policy namespace>.
    
    The label doesn't set a value because the value could be too long.
    
    This will allow querying for subscriptions managed by operator policies.
    
    Signed-off-by: mprahl <mprahl@users.noreply.github.com>
    mprahl committed Jun 6, 2024
    Configuration menu
    Copy the full SHA
    2f0a380 View commit details
    Browse the repository at this point in the history

Commits on Jun 7, 2024

  1. Add support for approving InstallPlans with multiple CSVs

    When an InstallPlan contains multiple CSVs, it will be approved if all
    CSVs can be approved by the current operator policies.
    
    Rather than query the operator policies directly, the subscriptions in
    the namespace that are managed by an operator policy are queried and
    then their associated operator policies are queried to avoid having to
    resolve templates to see if they manage a subscription in the namespace.
    
    Relates:
    https://issues.redhat.com/browse/ACM-11981
    
    Signed-off-by: mprahl <mprahl@users.noreply.github.com>
    mprahl committed Jun 7, 2024
    Configuration menu
    Copy the full SHA
    4f07c82 View commit details
    Browse the repository at this point in the history