Merge pull request #1575 from desmos-labs/paul/fix-wasmvm-cache-metric-collector

alpe · web-flow · commit 7c6069d2d15a · 2023-08-28T13:23:44.000+02:00
fix: make sure wasmvm cache metrics collector registration after VM setup
diff --git a/x/wasm/keeper/options.go b/x/wasm/keeper/options.go
@@ -126,7 +126,7 @@ func WithAccountPruner(x AccountPruner) Option {
 }
 
 func WithVMCacheMetrics(r prometheus.Registerer) Option {
-	return optsFn(func(k *Keeper) {
+	return postOptsFn(func(k *Keeper) {
 		NewWasmVMMetricsCollector(k.wasmVM).Register(r)
 	})
 }

Original file line number	Diff line number	Diff line change
`@@ -126,7 +126,7 @@ func WithAccountPruner(x AccountPruner) Option {`
`126`	`126`	`}`
`127`	`127`
`128`	`128`	`func WithVMCacheMetrics(r prometheus.Registerer) Option {`
`129`		`- return optsFn(func(k *Keeper) {`
	`129`	`+ return postOptsFn(func(k *Keeper) {`
`130`	`130`	`NewWasmVMMetricsCollector(k.wasmVM).Register(r)`
`131`	`131`	`})`
`132`	`132`	`}`