Discussion :

[IdiCoda]

Bonjour, j'ai besoin de tester deux paramètres dans /sys/class/net/. Le but de ce script est pour le moment de vérifier que les interfaces sont des interfaces réseau (pas de loopback ou de bond) et qu'elles sont PHYSICAL UP.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
#!/usr/bin/env python3
 
import  os
 
path = '/sys/class/net/'
interface = os.listdir(path)
 
for f in interface:
        index = os.path.join(path, interface, 'ifindex')
        carrier = os.path.join(path, interface, 'carrier')
 
        if index == '6' and carrier == '1':
                print('Bonjour')

Le souci étant :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
Traceback (most recent call last):
  File "/root/test/./test.py", line 10, in <module>
    index = os.path.join(path, interface, 'ifindex')
  File "/usr/lib/python3.9/posixpath.py", line 90, in join
    genericpath._check_arg_types('join', a, *p)
  File "/usr/lib/python3.9/genericpath.py", line 152, in _check_arg_types
    raise TypeError(f'{funcname}() argument must be str, bytes, or '
TypeError: join() argument must be str, bytes, or os.PathLike object, not 'list'

Je le comprends et pensais pouvoir le résoudre avec la boucle, sans succès. Avez-vous une idée ?

[IdiCoda]

De plus, mon script est incomplet car je ne lis pas la valeur de ifindex ou de carrier, y a-t-il une autre façon qu'avec open() et read() ?

[wiztricks]

Salut,

Je le comprends et pensais pouvoir le résoudre avec la boucle, sans succès.

Avant de partir à écrire de "vrais" codes, on fait les exercices proposés dans le tuto qu'on a pris le temps de choisir...
Et comme on a passé du temps avec le tuto., on se souvient d'exercices similaires et des chapitres où ça parle de boucle qu'on peut aller revoir (si on constate qu'on n'a pas tout compris).

- W

[papajoker]

bonjour

et pensais pouvoir le résoudre avec la boucle, sans succès.

Il faut comprendre ce que fait une boucle Pourquoi créer une variable f mais ??? ne jamais utiliser cette variable

TypeError: join() argument must be str, bytes, or os.PathLike object, not 'list'

Python te donne bien l'erreur ("TypeError: join() ne prend PAS une liste en paramètre"), c'est malheureux de demander de l'aide pour cela ...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if index == '6' and carrier == '1':

Difficile à comprendre ta logique, tu penses que os.path.join() retourne le dernier fichier ?

[popo]

Sur ta ligne 10, interface est une liste (celle renvoyée par os.listdir).

A la rigueur, tu peux utiliser f.
Comme ceci : index = os.path.join(path, f, 'ifindex').


os.path.join va simplement ajouter un séparateur de répertoire entre les différents éléments que tu passes.
Tu ne peux pas avoir '6' dedans, tu auras un chemin.

[wiztricks]

De plus, mon script est incomplet car je ne lis pas la valeur de ifindex ou de carrier, y a-t-il une autre façon qu'avec open() et read() ?

Et non....
Si vous en êtes à vous mélanger les pinceaux entre nom de fichier et son contenu, vous avez du temps à passer dans les tutos (il y a souvent/toujours un chapitre sur les fichiers...)avant de programmer.

- W

[BufferBob]

(...)

salut,

un truc comme ça devrait fonctionner :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
import os
 
path = '/sys/class/net/'
for iface in os.listdir(path):
   index_file = os.path.join(path, iface, 'ifindex')
   carrier_file = os.path.join(path, iface, 'carrier')
   with open(index_file) as ifile, open(carrier_file) as cfile:
      index = int(ifile.read())
      carrier = int(cfile.read())
   if index == 6 and carrier == 1:
      print(f'{iface}: bonjour')

à tester/adapter.