Uploaded image for project: 'OpenShift Bugs'
  1. OpenShift Bugs
  2. OCPBUGS-17157

OLMv0 Is Inefficient In Memory Use

    XMLWordPrintable

Details

    • Bug
    • Resolution: Not a Bug
    • Normal
    • 4.14.0
    • 4.15.0
    • OLM
    • No
    • Rejected
    • False
    • Hide

      None

      Show
      None

    Description

      Description of problem:

      OLMv0 over-uses listers and consumes too much memory. Also, $GOMEMLIMIT is not used and the runtime overcommits on RSS. See the following doc for more detail:

https://docs.google.com/document/d/11J7lv1HtEq_c3l6fLTWfsom8v1-7guuG4DziNQDU6cY/edit#heading=h.ttj9tfltxgzt

      Version-Release number of selected component (if applicable):

      How reproducible:

      Steps to Reproduce:

      1.
2.
3.

      Actual results:

      Expected results:

      Additional info:

      Attachments

        Issue Links

          causes

          Bug - A problem that impairs or prevents the functions of the product. OCPBUGS-17695 opm serve is inefficient in memory use

          • Normal - To be worked after urgent and high priorities are resolved. This term is chosen when the severity of an issue is relatively close to the level of effort to fix it. The existence of an easily implemented workaround can also lead to this priority level instead of a higher priority.
          • Closed
          is depended on by

          Bug - A problem that impairs or prevents the functions of the product. OCPBUGS-17804 OLM v0 catalog-operator unnecessarily watches copiedCSVs

          • Major - To be worked after higher priority work items are addressed. When the severity is high and the effort to change it is low to moderate, this term is chosen. Issues in this category likely have an existing workaround but implementation or execution may be non-trivial.
          • Closed
          relates to

          Bug - A problem that impairs or prevents the functions of the product. OCPBUGS-25448 olm-operator pod always restart due to "detected that every object is labelled, exiting to re-start the process..." when upgrading OCP to 4.15 from 4.14.6

          • Undefined - Priority not specified.
          • Verified

          Bug - A problem that impairs or prevents the functions of the product. OCPBUGS-17706 should use opm 4.14 version instead of 4.13 in nightly payload

          • Normal - To be worked after urgent and high priorities are resolved. This term is chosen when the severity of an issue is relatively close to the level of effort to fix it. The existence of an easily implemented workaround can also lead to this priority level instead of a higher priority.
          • Closed
          links to

          GitHub openshift/operator-framework-olm#526: OCPBUGS-17157: operator-registry: cherry-pick to be up-to-date

          GitHub openshift/operator-framework-olm#527: OCPBUGS-17157: cmd/package-server-manager: add pprof endpoints

          GitHub openshift/operator-framework-olm#531: OCPBUGS-17157: sync

          GitHub openshift/operator-framework-olm#534: OCPBUGS-17157: scripts: add a Go-based bumper, sync upstream

          GitHub openshift/operator-framework-olm#551: OCPBUGS-17157: manifests: add $GOMEMLIMIT to PSM

          Web Link operator-framework/api#288: OCPBUGS-17157: CatalogSource: add configurations for gRPC server memory footprint

          GitHub operator-framework/operator-lifecycle-manager#3003: OCPBUGS-17157: improve watching semantics around CSVs

          GitHub operator-framework/operator-lifecycle-manager#3004: OCPBUGS-17157: registry: implement memory targets on gRPC servers

          GitHub operator-framework/operator-lifecycle-manager#3005: OCPBUGS-17157: operators/olm: record and expose infromers

          GitHub operator-framework/operator-lifecycle-manager#3007: OCPBUGS-17157: queueinformer: don't double register informers

          GitHub operator-framework/operator-lifecycle-manager#3014: OCPBUGS-17157: pkg/controller: use a metadata watch for CRDs

          GitHub operator-framework/operator-lifecycle-manager#3015: OCPBUGS-17157: controller/registry: don't set memory limits for target

          GitHub operator-framework/operator-lifecycle-manager#3017: OCPBUGS-17157: label non-OLM resources

          GitHub operator-framework/operator-lifecycle-manager#3020: OCPBUGS-17157: *: label k8s objects we own

          GitHub operator-framework/operator-lifecycle-manager#3021: OCPBUGS-17157: *: filter informers when preconditions are met

          GitHub operator-framework/operator-lifecycle-manager#3028: OCPBUGS-17157: *: detect when all objects are labelled, restart

          GitHub operator-framework/operator-lifecycle-manager#3034: OCPBUGS-17157: pkg/controller: label RBAC with content hash

          GitHub operator-framework/operator-marketplace#525: OCPBUGS-17157: bump controller-runtime, expose pprof, watch only one configmap

          GitHub operator-framework/operator-marketplace#526: OCPBUGS-17157: manifests: tune memory use

          GitHub operator-framework/operator-marketplace#527: OCPBUGS-17157: defaults: add memory targets to CatalogSources

          Web Link operator-framework/operator-registry#1129: OCPBUGS-17157: opm: always serve pprof endpoints, improve server allocations

          Web Link operator-framework/operator-registry#1131: OCPBUGS-17157: go.mod: update API to latest tag

          GitHub operator-framework/operator-registry#1140: OCPBUGS-17157: cache/json: use shared buffers for JSON decoding

          Web Link RHEA-2023:123645 OpenShift Container Platform 4.15.0 CNF vRAN extras update

          mentioned on

          GitLab Merge request - Updated 2 upstream sources

          GitLab Merge request - Updated US source to: 0cc557b Bump open-cluster-management.io/governance-policy-propagator from 0.8.0 to 0.12.0 (#689)

          GitLab Merge request - Updated US source to: 0e98bac OCPBUGS-20204: Invalid ConfigurationPolicy names (#721)

          GitLab Merge request - Updated US source to: 8bbb090 Bump github.com/spf13/cobra from 1.7.0 to 1.8.0 (#717)

          GitLab Merge request - Updated US source to: 55aea28 Bump github.com/go-logr/logr from 1.2.4 to 1.3.0 (#389)

          GitLab Merge request - Updated US source to: 55aea28 Bump github.com/go-logr/logr from 1.2.4 to 1.3.0 (#389)

          GitLab Merge request - Updated US source to: 55aea28 Bump github.com/go-logr/logr from 1.2.4 to 1.3.0 (#389)

          GitLab Merge request - Updated US source to: 87ad959 Update codegen so it doesn't alter go.mod (#718)

          GitLab Merge request - Updated US source to: 606f64a Add script to fix path structure (#725)

          GitLab Merge request - Updated US source to: 929feca Bump github.com/onsi/gomega from 1.29.0 to 1.30.0 (#724)

          GitLab Merge request - Updated US source to: 7860bf2 Bump github.com/operator-framework/api from 0.18.0 to 0.20.0 (#399)

          GitLab Merge request - Updated US source to: c0f884b Merge pull request #444 from openstack-k8s-operators/renovate/main-misc

          GitLab Merge request - Updated US source to: e50b230 Exclude vendor dir from snyk scan (#728)

          Activity

            People

              skuznets@redhat.com Steve Kuznetsov
              skuznets@redhat.com Steve Kuznetsov
              Jian Zhang Jian Zhang
              Votes:
              0 Vote for this issue
              Watchers:
              10 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: