This may be a breaking change, so fix this on the next major release. The reconcile-interval-seconds flag has a misleading name and description for time.Duration type, remove "seconds" from flag name and description and update netd.yaml as well.
Note: #86 fixes the default value for the reconcile-interval-seconds flag.
|
fs.DurationVar(&nc.ReconcileIntervalSeconds, "reconcile-interval-seconds", 10, |
|
"Reconcile interval in seconds.") |
|
reconcile_interval_seconds: "60s" |
|
- --reconcile-interval-seconds=$(RECONCILE_INTERVAL_SECONDS) |
|
- name: RECONCILE_INTERVAL_SECONDS |
|
key: reconcile_interval_seconds |
This may be a breaking change, so fix this on the next major release. The
reconcile-interval-secondsflag has a misleading name and description fortime.Durationtype, remove "seconds" from flag name and description and updatenetd.yamlas well.Note: #86 fixes the default value for the
reconcile-interval-secondsflag.netd/pkg/options/options.go
Lines 40 to 41 in fef8459
netd/netd.yaml
Line 100 in fef8459
netd/netd.yaml
Line 204 in fef8459
netd/netd.yaml
Line 213 in fef8459
netd/netd.yaml
Line 217 in fef8459