Project

General

Profile

Anomalie #2614

au montage (USB par exemple), chown -R du point de montage

Added by Fabrice Barconnière almost 10 years ago. Updated almost 10 years ago.

Status:
Fermé
Priority:
Haut
Assigned To:
Category:
-
Start date:
12/16/2011
Due date:
% Done:

100%

Spent time:
Distribution:
EOLE 2.3

Description

Test sur une clé USB contenant déjà des répertoires et fichiers.
Le chown se fait sur toute la clé.
Peut-on prévoir un répertoire dédié à la sauvegarde sur le support et ainsi faire le chown uniquement sur ce répertoire ?

pas_chown.diff View (1.26 KB) Emmanuel GARETTE, 12/20/2011 11:21 AM


Related issues

Related to python-pyeole - Anomalie #2632: bacula : erreur au test en cas de support manuel Fermé 12/20/2011
Related to python-pyeole - Anomalie #2447: Montage disque dur externe bacula Fermé 11/22/2011
Related to eole-bacula - Anomalie #2407: Erreur provoqué par chown sur le montage /mnt/sauvegardes (non démonté) Fermé 11/16/2011
Related to eole-bacula - Anomalie #4085: Scribe 2.3 problème de montage bacula Fermé 11/08/2013

Associated revisions

Revision 0662cd3d (diff)
Added by Daniel Dehennin almost 10 years ago

Ne pas changer de propiétaire au montage d’un disque pour bacula.

Le montage d’un périphérique de sauvegarde entraîne un 'chown -R' sur
tout le périphérique.
Nous partirons du principe que la mise en place de la sauvegarde doit
être faite lors du montage et que l’utilisateur de bacula-sd a le droit
d’écrire sur le disque.

Cela permettra :

  • De mettre en place des tests afin de détecter tout changement au lieu
    de les passer sous silence ;
  • De ne pas modifier les données autres que celles de bacula, par exemple
    lors de l’utilisation d’un périphérique temporaire.
  • pyeole/bacula.py (mount_bacula_support): Mise en commentaire des lignes
    concernant le 'chown'.

Fixes: #2614 @10m
Thanks: Emmanuel GARETTE pour le patch.

History

#1 Updated by Emmanuel GARETTE almost 10 years ago

Dans la configuration de bacula, on définit le point de montage"." Pas le répertoire où la sauvegarde est effectuée.

Par contre, il faudrait peut etre améliorer le chown en changeant les droits que pour les fichiers nécessaires (et seulement si c'est nécessaire).

#2 Updated by Daniel Dehennin almost 10 years ago

Ne peut-on pas considérer que le chown est un traitement d’exception et non quelque chose de normal ?

Si on a besoin de faire un chown, c’est lors de l’initialisation du périphérique de sauvegarde.

Toute modification du propriétaire des fichiers de la sauvegarde entraînant une erreur de sauvegarde doit être remonté et non passé sous silence par un chown en tant que root.

De toute façon la sauvegarde se faisant en tant que root, a-t-on vraiment besoin du changement de propriétaire ?

#3 Updated by Emmanuel GARETTE almost 10 years ago

Ne peut-on pas considérer que le chown est un traitement d’exception et non quelque chose de normal ?

Surtout que durant le montage, il y a un test. On pourrait considéré que c'est superflux (si les droits ne sont pas bon -> erreur).

De toute façon la sauvegarde se faisant en tant que root, a-t-on vraiment besoin du changement de propriétaire ?

Non, la sauvegarde est faite en tant que bacula en 2.3 (on ne modifie plus les sources justement).

#4 Updated by Daniel Dehennin almost 10 years ago

Je ne vois pas où est fait le chown dans eole-bacula.

Le seul chown que je vois est eole-bacula:source:eole/posttemplate/00-bacula#L83 mais le point de montage est démonté à ce moment là.

Il est par ailleurs amusant de faire un mkdir après avoir testé si quelque chose était monté.
De plus il y a un chemin codé en dur.

#5 Updated by Emmanuel GARETTE almost 10 years ago

  • Project changed from eole-bacula to python-pyeole

Voilà.

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

  • Status changed from Nouveau to Accepté
  • Priority changed from Normal to Haut
  • Target version changed from Mises à jour 2.3.4 RC to Mises à jour 2.3 - 03 RC

#7 Updated by Emmanuel GARETTE almost 10 years ago

Voici un patch pour désactiver le chown.

Je ne sais pas si cela fonctionne (notamment et principalement en NTFS).

A tester donc avec un disque formatté en NTFS et en ext3 (même si cela ne devrait pas poser de problème en ext3).

Le chown n'est pas fait en vfat.

#8 Updated by Emmanuel GARETTE almost 10 years ago

A priori il est possible de mettre l'UID pour les partitions formatées en NTFS (mais pas celles en ext3).

Faudrait voir ce que donne "/usr/bin/lshw -short -quiet -class volume" quand le disque est formaté en NTFS.

Et modifier en conséquence la ligne "if ' '.join(line[-3:]) == 'Windows FAT volume':" de bacula.py.

Il serait également bien d'ajouter, à la ligne de commande cette option : "umask=0077".

USB_MOUNT = '/bin/mount {0} {1} -o noexec,nosuid,nodev,uid={2},umask=0077'

#9 Updated by Daniel Dehennin almost 10 years ago

  • Status changed from Accepté to Résolu
  • % Done changed from 0 to 100

#10 Updated by Joël Cuissinat almost 10 years ago

  • Status changed from Résolu to Fermé

Ok

Bien sûr il faut faire un chown bacula sur le point de montage monté !

#11 Updated by Emmanuel GARETTE almost 10 years ago

Ok pour moi, le montage fonctionne avec une partition en :

  • VFAT
  • NTFS
  • ext4

Also available in: Atom PDF