Re: [vdsm] Patch review process

2012-09-10 Thread Dan Kenigsberg
On Sun, Sep 09, 2012 at 12:27:30PM -0500, Adam Litke wrote: Hi, I want to open up a discussion about patch reviews in the vdsm project. I believe everyone will agree that more code review needs to happen for the betterment of the project. I want to ask everyone some questions and also

Re: [vdsm] Change in vdsm[master]: bootstrap: perform reboot asynchronously

2012-09-10 Thread Ryan Harper
* Alon Bar-Lev alo...@redhat.com [2012-09-07 15:46]: - Original Message - From: Ryan Harper ry...@us.ibm.com To: Alon Bar-Lev alo...@redhat.com Cc: Ryan Harper ry...@us.ibm.com, vdsm-devel@lists.fedorahosted.org Sent: Friday, September 7, 2012 10:47:10 PM Subject: Re: Change

Re: [vdsm] Change in vdsm[master]: Add uploadIso API call for pushing ISOs into an active iso_d...

2012-09-10 Thread Ryan Harper
* ih...@redhat.com ih...@redhat.com [2012-09-09 15:39]: Itamar Heim has posted comments on this change. Change subject: Add uploadIso API call for pushing ISOs into an active iso_domain via pool .. Patch Set 1: how

Re: [vdsm] Change in vdsm[master]: Add uploadIso API call for pushing ISOs into an active iso_d...

2012-09-10 Thread Itamar Heim
On 09/10/2012 05:45 PM, Ryan Harper wrote: * ih...@redhat.com ih...@redhat.com [2012-09-09 15:39]: Itamar Heim has posted comments on this change. Change subject: Add uploadIso API call for pushing ISOs into an active iso_domain via pool

Re: [vdsm] Change in vdsm[master]: Add uploadIso API call for pushing ISOs into an active iso_d...

2012-09-10 Thread Ryan Harper
* Itamar Heim ih...@redhat.com [2012-09-10 10:08]: On 09/10/2012 05:45 PM, Ryan Harper wrote: * ih...@redhat.com ih...@redhat.com [2012-09-09 15:39]: Itamar Heim has posted comments on this change. Change subject: Add uploadIso API call for pushing ISOs into an active iso_domain via pool

Re: [vdsm] Patch review process

2012-09-10 Thread Ryan Harper
* Adam Litke a...@us.ibm.com [2012-09-09 12:29]: Hi, I want to open up a discussion about patch reviews in the vdsm project. I believe everyone will agree that more code review needs to happen for the betterment of the project. I want to ask everyone some questions and also make some

Re: [vdsm] Change in vdsm[master]: Add uploadIso API call for pushing ISOs into an active iso_d...

2012-09-10 Thread Ryan Harper
* Itamar Heim ih...@redhat.com [2012-09-10 11:20]: On 09/10/2012 06:34 PM, Ryan Harper wrote: * Itamar Heim ih...@redhat.com [2012-09-10 10:08]: On 09/10/2012 05:45 PM, Ryan Harper wrote: * ih...@redhat.com ih...@redhat.com [2012-09-09 15:39]: Itamar Heim has posted comments on this change.

Re: [vdsm] Change in vdsm[master]: Add uploadIso API call for pushing ISOs into an active iso_d...

2012-09-10 Thread Ryan Harper
* Itamar Heim ih...@redhat.com [2012-09-10 11:33]: On 09/10/2012 07:22 PM, Ryan Harper wrote: * Itamar Heim ih...@redhat.com [2012-09-10 11:20]: On 09/10/2012 06:34 PM, Ryan Harper wrote: * Itamar Heim ih...@redhat.com [2012-09-10 10:08]: On 09/10/2012 05:45 PM, Ryan Harper wrote: *

Re: [vdsm] Patch review process

2012-09-10 Thread Alon Bar-Lev
- Original Message - From: Ryan Harper ry...@us.ibm.com To: Adam Litke a...@us.ibm.com Cc: Ryan Harper ry...@linux.vnet.ibm.com, Anthony Liguori aligu...@linux.vnet.ibm.com, vdsm-devel@lists.fedorahosted.org Sent: Monday, September 10, 2012 7:07:56 PM Subject: Re: [vdsm] Patch

Re: [vdsm] [RFC] GlusterFS domain specific changes

2012-09-10 Thread M. Mohan Kumar
On Fri, 7 Sep 2012 17:07:28 -0400 (EDT), Ayal Baron aba...@redhat.com wrote: - Original Message - As of now BD xlator supports only working with linear Logical volumes, they are thick provisioned. gluster cli command gluster volume create with option device=lv allows to work

Re: [vdsm] Patch review process

2012-09-10 Thread Ryan Harper
* Alon Bar-Lev alo...@redhat.com [2012-09-10 12:22]: - Original Message - From: Ryan Harper ry...@us.ibm.com To: Adam Litke a...@us.ibm.com Cc: Ryan Harper ry...@linux.vnet.ibm.com, Anthony Liguori aligu...@linux.vnet.ibm.com, vdsm-devel@lists.fedorahosted.org Sent:

Re: [vdsm] Patch review process

2012-09-10 Thread Itamar Heim
On 09/10/2012 08:33 PM, Ryan Harper wrote: What's the point of going to the list if not to be able to respond to email? to be able to see what's going on in bulk, in offline, via mail client. but go on gerrit to reply/discuss, or some of your comments will get lost from the patch activity. if

Re: [vdsm] Patch review process

2012-09-10 Thread Ryan Harper
* Itamar Heim ih...@redhat.com [2012-09-10 12:43]: On 09/10/2012 08:33 PM, Ryan Harper wrote: What's the point of going to the list if not to be able to respond to email? to be able to see what's going on in bulk, in offline, via mail client. but go on gerrit to reply/discuss, or some of

Re: [vdsm] Patch review process

2012-09-10 Thread Ryan Harper
* Alon Bar-Lev alo...@redhat.com [2012-09-10 12:44]: - Original Message - From: Ryan Harper ry...@us.ibm.com To: Alon Bar-Lev alo...@redhat.com Cc: Ryan Harper ry...@us.ibm.com, Ryan Harper ry...@linux.vnet.ibm.com, Anthony Liguori aligu...@linux.vnet.ibm.com,