[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [lvm-devel] [PATCH 0/3] Clvmd -E fixes



On 09/22/2011 11:52 AM, Zdenek Kabelac wrote:
> Patchset fixed restart of clvmd with -E option.
> 
> 1st. Adds support for easier restart of clvmd
>      (i.e.  -Isinglenode is preserved).
> 
> 2nd. Bug fixes support of -E arg.

ack these two.

(maybe cluster lock type is on of the attribute we can store
in udev db? to read it later :-)


> 3rd. Allows to build test suite script for checking clvmd restart.
>      (Any better idea here ?)

We need something for testsuite but I would like to avoid any online
reconfiguration of daemon paths, this is tricky and bug there is
security problem (clvms -S must not start arbitrary program
- it run with root privileges.)

IMHO that popen() should go before this patch.

I am not keen for committing such change now...

Milan


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]