Re: [Xen-devel] [PATCH 2 of 2 V6] libxl: Remus - xl remus command

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Thu, 2012-05-17 at 20:48 +0100, Shriram Rajagopalan wrote:
> diff -r 496ff6ce5bb6 -r 92bf8bd9ae57 docs/man/xl.pod.1
> --- a/docs/man/xl.pod.1       Thu May 17 12:37:07 2012 -0700
> +++ b/docs/man/xl.pod.1       Thu May 17 12:37:10 2012 -0700
> @@ -381,6 +381,41 @@
>  
>  =back
>  
> +=item B<remus> [I<OPTIONS>] I<domain-id> I<host>
> +
> +Enable Remus HA for domain. By default B<xl> relies on ssh as a transport
> +mechanism between the two hosts.
> +
> [...]

> +
> +=item B<-b>
> +
> +Do not checkpoint the disk. Replicate memory checkpoints to /dev/null
> +(blackhole).  Network output buffering remains enabled (unless --no-net is
> +supplied).  Generally useful for debugging.

Unless I'm mistaken the current remus support in (lib)xl doesn't
implement either disk or networking replication (and --no-net doesn't
seem to exist), at least there as several TODOs to that effect in the
code.

Please can you send an incremental patch which corrects this.

I also think it would be worth mentioning in the intro that "xl remus"
as it stands is "proof-of-concept" or "early preview", "experimental" or
something along these lines, otherwise people will expect it to be a
complete solution, which it isn't.

More importantly I think the lack of STONITH functionality should be
highlighted, since it would be rather dangerous to deploy remus without
it.

Ian.



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel