[ldap_crans.py] On deplace le test d'adresse mac deja utilisee

Ignore-this: 658828a201b282e06a0884ceaa34d573
Ca permet d'eviter de se prendre la tete avec une mac non valide

darcs-hash:20110907100351-ddb99-46ae34deace1b89f59b2ab96479968a98c58a9b3.gz
This commit is contained in:
Michel Blockelet 2011-09-07 12:03:51 +02:00
parent f2b46b1d87
commit 2d53246bb7

View file

@ -2736,10 +2736,6 @@ class Machine(BaseClasseCrans):
mac = format_mac(mac) mac = format_mac(mac)
# La mac serait-elle déjà connue ?
if not multi_ok and self.exist('macAddress=%s' % mac):
raise ValueError(u"Mac déjà utilisée sur le réseau.", 1)
# La MAC serait-elle une MAC à la con ? # La MAC serait-elle une MAC à la con ?
if mac == "00:04:4b:80:80:03": if mac == "00:04:4b:80:80:03":
raise ValueError(u"Il s'agit de l'unique adresse MAC achetée par nVidia pour ses cartes réseau. Il faut changer cette adresse.", 2) raise ValueError(u"Il s'agit de l'unique adresse MAC achetée par nVidia pour ses cartes réseau. Il faut changer cette adresse.", 2)
@ -2755,7 +2751,7 @@ class Machine(BaseClasseCrans):
vendor = line.replace(prefix, '').replace('( )', '').strip() vendor = line.replace(prefix, '').replace('( )', '').strip()
break break
except IOError: except IOError:
# Le fichier existe pas, on sors # Le fichier existe pas, on sort
raise RuntimeError(u"Fichier de fabriquants de MAC non trouvé !") raise RuntimeError(u"Fichier de fabriquants de MAC non trouvé !")
if not multi_ok and not vendor: if not multi_ok and not vendor:
@ -2764,6 +2760,10 @@ class Machine(BaseClasseCrans):
L'adresse MAC correspond peut-être à un pont réseau, désactivez ce pont réseau. L'adresse MAC correspond peut-être à un pont réseau, désactivez ce pont réseau.
Contactez nounou si la MAC est bien celle d'une carte.""", 3) Contactez nounou si la MAC est bien celle d'une carte.""", 3)
# La mac serait-elle déjà connue ?
if not multi_ok and self.exist('macAddress=%s' % mac):
raise ValueError(u"Mac déjà utilisée sur le réseau.", 1)
# Lock de la mac # Lock de la mac
self.lock('macAddress', mac) self.lock('macAddress', mac)