Skip to content

Commit 0674758

Browse files
author
Kate Osborn
committed
Some more name changes
1 parent 9bf4fc4 commit 0674758

File tree

2 files changed

+10
-10
lines changed

2 files changed

+10
-10
lines changed

internal/mode/static/handler.go

+2-2
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ import (
2424
"github.com/nginxinc/nginx-gateway-fabric/internal/mode/static/state/resolver"
2525
)
2626

27-
type controllerMetricsCollector interface {
27+
type handlerMetricsCollector interface {
2828
ObserveLastEventBatchProcessTime(time.Duration)
2929
}
3030

@@ -51,7 +51,7 @@ type eventHandlerConfig struct {
5151
// controlConfigNSName is the NamespacedName of the NginxGateway config for this controller.
5252
controlConfigNSName types.NamespacedName
5353
// metricsCollector collects metrics for this controller.
54-
metricsCollector controllerMetricsCollector
54+
metricsCollector handlerMetricsCollector
5555
// logger is the logger to be used by the EventHandler.
5656
logger logr.Logger
5757
// version is the current version number of the nginx config.

internal/mode/static/manager.go

+8-8
Original file line numberDiff line numberDiff line change
@@ -132,9 +132,9 @@ func StartManager(cfg config.Config) error {
132132
}
133133

134134
var (
135-
mgrCollector ngxruntime.MetricsCollector = collectors.NewManagerNoopCollector()
135+
ngxruntimeCollector ngxruntime.MetricsCollector = collectors.NewManagerNoopCollector()
136136
// nolint:ineffassign // not an ineffectual assignment. Will be used if metrics are disabled.
137-
ctlrCollector controllerMetricsCollector = collectors.NewControllerNoopCollector()
137+
handlerCollector handlerMetricsCollector = collectors.NewControllerNoopCollector()
138138
)
139139

140140
if cfg.MetricsConfig.Enabled {
@@ -144,12 +144,12 @@ func StartManager(cfg config.Config) error {
144144
return fmt.Errorf("cannot create nginx metrics collector: %w", err)
145145
}
146146

147-
mgrCollector = collectors.NewManagerMetricsCollector(constLabels)
148-
ctlrCollector = collectors.NewControllerCollector(constLabels)
147+
ngxruntimeCollector = collectors.NewManagerMetricsCollector(constLabels)
148+
handlerCollector = collectors.NewControllerCollector(constLabels)
149149
metrics.Registry.MustRegister(
150150
ngxCollector,
151-
mgrCollector.(prometheus.Collector),
152-
ctlrCollector.(prometheus.Collector),
151+
ngxruntimeCollector.(prometheus.Collector),
152+
handlerCollector.(prometheus.Collector),
153153
)
154154
}
155155

@@ -174,12 +174,12 @@ func StartManager(cfg config.Config) error {
174174
cfg.Logger.WithName("nginxFileManager"),
175175
file.NewStdLibOSFileManager(),
176176
),
177-
nginxRuntimeMgr: ngxruntime.NewManagerImpl(mgrCollector),
177+
nginxRuntimeMgr: ngxruntime.NewManagerImpl(ngxruntimeCollector),
178178
statusUpdater: statusUpdater,
179179
eventRecorder: recorder,
180180
healthChecker: hc,
181181
controlConfigNSName: controlConfigNSName,
182-
metricsCollector: ctlrCollector,
182+
metricsCollector: handlerCollector,
183183
})
184184

185185
objects, objectLists := prepareFirstEventBatchPreparerArgs(cfg.GatewayClassName, cfg.GatewayNsName)

0 commit comments

Comments
 (0)