chore: add details of fp pk to logs (backport #737) (#744)

HAL FIND-005

# Description
## Vigilante

Some error logs in vigilante are missing the delegation identifier
(staking_tx_hash), making it difficult for operators to correlate
operational failures to specific delegations during troubleshooting.


## Impact:

Debugging difficulty: Operators cannot identify which delegation caused
the error
Operational overhead: Must correlate logs with other events to
troubleshoot issues
Inconsistent logging: Some errors include context, others don't<hr>This
is an automatic backport of pull request #737 done by
[Mergify](https://mergify.com).

Co-authored-by: RafilxTenfen <[email protected]>

Author: mergify[bot]

CHANGELOG.md

@@ -47,6 +47,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
 * [#734](https://github.com/babylonlabs-io/finality-provider/pull/734) chore: rm unused cfg val
 * [#735](https://github.com/babylonlabs-io/finality-provider/pull/735) chore: use counter prometeus
 * [#736](https://github.com/babylonlabs-io/finality-provider/pull/736) chore: disable flag on unsafe/test RPCs
+* [#737](https://github.com/babylonlabs-io/finality-provider/pull/737) chore: add fp identifier to logs
 * [#739](https://github.com/babylonlabs-io/finality-provider/pull/739) chore: bump babylon to `v4.0.0`
 
 ## v2.0.0-rc.5

finality-provider/service/fp_instance.go

@@ -236,7 +236,10 @@ func (fp *FinalityProviderInstance) finalitySigSubmissionLoop(ctx context.Contex
 		case <-ticker.C:
 			fp.processAndSubmitSignatures(ctx)
 		case <-fp.quit:
-			fp.logger.Info("the finality signature submission loop is closing")
+			fp.logger.Info(
+				"the finality signature submission loop is closing",
+				zap.String("pk", fp.GetBtcPkHex()),
+			)
 
 			return
 		}
@@ -248,7 +251,10 @@ func (fp *FinalityProviderInstance) finalitySigSubmissionLoop(ctx context.Contex
 func (fp *FinalityProviderInstance) processAndSubmitSignatures(ctx context.Context) {
 	select {
 	case <-fp.quit:
-		fp.logger.Debug("processAndSubmitSignatures: the finality signature submission loop is closing")
+		fp.logger.Debug(
+			"processAndSubmitSignatures: the finality signature submission loop is closing",
+			zap.String("pk", fp.GetBtcPkHex()),
+		)
 
 		return
 	default:
@@ -342,7 +348,10 @@ func (fp *FinalityProviderInstance) randomnessCommitmentLoop(ctx context.Context
 		case <-ticker.C:
 			fp.processRandomnessCommitment(ctx)
 		case <-fp.quit:
-			fp.logger.Info("the randomness commitment loop is closing")
+			fp.logger.Info(
+				"the randomness commitment loop is closing",
+				zap.String("pk", fp.GetBtcPkHex()),
+			)
 
 			return
 		}
@@ -390,9 +399,17 @@ func (fp *FinalityProviderInstance) reportCriticalErr(err error) {
 		fpBtcPk: fp.GetBtcPkBIP340(),
 	}:
 	case <-fp.quit:
-		fp.logger.Debug("skipping error report due to context cancellation", zap.Error(err))
+		fp.logger.Debug(
+			"skipping error report due to context cancellation",
+			zap.String("pk", fp.GetBtcPkHex()),
+			zap.Error(err),
+		)
 	default:
-		fp.logger.Error("failed to report critical error (channel full)", zap.Error(err))
+		fp.logger.Error(
+			"failed to report critical error (channel full)",
+			zap.String("pk", fp.GetBtcPkHex()),
+			zap.Error(err),
+		)
 	}
 }
 
@@ -450,6 +467,7 @@ func (fp *FinalityProviderInstance) GetVotingPowerWithRetry(height uint64) (bool
 	}, RtyAtt, RtyDel, RtyErr, retry.OnRetry(func(n uint, err error) {
 		fp.logger.Debug(
 			"failed to query the voting power",
+			zap.String("pk", fp.GetBtcPkHex()),
 			zap.Uint("attempt", n+1),
 			zap.Uint("max_attempts", RtyAttNum),
 			zap.Error(err),