Merge pull request #545 from application-stacks/monitor-https

prometheus insists scheme is 'https' not 'HTTPS'

Author: idlewis

bundle/tests/scorecard/kuttl/monitor/02-assert.yaml

@@ -21,7 +21,7 @@ spec:
       app.kubernetes.io/instance: service-monitor-rc
   endpoints:
   - path: "/path"
-    scheme: "myScheme"
+    scheme: "https"
     port: 3000-tcp
     params:
       params:

bundle/tests/scorecard/kuttl/monitor/02-runtime-update-monitor.yaml

@@ -6,7 +6,7 @@ spec:
   monitoring:
     endpoints:
     - path: "/path"
-      scheme: "myScheme"
+      scheme: "https"
       params:
         params:
         - "param1"

utils/utils.go

@@ -1040,7 +1040,7 @@ func CustomizeServiceMonitor(sm *prometheusv1.ServiceMonitor, ba common.BaseComp
 	}
 	if ba.GetManageTLS() == nil || *ba.GetManageTLS() {
 		if len(ba.GetMonitoring().GetEndpoints()) == 0 || ba.GetMonitoring().GetEndpoints()[0].TLSConfig == nil {
-			sm.Spec.Endpoints[0].Scheme = "HTTPS"
+			sm.Spec.Endpoints[0].Scheme = "https"
 			if sm.Spec.Endpoints[0].TLSConfig == nil {
 				sm.Spec.Endpoints[0].TLSConfig = &prometheusv1.TLSConfig{}
 			}