fix: orchestrator sign up to last unbonding period

orchestrator/orchestrator.go

@@ -208,15 +208,29 @@ func (orch Orchestrator) EnqueueMissingEvents(
 	if err != nil {
 		return err
 	}
+	var startingNonce uint64
+	if lastUnbondingHeight == 0 {
+		startingNonce = 1
+	} else {
+		dc, err := orch.AppQuerier.QueryDataCommitmentForHeight(ctx, uint64(lastUnbondingHeight))
+		if err != nil {
+			return err
+		}
+		startingValset, err := orch.AppQuerier.QueryLastValsetBeforeNonce(ctx, dc.Nonce)
+		if err != nil {
+			return err
+		}
+		startingNonce = startingValset.Nonce
+	}
 
-	orch.Logger.Info("syncing missing nonces", "latest_nonce", latestNonce, "last_unbonding_height", lastUnbondingHeight)
+	orch.Logger.Info("syncing missing nonces", "latest_nonce", latestNonce, "first_nonce", startingNonce)
 
 	// To accommodate the delay that might happen between starting the two go routines above.
 	// Probably, it would be a good idea to further refactor the orchestrator to the relayer style
 	// as it is entirely synchronous. Probably, enqueuing separately old nonces and new ones, is not
 	// the best design.
 	// TODO decide on this later
-	for i := uint64(lastUnbondingHeight); i < latestNonce; i++ {
+	for i := uint64(0); i < latestNonce-startingNonce+1; i++ {
 		select {
 		case <-signalChan:
 			return ErrSignalChanNotif
@@ -233,7 +247,7 @@ func (orch Orchestrator) EnqueueMissingEvents(
 			}
 		}
 	}
-	orch.Logger.Info("finished syncing missing nonces", "latest_nonce", latestNonce, "last_unbonding_height", lastUnbondingHeight)
+	orch.Logger.Info("finished syncing missing nonces", "latest_nonce", latestNonce, "first_nonce", startingNonce)
 	return nil
 }