feat: metrics for bumped gas price

yetanotherco · Oct 22, 2024 · 96f2ace · 96f2ace
1 parent 6f85dda
commit 96f2ace
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 6 deletions.
diff --git a/aggregator/internal/pkg/aggregator.go b/aggregator/internal/pkg/aggregator.go
@@ -301,7 +301,8 @@ func (agg *Aggregator) sendAggregatedResponse(batchIdentifierHash [32]byte, batc
 		"senderAddress", hex.EncodeToString(senderAddress[:]),
 		"batchIdentifierHash", hex.EncodeToString(batchIdentifierHash[:]))
 
-	receipt, err := agg.avsWriter.SendAggregatedResponse(batchIdentifierHash, batchMerkleRoot, senderAddress, nonSignerStakesAndSignature)
+	onRetry := func() { agg.metrics.IncBumpedGasPriceForAggregatedResponse() }
+	receipt, err := agg.avsWriter.SendAggregatedResponse(batchIdentifierHash, batchMerkleRoot, senderAddress, nonSignerStakesAndSignature, onRetry)
 	if err != nil {
 		agg.walletMutex.Unlock()
 		agg.logger.Infof("- Unlocked Wallet Resources: Error sending aggregated response for batch %s. Error: %s", hex.EncodeToString(batchIdentifierHash[:]), err)

diff --git a/core/chainio/avs_writer.go b/core/chainio/avs_writer.go
@@ -103,7 +103,13 @@ func (w *AvsWriter) SendAggregatedResponse(batchIdentifierHash [32]byte, batchMe
 	txOpts.Nonce = txNonce
 
 	lastTxGasPrice := tx.GasPrice()
+	i := 0
 	sendTransaction := func() (*types.Receipt, error) {
+		if i > 0 {
+			onRetry()
+		}
+		i++
+
 		bumpedGasPrice := utils.CalculateGasPriceBump(lastTxGasPrice, gasBumpPercentage)
 		lastTxGasPrice = bumpedGasPrice
 		txOpts.GasPrice = bumpedGasPrice

diff --git a/metrics/metrics.go b/metrics/metrics.go
@@ -12,11 +12,12 @@ import (
 )
 
 type Metrics struct {
-	ipPortAddress              string
-	logger                     logging.Logger
-	numAggregatedResponses     prometheus.Counter
-	numAggregatorReceivedTasks prometheus.Counter
-	numOperatorTaskResponses   prometheus.Counter
+	ipPortAddress                          string
+	logger                                 logging.Logger
+	numAggregatedResponses                 prometheus.Counter
+	numAggregatorReceivedTasks             prometheus.Counter
+	numOperatorTaskResponses               prometheus.Counter
+	numBumpedGasPriceForAggregatedResponse prometheus.Counter
 }
 
 const alignedNamespace = "aligned"
@@ -40,6 +41,11 @@ func NewMetrics(ipPortAddress string, reg prometheus.Registerer, logger logging.
 			Name:      "aggregator_received_tasks",
 			Help:      "Number of tasks received by the Service Manager",
 		}),
+		numBumpedGasPriceForAggregatedResponse: promauto.With(reg).NewCounter(prometheus.CounterOpts{
+			Namespace: alignedNamespace,
+			Name:      "respond_to_task_gas_price_bumped",
+			Help:      "Number of times gas price was bumped while sending aggregated response",
+		}),
 	}
 }
 
@@ -74,3 +80,7 @@ func (m *Metrics) IncAggregatedResponses() {
 func (m *Metrics) IncOperatorTaskResponses() {
 	m.numOperatorTaskResponses.Inc()
 }
+
+func (m *Metrics) IncBumpedGasPriceForAggregatedResponse() {
+	m.numBumpedGasPriceForAggregatedResponse.Inc()
+}