Re: [REVIEW] CLOUDSTACK-7468 in master

2014-09-04 Thread Will Stevens
I found the .travis.yml file. I still need to figure out how to use travis, so I will have to research that... ws *Will STEVENS* Lead Developer *CloudOps* *| *Cloud Solutions Experts 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6 w cloudops.com *|* tw @CloudOps_ On Thu, Sep 4, 2014 at 1:49

Re: [REVIEW] CLOUDSTACK-7468 in master

2014-09-04 Thread Will Stevens
yes, those are both the same fix. One is targeting the 4.4 branch for Daan to merge. The other is targeting the master branch... as for the tests, I can look into travis. where is this .travis.yml you speak of? is there docs anywhere on this? I did real tests on an actual environment with my

Re: [REVIEW] CLOUDSTACK-7468 in master

2014-09-04 Thread sebgoa
On Sep 4, 2014, at 7:17 PM, Will Stevens wrote: > Following up on this... > > Who should I be asking to merge this into master? Should I just be pushing > my changes directly into master? I thought we wanted features and fixes to > go through the RM for the release, is this not the case. > >

Re: [REVIEW] CLOUDSTACK-7468 in master

2014-09-04 Thread Will Stevens
Following up on this... Who should I be asking to merge this into master? Should I just be pushing my changes directly into master? I thought we wanted features and fixes to go through the RM for the release, is this not the case. I have the following fixes waiting for commit: CLOUDSTACK-7468: