Distribution EOLE

J'ai déjà lu ça quelque part... https://pcll.ac-dijon.fr/listes/arc/scribe/2023-03/msg00005.html

root@scribe:~# dpkg -S /usr/share/eole/sbin/run_samba_tool_dbcheck 
eole-ad-dc-scribe: /usr/share/eole/sbin/run_samba_tool_dbcheck

root@scribe:/usr/share/eole/diagnose# cat 151-AD-scribe 
#!/bin/bash

[ -x /var/lib/lxc/addc/rootfs/usr/share/eole/diagnose/151-AD ] && /var/lib/lxc/addc/rootfs/usr/share/eole/diagnose/151-AD
exit 0

Fonctionnalité ajoutée sur EOLE ≥ 2.8.1

à défaut de trouver la source du problème, il faudrait que le test diagnose supporte le "mauvais" format...

La lecture du contenu du rapport de samba-tool dbcheck est assurée par

read -a db_status <<< $(cat /var/lib/eole/reports/samba-dbcheck.log)
db_errors=${db_status[0]}

Le doublon est présent à cause de

EchoRouge "${db_errors}" 
EchoRouge "${db_status[@]}" 

db_status est un tableau qui ne contient que la partie du retour de samba-tool dbcheck qui est entre parenthèses, par exemple "0 errors". Il faudrait un exemple avec au moins une erreur dans la base pour sur la pertinence d’afficher ce contenu.

Pour tester, il faudrait peut-être regarder comment samba fait ces tests dbcheck ?

Sur un Seth avec import :
dbcheck.ldif

dn: CN=c42e15,CN=Users,DC=domseth,DC=ac-test,DC=fr
changetype: modify
add: otherHomePhone
otherHomePhone: 1
otherHomePhone: 2
otherHomePhone: 1
otherHomePhone: 3
otherHomePhone: 2

ldbmodify -H /var/lib/samba/private/sam.ldb.d/DC%3DDOMSETH%2CDC%3DAC-TEST%2CDC%3DFR.ldb dbcheck.ldif

# samba-tool dbcheck
Checking 321 objects
ERROR: Duplicate values for attribute 'otherHomePhone' in 'CN=c42e15,CN=Users,DC=domseth,DC=ac-test,DC=fr'
Values contain a duplicate: [1,2,1,3,2]/[2,1,3]!
Not fixing attribute 'otherHomePhone'
On object CN=c42e15,CN=Users,DC=domseth,DC=ac-test,DC=fr
ERROR: Attribute otherhomephone not present in replication metadata
Not fixing missing replPropertyMetaData element 'otherhomephone'
Please use --fix to fix these errors
Checked 321 objects (2 errors)

Une première lecture du code confirme également que la seule information entre parenthèses devrait être le nombre d’erreurs.
Il ne semble pas nécessaire de répéter cette information. Peut-être afficher un conseil sur l’exécution de samba-tool dbcheck pour avoir plus de détails ?

Au passage, on peut voir que la commande awk utilisée pour extraire les informations pour le rapport doit intercepter la phrase "Please use --fix to fix these errors". On peut filtrer sur "errors)" plutôt que sur "errors" ou encore se fier au fait que le nombre d’erreurs est toujours sur la dernière ligne.

Actuellement, le script run_samba_tool_dbcheck filtre les résultats écrits dans le fichier de rapport.
Ce rapport, de ce fait, ne contient pas de détails sur les erreurs identifiées par "samba-tool dbcheck".

L’idée proposée précédemment de conseiller à l’administrateur de relancer "samba-tool dbcheck" pour avoir des détails sur les erreurs ne semble pas judicieuse si cette commande peut mettre du temps.

Il semble finalement plus pertinent d’inclure dans le rapport /var/lib/eole/reports/samba-dbcheck.log la totalité du retour de la commande "samba-tool dbcheck" et de déléguer le filtrage aux scripts qui le lisent (diagnose et agent zéphir).

Pour éviter les effets de bords d’un changement conséquent du contenu du fichier de rapport, le résultat de la commande peut être envoyé dans un autre fichier (/var/lib/eole/reports/samba-dbcheck.full_report par exemple) et le contenu de ce dernier filtré pour construire le fichier /var/lib/eole/reports/samba-dbcheck.log. Les scripts peuvent continuer à le lire et renvoyer l’administrateur à la lecture de l’autre fichier pour plus de détails).

eole-ad-dc$ git branch -r --contains  6d91ac7c  
  origin/2.8.1/master
  origin/HEAD -> origin/master
  origin/dist/eole/2.8.1/master
  origin/dist/eole/2.9.0/master
  origin/master