RE: [MERGE] Review Request 22270: Xen 2 XenServer refactoring

2014-06-08 Thread Santhosh Edukulla
Logged a bug to track and fix devcloud issue. https://issues.apache.org/jira/browse/CLOUDSTACK-6860 Santhosh On Jun 6, 2014, at 3:53 PM, sebgoa wrote: > > On Jun 6, 2014, at 9:35 PM, Rohit Yadav wrote: > >> >> >> >> On Sat, Jun 7, 2014 at 12:51 AM, sebgoa wrote: >> >> On Jun 6, 2014, at 8:4

Re: [MERGE] Review Request 22270: Xen 2 XenServer refactoring

2014-06-07 Thread Daan Hoogland
+1, If master builds it is alright by me. We will have to fix forward on this one. On Sat, Jun 7, 2014 at 10:55 AM, Sebastien Goasguen wrote: > I applied this patch to master today, it's not a merge of the xen2server > branch per se but the work was done in that branch. > > This renames the xen

Re: [MERGE] Review Request 22270: Xen 2 XenServer refactoring

2014-06-07 Thread Sebastien Goasguen
I applied this patch to master today, it's not a merge of the xen2server branch per se but the work was done in that branch. This renames the xen plugin as xenserver. I know that I jumped the gun a bit on the merge window, but I am willing to revert it if people are not OK with this. I just wan

Re: [MERGE] Review Request 22270: Xen 2 XenServer refactoring

2014-06-06 Thread sebgoa
On Jun 6, 2014, at 9:35 PM, Rohit Yadav wrote: > > > > On Sat, Jun 7, 2014 at 12:51 AM, sebgoa wrote: > > On Jun 6, 2014, at 8:41 PM, Rohit Yadav wrote: > > > Hi again, > > > > I was unable to deploy basic zone, I did: > > > > # marvin and python dependencies were pre-installed > > cd too

Re: [MERGE] Review Request 22270: Xen 2 XenServer refactoring

2014-06-06 Thread Rohit Yadav
On Sat, Jun 7, 2014 at 12:51 AM, sebgoa wrote: > > On Jun 6, 2014, at 8:41 PM, Rohit Yadav wrote: > > > Hi again, > > > > I was unable to deploy basic zone, I did: > > > > # marvin and python dependencies were pre-installed > > cd tools/devcloud; > > python ../marvin/marvin/deployDataCenter.py -

Re: [MERGE] Review Request 22270: Xen 2 XenServer refactoring

2014-06-06 Thread sebgoa
On Jun 6, 2014, at 8:41 PM, Rohit Yadav wrote: > Hi again, > > I was unable to deploy basic zone, I did: > > # marvin and python dependencies were pre-installed > cd tools/devcloud; > python ../marvin/marvin/deployDataCenter.py -i devcloud.cfg > > The error I got was related to agent communic

Re: [MERGE] Review Request 22270: Xen 2 XenServer refactoring

2014-06-06 Thread Rohit Yadav
Hi again, I was unable to deploy basic zone, I did: # marvin and python dependencies were pre-installed cd tools/devcloud; python ../marvin/marvin/deployDataCenter.py -i devcloud.cfg The error I got was related to agent communication, if this can help resolve the issue: Exception Occurred :['Tr

Re: Merge Review Request 22270: Xen 2 XenServer refactoring

2014-06-06 Thread Rohit Yadav
I just read the design document on the wiki https://cwiki.apache.org/confluence/display/CLOUDSTACK/Convert+Xen+usage+to+XenServer and got my answers regarding the work. Thanks. Regards. On Fri, Jun 6, 2014 at 11:48 PM, Rohit Yadav wrote: > Hi Sebastien, > > On Fri, Jun 6, 2014 at 9:51 PM, sebg

Re: Merge Review Request 22270: Xen 2 XenServer refactoring

2014-06-06 Thread Rohit Yadav
Hi Sebastien, On Fri, Jun 6, 2014 at 9:51 PM, sebgoa wrote: > Rohit, > > Any chance you have time to check this one review. > My git kungfu is limited and the patch seems to have conflicts. > > Tim did quite a bit of work to get there, so a merge could be call > quickly, if this patch could appl

Re: [MERGE] Review Request 22270: Xen 2 XenServer refactoring

2014-06-06 Thread Rohit Yadav
On Fri, Jun 6, 2014 at 10:14 PM, sebgoa wrote: > Folks, > > Tim has prepared a pretty significant patch in the review it lists below. > There is also a wiki page describing the change. > > Basically it splits XenServer and XenProject (pure Xen) in two separate > hypervisors. Until now we had used

[MERGE] Review Request 22270: Xen 2 XenServer refactoring

2014-06-06 Thread sebgoa
Folks, Tim has prepared a pretty significant patch in the review it lists below. There is also a wiki page describing the change. Basically it splits XenServer and XenProject (pure Xen) in two separate hypervisors. Until now we had used XenServer to also handle XenProject. It will allow to split

Re: Merge Review Request 22270: Xen 2 XenServer refactoring

2014-06-06 Thread sebgoa
Rohit, Any chance you have time to check this one review. My git kungfu is limited and the patch seems to have conflicts. Tim did quite a bit of work to get there, so a merge could be call quickly, if this patch could apply, that would ease another round of rebasing.. thanks,, -sebastien On J

Merge Review Request 22270: Xen 2 XenServer refactoring

2014-06-05 Thread Tim Mackey
I've just submitted a review request which is essentially a merge of the xen2server feature branch back into master. Since this is a refactoring of the Xen plugin to make it more explicitly a XenServer plugin per the feature: https://cwiki.apache.org/confluence/display/CLOUDSTACK/Convert+Xen+usage