cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Pranav Saxena <>
Subject RE: [Merged] - CloudStack -664:AWS Health Check feature into Master
Date Mon, 18 Mar 2013 18:35:33 GMT
Hi Chip ,

I guess there was a discussion about the reason behind not having unit tests for this feature
now - (
) . Not sure , if they hold good now as well  otherwise Rajesh would need to add them .


-----Original Message-----
From: Chip Childers [] 
Sent: Monday, March 18, 2013 11:22 PM
Cc: Rajesh Battala; Brian Federle; Animesh Chaturvedi; Haroon Abdelrahman
Subject: Re: [Merged] - CloudStack -664:AWS Health Check feature into Master

On Mon, Mar 18, 2013 at 4:35 AM, Pranav Saxena <> wrote:
> The AWS health Check feature backend code has been merged into master . The patch applied
cleanly and the RAT build was successful . I'll be soon merging the UI support for this feature
as well.

Rajesh and Pranav,

I don't see any unit tests or marvin tests for this feature.  Am I missing something?

I *do* see a good description of what I would expect to have seen made into a marvin test
included in the "Testing" field of review board.

We had agreed that tests would be included in new feature merges (documented in the feature
branch merge expectations [1] and referenced in the review board guidelines [2]).

Please explain when we will see these.  I'd like to avoid reverting the commits.



View raw message