<div dir="ltr">I apologise if this is not the place to ask, but this question was actually bugging me for a long time now.<div>Is there any particular reason fast-diff was disabled for backup images in the first place? Like, any actual technical limitation, i.e. "don't enable it or it will break your backups"?</div><div>Or is it only because earlier versions of CEPH did not support fast-diff, so it was disabled for compatibility reasons?</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">ср, 13 янв. 2021 г. в 16:40, Brian Rosmaita <<a href="mailto:rosmaita.fossdev@gmail.com">rosmaita.fossdev@gmail.com</a>>:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 1/13/21 4:37 AM, Christian Rohmann wrote:<br>
> Hey everyone,<br>
> <br>
> I wrote a tiny patch to add the Ceph RDB feature of fast-diff to backups <br>
> created by cinder-backup:<br>
> <br>
>   * <a href="https://review.opendev.org/c/openstack/cinder/+/766856/" rel="noreferrer" target="_blank">https://review.opendev.org/c/openstack/cinder/+/766856/</a><br>
> <br>
> <br>
> Could someone please take a peek and let me know of this is sufficient <br>
> to be merged?<br>
<br>
Thanks for raising awareness of your patch.  Right now, the cinder team <br>
is prioritizing new driver reviews in light of the impending merge <br>
deadline at wallaby milestone-2 next week:<br>
<br>
<a href="http://lists.openstack.org/pipermail/openstack-discuss/2021-January/019720.html" rel="noreferrer" target="_blank">http://lists.openstack.org/pipermail/openstack-discuss/2021-January/019720.html</a><br>
<br>
So there may be a slight delay to your patch being reviewed.  If you <br>
have some time, you can always help things along by reviewing some small <br>
patches by other authors.<br>
<br>
cheers,<br>
brian<br>
<br>
> <br>
> <br>
> <br>
> Thanks and with kind regards,<br>
> <br>
> <br>
> Christian<br>
> <br>
> <br>
> <br>
<br>
<br>
</blockquote></div>