Add Cron interval to config

cmd/server.go

@@ -2,7 +2,6 @@ package cmd
 
 import (
 	"net/http"
-	"time"
 
 	wisski_distillery "github.com/FAU-CDI/wisski-distillery"
 	"github.com/FAU-CDI/wisski-distillery/internal/cli"
@@ -51,7 +50,7 @@ func (s server) Run(context wisski_distillery.Context) error {
 
 		// start the cron tasks
 		context.Printf("Starting cron tasks %s\n", s.Bind)
-		done := dis.Cron().Start(context.Context, time.Minute, notify)
+		done := dis.Cron().Start(context.Context, notify)
 		defer func() {
 			<-done
 		}()

internal/config/config.go

@@ -6,6 +6,7 @@ import (
 	"net/url"
 	"reflect"
 	"strings"
+	"time"
 )
 
 // Config represents the configuration of a WissKI Distillery.
@@ -89,6 +90,9 @@ type Config struct {
 	// name of docker network to use
 	DockerNetworkName string `env:"DOCKER_NETWORK_NAME" default:"distillery" parser:"nonempty"`
 
+	// interval to trigger distillery cron tasks in
+	CronInterval time.Duration `env:"CRON_INTERVAL" default:"10m" parser:"duration"`
+
 	// ConfigPath is the path this configuration was loaded from (if any)
 	ConfigPath string
 }

internal/config/config_template

@@ -76,3 +76,6 @@ KEYCLOAK_ADMIN_PASSWORD=${KEYCLOAK_ADMIN_PASSWORD}
 # The admin user and password required to access the /dis/ server and api
 DIS_ADMIN_USER=${DIS_ADMIN_USER}
 DIS_ADMIN_PASSWORD=${DIS_ADMIN_PASSWORD}
+
+# the interval to run cron in
+CRON_INTERVAL=10m

internal/dis/component/control/cron/cron.go

@@ -91,16 +91,18 @@ func (control *Cron) Once(ctx context.Context) {
 	zerolog.Ctx(ctx).Info().Time("time", time.Now()).Msg("Finished Cron")
 }
 
-// Start invokes all cron jobs regularly, waiting interval between invocations.
+// Start invokes all cron jobs regularly, waiting between invocations as specified in configuration.
 //
 // The first run is invoked immediatly.
 // The call to Start returns after the first invocation of all cron tasks.
 //
 // The returned channel is closed once no more cron tasks are active.
-func (control *Cron) Start(ctx context.Context, interval time.Duration, signal <-chan struct{}) <-chan struct{} {
+func (control *Cron) Start(ctx context.Context, signal <-chan struct{}) <-chan struct{} {
+	zerolog.Ctx(ctx).Info().Dur("interval", control.Config.CronInterval).Msg("Scheduling Cron() tasks")
+
 	// run runs cron tasks with the configured timeout
 	run := func() {
-		ctx, done := context.WithTimeout(ctx, interval)
+		ctx, done := context.WithTimeout(ctx, control.Config.CronInterval)
 		defer done()
 
 		control.Once(ctx)
@@ -117,7 +119,7 @@ func (control *Cron) Start(ctx context.Context, interval time.Duration, signal <
 		timer := timex.NewTimer()
 		for {
 			timex.StopTimer(timer)
-			timer.Reset(interval)
+			timer.Reset(control.Config.CronInterval)
 
 			select {
 			case <-timer.C:

pkg/stringparser/parse.go

@@ -47,6 +47,7 @@ var knownParsers map[string]Parser[any] = map[string]Parser[any]{
 	"abspath":   asGenericParser(ParseAbspath),
 	"domain":    asGenericParser(ParseValidDomain),
 	"domains":   asGenericParser(ParseValidDomains),
+	"duration":  asGenericParser(ParseDuration),
 	"number":    asGenericParser(ParseNumber),
 	"port":      asGenericParser(ParsePort),
 	"https_url": asGenericParser(ParseHttpsURL),

pkg/stringparser/stringparser.go

@@ -6,6 +6,7 @@ import (
 	"regexp"
 	"strconv"
 	"strings"
+	"time"
 
 	"github.com/FAU-CDI/wisski-distillery/pkg/environment"
 	"github.com/FAU-CDI/wisski-distillery/pkg/fsx"
@@ -116,3 +117,8 @@ func ParseSlug(env environment.Environment, s string) (string, error) {
 	}
 	return s, nil
 }
+
+// ParseDuration parses a time.Duration
+func ParseDuration(env environment.Environment, s string) (time.Duration, error) {
+	return time.ParseDuration(s)
+}