You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Devdeep Singh <de...@citrix.com> on 2013/11/04 15:26:06 UTC

RE: [Merge] hyperv branch to master

The changes have been merged to the master branch.

Regards,
Devdeep

-----Original Message-----
From: Devdeep Singh [mailto:devdeep.singh@citrix.com] 
Sent: Thursday, October 31, 2013 3:38 PM
To: dev@cloudstack.apache.org
Cc: Donal Lafferty; Rajesh Battala; Anshul Gangwar
Subject: RE: [Merge] hyperv branch to master

Thanks Darren. I'll start working on getting it merged into the master. I hope to finish it by Friday.

Regards,
Devdeep

-----Original Message-----
From: Darren Shepherd [mailto:darren.s.shepherd@gmail.com]
Sent: Tuesday, October 29, 2013 10:45 PM
To: dev@cloudstack.apache.org
Cc: Donal Lafferty; Rajesh Battala; Anshul Gangwar
Subject: Re: [Merge] hyperv branch to master

+1

When you merge you will get a conflict in the spring configuration.
componentContext.xml and others are gone now in master.  You will need to update to the new style.  The easiest will be to refer to XenServer spring config for an example of how to do it.  You can largely copy and paste the contents of plugins/hypervisors/xen/resources/META-INF/cloudstack and replace xenserver with hyperv.  You can also refer to https://cwiki.apache.org/confluence/display/CLOUDSTACK/Plug-ins%2C+Modules%2C+and+Extensions
for more in depth explaination of the changes.

Darren

On Tue, Oct 29, 2013 at 9:35 AM, David Nalley <da...@gnsa.us> wrote:
> On Mon, Oct 28, 2013 at 7:41 AM, Devdeep Singh <de...@citrix.com> wrote:
>> Hi,
>>
>> I would like to merge the support for Hyperv to the master branch. Development for this has been done by Donal, Rajesh, Anshul and I on branch [1]. The feature was proposed for merge earlier [3] but unit tests for hyperv agent code were requested [4].
>>
>
> I know this has been discussed several times, but what was the 
> decision about making the agent code live in its own repo?
> It's C#, might need to be signed to actually run easily on Windows, etc.
>
> --David