Merge pull request #1360 from swghosh/bump-lib-go-latency-profile

Bug 2100347: Bump(openshift/library-go) latency profile observer, controller

Author: openshift-ci[bot]

go.mod

@@ -15,7 +15,7 @@ require (
 	github.com/openshift/api v0.0.0-20220525145417-ee5b62754c68
 	github.com/openshift/build-machinery-go v0.0.0-20211213093930-7e33a7eb4ce3
 	github.com/openshift/client-go v0.0.0-20220525160904-9e1acff93e4a
-	github.com/openshift/library-go v0.0.0-20220525173854-9b950a41acdc
+	github.com/openshift/library-go v0.0.0-20220622115547-84d884f4c9f6
 	github.com/pkg/profile v1.5.0 // indirect
 	github.com/prometheus-operator/prometheus-operator/pkg/client v0.45.0
 	github.com/prometheus/client_golang v1.12.1

go.sum

@@ -524,8 +524,8 @@ github.com/openshift/build-machinery-go v0.0.0-20211213093930-7e33a7eb4ce3 h1:65
 github.com/openshift/build-machinery-go v0.0.0-20211213093930-7e33a7eb4ce3/go.mod h1:b1BuldmJlbA/xYtdZvKi+7j5YGB44qJUJDZ9zwiNCfE=
 github.com/openshift/client-go v0.0.0-20220525160904-9e1acff93e4a h1:ylsEgoC8Dlg4A0C1TLH0A4x/TZao7k1YveLwROhRUdk=
 github.com/openshift/client-go v0.0.0-20220525160904-9e1acff93e4a/go.mod h1:eDO5QeVi2IiXmDwB0e2z1DpAznWroZKe978pzZwFBzg=
-github.com/openshift/library-go v0.0.0-20220525173854-9b950a41acdc h1:j+upvKc1uLzuL+q/JXie8+IMohOooTCaEC9w+4d1Ztk=
-github.com/openshift/library-go v0.0.0-20220525173854-9b950a41acdc/go.mod h1:AMZwYwSdbvALDl3QobEzcJ2IeDO7DYLsr42izKzh524=
+github.com/openshift/library-go v0.0.0-20220622115547-84d884f4c9f6 h1:lmfmsIGq62lmj17qrZh4Gbbb86WvJw6pLhCNwNjB2Yk=
+github.com/openshift/library-go v0.0.0-20220622115547-84d884f4c9f6/go.mod h1:AMZwYwSdbvALDl3QobEzcJ2IeDO7DYLsr42izKzh524=
 github.com/opentracing/opentracing-go v1.1.0/go.mod h1:UkNAQd3GIcIGf0SeVgPpRdFStlNbqXla1AfSYxPUl2o=
 github.com/pascaldekloe/goe v0.0.0-20180627143212-57f6aae5913c/go.mod h1:lzWF7FIEvWOWxwDKqyGYQf6ZUaNfKdP144TG7ZOy1lc=
 github.com/pborman/uuid v1.2.0/go.mod h1:X/NO0urCmaxf9VXbdlT7C2Yzkj2IKimNn4k+gtPdI/k=

pkg/operator/configobservation/configobservercontroller/observe_config_controller.go

@@ -145,11 +145,13 @@ func NewConfigObserver(
 			network.ObserveServicesNodePortRange,
 			nodeobserver.NewLatencyProfileObserver(
 				node.LatencyConfigs,
-				nodeobserver.NewSuppressConfigUpdateUntilSameProfileFunc(
-					operatorClient,
-					kubeInformersForNamespaces.ConfigMapLister().ConfigMaps(operatorclient.TargetNamespace),
-					node.LatencyConfigs,
-				),
+				[]nodeobserver.ShouldSuppressConfigUpdatesFunc{
+					nodeobserver.NewSuppressConfigUpdateUntilSameProfileFunc(
+						operatorClient,
+						kubeInformersForNamespaces.ConfigMapLister().ConfigMaps(operatorclient.TargetNamespace),
+						node.LatencyConfigs,
+					),
+				},
 			),
 			proxy.NewProxyObserveFunc([]string{"targetconfigcontroller", "proxy"}),
 			images.ObserveInternalRegistryHostname,

pkg/operator/starter.go

@@ -375,7 +375,7 @@ func RunOperator(ctx context.Context, controllerContext *controllercmd.Controlle
 	latencyProfileController := latencyprofilecontroller.NewLatencyProfileController(
 		operatorClient,
 		operatorclient.TargetNamespace,
-		node.LatencyConfigs,
+		nil, // profile rejection logic is not required for this operator
 		latencyprofilecontroller.NewInstallerRevisionConfigMatcher(
 			kubeInformersForNamespaces.ConfigMapLister().ConfigMaps(operatorclient.TargetNamespace),
 			node.LatencyConfigs,