BUG/MINOR: redispatch: allow interval of 0

Author: mjuraga

configuration/configuration.go

@@ -976,9 +976,8 @@ func (s *SectionParser) redispatch() interface{} {
 		return nil
 	}
 	d := data.(*types.OptionRedispatch)
-	br := &models.Redispatch{}
-	if d.Interval != nil {
-		br.Interval = *d.Interval
+	br := &models.Redispatch{
+		Interval: d.Interval,
 	}
 	if d.NoOption {
 		d := "disabled"
@@ -2350,15 +2349,12 @@ func (s *SectionObject) redispatch(field reflect.Value) error {
 			return misc.CreateTypeAssertError("option redispatch")
 		}
 		d := &types.OptionRedispatch{
-			Interval: &br.Interval,
+			Interval: br.Interval,
 			NoOption: false,
 		}
 		if *br.Enabled == "disabled" {
 			d.NoOption = true
 		}
-		if br.Interval == 0 {
-			d = nil
-		}
 		if err := s.set("option redispatch", d); err != nil {
 			return err
 		}

models/redispatch.go

@@ -9722,7 +9722,7 @@ definitions:
         type: string
       interval:
         type: integer
-        x-nullable: false
+        x-nullable: true
     required:
       - enabled
     type: object

models/redispatch_compare.go

@@ -149,7 +149,7 @@ redispatch:
       enum: [enabled, disabled]
     interval:
       type: integer
-      x-nullable: false
+      x-nullable: true
 errorfile:
   type: object
   x-display-name: Error File

models/redispatch_compare_test.go

@@ -219,7 +219,7 @@ func TestCreateEditDeleteBackend(t *testing.T) {
 			Disabled: true,
 			Redispatch: &models.Redispatch{
 				Enabled:  misc.StringP("enabled"),
-				Interval: 0,
+				Interval: misc.Int64P(0),
 			},
 		},
 	}
@@ -236,9 +236,6 @@ func TestCreateEditDeleteBackend(t *testing.T) {
 		t.Error(err.Error())
 	}
 
-	// A redispatch with a zero interval is automatically removed from configuration.
-	b.Redispatch = nil
-
 	var givenJSONB []byte
 	givenJSONB, err = b.MarshalBinary()
 	if err != nil {