diff --git a/config/crd/bases/config.nri_balloonspolicies.yaml b/config/crd/bases/config.nri_balloonspolicies.yaml index da3b852c2..33d1d11dc 100644 --- a/config/crd/bases/config.nri_balloonspolicies.yaml +++ b/config/crd/bases/config.nri_balloonspolicies.yaml @@ -324,6 +324,7 @@ spec: default: enabled: - policy + - buildinfo description: Metrics defines which metrics to collect. properties: enabled: diff --git a/config/crd/bases/config.nri_templatepolicies.yaml b/config/crd/bases/config.nri_templatepolicies.yaml index 2b1d4db9a..a5a5b5511 100644 --- a/config/crd/bases/config.nri_templatepolicies.yaml +++ b/config/crd/bases/config.nri_templatepolicies.yaml @@ -96,6 +96,7 @@ spec: default: enabled: - policy + - buildinfo description: Metrics defines which metrics to collect. properties: enabled: diff --git a/config/crd/bases/config.nri_topologyawarepolicies.yaml b/config/crd/bases/config.nri_topologyawarepolicies.yaml index af76a8e9a..9e7f8fa7c 100644 --- a/config/crd/bases/config.nri_topologyawarepolicies.yaml +++ b/config/crd/bases/config.nri_topologyawarepolicies.yaml @@ -123,6 +123,7 @@ spec: default: enabled: - policy + - buildinfo description: Metrics defines which metrics to collect. properties: enabled: diff --git a/deployment/helm/balloons/crds/config.nri_balloonspolicies.yaml b/deployment/helm/balloons/crds/config.nri_balloonspolicies.yaml index da3b852c2..33d1d11dc 100644 --- a/deployment/helm/balloons/crds/config.nri_balloonspolicies.yaml +++ b/deployment/helm/balloons/crds/config.nri_balloonspolicies.yaml @@ -324,6 +324,7 @@ spec: default: enabled: - policy + - buildinfo description: Metrics defines which metrics to collect. properties: enabled: diff --git a/deployment/helm/template/crds/config.nri_templatepolicies.yaml b/deployment/helm/template/crds/config.nri_templatepolicies.yaml index 2b1d4db9a..a5a5b5511 100644 --- a/deployment/helm/template/crds/config.nri_templatepolicies.yaml +++ b/deployment/helm/template/crds/config.nri_templatepolicies.yaml @@ -96,6 +96,7 @@ spec: default: enabled: - policy + - buildinfo description: Metrics defines which metrics to collect. properties: enabled: diff --git a/deployment/helm/topology-aware/crds/config.nri_topologyawarepolicies.yaml b/deployment/helm/topology-aware/crds/config.nri_topologyawarepolicies.yaml index af76a8e9a..9e7f8fa7c 100644 --- a/deployment/helm/topology-aware/crds/config.nri_topologyawarepolicies.yaml +++ b/deployment/helm/topology-aware/crds/config.nri_topologyawarepolicies.yaml @@ -123,6 +123,7 @@ spec: default: enabled: - policy + - buildinfo description: Metrics defines which metrics to collect. properties: enabled: diff --git a/pkg/apis/config/v1alpha1/instrumentation/config.go b/pkg/apis/config/v1alpha1/instrumentation/config.go index 1e387dc20..f8cabb767 100644 --- a/pkg/apis/config/v1alpha1/instrumentation/config.go +++ b/pkg/apis/config/v1alpha1/instrumentation/config.go @@ -49,6 +49,6 @@ type Config struct { // +optional PrometheusExport bool `json:"prometheusExport,omitempty"` // Metrics defines which metrics to collect. - // +kubebuilder:default={"enabled": {"policy"}} + // +kubebuilder:default={"enabled": {"policy", "buildinfo"}} Metrics *metrics.Config `json:"metrics,omitempty"` } diff --git a/pkg/metrics/collectors/collectors.go b/pkg/metrics/collectors/collectors.go new file mode 100644 index 000000000..68518757d --- /dev/null +++ b/pkg/metrics/collectors/collectors.go @@ -0,0 +1,50 @@ +// Copyright The NRI Plugins Authors. All Rights Reserved. +// +// Licensed under the Apache License, Version 2.0 (the "License"); +// you may not use this file except in compliance with the License. +// You may obtain a copy of the License at +// +// http://www.apache.org/licenses/LICENSE-2.0 +// +// Unless required by applicable law or agreed to in writing, software +// distributed under the License is distributed on an "AS IS" BASIS, +// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +// See the License for the specific language governing permissions and +// limitations under the License. + +package collectors + +import ( + "github.com/prometheus/client_golang/prometheus" + "github.com/prometheus/client_golang/prometheus/collectors" + + logger "github.com/containers/nri-plugins/pkg/log" + "github.com/containers/nri-plugins/pkg/metrics" +) + +var ( + log = logger.Get("metrics") +) + +func init() { + var ( + collectors = map[string]prometheus.Collector{ + "buildinfo": collectors.NewBuildInfoCollector(), + "golang": collectors.NewGoCollector(), + "process": collectors.NewProcessCollector(collectors.ProcessCollectorOpts{}), + } + options = []metrics.RegisterOption{ + metrics.WithGroup("standard"), + metrics.WithCollectorOptions( + metrics.WithoutNamespace(), + metrics.WithoutSubsystem(), + ), + } + ) + + for name, collector := range collectors { + if err := metrics.Register(name, collector, options...); err != nil { + log.Error("failed to register %s collector: %v", name, err) + } + } +}