I trust you did and I trust you can. its just that i have some test scripts of my own that didn't work anymore after upgrading. No big deal I just want us to be prepared when others come to the list with the same issue. The scripts you don't need to look at, just a ref if anybody else find the same issue unknowingly.
On Fri, May 16, 2014 at 10:36 AM, Santhosh Edukulla <santhosh.eduku...@citrix.com> wrote: > 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 -- Daan
