All of lore.kernel.org
 help / color / mirror / Atom feed
From: Lars Ellenberg <lars.ellenberg@linbit.com>
To: drbd-dev@lists.linbit.com, xen-devel@lists.xenproject.org,
	drbd-user@lists.linbit.com
Subject: Re: [Drbd-dev] [PATCH] block-drbd: type is "phy" for drbd backends
Date: Mon, 7 Apr 2014 16:31:32 +0200	[thread overview]
Message-ID: <20140407143132.GG2209__20527.3533043966$1396881216$gmane$org@soda.linbit> (raw)
In-Reply-To: <533A66D5.1020103@citrix.com>

On Tue, Apr 01, 2014 at 09:12:21AM +0200, Roger Pau Monné wrote:
> Ping?

Has been committed to our internal repositories,
will find its way to the public git and
into the tarballs and packages "soon".

> 
> On 26/02/14 18:01, Roger Pau Monné wrote:
> > On 26/02/14 17:30, Lars Ellenberg wrote:
> >> On Wed, Feb 26, 2014 at 04:18:50PM +0100, Roger Pau Monné wrote:
> >>>
> >>> I've realized I haven't Cc'ed the drbd developers lists, so adding it now.
> >>
> >> We do read the user's list just as well.
> >> Our -dev is actually mainly intended for "coordination".
> >>
> >>> On 20/02/14 16:59, Roger Pau Monne wrote:
> >>>> The type written to xenstore by libxl when attaching a drbd backend is
> >>>> "phy", not "drbd", so handle this case also.
> >>
> >> If you say so; you should know ;-)
> >> Did this change at some point?
> > 
> > I'm sure libxl (the new Xen toolstack) uses "phy" and not "drbd" as
> > type. Probably the old xend toolstack was setting type to "drbd", and
> > that's why the script checked for drbd.
> > 
> > IMHO, setting the type to phy is the right thing to do, since drbd ends
> > up presenting a block device which is handled by blkback.
> > 
> >> I personally have not used this script in a long while...
> >>
> >> Do you ("citrix") have this in some
> >> regression/integration testing yourself?
> > 
> > Not in Xen OSS Test suite [0], although I have no idea if other products
> > based on Xen perform any kind of tests with drbd.
> > 
> > [0] http://xenbits.xen.org/gitweb/?p=osstest.git;a=summary

-- 
: Lars Ellenberg
: LINBIT | Your Way to High Availability
: DRBD/HA support and consulting http://www.linbit.com

DRBD® and LINBIT® are registered trademarks of LINBIT, Austria.
__
please don't Cc me, but send to list   --   I'm subscribed

  reply	other threads:[~2014-04-07 14:31 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-02-20 15:59 [PATCH] block-drbd: type is "phy" for drbd backends Roger Pau Monne
2014-02-26 15:18 ` Roger Pau Monné
2014-02-26 16:30   ` [Drbd-dev] " Lars Ellenberg
     [not found]   ` <20140226163036.GE3154@soda.linbit>
2014-02-26 17:01     ` Roger Pau Monné
2014-04-01  7:12       ` Roger Pau Monné
2014-04-07 14:31         ` Lars Ellenberg [this message]
     [not found]         ` <20140407143132.GG2209@soda.linbit>
2014-04-07 14:40           ` Roger Pau Monné

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='20140407143132.GG2209__20527.3533043966$1396881216$gmane$org@soda.linbit' \
    --to=lars.ellenberg@linbit.com \
    --cc=drbd-dev@lists.linbit.com \
    --cc=drbd-user@lists.linbit.com \
    --cc=xen-devel@lists.xenproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.