<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<meta name="Generator" content="Microsoft Word 14 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
        {font-family:"Cambria Math";
        panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
        {font-family:Calibri;
        panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
        {font-family:Tahoma;
        panose-1:2 11 6 4 3 5 4 4 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
        {margin:0in;
        margin-bottom:.0001pt;
        font-size:12.0pt;
        font-family:"Times New Roman","serif";}
a:link, span.MsoHyperlink
        {mso-style-priority:99;
        color:blue;
        text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
        {mso-style-priority:99;
        color:purple;
        text-decoration:underline;}
p
        {mso-style-priority:99;
        mso-margin-top-alt:auto;
        margin-right:0in;
        mso-margin-bottom-alt:auto;
        margin-left:0in;
        font-size:12.0pt;
        font-family:"Times New Roman","serif";}
span.im
        {mso-style-name:im;}
span.EmailStyle19
        {mso-style-type:personal-reply;
        font-family:"Calibri","sans-serif";
        color:#1F497D;}
.MsoChpDefault
        {mso-style-type:export-only;
        font-family:"Calibri","sans-serif";}
@page WordSection1
        {size:8.5in 11.0in;
        margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
        {page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="EN-US" link="blue" vlink="purple">
<div class="WordSection1">
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">Hi John,<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">Okay, it sounds like we’ll be okay to implement the replication V2 spec. I believe the failover aspect was the only API that we were seeing a problem with.
 It also sounds like there might be some areas for improvement around documentation, etc. Let me know if there’s anything I/we can do to help on that.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">Thanks,<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">Michael<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal"><b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif"">From:</span></b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif""> John Griffith [mailto:john.griffith@solidfire.com]
<br>
<b>Sent:</b> Thursday, September 24, 2015 2:26 PM<br>
<b>To:</b> Price, Loren<br>
<b>Cc:</b> openstack@lists.openstack.org<br>
<b>Subject:</b> Re: [Openstack] [Cinder] Questions on implementing the Replication V2 spec<o:p></o:p></span></p>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<div>
<p class="MsoNormal"><span style="font-family:"Courier New""><o:p> </o:p></span></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<p class="MsoNormal">On Thu, Sep 24, 2015 at 11:48 AM, Price, Loren <<a href="mailto:Michael.Price@netapp.com" target="_blank">Michael.Price@netapp.com</a>> wrote:<o:p></o:p></p>
<div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span style="color:black">Hey,</span><o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span style="color:black"> </span><o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span style="color:black">We’re looking into implementing the
<a href="https://github.com/openstack/cinder-specs/blob/master/specs/liberty/replication_v2.rst" target="_blank">
VolumeReplication_V2</a> spec for our NetApp E-Series volume driver. Looking at the specification, I can foresee a problem with implementing the new API call “</span><span style="color:#333333;background:white">failover_replicated_volume(volume) “ with an unmanaged
 replication target. I believe with a managed target we can provide it, if I’m understanding correctly that it merely requires updating the host id for the volume. Based on that, I have two questions:</span><o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span style="color:#333333;background:white"> </span><o:p></o:p></p>
<p><span style="color:#333333">1.</span><span style="font-size:7.0pt;color:#333333">     
</span><span style="color:black">Is it acceptable, in implementing this spec, to only provide this API for managed targets (and either throw an exception or essentially make a no-op) for an unmanaged replication target?</span><o:p></o:p></p>
<p><span style="color:#333333">2.</span><span style="font-size:7.0pt;color:#333333">     
</span><span style="color:black">In general, if a storage backend is incapable of performing a certain operation, what is the correct way to handle it? Can the driver implement the spec at all? Should it throw a NotImplementedError? No-op?</span><o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span style="color:black"> </span><o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span style="color:black">Thanks,</span><o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span style="color:black"> </span><o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span style="color:black">Michael Price</span><o:p></o:p></p>
</div>
</div>
<p class="MsoNormal" style="margin-bottom:12.0pt"><br>
_______________________________________________<br>
Mailing list: <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack" target="_blank">
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack</a><br>
Post to     : <a href="mailto:openstack@lists.openstack.org">openstack@lists.openstack.org</a><br>
Unsubscribe : <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack" target="_blank">
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack</a><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><span style="font-family:"Cambria Math","serif"">​</span><span style="font-family:"Courier New"">Ooops, did I not respond to the list on that last response?  Just incase here it is again:<o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-family:"Cambria Math","serif"">​</span><span style="font-family:"Courier New""><o:p></o:p></span></p>
</div>
<div>
<blockquote style="border:none;border-left:solid #CCCCCC 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-right:0in">
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span style="color:#333333"> </span><o:p></o:p></p>
<p><span style="font-size:9.5pt;font-family:"Arial","sans-serif";color:#333333">1.</span><span style="font-size:7.0pt;color:#333333">      </span><span style="font-size:9.5pt;font-family:"Arial","sans-serif";color:black">Is it acceptable, in implementing this
 spec, to only provide this API for managed targets (and either throw an exception or essentially make a no-op) for an unmanaged replication target?</span><span style="font-size:9.5pt;font-family:"Arial","sans-serif""><o:p></o:p></span></p>
</div>
</blockquote>
<div>
<div>
<p class="MsoNormal"><span style="font-size:9.5pt;font-family:"Cambria Math","serif"">​</span><span style="font-size:9.5pt;font-family:"Courier New"">Yes by
</span><span style="font-size:9.5pt;font-family:"Cambria Math","serif"">​</span><span style="font-size:9.5pt;font-family:"Courier New""><o:p></o:p></span></p>
</div>
<p class="MsoNormal"><span style="font-size:9.5pt;font-family:"Arial","sans-serif""> <o:p></o:p></span></p>
<div>
<p class="MsoNormal"><span style="font-size:9.5pt;font-family:"Cambria Math","serif"">​</span><span style="font-size:9.5pt;font-family:"Courier New"">design it's set up such that
</span><span style="font-size:9.5pt;font-family:"Cambria Math","serif"">​</span><span style="font-size:9.5pt;font-family:"Courier New"">it's left up to configuration.  In other words the idea is that we have fairly loose definitions around the API calls themselves
 to allow for differing implementations.<o:p></o:p></span></p>
</div>
</div>
<blockquote style="border:none;border-left:solid #CCCCCC 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-right:0in">
<div>
<p><span style="font-size:9.5pt;font-family:"Arial","sans-serif";color:#333333">2.</span><span style="font-size:7.0pt;color:#333333">      </span><span style="font-size:9.5pt;font-family:"Arial","sans-serif";color:black">In general, if a storage backend is
 incapable of performing a certain operation, what is the correct way to handle it? Can the driver implement the spec at all? Should it throw a NotImplementedError? No-op?</span><span style="font-size:9.5pt;font-family:"Arial","sans-serif""><o:p></o:p></span></p>
</div>
</blockquote>
<div>
<div>
<p class="MsoNormal"><span style="font-size:9.5pt;font-family:"Cambria Math","serif"">​</span><span style="font-size:9.5pt;font-family:"Courier New"">Depends on who you ask :)  IMO we need to do a better job of this, this could be documenting in the deployment
 guides how to enable/disable API calls in certain deployments so that unsupported calls are just flat out not available.  My true belief is that we shouldn't be implementing features that you can't run with every/any backend device in the first place, but
 that's my usual rant and somewhat off topic here :)<o:p></o:p></span></p>
</div>
</div>
<div>
<div>
<p class="MsoNormal"><span style="font-size:9.5pt;font-family:"Courier New""><o:p> </o:p></span></p>
</div>
</div>
<div>
<div>
<p class="MsoNormal"><span style="font-size:9.5pt;font-family:"Courier New"">Note that a lot of the logic for replication in V2 was moved into the volume-type and the conf file precisely to address some of the issues you mention above.  The idea being that
 if the capabilities of the backend don't match replication specs in the type then the command fails for no-valid host.  The one thing I don't like about this is how we relay that info to the end user (or more accurately the fact that we don't).  We just put
 the volume in error state and the only info regarding why is in the logs which the end user doesn't have.  This is where something like a better more clear policy file would help as well as providing a capabilities call in the API.<o:p></o:p></span></p>
</div>
</div>
<div>
<div>
<p class="MsoNormal"><span style="font-size:9.5pt;font-family:"Courier New""><o:p> </o:p></span></p>
</div>
</div>
<div>
<div>
<p class="MsoNormal"><span style="font-size:9.5pt;font-family:"Courier New"">By the way, I'm glad you asked these questions here.  This is part of the reason why I was so strongly opposed to merging an implementation of the V2 replication in Liberty.  I think
 it's important to have more than one or two vendors looking at this and working out details so we release something that is stable and usable.  My philosophy is that now for M we have a foundation in the core code that will likely evolve as drivers begin implementing
 the feature.<o:p></o:p></span></p>
</div>
</div>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
</div>
</div>
</body>
</html>