Univention Bugzilla – Bug 29399
samba/shares.conf sollte nicht erst im postrun aktualisiert werden
Last modified: 2013-11-19 06:44:10 CET
Bei ucs-test samba fiel auf, dass man kurze Zeit nach Löschen eines shares in UDM Fehler beim Aufruf von samba client tools Fehlermeldungen erhält, die darauf Hinweisen, dass die smb.conf nicht korrekt ausgewertet werden kann. ein strace zeigte: stat("/etc/samba/shares.conf.d/E5upV01D", 0x7fff0e0ec3c0) = -1 ENOENT (No such file or directory) Die Ursache scheint zu sein, dass das samba-shares Listenermodul im Handler direkt share-Definitionsdateien löscht/anlegt, aber die shares.conf erst im postrun aktualisiert, direkt vor dem samba reload. Es wäre konsistenter, die shares.conf direkt zu aktualisieren, falls es notwendig ist.
Created attachment 4838 [details] Vorschlag für einen Patch
Die shares.conf wird nun direkt im Handler aktualisiert. Samba-tool wirft nun nicht mehr den Fehler.
Wie besprochen, jetzt tritt der folgender Fehler in der listener.log auf: Traceback (most recent call last): File "/usr/lib/univention-directory-listener/system/samba-shares.py", line 235, in handler print >>fp, 'include = %s' % os.path.join('/etc/samba/shares.conf.d', f) ValueError: I/O operation on closed file
(In reply to comment #3) > Wie besprochen, jetzt tritt der folgender Fehler in der listener.log auf: > > Traceback (most recent call last): > File "/usr/lib/univention-directory-listener/system/samba-shares.py", line > 235, in handler > print >>fp, 'include = %s' % os.path.join('/etc/samba/shares.conf.d', f) > ValueError: I/O operation on closed file Der Fehler tritt nun nicht mehr auf.
We will not ship a UCS 3.1-2 release; the next UCS release will be UCS 3.2. As such, this bug is moved to the new target milestone.
Maybe we should update shares.conf in an atomic operation, see patch proposal for Bug 31800.
Now, shares.conf is updated in an atomic operation
Ok.
UCS 3.2 has been released: http://docs.univention.de/release-notes-3.2-en.html http://docs.univention.de/release-notes-3.2-de.html If this error occurs again, please use "Clone This Bug".