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

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