Project

General

Profile

Tâche #30348

Scénario #25298: Ne pas enregistrer des valeurs de configuration qui lèvent des erreurs DHCP-02 - réservation d’IP dans l'EAD 2 (2.7.0-b1)

Valider les règles de contrôle d’une réservation d’IP

Added by Benjamin Bohard almost 4 years ago. Updated almost 4 years ago.

Status:
Fermé
Priority:
Normal
Assigned To:
Start date:
06/18/2020
Due date:
% Done:

100%

Remaining (hours):
0.0

Description

Les règles de validation du formulaire Gestion du DHCP de l’EAD 2 sont sommaires et moindres que celles du script gen_dhcp.

Pour pallier la différence, le script gen_dhcp édite le fichier /var/lib/eole/config/dhcp.conf après sa propre étape de validation.
Pour déterminer si quels contrôles effectués par le script gen_dhcp doivent impliquer le nettoyage du fichier /var/lib/eole/config/dhcp.conf, il existe une liste d’erreurs. Dans cette liste, l’erreur de l’IP hors subnet a été commentée, ce qui a pour conséquence que la réservation incriminée n’est pas supprimée du fichier /var/lib/eole/config/dhcp.conf.

Associated revisions

Revision e3a18e86 (diff)
Added by Benjamin Bohard almost 4 years ago

Rendre bloquante l’erreur d’une IP hors sous-réseau.

Ref #30348

History

#1 Updated by Benjamin Bohard almost 4 years ago

Est-ce que le motif pour commenter cette règle de validation est toujours valable (code isodysfonctionnel avec le fonctionnement historique du formulaire pour ne pas casser tous les usages) ?

        elif IP(ip) not in IPSet(list(conf['dynamic'].keys())):  # IP defined outside declared subnets
            print("Aucun subnet n’est déclaré pour cette adresse IP : {0} ({1})".format(ip, name))
            errors.append('IP hors sous-réseau déclaré')

Sinon, la solution la plus simple consiste à la rétablir en décommentant la ligne.

CRITICAL_ALLOC_ERRORS = set([
    'incohérence couples MAC, nom',
    'format IP invalide',
    # 'IP hors sous-réseau déclaré',
])

Si il doit être possible de déclarer une IP hors d’un subnet déclaré, il faut soit modifier l’affichage pour qu’il apparaisse bien comme un avertissement, soit ne plus tester ce critère et éventuellement le remplacer par un critère plus adapté. Le problème posé est celui de l’introduction d’une contrainte de validation plus forte qui pourrait supprimer des déclarations de réservations saisies précédemment.

#2 Updated by Joël Cuissinat almost 4 years ago

Nous pouvons assumer qu'à partir d'EOLE 2.7.1, les réservations d'IP hors plage sont interdites.

#3 Updated by Benjamin Bohard almost 4 years ago

  • Status changed from Nouveau to En cours

#4 Updated by Benjamin Bohard almost 4 years ago

  • Status changed from En cours to Résolu

#5 Updated by Joël Cuissinat almost 4 years ago

  • Assigned To set to Benjamin Bohard
/git/eole-dhcp (master=)$ git branch -r --contains e3a18e86 
  origin/2.7.1/master
  origin/2.7.2/master
  origin/HEAD -> origin/master
  origin/dist/eole/2.7.1/master
  origin/dist/eole/2.7.2/master
  origin/dist/eole/2.8.0/master
  origin/master

#6 Updated by Joël Cuissinat almost 4 years ago

  • % Done changed from 0 to 100

Validé par le test squash.

#7 Updated by Joël Cuissinat almost 4 years ago

  • Status changed from Résolu to Fermé
  • Remaining (hours) set to 0.0

Also available in: Atom PDF