hotfix: Remove broadcasting btc timestamps (#633)

babylonchain · May 7, 2024 · f63e951 · f63e951
2 parents 42dd3a7 + d4a7082
commit f63e951
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/test/e2e/e2e_test.go b/test/e2e/e2e_test.go
@@ -28,9 +28,10 @@ func TestBTCTimestampingTestSuite(t *testing.T) {
 
 // TestBTCTimestampingPhase2RlyTestSuite tests BTC timestamping phase 2 protocol end-to-end,
 // with the Go relayer
-func TestBTCTimestampingPhase2RlyTestSuite(t *testing.T) {
-	suite.Run(t, new(BTCTimestampingPhase2RlyTestSuite))
-}
+// TODO: Uncomment once we have fix broadcasting of timestamps
+// func TestBTCTimestampingPhase2RlyTestSuite(t *testing.T) {
+// 	suite.Run(t, new(BTCTimestampingPhase2RlyTestSuite))
+// }
 
 // TestBTCStakingTestSuite tests BTC staking protocol end-to-end
 func TestBTCStakingTestSuite(t *testing.T) {

diff --git a/x/zoneconcierge/keeper/hooks.go b/x/zoneconcierge/keeper/hooks.go
@@ -36,7 +36,10 @@ func (h Hooks) AfterRawCheckpointFinalized(ctx context.Context, epoch uint64) er
 	headersToBroadcast := h.k.getHeadersToBroadcast(ctx)
 
 	// send BTC timestamp to all open channels with ZoneConcierge
-	h.k.BroadcastBTCTimestamps(ctx, epoch, headersToBroadcast)
+	// TODO: BroadcastBTCTimestamps is non-deterministic due to generating proofs
+	// which are affected by pruning. Re-enable after improving BroadcastBTCTimestamps
+	// methods
+	// h.k.BroadcastBTCTimestamps(ctx, epoch, headersToBroadcast)
 
 	// Update the last broadcasted segment
 	h.k.setLastSentSegment(ctx, &types.BTCChainSegment{