Bug 26392 - Computerraum: Raumeinstellungen (Shares, Filter, Drucker)
Summary: Computerraum: Raumeinstellungen (Shares, Filter, Drucker)
Status: CLOSED FIXED
Alias: None
Product: UCS@school
Classification: Unclassified
Component: UMC - Computer room administration
Version: UCS@school 3.0
Hardware: Other Linux
: P5 normal
Target Milestone: UCS@school 3.0 MS2
Assignee: Andreas Büsching
QA Contact: Florian Best
URL:
Keywords:
Depends on: 27252
Blocks:
  Show dependency treegraph
 
Reported: 2012-03-07 12:59 CET by Alexander Kläser
Modified: 2012-06-11 06:29 CEST (History)
4 users (show)

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Customer ID:
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Kläser univentionstaff 2012-03-07 12:59:15 CET
In diesem Bug wird die generelle Umsetzung der Raumeinstellungen bearbeitet, also Shares, Web-Filter und Druckmodus. Das Erstellen von Ad-hoc-Whitelists wird über Bug 26351 geregelt.
Comment 1 Alexander Kläser univentionstaff 2012-03-08 12:33:43 CET
Ziel ist es, das Zurücksetzen der Raumeinstellungen über AT-Jobs zu implementieren, da dies transparenter wäre und somit das alte Scheduling ersetzen würde.
Comment 2 Andreas Büsching univentionstaff 2012-05-07 15:07:49 CEST
Das ist jetzt umgesetzt und ChangeLog-Eintrag wurde hinzugefügt
Comment 3 Florian Best univentionstaff 2012-05-21 15:33:55 CEST
Shares funktionieren nicht, das liegt wohl an einem typo im Namen der UCR-Variable. Andreas weiß bescheid.
Comment 4 Andreas Büsching univentionstaff 2012-05-21 15:38:42 CEST
(In reply to comment #3)
> Shares funktionieren nicht, das liegt wohl an einem typo im Namen der
> UCR-Variable. Andreas weiß bescheid.

Der Typo ist behoben und gleichzeitig wurde noch ein Logikfehler behoben für den ShareMode none

Neues Paket baut
Comment 5 Florian Best univentionstaff 2012-05-21 16:00:52 CEST
Wieder auf, weil nur die Home-Freigabe gesperrt wird: Marktplatz, etc. bleibt ungesperrt
Comment 6 Andreas Büsching univentionstaff 2012-05-22 16:25:42 CEST
(In reply to comment #5)
> Wieder auf, weil nur die Home-Freigabe gesperrt wird: Marktplatz, etc. bleibt
> ungesperrt

Das ist jetzt korrigiert. Auch der marktplatz wird gesperrt.
Comment 7 Florian Best univentionstaff 2012-05-23 10:57:16 CEST
Die Ausf�hrung des Kommandos computerroom/settings/set ist fehlgeschlagen: Traceback (most recent call last): File '/usr/lib/pymodules/python2.6/univention/management/console/modules/__init__.py', line 102, in execute func( request ) File '/usr/lib/pymodules/python2.6/univention/management/console/modules/computerroom/__init__.py', line 587, in settings_set MODULE.info( 'SMB process: %s' % str( process ) ) File '/usr/lib/pymodules/python2.6/ucsschool/lib/smbstatus.py', line 85, in __str__ title = 'Process %(pid)s: User: %(username)s' % self KeyError: 'username'
Comment 8 Andreas Büsching univentionstaff 2012-05-23 11:26:21 CEST
(In reply to comment #7)
> Die Ausf�hrung des Kommandos computerroom/settings/set ist fehlgeschlagen:
> Traceback (most recent call last): File
> '/usr/lib/pymodules/python2.6/univention/management/console/modules/__init__.py',
> line 102, in execute func( request ) File
> '/usr/lib/pymodules/python2.6/univention/management/console/modules/computerroom/__init__.py',
> line 587, in settings_set MODULE.info( 'SMB process: %s' % str( process ) )
> File '/usr/lib/pymodules/python2.6/ucsschool/lib/smbstatus.py', line 85, in
> __str__ title = 'Process %(pid)s: User: %(username)s' % self KeyError:
> 'username'

Das lag an einer besonderen Situation/Ausgabe. Ich habe den Parser dahingehend jetzt verbessert.
Comment 9 Andreas Büsching univentionstaff 2012-05-24 13:34:26 CEST
(In reply to comment #8)
> (In reply to comment #7)
> > Die Ausf�hrung des Kommandos computerroom/settings/set ist fehlgeschlagen:
> > Traceback (most recent call last): File
> 
> Das lag an einer besonderen Situation/Ausgabe. Ich habe den Parser dahingehend
> jetzt verbessert.

Das ist korrigiert. Durch die Anpassung der atjobs-Lib funktioniert jetzt auch wieder das Ändern der Einstellungen
Comment 10 Florian Best univentionstaff 2012-05-24 15:10:40 CEST
Die Ausf�hrung des Kommandos computerroom/settings/set ist fehlgeschlagen:

Traceback (most recent call last):
  File '/usr/lib/pymodules/python2.6/univention/management/console/modules/__init__.py', line 102, in execute
    func( request )
  File '/usr/lib/pymodules/python2.6/univention/management/console/modules/computerroom/__init__.py', line 626, in settings_set
    MODULE.info( 'SMB process: %s' % str( process ) )
  File '/usr/lib/pymodules/python2.6/ucsschool/lib/smbstatus.py', line 85, in __str__
    title = 'Process %(pid)s: User: %(username)s (group: %(group)s)' % self
KeyError: 'username'
Comment 11 Andreas Büsching univentionstaff 2012-05-24 15:16:45 CEST
(In reply to comment #10)
> Die Ausf�hrung des Kommandos computerroom/settings/set ist

Um das zu finden benötige ich den Output von smbstatus, damit ich sehen kann wo der Parser aussteigt.

Alternative kann ich nur die Exception abfangen. Dadurch würde ich dann aber einige Verbindungen zu Freigaben nicht sauber trennen.
Comment 12 Andreas Büsching univentionstaff 2012-05-24 16:23:42 CEST
(In reply to comment #11)
> (In reply to comment #10)
> > Die Ausf�hrung des Kommandos computerroom/settings/set ist
> 
> Um das zu finden benötige ich den Output von smbstatus, damit ich sehen kann wo
> der Parser aussteigt.
> 
> Alternative kann ich nur die Exception abfangen. Dadurch würde ich dann aber
> einige Verbindungen zu Freigaben nicht sauber trennen.

Da ich das nicht reproduzieren kann hab ich diese Variante implementiert.

Sobald das auftritt wird ein Vermerk in das Logfile geschrieben. Bei Level 4 wird der Output von smbstatus noch hinzugefügt
Comment 13 Andreas Büsching univentionstaff 2012-05-25 08:38:46 CEST
(In reply to comment #12)
> Sobald das auftritt wird ein Vermerk in das Logfile geschrieben. Bei Level 4
> wird der Output von smbstatus noch hinzugefügt

Wenn das Probelm auftritt, dann kann man den Parser auch direkt aufrufen

python /usr/share/pyshared/ucsschool/lib/smbstatus.py

Anschließend gibt es eine Logdatei /var/log/univention/smbstatus.log

Diese dann bitte an den Bug hängen.
Comment 14 Florian Best univentionstaff 2012-05-25 10:10:32 CEST
Ich kann den Traceback nicht reproduzieren dafür aber einige Tests mit gefälschtem smbstatus output gemacht.
Also VERIFIED.
Comment 15 Felix Botner univentionstaff 2012-05-25 17:06:26 CEST
Wenn ich für einen Raum DruckModus = Drucken verboten erhalte ich folgendes:

-> ucr search --brief printmode
samba/printmode/hosts/none: 10.200.7.65
samba/printmode/room/room1: none

-> ucr search --brief hosts/de

-> more /etc/samba/local.config.d/printer.*
::::::::::::::
/etc/samba/local.config.d/printer.new.local.config.conf
::::::::::::::
[None]
::::::::::::::
/etc/samba/local.config.d/printer.PDFDrucker.local.config.conf
::::::::::::::
[None]


Das sieht nicht richtig aus.
Comment 16 Andreas Büsching univentionstaff 2012-05-25 18:44:56 CEST
(In reply to comment #15)
> Wenn ich für einen Raum DruckModus = Drucken verboten erhalte ich folgendes:
> 
> -> ucr search --brief printmode
> samba/printmode/hosts/none: 10.200.7.65
> samba/printmode/room/room1: none

Das ist soweit richtig, wenn es nur einen Rechner gab an dem ein Schüler sass

> ::::::::::::::
> /etc/samba/local.config.d/printer.new.local.config.conf
> ::::::::::::::
> [None]
> ::::::::::::::
> /etc/samba/local.config.d/printer.PDFDrucker.local.config.conf
> ::::::::::::::
> [None]

Hier gab es ein Problem mit Druckern, die keinen extra Samba-Namen hatten.

Das hat jetzt bei meinen Tests funktioniert. Zusätzlichen werden auch nicht genutzte Dateien (stehen nicht im include) gelöscht.
Comment 17 Florian Best univentionstaff 2012-05-29 09:51:17 CEST
(In reply to comment #16)
> Hier gab es ein Problem mit Druckern, die keinen extra Samba-Namen hatten.
Ja, Druckfreigabe/sperrung geht auch für Drucker ohne Samba-Name.
Comment 18 Stefan Gohmann univentionstaff 2012-06-11 06:29:26 CEST
UCS@school 3.0 wurde veröffentlicht. Sollte der hier beschriebene Bug mit einer  neueren Version von UCS@school erneut auftreten, so sollte dieser Bug dupliziert werden: "Clone This Bug"