RE: Disabling failed test cases (was RE: Review Request 23605: CLOUDSTACK-7107: Disabling failed test cases)

2014-07-22 Thread Santhosh Edukulla
Ponnaganti [sudha.ponnaga...@citrix.com] Sent: Monday, July 21, 2014 7:42 AM To: dev@cloudstack.apache.org Subject: RE: Disabling failed test cases (was RE: Review Request 23605: CLOUDSTACK-7107: Disabling failed test cases) Hugo, I absolutely agree with you that tests should not be disabled and fixes

RE: Disabling failed test cases (was RE: Review Request 23605: CLOUDSTACK-7107: Disabling failed test cases)

2014-07-22 Thread Sudha Ponnaganti
/CLOUDSTACK/Development+Process Thanks /Sudha -Original Message- From: Santhosh Edukulla [mailto:santhosh.eduku...@citrix.com] Sent: Tuesday, July 22, 2014 4:14 AM To: dev@cloudstack.apache.org Subject: RE: Disabling failed test cases (was RE: Review Request 23605: CLOUDSTACK-7107

RE: Disabling failed test cases (was RE: Review Request 23605: CLOUDSTACK-7107: Disabling failed test cases)

2014-07-21 Thread Stephen Turner
: Review Request 23605: CLOUDSTACK-7107: Disabling failed test cases On July 21, 2014, 1:03 p.m., Hugo Trippaers wrote: Why would we want to disable test cases that fail? Doesn't this mean we need to fix something else so they don't fail anymore? Hi Hugo, Whenever we found a test case failing

Re: Disabling failed test cases (was RE: Review Request 23605: CLOUDSTACK-7107: Disabling failed test cases)

2014-07-21 Thread Gaurav Aradhye
is going up or down. -- Stephen Turner -Original Message- From: Gaurav Aradhye [mailto:nore...@reviews.apache.org] On Behalf Of Gaurav Aradhye Sent: 21 July 2014 09:58 To: Girish Shilamkar Cc: Gaurav Aradhye; Hugo Trippaers; cloudstack Subject: Re: Review Request 23605: CLOUDSTACK-7107

RE: Disabling failed test cases (was RE: Review Request 23605: CLOUDSTACK-7107: Disabling failed test cases)

2014-07-21 Thread Santhosh Edukulla
cases (was RE: Review Request 23605: CLOUDSTACK-7107: Disabling failed test cases) Hugo, Stephen, We have been following this practice as part of Continuous Integration changes as defined in doc [1]. I personally think that tagging test case with BugId is good idea to map the test cases with bugs

Re: Disabling failed test cases (was RE: Review Request 23605: CLOUDSTACK-7107: Disabling failed test cases)

2014-07-21 Thread Hugo Trippaers
: Girish Shilamkar Subject: Re: Disabling failed test cases (was RE: Review Request 23605: CLOUDSTACK-7107: Disabling failed test cases) Hugo, Stephen, We have been following this practice as part of Continuous Integration changes as defined in doc [1]. I personally think that tagging

RE: Disabling failed test cases (was RE: Review Request 23605: CLOUDSTACK-7107: Disabling failed test cases)

2014-07-21 Thread Sudha Ponnaganti
To: Gaurav Aradhye; Stephen Turner; Hugo Trippaers; dev@cloudstack.apache.org Cc: Girish Shilamkar Subject: RE: Disabling failed test cases (was RE: Review Request 23605: CLOUDSTACK-7107: Disabling failed test cases) All, Alex, wanted to disable test cases in between CI( continuous integration

Re: Disabling failed test cases (was RE: Review Request 23605: CLOUDSTACK-7107: Disabling failed test cases)

2014-07-21 Thread Hugo Trippaers
; Hugo Trippaers; dev@cloudstack.apache.org Cc: Girish Shilamkar Subject: RE: Disabling failed test cases (was RE: Review Request 23605: CLOUDSTACK-7107: Disabling failed test cases) All, Alex, wanted to disable test cases in between CI( continuous integration) runs for the below reason

RE: Disabling failed test cases (was RE: Review Request 23605: CLOUDSTACK-7107: Disabling failed test cases)

2014-07-21 Thread Stephen Turner
. -- Stephen Turner From: Gaurav Aradhye [mailto:gaurav.arad...@clogeny.com] Sent: 21 July 2014 10:41 To: Stephen Turner; Hugo Trippaers; dev@cloudstack.apache.org; Santhosh Edukulla Cc: Girish Shilamkar Subject: Re: Disabling failed test cases (was RE: Review Request 23605: CLOUDSTACK-7107: Disabling

RE: Disabling failed test cases (was RE: Review Request 23605: CLOUDSTACK-7107: Disabling failed test cases)

2014-07-21 Thread Sudha Ponnaganti
+Process -Original Message- From: Trippie [mailto:trip...@gmail.com] On Behalf Of Hugo Trippaers Sent: Monday, July 21, 2014 3:32 AM To: dev@cloudstack.apache.org Subject: Re: Disabling failed test cases (was RE: Review Request 23605: CLOUDSTACK-7107: Disabling failed test cases) Hey Sudha

RE: Disabling failed test cases (was RE: Review Request 23605: CLOUDSTACK-7107: Disabling failed test cases)

2014-07-21 Thread Stephen Turner
:43 To: dev@cloudstack.apache.org Subject: RE: Disabling failed test cases (was RE: Review Request 23605: CLOUDSTACK-7107: Disabling failed test cases) Hugo, I absolutely agree with you that tests should not be disabled and fixes should be made before check in. As per what Alex has mentioned

Re: Disabling failed test cases (was RE: Review Request 23605: CLOUDSTACK-7107: Disabling failed test cases)

2014-07-21 Thread Hugo Trippaers
-Original Message- From: Trippie [mailto:trip...@gmail.com] On Behalf Of Hugo Trippaers Sent: Monday, July 21, 2014 3:32 AM To: dev@cloudstack.apache.org Subject: Re: Disabling failed test cases (was RE: Review Request 23605: CLOUDSTACK-7107: Disabling failed test cases) Hey