Merge pull request #15179 from bednar/check-cron

fix(alerts): remove the cron option from checks/rules

http/swagger.yml

@@ -9361,9 +9361,6 @@ components:
         offset:
           description: Duration to delay after the schedule, before executing check.
           type: string
-        cron:
-          description: The check repetition interval in the form `* * * * * *`.
-          type: string
         tags:
           description: List of tags to write to each status.
           type: array
@@ -9545,9 +9542,6 @@ components:
         offset:
           description: Duration to delay after the schedule, before executing check.
           type: string
-        cron:
-          description: The notification repetition interval in the form `* * * * * *`.
-          type: string
         runbookLink:
           type: string
         limitEvery:

notification/check/check.go

@@ -84,10 +84,6 @@ func (b Base) generateTaskOption() ast.Statement {
 
 	props = append(props, flux.Property("name", flux.String(b.Name)))
 
-	if b.Cron != "" {
-		props = append(props, flux.Property("cron", flux.String(b.Cron)))
-	}
-
 	if b.Every != nil {
 		props = append(props, flux.Property("every", (*ast.DurationLiteral)(b.Every)))
 	}

notification/rule/rule.go

@@ -52,7 +52,6 @@ type Base struct {
 	TaskID      influxdb.ID `json:"taskID,omitempty"`
 	// SleepUntil is an optional sleeptime to start a task.
 	SleepUntil *time.Time             `json:"sleepUntil,omitempty"`
-	Cron       string                 `json:"cron,omitempty"`
 	Every      *notification.Duration `json:"every,omitempty"`
 	// Offset represents a delay before execution.
 	// It gets marshalled from a string duration, i.e.: "10s" is 10 seconds
@@ -263,10 +262,6 @@ func (b *Base) generateTaskOption() ast.Statement {
 
 	props = append(props, flux.Property("name", flux.String(b.Name)))
 
-	if b.Cron != "" {
-		props = append(props, flux.Property("cron", flux.String(b.Cron)))
-	}
-
 	if b.Every != nil {
 		// Make the windows overlap and filter records from previous queries.
 		// This is so that we wont miss the first points possible state change.