Fixing invalid name for metrics component (#33)

* Fixing invalid name for metrics component and updating metrics settings on the deployment yaml * 💄 fixing typo
knative · Nov 14, 2019 · 6126d24 · 6126d24
1 parent e5cfad2
commit 6126d24
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 6 deletions.
diff --git a/cmd/manager/main.go b/cmd/manager/main.go
@@ -20,9 +20,9 @@ import (
 	"log"
 
 	"k8s.io/client-go/tools/clientcmd"
+	"knative.dev/eventing-operator/pkg/reconciler/knativeeventing"
 	"knative.dev/pkg/injection/sharedmain"
 	"knative.dev/pkg/signals"
-	"knative.dev/eventing-operator/pkg/reconciler/knativeeventing"
 )
 
 func main() {
@@ -32,5 +32,5 @@ func main() {
 	if err != nil {
 		log.Fatal("Error building kubeconfig", err)
 	}
-	sharedmain.MainWithConfig(signals.NewContext(), "eventing-operator", cfg, knativeeventing.NewController)
+	sharedmain.MainWithConfig(signals.NewContext(), "eventing_operator", cfg, knativeeventing.NewController)
 }
diff --git a/config/operator.yaml b/config/operator.yaml
@@ -25,12 +25,17 @@ spec:
                 fieldRef:
                   fieldPath: metadata.name
             - name: OPERATOR_NAME
-              valueFrom:
-                fieldRef:
-                  fieldPath: metadata.name
+              value: "knative-eventing-operator"
             - name: SYSTEM_NAMESPACE
               valueFrom:
                 fieldRef:
                   fieldPath: metadata.namespace
+            - name: CONFIG_LOGGING_NAME
+              value: config-logging
+            - name: CONFIG_OBSERVABILITY_NAME
+              value: config-observability
             - name: METRICS_DOMAIN
-              value: knative.dev/operator-operator
+              value: knative.dev/eventing-operator
+          ports:
+            - name: metrics
+              containerPort: 9090