Distribution EOLE » Modules » Amon » conf-amon

Dans le même ordre d'idée; je trouve étrange les commandes suivantes

# Vidage des chaines MANGLE
check_T1=$(iptables-save |grep "RESTOREMARK" |wc -l)
check_T2=$(iptables-save |grep "T2" | wc -l)
check_RESTOREMARK=$(iptables-save |grep "T1" |wc -l)
check_PREROUTING=$(iptables-save |grep "T1" |wc -l)

J'aurais plutôt vu,

check_T1=$(iptables-save |grep "T1" |wc -l)
check_T2=$(iptables-save |grep "T2" | wc -l)
check_RESTOREMARK=$(iptables-save |grep "RESTOREMARK" |wc -l)
check_PREROUTING=$(iptables-save |grep "PREROUTING" |wc -l)

mais je me trompe peut être :)

bien vu :) je devais être amoureux ce jour là ^^

ok pour correction remplacer une des lignes

check_PREROUTING=$(iptables-save |grep "T1" |wc -l)

par

check_PREROUTING=$(iptables-save |grep "PREROUTING" |wc -l)

Merci pour la prise en compte de ce signalement, cependant voici d'une manière plus précise l'objet de cette demande

http://dev-eole.ac-dijon.fr/projects/conf-amon/repository/revisions/2.3/entry/bin/agregation.sh

Ligne 240: il y a

if [ -z "$chaine_T1" ] ; then

C'est la même ligne que 230, je pense plutôt que c'est if [ -z "$chaine_T2" ] ; then

Ligne 211: il y a

check_T1=$(iptables-save |grep "RESTOREMARK" |wc -l)

et ligne 213: il y a

check_RESTOREMARK=$(iptables-save |grep "T1" |wc -l)

Vu que check_T1 est utilisé plus bas pour vider les regles de la table T1 et
Vu que check_RESTOREMARK est utilisé plus bas pour vider les regles RESTOREMARK

J'aurais pour plus de cohérence modifié comme suit

ligne 211 check_T1=$(iptables-save |grep "T1" |wc -l)
ligne 213 check_RESTOREMARK=$(iptables-save |grep "RESTOREMARK" |wc -l)

j'aurais aussi une modification a apporter dans le /etc/init.d/agregation, je ferais un autre signalement :)

oui effectivement j'avais pas compris et encore moins ouvert grand mes yeux. je prépare un patch pour toutes ces modifs.

Paquet candidat 2.3 : conf-amon >= 2.3-eole103~1
Paquet unstable 2.4 : conf-amon >= TODO