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

OCPBUGS-17157: label non-OLM resources #3017

Commits on Aug 24, 2023

  1. add a round-tripper to ensure we label non-OLM resources

    This round-tripper is added to our *rest.Config when it's possible to
    detect that we're in a CI environment. Developers should set $CI=true to
    get this behavior locally.
    
    Signed-off-by: Steve Kuznetsov <[email protected]>
    stevekuznetsov committed Aug 24, 2023
    Configuration menu
    Copy the full SHA
    3b6c7a2 View commit details
    Browse the repository at this point in the history
  2. *: label non-OLM resources

    Today, our controllers use un-filtered LIST+WATCH calls to monitor the
    state of the cluster. For OLM-specific resource types, that's fine,
    since we need to know (for instance) about every CSV. For non-OLM
    resource groups, though, that is needlessly wasteful in memory
    consumption and makes our controller's footprint scale with the size of
    the cluster itself, irrespective of the usage of OLM. Adding a label to
    every resource we create is the first step in being able to filter down
    all of those requests to only those objects with our label.
    
    Signed-off-by: Steve Kuznetsov <[email protected]>
    stevekuznetsov committed Aug 24, 2023
    Configuration menu
    Copy the full SHA
    561b097 View commit details
    Browse the repository at this point in the history