set default StartLimitIntervalSec of 1 day

aksiksi · Nov 6, 2023 · 03fb8a3 · 03fb8a3
1 parent d72b0ae
commit 03fb8a3
Show file tree

Hide file tree

Showing 5 changed files with 13 additions and 2 deletions.
diff --git a/compose.go b/compose.go
@@ -76,8 +76,11 @@ func parseRestartPolicyAndSystemdLabels(service *types.ServiceConfig) (*NixConta
 			if maxAttempts, err := strconv.ParseInt(maxAttemptsString, 10, 64); err != nil {
 				return nil, fmt.Errorf("failed to parse on-failure attempts: %q: %w", maxAttemptsString, err)
 			} else {
-				v := int(maxAttempts)
-				p.StartLimitBurst = &v
+				burst := int(maxAttempts)
+				// Retry limit resets once per day.
+				interval := int((time.Hour * 24).Seconds())
+				p.StartLimitBurst = &burst
+				p.StartLimitIntervalSec = &interval
 			}
 		} else {
 			return nil, fmt.Errorf("unsupported restart: %q", restart)
@@ -108,6 +111,10 @@ func parseRestartPolicyAndSystemdLabels(service *types.ServiceConfig) (*NixConta
 			if window := restartPolicy.Window; window != nil {
 				windowSecs := int(time.Duration(*window).Seconds())
 				p.StartLimitIntervalSec = &windowSecs
+			} else if p.StartLimitBurst != nil {
+				// Retry limit resets once per day by default.
+				windowSecs := int((time.Hour * 24).Seconds())
+				p.StartLimitIntervalSec = &windowSecs
 			}
 		}
 	}

diff --git a/testdata/TestDocker_WithProject_out.nix b/testdata/TestDocker_WithProject_out.nix
@@ -164,6 +164,7 @@
       Restart = "on-failure";
     };
     startLimitBurst = 3;
+    startLimitIntervalSec = 86400;
   };
   virtualisation.oci-containers.containers."traefik" = {
     image = "docker.io/library/traefik";

diff --git a/testdata/TestDocker_out.nix b/testdata/TestDocker_out.nix
@@ -164,6 +164,7 @@
       Restart = "on-failure";
     };
     startLimitBurst = 3;
+    startLimitIntervalSec = 86400;
   };
   virtualisation.oci-containers.containers."traefik" = {
     image = "docker.io/library/traefik";

diff --git a/testdata/TestPodman_WithProject_out.nix b/testdata/TestPodman_WithProject_out.nix
@@ -169,6 +169,7 @@
       Restart = "on-failure";
     };
     startLimitBurst = 3;
+    startLimitIntervalSec = 86400;
   };
   virtualisation.oci-containers.containers."traefik" = {
     image = "docker.io/library/traefik";

diff --git a/testdata/TestPodman_out.nix b/testdata/TestPodman_out.nix
@@ -169,6 +169,7 @@
       Restart = "on-failure";
     };
     startLimitBurst = 3;
+    startLimitIntervalSec = 86400;
   };
   virtualisation.oci-containers.containers."traefik" = {
     image = "docker.io/library/traefik";