Daan, We kept a wiki page, i can check whether this change was mentioned there. We verified all the references before changing and then changed, tested. All scripts now runs with no issues post the changes.
We were receiving many arguments, which can be part of one structure. It was receiving all management details from json as one structure and then passed as it is in deserialized format. Earlier, there were many arguments which are part of this structure mentioned individually and assigned back individually. Where are these scripts which has earlier references? I can look at them. Santhosh ________________________________________ From: Daan Hoogland [daan.hoogl...@gmail.com] Sent: Friday, May 16, 2014 4:09 AM To: Santhosh Edukulla Cc: Girish Shilamkar; dev Subject: Re: marvin cloudstackConnection init No I mean you replaced the individual arguments with a mgmtDetails structure, did you describe this change somewhere? On Fri, May 16, 2014 at 8:52 AM, Santhosh Edukulla <santhosh.eduku...@citrix.com> wrote: > If we are referring to scheme argument related to http\https, there was one > more argument, passed i.e., mgmtDetails structure which serves this purpose > and so removed it. > > > Santhosh > ________________________________________ > From: Daan Hoogland [daan.hoogl...@gmail.com] > Sent: Thursday, May 15, 2014 4:15 PM > To: Santhosh Edukulla; Girish Shilamkar; dev > Subject: marvin cloudstackConnection init > > Santhosh, Girish, > > in 980e66d501f525672827c207d8b0921f819e5fcd you changed the interface > to cloudstackConnection. My test scripts don't work anymore after > updating marvin, do you have a link to an fs and/or a rationale? I can > read the source but I think more people are going to run into this. > > thanks, > -- > Daan -- Daan
