<p dir="ltr">Given the the patch is up and well on its way I don't know why this would be a problem.  FWIW ya get my +1</p>
<div class="gmail_extra"><br><div class="gmail_quote">On Sep 9, 2016 1:34 PM, "Gorka Eguileor" <<a href="mailto:geguileo@redhat.com">geguileo@redhat.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
As some of you may know, Jon Bernard (jbernard on IRC) has been working<br>
on the RBD v2.1 replication implementation [1] for a while, and we would<br>
like to request a Feature Freeze Exception for that work, as we believe<br>
it is a good candidate being a low risk change for the integrity of<br>
the existing functionality in the driver:<br>
<br>
- It's non intrusive if it's not enabled (enabled using<br>
  replication_device configuration option).<br>
- It doesn't affect existing deployments (disabled by default).<br>
- Changes are localized to the driver itself (rbd.py) and the driver<br>
  unit tests file (test_rbd.py).<br>
<br>
Jon would have liked to make this request himself, but due to the<br>
untimely arrival of his newborn baby this is not possible.<br>
<br>
For obvious reasons Jon will not be available for a little while, but<br>
this will not be a problem, as I am well acquainted with the code -and<br>
I'll be able to reach Jon if necessary- and will be taking care of the<br>
final steps of the review process of his patch: replying to comments in<br>
a timely fashion, making changes to the code as required, and answering<br>
pings on IRC regarding the patch.<br>
<br>
Since some people may be interested in testing this functionality during<br>
the reviewing process -or just for fun- I'll be publishing a post with<br>
detailed explanation on how to deploy and test this feature as well as<br>
an automated way to deploy 2 Ceph clusters -linked to be mirroring one<br>
another-, and one devstack node with everything ready to test the<br>
functionality (configuration and keys for the Ceph clusters, cinder<br>
configuration, the latest upstream patch, and a volume type with the<br>
right configuration).<br>
<br>
Please, do not hesitate to ask if there are any questions to or concerns<br>
related to this request.<br>
<br>
Thank you for taking the time to evaluate this request.<br>
<br>
Cheers,<br>
Gorka.<br>
<br>
[1]: <a href="https://review.openstack.org/333565" rel="noreferrer" target="_blank">https://review.openstack.org/<wbr>333565</a><br>
<br>
______________________________<wbr>______________________________<wbr>______________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.<wbr>openstack.org?subject:<wbr>unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/<wbr>cgi-bin/mailman/listinfo/<wbr>openstack-dev</a><br>
</blockquote></div></div>