<div dir="ltr"><div>Hi All,<br></div>    With respect to the comment made by xian-yang @ <br><div><a href="https://review.openstack.org/#/c/102496/1/manila/share/drivers/glusterfs.py">https://review.openstack.org/#/c/102496/1/manila/share/drivers/glusterfs.py</a><br>
<br>for _update_share_status and the docstring that the method has, which is<br><span class="">"""Retrieve status info from share volume group."""<br><br></span></div><div><span class="">I have few questions based on the above...<br>
<br></span></div><div><span class="">1) "share volume group" in docstring is incorrect, since its a glusterfs driver. But I think i know why it says volume group, probably because it came from lvm.py to begin with. I see that all other drivers also say volume group, tho' it may not be the right thing to say for their respective case.<br>
<br>Do we want to ensure that the docstrings are put in a way thats meaningful to the driver ?<br><br></span></div><div><span class="">2) _update_share_status method - I see the same issue here.. it says the same in all other drivers.. but as xian pointed, it should be rightfully called _update_share_stats. So should we wait for all driver to follow suit or start changing in the driver specific code as and when we touch that part of code ?<br>
<br></span></div><div><span class="">thanx,<br>deepak<br></span></div></div>