Re: [PATCH 3/3] Ktest: add email options to sample.config

[Steven Rostedt]

On Fri, 15 Dec 2017 15:20:11 -0800
Tim Tianyang Chen <tianyang.chen@xxxxxxxxxx> wrote:

> A block of email options is added under the optional config section.
> 
> Suggested-by: Dhaval Giani <dhaval.giani@xxxxxxxxxx>
> Signed-off-by: Tim Tianyang Chen <tianyang.chen@xxxxxxxxxx>
> ---
>  sample.conf | 10 ++++++++++
>  1 file changed, 10 insertions(+)
> 
> diff --git a/sample.conf b/sample.conf
> index 6c58cd8bbbae..0571acf2179b 100644
> --- a/sample.conf
> +++ b/sample.conf
> @@ -396,6 +396,16 @@
>  
>  #### Optional Config Options (all have defaults) ####
>  
> +# Email options for receiving notifications. Users must setup
> +# the specified mailer prior to using this feature.

Each one should have a separate comment.

> +#
> +#MAILTO = ""

as this one:

# (default undefined)
#MAILTO =

-- Steve

> +#MAILER = sendmail
> +#EMAIL_ON_ERROR = 1
> +#EMAIL_WHEN_FINISHED = 1
> +#EMAIL_WHEN_STARTED = 0
> +#EMAIL_WHEN_CANCELED = 0
> +
>  # Start a test setup. If you leave this off, all options
>  # will be default and the test will run once.
>  # This is a label and not really an option (it takes no value).