<div dir="ltr"><div class="gmail-boardCommentBody" id="gmail-yui_3_10_3_1_1571143074828_2515">
    

    

    

  
  <div class="gmail-comment-text"><p>I proposed some 
patches through heat templates and puppet-cinder to remove lock files 
older than 1 week and avoid file system growing.</p>
<p>This is a solution based on a cron job, to fix that on stable 
branches, in a second time I'll help the fasteners project to fix the 
root cause by reviewing and testing the proposed patch (lock based on 
file offset). In next versions I hope we will use a patched fasteners 
and so we could drop the cron based solution.</p>
<p>Please can you give /reviews/feedbacks:<br>
- <a rel="nofollow" href="https://review.opendev.org/688413">https://review.opendev.org/688413</a><br>
- <a rel="nofollow" href="https://review.opendev.org/688414">https://review.opendev.org/688414</a><br>
- <a rel="nofollow" href="https://review.opendev.org/688415">https://review.opendev.org/688415</a></p><p>Thanks<br></p></div>
  
  
  


  </div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Le lun. 30 sept. 2019 à 03:35, Rikimaru Honjo <<a href="mailto:honjo.rikimaru@ntt-tx.co.jp">honjo.rikimaru@ntt-tx.co.jp</a>> a écrit :<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 2019/09/28 1:44, Ben Nemec wrote:<br>
> <br>
> <br>
> On 9/23/19 11:42 PM, Rikimaru Honjo wrote:<br>
>> Hi Eric,<br>
>><br>
>> On 2019/09/20 23:10, Eric Harney wrote:<br>
>>> On 9/20/19 1:52 AM, Rikimaru Honjo wrote:<br>
>>>> Hi,<br>
>>>><br>
>>>> I'm using Queens cinder with the following setting.<br>
>>>><br>
>>>> ---------------------------------<br>
>>>> [coordination]<br>
>>>> backend_url = file://$state_path<br>
>>>> ---------------------------------<br>
>>>><br>
>>>> As a result, the files like the following were remained under the state path after some operations.[1]<br>
>>>><br>
>>>> cinder-63dacb3d-bd4d-42bb-88fe-6e4180164765-delete_volume<br>
>>>> cinder-32c426af-82b4-41de-b637-7d76fed69e83-delete_snapshot<br>
>>>><br>
>>>> In my understanding, these are lock-files created for synchronization by tooz.<br>
>>>> But, these lock-files were not deleted after finishing operations.<br>
>>>> Is this behaviour correct?<br>
>>>><br>
>>>> [1]<br>
>>>> e.g. Delete volume, Delete snapshot<br>
>>><br>
>>> This is a known bug that's described here:<br>
>>><br>
>>> <a href="https://github.com/harlowja/fasteners/issues/26" rel="noreferrer" target="_blank">https://github.com/harlowja/fasteners/issues/26</a><br>
>>><br>
>>> (The fasteners library is used by tooz, which is used by Cinder for managing these lock files.)<br>
>>><br>
>>> There's an old Cinder bug for it here:<br>
>>> <a href="https://bugs.launchpad.net/cinder/+bug/1432387" rel="noreferrer" target="_blank">https://bugs.launchpad.net/cinder/+bug/1432387</a><br>
>>><br>
>>> but that's marked as "Won't Fix" because Cinder needs it to be fixed in the underlying libraries.<br>
>> Thank you for your explanation.<br>
>> I understood the state.<br>
>><br>
>> But, I have one more question.<br>
>> Can I think this bug doesn't affect synchronization?<br>
> <br>
> It does not. In fact, it's important to not remove lock files while a service is running or you can end up with synchronization issues.<br>
> <br>
> To clean up the leftover lock files, we generally recommend clearing the lock_path for each service on reboot before the services have started.<br>
<br>
Thank you for your information.<br>
I think that I understood this issue completely.<br>
<br>
Best Regards,<br>
<br>
<br>
>><br>
>> Best regards,<br>
>><br>
>>> Thanks,<br>
>>> Eric<br>
>>><br>
>><br>
> <br>
<br>
-- <br>
_/_/_/_/_/_/_/_/_/_/_/_/_/_/_/_/_/_/_/_/_/_/_/_/_/<br>
Rikimaru Honjo<br>
<a href="mailto:E-mail%3Ahonjo.rikimaru@ntt-tx.co.jp" target="_blank">E-mail:honjo.rikimaru@ntt-tx.co.jp</a><br>
<br>
<br>
</blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div>Hervé Beraud</div><div>Senior Software Engineer<br></div><div>Red Hat - Openstack Oslo</div><div>irc: hberaud</div><div>-----BEGIN PGP SIGNATURE-----<br><br>wsFcBAABCAAQBQJb4AwCCRAHwXRBNkGNegAALSkQAHrotwCiL3VMwDR0vcja10Q+<br>Kf31yCutl5bAlS7tOKpPQ9XN4oC0ZSThyNNFVrg8ail0SczHXsC4rOrsPblgGRN+<br>RQLoCm2eO1AkB0ubCYLaq0XqSaO+Uk81QxAPkyPCEGT6SRxXr2lhADK0T86kBnMP<br>F8RvGolu3EFjlqCVgeOZaR51PqwUlEhZXZuuNKrWZXg/oRiY4811GmnvzmUhgK5G<br>5+f8mUg74hfjDbR2VhjTeaLKp0PhskjOIKY3vqHXofLuaqFDD+WrAy/NgDGvN22g<br>glGfj472T3xyHnUzM8ILgAGSghfzZF5Skj2qEeci9cB6K3Hm3osj+PbvfsXE/7Kw<br>m/xtm+FjnaywZEv54uCmVIzQsRIm1qJscu20Qw6Q0UiPpDFqD7O6tWSRKdX11UTZ<br>hwVQTMh9AKQDBEh2W9nnFi9kzSSNu4OQ1dRMcYHWfd9BEkccezxHwUM4Xyov5Fe0<br>qnbfzTB1tYkjU78loMWFaLa00ftSxP/DtQ//iYVyfVNfcCwfDszXLOqlkvGmY1/Y<br>F1ON0ONekDZkGJsDoS6QdiUSn8RZ2mHArGEWMV00EV5DCIbCXRvywXV43ckx8Z+3<br>B8qUJhBqJ8RS2F+vTs3DTaXqcktgJ4UkhYC2c1gImcPRyGrK9VY0sCT+1iA+wp/O<br>v6rDpkeNksZ9fFSyoY2o<br>=ECSj<br>-----END PGP SIGNATURE-----<br><br></div></div></div></div></div></div></div></div></div></div></div></div></div>