Uploaded image for project: 'OpenShift Logging'
  1. OpenShift Logging
  2. LOG-4072

Refactor reconciliation to better encapsulate CR loading, migration, validation

XMLWordPrintable

    • Icon: Task Task
    • Resolution: Done
    • Icon: Major Major
    • None
    • None
    • Log Collection
    • 3
    • False
    • Hide

      None

      Show
      None
    • False
    • NEW
    • NEW
    • Log Collection - Sprint 236, Log Collection - Sprint 237

      Summary

      Refactor the reconciliation of ClusterLogging and ClusterLogforwarder to a "higher" level in the call stack in preparation to support multi cluster logforwarding. The goal of this change is to reduce code duplication and to minimize the dependency between both objects since this feature will introduce ClusterLogforwarding objects that do not "pair" with a ClusterLogging object.

      Acceptance Criteria

      • Verify there is no regressions between before and after this change:
      • Verify ClusterLogging(ns=openshift-logging,name=instance) deploys collectors and logstorage
      • Verify ClusterLogging(ns=openshift-logging,name=instance) with collection and ClusterLogforwarder(ns=openshift-logging,name=instance) deploys collector
      • Verify ClusterLogforwarder(ns=openshift-logging,name=instance) with pipeline ref to default deploys nothing given there is not a deployed default logstore
      • Verify deleting ClusterLogforwarder, removes the collectors when there is no default logStore specified

      Notes

              jcantril@redhat.com Jeffrey Cantrill
              jcantril@redhat.com Jeffrey Cantrill
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Created:
                Updated:
                Resolved: