Skip to content

Commit e658b72

Browse files
authored
Merge pull request #1106 from mm4tt/kubemark_prom_fix
Fix confusing log line in exposeAPIServerMetrics
2 parents 11ac5fc + 234388f commit e658b72

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

clusterloader2/pkg/prometheus/prometheus.go

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -194,8 +194,11 @@ func (pc *PrometheusController) applyManifests(manifestGlob string) error {
194194

195195
// exposeAPIServerMetrics configures anonymous access to the apiserver metrics.
196196
func (pc *PrometheusController) exposeAPIServerMetrics() error {
197-
klog.Info("Exposing kube-apiserver metrics in kubemark cluster")
198-
// This has to be done in the kubemark cluster, thus we need to create a new client.
197+
klog.Info("Exposing kube-apiserver metrics in the cluster")
198+
// We need to get a client to the cluster where the test is being executed on,
199+
// not the cluster that the prometheus is running in. Usually, there is only
200+
// once cluster, but in case of kubemark we have two and thus we need to
201+
// create a new client here.
199202
clientSet, err := framework.NewMultiClientSet(
200203
pc.clusterLoaderConfig.ClusterConfig.KubeConfigPath, numK8sClients)
201204
if err != nil {

0 commit comments

Comments
 (0)