Hi Dejan,
I agree to your patch.
Best Regards,
Hideo Yamauchi.
--- On Thu, 2012/4/5, Dejan Muhamedagic wrote:
> Hi all,
>
> This is a proposed set of two patches which would eliminate use
> of LVM commands in the monitor path. We already discussed the
> issue elsewhere and I don't see any poi
On Wed, Apr 4, 2012 at 1:52 AM, Christian Franke wrote:
> Hello Florian,
>
> Your question is fully justified - I sincerely apologize for ignoring
> that comprehensive documentation.
>
> I rewrote the patch trying to adhere to the requirements given in the
> documentation.
Wow, that is a response
Hi all,
This is a proposed set of two patches which would eliminate use
of LVM commands in the monitor path. We already discussed the
issue elsewhere and I don't see any point in keeping
vgck/vgdisplay given that they don't result in better monitoring
under normal circumstances. And if the circums
Hi Christian,
On Wed, Apr 04, 2012 at 06:00:45PM +0200, Christian Franke wrote:
> On 04/04/2012 02:55 PM, Lars Marowsky-Bree wrote:
> > that patch shares a *lot* of code with IPaddr(2), doesn't it?
> >
> > I wonder if that wouldn't benefit from refactoring, moving shared
> > functions to an inclu
On 04/04/2012 02:55 PM, Lars Marowsky-Bree wrote:
> that patch shares a *lot* of code with IPaddr(2), doesn't it?
>
> I wonder if that wouldn't benefit from refactoring, moving shared
> functions to an include?
Thanks for your feedback, I will have a look at that - the gratuitous
arp and also som
On Wed, Apr 04, 2012 at 02:55:52PM +0200, Lars Marowsky-Bree wrote:
> On 2012-04-04T01:52:12, Christian Franke wrote:
>
> > Hello Florian,
> >
> > Your question is fully justified - I sincerely apologize for ignoring
> > that comprehensive documentation.
> >
> > I rewrote the patch trying to ad
On 2012-04-04T01:52:12, Christian Franke wrote:
> Hello Florian,
>
> Your question is fully justified - I sincerely apologize for ignoring
> that comprehensive documentation.
>
> I rewrote the patch trying to adhere to the requirements given in the
> documentation.
Hi Christian,
that patch sh