You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Daan Hoogland <da...@gmail.com> on 2014/05/15 22:15:48 UTC
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
Re: marvin cloudstackConnection init
Posted by Daan Hoogland <da...@gmail.com>.
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
<sa...@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.hoogland@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
> <sa...@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.hoogland@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
RE: marvin cloudstackConnection init
Posted by Santhosh Edukulla <sa...@citrix.com>.
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.hoogland@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
<sa...@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.hoogland@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
Re: marvin cloudstackConnection init
Posted by Daan Hoogland <da...@gmail.com>.
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
<sa...@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.hoogland@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
RE: marvin cloudstackConnection init
Posted by Santhosh Edukulla <sa...@citrix.com>.
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.hoogland@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