Re: [PATCH v3 0/4] Ktest: add email support

2018-04-04 Thread Steven Rostedt
On Tue, 3 Apr 2018 18:38:19 -0400
Steven Rostedt  wrote:

> On Tue, 3 Apr 2018 12:52:20 -0700
> Tim Tianyang Chen  wrote:
> 
> > Hi Steve, did you get a chance to take a look? cc'ing myself and a 
> > personal email. When I sent the patchset I forgot to include.  
> 
> Sorry, I'm trying to get my own ktest patches in (I have a bunch of non
> committed changes) and I need to clean them up before adding yours.
> 
> I'll take a look at them tonight or tomorrow.

I started working on ktest again, but didn't get as far as I wanted.
I'll be continuing more on it tomorrow. I want this ready for this
merge window, so it is high on my priority list (including your
patches).

-- Steve


Re: [PATCH v3 0/4] Ktest: add email support

2018-04-04 Thread Steven Rostedt
On Tue, 3 Apr 2018 18:38:19 -0400
Steven Rostedt  wrote:

> On Tue, 3 Apr 2018 12:52:20 -0700
> Tim Tianyang Chen  wrote:
> 
> > Hi Steve, did you get a chance to take a look? cc'ing myself and a 
> > personal email. When I sent the patchset I forgot to include.  
> 
> Sorry, I'm trying to get my own ktest patches in (I have a bunch of non
> committed changes) and I need to clean them up before adding yours.
> 
> I'll take a look at them tonight or tomorrow.

I started working on ktest again, but didn't get as far as I wanted.
I'll be continuing more on it tomorrow. I want this ready for this
merge window, so it is high on my priority list (including your
patches).

-- Steve


Re: [PATCH v3 0/4] Ktest: add email support

2018-04-03 Thread Steven Rostedt
On Tue, 3 Apr 2018 12:52:20 -0700
Tim Tianyang Chen  wrote:

> Hi Steve, did you get a chance to take a look? cc'ing myself and a 
> personal email. When I sent the patchset I forgot to include.

Sorry, I'm trying to get my own ktest patches in (I have a bunch of non
committed changes) and I need to clean them up before adding yours.

I'll take a look at them tonight or tomorrow.

Thanks!

-- Steve


Re: [PATCH v3 0/4] Ktest: add email support

2018-04-03 Thread Steven Rostedt
On Tue, 3 Apr 2018 12:52:20 -0700
Tim Tianyang Chen  wrote:

> Hi Steve, did you get a chance to take a look? cc'ing myself and a 
> personal email. When I sent the patchset I forgot to include.

Sorry, I'm trying to get my own ktest patches in (I have a bunch of non
committed changes) and I need to clean them up before adding yours.

I'll take a look at them tonight or tomorrow.

Thanks!

-- Steve


Re: [PATCH v3 0/4] Ktest: add email support

2018-04-03 Thread Dhaval Giani
On 2018-03-26 04:08 PM, Tim Tianyang Chen wrote:
> This patch set will let users define a mailer, an email address and when to 
> receive
> notifications during automated testings. Users need to setup the specified 
> mailer
> prior to using this feature.
> 
> Tim Tianyang Chen (4):
>   Ktest: add email support
>   Ktest: add SigInt handling
>   Ktest: use dodie for critical falures
>   Ktest: add email options to sample.config
> 
>  ktest.pl| 125 
> +---
>  sample.conf |  22 +++
>  2 files changed, 117 insertions(+), 30 deletions(-)
> 

Steve,

Any thoughts?

Thanks!
Dhaval


Re: [PATCH v3 0/4] Ktest: add email support

2018-04-03 Thread Dhaval Giani
On 2018-03-26 04:08 PM, Tim Tianyang Chen wrote:
> This patch set will let users define a mailer, an email address and when to 
> receive
> notifications during automated testings. Users need to setup the specified 
> mailer
> prior to using this feature.
> 
> Tim Tianyang Chen (4):
>   Ktest: add email support
>   Ktest: add SigInt handling
>   Ktest: use dodie for critical falures
>   Ktest: add email options to sample.config
> 
>  ktest.pl| 125 
> +---
>  sample.conf |  22 +++
>  2 files changed, 117 insertions(+), 30 deletions(-)
> 

Steve,

Any thoughts?

Thanks!
Dhaval


Re: [PATCH v3 0/4] Ktest: add email support

2018-04-03 Thread Tim Tianyang Chen
Hi Steve, did you get a chance to take a look? cc'ing myself and a 
personal email. When I sent the patchset I forgot to include.


Tianyang

On 2018-03-26 04:08 PM, Tim Tianyang Chen wrote:


This patch set will let users define a mailer, an email address and when to 
receive
notifications during automated testings. Users need to setup the specified 
mailer
prior to using this feature.

Tim Tianyang Chen (4):
   Ktest: add email support
   Ktest: add SigInt handling
   Ktest: use dodie for critical falures
   Ktest: add email options to sample.config

  ktest.pl| 125 +---
  sample.conf |  22 +++
  2 files changed, 117 insertions(+), 30 deletions(-)



Re: [PATCH v3 0/4] Ktest: add email support

2018-04-03 Thread Tim Tianyang Chen
Hi Steve, did you get a chance to take a look? cc'ing myself and a 
personal email. When I sent the patchset I forgot to include.


Tianyang

On 2018-03-26 04:08 PM, Tim Tianyang Chen wrote:


This patch set will let users define a mailer, an email address and when to 
receive
notifications during automated testings. Users need to setup the specified 
mailer
prior to using this feature.

Tim Tianyang Chen (4):
   Ktest: add email support
   Ktest: add SigInt handling
   Ktest: use dodie for critical falures
   Ktest: add email options to sample.config

  ktest.pl| 125 +---
  sample.conf |  22 +++
  2 files changed, 117 insertions(+), 30 deletions(-)