Implement periodic timeout checking for instances

pkg/instance/timeout.go

@@ -15,8 +15,7 @@ func (i *Process) ShouldTimeout() bool {
 	i.mu.RLock()
 	defer i.mu.RUnlock()
 
-	// If idle timeout is not set, no timeout
-	if i.options.IdleTimeout == nil || *i.options.IdleTimeout <= 0 {
+	if !i.Running || i.options.IdleTimeout == nil || *i.options.IdleTimeout <= 0 {
 		return false
 	}
 

pkg/manager/manager.go

@@ -10,6 +10,7 @@ import (
 	"path/filepath"
 	"strings"
 	"sync"
+	"time"
 )
 
 // InstanceManager defines the interface for managing instances of the llama server.
@@ -31,6 +32,9 @@ type instanceManager struct {
 	instances       map[string]*instance.Process
 	ports           map[int]bool
 	instancesConfig config.InstancesConfig
+
+	// Timeout checker
+	timeoutChecker *time.Ticker
 }
 
 // NewInstanceManager creates a new instance of InstanceManager.
@@ -39,12 +43,21 @@ func NewInstanceManager(instancesConfig config.InstancesConfig) InstanceManager
 		instances:       make(map[string]*instance.Process),
 		ports:           make(map[int]bool),
 		instancesConfig: instancesConfig,
+
+		timeoutChecker: time.NewTicker(time.Duration(instancesConfig.TimeoutCheckInterval) * time.Minute),
 	}
 
 	// Load existing instances from disk
 	if err := im.loadInstances(); err != nil {
 		log.Printf("Error loading instances: %v", err)
 	}
+
+	go func() {
+		for range im.timeoutChecker.C {
+			im.checkAllTimeouts()
+		}
+	}()
+
 	return im
 }
 
@@ -94,6 +107,12 @@ func (im *instanceManager) Shutdown() {
 	im.mu.Lock()
 	defer im.mu.Unlock()
 
+	// Stop the timeout checker
+	if im.timeoutChecker != nil {
+		im.timeoutChecker.Stop()
+		im.timeoutChecker = nil
+	}
+
 	var wg sync.WaitGroup
 	wg.Add(len(im.instances))
 

pkg/manager/timeout.go

@@ -0,0 +1,19 @@
+package manager
+
+import "log"
+
+func (im *instanceManager) checkAllTimeouts() {
+	im.mu.RLock()
+	defer im.mu.RUnlock()
+
+	for _, inst := range im.instances {
+		if inst.ShouldTimeout() {
+			log.Printf("Instance %s has timed out, stopping it", inst.Name)
+			if proc, err := im.StopInstance(inst.Name); err != nil {
+				log.Printf("Error stopping instance %s: %v", inst.Name, err)
+			} else {
+				log.Printf("Instance %s stopped successfully, process: %v", inst.Name, proc)
+			}
+		}
+	}
+}