add comment to explain grace period

Emyrk · Emyrk · commit 295b7557260b · 2024-03-06T15:28:12.000-06:00
diff --git a/enterprise/coderd/schedule/template.go b/enterprise/coderd/schedule/template.go
@@ -277,6 +277,9 @@ func (s *EnterpriseTemplateScheduleStore) updateWorkspaceBuild(ctx context.Conte
 	}
 
 	// If max deadline is before now()+2h, then set it to that.
+	// This is intended to give ample warning to this workspace about an upcoming auto-stop.
+	// If we were to omit this "grace" period, then this workspace could be set to be stopped "now".
+	// The "2 hours" was an arbitrary decision for this window.
 	now := s.now()
 	if !autostop.MaxDeadline.IsZero() && autostop.MaxDeadline.Before(now.Add(2*time.Hour)) {
 		autostop.MaxDeadline = now.Add(time.Hour * 2)

Original file line number	Diff line number	Diff line change
`@@ -277,6 +277,9 @@ func (s *EnterpriseTemplateScheduleStore) updateWorkspaceBuild(ctx context.Conte`
`277`	`277`	`}`
`278`	`278`
`279`	`279`	`// If max deadline is before now()+2h, then set it to that.`
	`280`	`+ // This is intended to give ample warning to this workspace about an upcoming auto-stop.`
	`281`	`+ // If we were to omit this "grace" period, then this workspace could be set to be stopped "now".`
	`282`	`+ // The "2 hours" was an arbitrary decision for this window.`
`280`	`283`	`now := s.now()`
`281`	`284`	`if !autostop.MaxDeadline.IsZero() && autostop.MaxDeadline.Before(now.Add(2*time.Hour)) {`
`282`	`285`	`autostop.MaxDeadline = now.Add(time.Hour * 2)`