<div dir="ltr">Much appreciated. I'll integrated your fix into the build I have. <br><br>I looked through Bugzilla and didn't see anything and decided it would be more straight forward to ask if there was an existing method to solve the problem before suggesting it was a bug.<div><br></div><div>-Mike Schmidt</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Apr 7, 2015 at 3:54 PM, Tony Asleson <span dir="ltr"><<a href="mailto:tasleson@redhat.com" target="_blank">tasleson@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 04/06/2015 12:08 PM, Michael Schmidt wrote:<br>
> Hi all,<br>
><br>
> While working with the Python bindings and attempting to add tags to<br>
> logical volume lvm_vg_write seems not to be called. This means if add_tag<br>
> is called and the virtual group handle is closed the tags disappear.<br>
<br>
</span>Thanks for reporting this!<br>
<span class=""><br>
> The<br>
> action in _liblvm_lvm_vg_add_tag solves this by calling lvm_vg_write on the<br>
> handle itself. Since the lv handle does not seem to have access to it's vg<br>
> handle I added a method to the vg just to trigger a write:<br>
<br>
</span>The python bindings were written with the expectation that the state<br>
changes would be saved automatically.  So adding a call for the user to<br>
issue a write on the vg goes against the overall design.<br>
<br>
...<br>
<span class=""><br>
> My question is: is there a better way to accomplish this?<br>
<br>
</span>In this case we should issue the lvm_vg_write in the add/remove tag<br>
calls for the lv.  The lv structure has a pointer to the parent vg in<br>
the python binding.<br>
<br>
I submitted a patch which does this and improves the unit test to test<br>
for this.<br>
<br>
Any bugzilla submitted for this?<br>
<br>
Regards,<br>
Tony<br>
<br>
_______________________________________________<br>
linux-lvm mailing list<br>
<a href="mailto:linux-lvm@redhat.com">linux-lvm@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/linux-lvm" target="_blank">https://www.redhat.com/mailman/listinfo/linux-lvm</a><br>
read the LVM HOW-TO at <a href="http://tldp.org/HOWTO/LVM-HOWTO/" target="_blank">http://tldp.org/HOWTO/LVM-HOWTO/</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">Michael T Schmidt | <a href="http://schmidmt.com" target="_blank">schmidmt.com</a> | <a href="http://duality.io" target="_blank">duality.io</a></div></div>
</div>