Re: [vdsm] Move some of code from spec file into vdsm-tool function issue

2012-05-28 Thread Dan Kenigsberg
On Mon, May 28, 2012 at 04:35:26PM +0100, Daniel P. Berrange wrote: > On Mon, May 28, 2012 at 11:33:15AM -0400, Federico Simoncelli wrote: > > - Original Message - > > > From: "Daniel P. Berrange" > > > To: "Federico Simoncelli" > > > Cc: "Lei Li" , "Adam Litke" , > > > "Dan Kenigsberg"

Re: [vdsm] Move some of code from spec file into vdsm-tool function issue

2012-05-28 Thread Daniel P. Berrange
On Mon, May 28, 2012 at 11:33:15AM -0400, Federico Simoncelli wrote: > - Original Message - > > From: "Daniel P. Berrange" > > To: "Federico Simoncelli" > > Cc: "Lei Li" , "Adam Litke" , > > "Dan Kenigsberg" , "Ryan > > Harper" , vdsm-devel@lists.fedorahosted.org, > > "Ayal Baron" > >

Re: [vdsm] Move some of code from spec file into vdsm-tool function issue

2012-05-28 Thread Federico Simoncelli
- Original Message - > From: "Daniel P. Berrange" > To: "Federico Simoncelli" > Cc: "Lei Li" , "Adam Litke" , "Dan > Kenigsberg" , "Ryan > Harper" , vdsm-devel@lists.fedorahosted.org, "Ayal > Baron" > Sent: Monday, May 28, 2012 4:52:38 PM > Subject: Re: Move some of code from spec file

Re: [vdsm] Move some of code from spec file into vdsm-tool function issue

2012-05-28 Thread Daniel P. Berrange
On Mon, May 28, 2012 at 10:39:08AM -0400, Federico Simoncelli wrote: > - Original Message - > > From: "Lei Li" > > To: vdsm-devel@lists.fedorahosted.org > > Cc: "Adam Litke" , "Dan Kenigsberg" , > > "Federico Simoncelli" , > > "Ryan Harper" > > Sent: Monday, May 28, 2012 11:18:03 AM > >

Re: [vdsm] Move some of code from spec file into vdsm-tool function issue

2012-05-28 Thread Federico Simoncelli
- Original Message - > From: "Lei Li" > To: vdsm-devel@lists.fedorahosted.org > Cc: "Adam Litke" , "Dan Kenigsberg" , > "Federico Simoncelli" , > "Ryan Harper" > Sent: Monday, May 28, 2012 11:18:03 AM > Subject: Move some of code from spec file into vdsm-tool function issue > > Hi guys,

[vdsm] Move some of code from spec file into vdsm-tool function issue

2012-05-28 Thread Lei Li
Hi guys, Adam point out a problem about my patch moving some of the post and preun section in vdsm spec file into vdsm-tool, and I have the same concern. After some discussion, I'd like to ask for your suggestion on the patch as link below. http://gerrit.ovirt.org/#patch,sidebyside,4528,3,vdsm.