operator: remove metrics server

Author: djoshy

cmd/machine-config-operator/start.go

@@ -25,17 +25,15 @@ var (
 	}
 
 	startOpts struct {
-		kubeconfig     string
-		imagesFile     string
-		promMetricsURL string
+		kubeconfig string
+		imagesFile string
 	}
 )
 
 func init() {
 	rootCmd.AddCommand(startCmd)
 	startCmd.PersistentFlags().StringVar(&startOpts.kubeconfig, "kubeconfig", "", "Kubeconfig file to access a remote cluster (testing only)")
 	startCmd.PersistentFlags().StringVar(&startOpts.imagesFile, "images-json", "", "images.json file for MCO.")
-	startCmd.PersistentFlags().StringVar(&startOpts.promMetricsURL, "metrics-listen-address", "127.0.0.1:8797", "Listen address for prometheus metrics listener")
 }
 
 func runStartCmd(_ *cobra.Command, _ []string) {
@@ -45,8 +43,6 @@ func runStartCmd(_ *cobra.Command, _ []string) {
 	// This is 'main' context that we thread through the controller context and
 	// the leader elections. Cancelling this is "stop everything, we are shutting down".
 	runContext, runCancel := context.WithCancel(context.Background())
-	stopCh := make(chan struct{})
-	defer close(stopCh)
 
 	// To help debugging, immediately log version
 	klog.Infof("Version: %s (Raw: %s, Hash: %s)", version.ReleaseVersion, version.Raw, version.Hash)
@@ -60,9 +56,6 @@ func runStartCmd(_ *cobra.Command, _ []string) {
 		klog.Fatalf("error creating clients: %v", err)
 	}
 
-	// start metrics listener
-	go ctrlcommon.StartMetricsListener(startOpts.promMetricsURL, stopCh, operator.RegisterMCOMetrics)
-
 	run := func(ctx context.Context) {
 		go common.SignalHandler(runCancel)
 		ctrlctx := ctrlcommon.CreateControllerContext(ctx, cb)

install/0000_80_machine-config_00_service.yaml

@@ -1,26 +1,5 @@
 apiVersion: v1
 kind: Service
-metadata:
-  name: machine-config-operator
-  namespace: openshift-machine-config-operator
-  labels:
-    k8s-app: machine-config-operator
-  annotations:
-    include.release.openshift.io/ibm-cloud-managed: "true"
-    include.release.openshift.io/self-managed-high-availability: "true"
-    include.release.openshift.io/single-node-developer: "true"
-    service.beta.openshift.io/serving-cert-secret-name: mco-proxy-tls
-spec:
-  type: ClusterIP
-  selector:
-    k8s-app: machine-config-operator
-  ports:
-  - name: metrics
-    port: 9001
-    protocol: TCP
----
-apiVersion: v1
-kind: Service
 metadata:
   name: machine-config-controller
   namespace: openshift-machine-config-operator

install/0000_80_machine-config_04_deployment.yaml

@@ -43,30 +43,6 @@ spec:
         volumeMounts:
         - name: images
           mountPath: /etc/mco/images
-      - name: kube-rbac-proxy
-        image: placeholder.url.oc.will.replace.this.org/placeholdernamespace:kube-rbac-proxy
-        ports:
-        - containerPort: 9001
-          name: metrics
-          protocol: TCP
-        args:
-        - --secure-listen-address=0.0.0.0:9001
-        - --config-file=/etc/kube-rbac-proxy/config-file.yaml
-        - --tls-cipher-suites=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256,TLS_ECDHE_ECDSA_WITH_AES_128_GCM_SHA256,TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384,TLS_ECDHE_ECDSA_WITH_AES_256_GCM_SHA384,TLS_ECDHE_RSA_WITH_CHACHA20_POLY1305,TLS_ECDHE_ECDSA_WITH_CHACHA20_POLY1305
-        - --upstream=http://127.0.0.1:8797
-        - --logtostderr=true
-        - --tls-cert-file=/etc/tls/private/tls.crt
-        - --tls-private-key-file=/etc/tls/private/tls.key
-        resources:
-          requests:
-            cpu: 20m
-            memory: 50Mi
-        terminationMessagePolicy: FallbackToLogsOnError
-        volumeMounts:
-        - mountPath: /etc/tls/private
-          name: proxy-tls
-        - mountPath: /etc/kube-rbac-proxy
-          name: auth-proxy-config
       serviceAccountName: machine-config-operator
       nodeSelector:
         node-role.kubernetes.io/master: ""
@@ -91,10 +67,4 @@ spec:
       - name: images
         configMap:
           name: machine-config-operator-images
-      - name: proxy-tls
-        secret:
-          secretName: mco-proxy-tls
-      - configMap:
-          name: kube-rbac-proxy
-        name: auth-proxy-config
 

install/0000_80_machine-config_04_kube_rbac_proxy_config.yaml

@@ -1,41 +1,5 @@
 apiVersion: monitoring.coreos.com/v1
 kind: ServiceMonitor
-metadata:
-  name: machine-config-operator
-  namespace: openshift-machine-config-operator
-  labels:
-    k8s-app: machine-config-operator
-  annotations:
-    include.release.openshift.io/self-managed-high-availability: "true"
-    include.release.openshift.io/single-node-developer: "true"
-spec:
-  endpoints:
-  - interval: 30s
-    bearerTokenFile: /var/run/secrets/kubernetes.io/serviceaccount/token
-    port: metrics
-    scheme: https
-    path: /metrics
-    relabelings:
-    - action: replace
-      regex: ;(.*)
-      replacement: $1
-      separator: ";"
-      sourceLabels:
-      - node
-      - __meta_kubernetes_pod_node_name
-      targetLabel: node
-    tlsConfig:
-      caFile: /etc/prometheus/configmaps/serving-certs-ca-bundle/service-ca.crt
-      serverName: machine-config-operator.openshift-machine-config-operator.svc
-  namespaceSelector:
-    matchNames:
-    - openshift-machine-config-operator
-  selector:
-    matchLabels:
-      k8s-app: machine-config-operator
----
-apiVersion: monitoring.coreos.com/v1
-kind: ServiceMonitor
 metadata:
   name: machine-config-controller
   namespace: openshift-machine-config-operator