Bug 25923 – Traceback bei Entfernen von (Samba4-)DNS-Alias

Bug 25923 - Traceback bei Entfernen von (Samba4-)DNS-Alias


Summary:	Traceback bei Entfernen von (Samba4-)DNS-Alias

Status:	CLOSED FIXED

Product:	UCS
Classification:	Unclassified
Component:	UMC - Computers
Version:	UCS 3.0
Hardware:	Other Linux

Importance:	P2 normal (vote)
Target Milestone:	UCS 3.0-2
Assigned To:	Lukas Walter
QA Contact:	Felix Botner

URL:
Keywords:	interim-1

Duplicates:	26483 (view as bug list)
Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2012-01-25 11:52 CET by Ingo Steuwer
Modified:	2012-07-20 15:24 CEST (History)
CC List:	5 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):
Max CVSS v3 score:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Ingo Steuwer

2012-01-25 11:52:44 CET

Am UCS Master gleichzeitig den von Samba4 gesetzten DNS-Alias und den Forward Zone Eintrag entfernt, Traceback:

Interner Modul-Fehler: Ein Fehler trat während des Ausführens des Befehls auf.

Fehlernachricht des Servers:

File '/usr/lib/pymodules/python2.6/notifier/threads.py', line 81, in _run
tmp = self._function()
File '/usr/lib/pymodules/python2.6/notifier/__init__.py', line 104, in __call__
return self._function( *tmp, **self._kwargs )
File '/usr/lib/pymodules/python2.6/univention/management/console/modules/udm/__init__.py', line 305, in _thread
module.modify( properties )
File '/usr/lib/pymodules/python2.6/univention/management/console/modules/udm/udm_ldap.py', line 130, in wrapper_func
ret = func( *args, **kwargs )
File '/usr/lib/pymodules/python2.6/univention/management/console/modules/udm/udm_ldap.py', line 372, in modify
obj.modify()
File '/usr/lib/pymodules/python2.6/univention/admin/handlers/__init__.py', line 344, in modify
return self._modify(modify_childs,ignore_license=ignore_license)
File '/usr/lib/pymodules/python2.6/univention/admin/handlers/__init__.py', line 743, in _modify
ml=self._ldap_modlist()
File '/usr/lib/pymodules/python2.6/univention/admin/handlers/computers/domaincontroller_master.py', line 583, in _ldap_modlist
ml=univention.admin.handlers.simpleComputer._ldap_modlist( self )
File '/usr/lib/pymodules/python2.6/univention/admin/handlers/__init__.py', line 2083, in _ldap_modlist
if not entry in self.info[ 'dnsEntryZoneAlias' ]:

KeyError: 'dnsEntryZoneAlias'

Comment 1 Ingo Steuwer

2012-01-25 11:54:05 CET

Es reicht aus den DNS-Alias zu entfernen (hier: de03a12f-eb2c-40a6-87c8-7769baed5f9c._msdcs)

Comment 2 Andreas Büsching

2012-01-25 20:55:47 CET

Das kommt durch eine API Änderung zu UCS 3.0

Attribute, die gelöscht werden, sind in dem Dictionary des Objektes gar nicht mehr enthalten. Wir sollten das für alle DNS-Attribute an diesem Bug prüfen.

Comment 3 Andreas Büsching

2012-03-14 13:56:30 CET

*** Bug 26483 has been marked as a duplicate of this bug. ***

Comment 4 Tobias Scherer

2012-03-28 11:01:19 CEST

Auch aufgetreten an Ticket#2012032721001551

Comment 5 Lukas Walter

2012-04-20 11:57:58 CEST

Das selbe Problem konnte gegebenenfalls ebenso beim Entfernen von dnsEntryZoneForward Einträgen auftreten da auch hier eine Überprüfung auf Vorhandensein des entsprechenden keys in self.info fehlte.

Jetzt gibt es ein einheitliches Vorgehen sowohl für dnsEntyZoneAlias als auch dnsEntryZoneFoward und dnsEntryZoneReverse das dieses Problem behebt und dabei aufveraltete Methoden wie "has_key" verzichtet.



univention-directory-manager-modules (7.0.243-1) unstable; urgency=low                         
  * Fixed keyerror when removing dns attributes from host objects (Bug #25923)

Comment 6 Alexander Kläser

2012-04-20 18:19:38 CEST

Folgender Codeblock ist wahrscheinlich nicht gewollt (zwei Mal for):

> +               if self.hasChanged('dnsEntryZoneReverse'):
> +                       for entry in self.oldinfo.get('dnsEntryZoneReverse', []):
> +                               for entry in self.oldinfo['dnsEntryZoneReverse']:
> +                                       if not entry in self.info.get('dnsEntryZoneReverse', []):

Comment 7 Lukas Walter

2012-04-25 15:54:35 CEST

Ja, das war ein Fehler. Ich habe die Zeile rausgenommen und das Paket neugebaut.

Comment 8 Felix Botner

2012-06-14 13:24:39 CEST

OK - Forward Zone, Reverse Zone und DNS-Alias können in diversen Kombinationen angelegt und gelöscht werden.

OK - Changelog Eintrag

Comment 9 Stefan Gohmann

2012-07-20 15:24:05 CEST

UCS 3.0-2 has been released: 
  http://forum.univention.de/viewtopic.php?f=54&t=1905

If this error occurs again, please use "Clone This Bug".