Discussion :

[Taylor08]

Bonjour,

Je suis sur une API Red Hat Satellite 5.5 (voir ici) que j'utilise pour implémenter plusieurs scripts python.

Voici mon code source :

test.py

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
 
#!/usr/bin/python
# -*-coding:Latin-1 -*
# Import lib
import xmlrpclib
import sys
import re
import datetime
import sys
from time import mktime, strptime
 
# Satellite connection
S_URL = "http://monSite.com"
S_LOGIN = "admin"
S_PASSWORD = "admin"
client = xmlrpclib.Server(S_URL, verbose=0)
key = client.auth.login(S_LOGIN, S_PASSWORD)
 
# Je liste mes stations 
list_systems = client.system.listSystems(key)
for system in list_systems:
# Nom de la station
 SYSNAME = system["name"]
 list_id = client.system.getId(key,SYSNAME)
# ID
 id = list_id[0]['id']
 # Je liste mes fichiers
 list_files = client.system.config.listFiles(key,id,1)
 for element in list_files:
 # Tous mes fichiers
  PATH = element["path"]
# Seulement les fichiers sudoers et users.allow  
  if PATH == "/etc/sudoers" or PATH == "/etc/centrifydc/users.allow" :
   # Resultat en sortie "
   print "%s : %s : %s : %s" % (SYSNAME, id, PATH, PATH)
 
 
client.auth.logout(key)

Voici mon resultat :



J'aimerais savoir comment fait on pour retirer les doublons puis obtenir un resultat de cette façon :

NomStation : IdStation : /etc/sudoers/ : /etc/centrifydc/users.alow


Quelqu'un aurait un idée svp?

Cdt,

[wiztricks]

Salut,

C'est cohérent avec ce qui est écrit à la ligne 35:

Code :

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

  print "%s : %s : %s : %s" % (SYSNAME, id, PATH, PATH)

- W

[Taylor08]

Oui c'est cohérent pourquoi?

[e-ric]

Salut

Pourtant, tout est dans la réponse de - W.
Tu demande imprimer deux fois la variable PATH.
Remplace par :

Code :

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

1
2
 
print "%s : %s : %s" % (SYSNAME, id, PATH)

ça devrait aller mieux.

@+

[wiztricks]

Oui c'est cohérent pourquoi?

C'est votre code: impossible de répondre à votre place.
Et le post d'e-ric, montre que c'est quand même trivial à corriger, non?

- W

[Taylor08]

Alors justement le fait de mettre deux fois la variable PATH me permettra de voir si la station contient le fichier "etc/sudoers" et le fichier "etc/centrifydc/user.allow/" aussi

C'est à dire dans l'idée:

Code :

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

1
2
3
4
5
6
7
8
9
10
 
 PATH = element["path"]
# Seulement les fichiers sudoers et users.allow  
  if PATH == "/etc/sudoers" or PATH == "/etc/centrifydc/users.allow" :
   # Resultat en sortie "
   print "%s : %s : %s : %s" % (SYSNAME, id, PATH, PATH)
 
#  print "%s : %s : %s : %s" % (SYSNAME, id, PATH, PATH)
#  si la station a ces deux fichiers alors le résultat  
#  en sortie serait  : NomStation (SYSNAME) : IdStation (id) : /etc/sudoers (PATH) : /etc/centrifydc/users.allow (PATH)

Maintenant ayant suivi vos conseils je n'ai quand même pas eu la réponse que je voulais car j'ai toujours ce doublon qui me gène :



Au final ce que je souhaiterais, c'est d'arriver à un resultat comme ceci : NomStation : IdStation : /etc/sudoers : /etc/centrifydc/users.allow

Sur une seule même ligne avec aucun doublon.

[e-ric]

Y a du boulot... Commence par lire les docs et un peu les bases de la programmation.
Une variable ne contient qu'une seule valeur qui peut être simple (nombre, chaîne) ou composée (liste), pour ton problème une liste devrait t'aider:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
 PATH = element["path"]
# Seulement les fichiers sudoers et users.allow  
  liste_de_paths = ["/etc/sudoers", "/etc/centrifydc/users.allow"]
  if PATH in liste_de_paths :
   # Resultat en sortie "
   print "%s : %s : %s" % (SYSNAME, id, " : ".join(liste_de_paths))
 
#  print "%s : %s : %s : %s" % (SYSNAME, id, PATH, PATH)
#  si la station a ces deux fichiers alors le résultat  
#  en sortie serait  : NomStation (SYSNAME) : IdStation (id) : /etc/sudoers (PATH) : /etc/centrifydc/users.allow (PATH)

Ce qui ne change pas le fait que ta variable PATH ne prendra qu'une valeur à la fois. Quel est l'intérêt d'afficher les deux valeurs alors qu'une seule a du sens ?

Pour les doublons, il faut peut-être analyser ce que retournent les instructions:

Code :

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

1
2
list_systems = client.system.listSystems(key)
for system in list_systems:

Cdlt

[Taylor08]

Oula Bon e-ric je pense qu'on ne s'est pas bien compris. Je n'avais pas envie d'expliquer tout mon code mais il va bien falloir tout expliquer!

Voici mon code avec plus de commentaires détailées :
P.S : Prenez le temps de bien lire mes commentaires car je ne code pas par pur bonheur !

test.py

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
 
#!/usr/bin/python
# -*-coding:Latin-1 -*
# Import lib
import xmlrpclib
 
# Satellite connection
SATELLITE_URL = "http://martinique.hq.ep.corp.local/rpc/api"
SATELLITE_LOGIN = "admin-integration"
SATELLITE_PASSWORD = "admin-integration"
client = xmlrpclib.Server(SATELLITE_URL, verbose=0)
key = client.auth.login(SATELLITE_LOGIN, SATELLITE_PASSWORD)
 
 # Je liste mes stations
 # "print list_systems" me donne en sortie : "NomStation45, NomStation2, NomStation78, NomStation1....etc"
list_systems = client.system.listSystems(key)
 # Pour chaque system de ma liste de systeme
 # je souhaite avoir le Nom de la station et son ID
for system in list_systems:
 # Nom de la station
 SYSNAME = system["name"]
 list_id = client.system.getId(key,SYSNAME)
 # ID
 id = list_id[0]['id']
 # Ensuite je liste mes fichiers
 # "print list_files" me donne en sortie tous les paramètres qui correspond à :
 # string "type"
    # file
    # directory
    # symlink
 # string "path" - File Path
 # string "channel_label"
 list_files = client.system.config.listFiles(key,id,1)
 # Pour chaque element de ma liste de fichiers
 # je souhaite avoir que la liste des fichiers de toutes les stations
 # Et parmi cette liste que les fichiers "etc/sudoers" et "etc/centrifydc/user.allow"
 for element in list_files:
 # Tous mes fichiers
  PATH = element["path"]
 # Seulement les fichiers "etc/sudoers" et users.allow  
  if PATH == "/etc/sudoers" or PATH == "/etc/centrifydc/users.allow" :
 # Resultat en sortie "
   print "%s : %s : %s " % (SYSNAME, id, PATH)
 
client.auth.logout(key)

Au final ce code m'affiche


Mais ce que je souhaite c'est qu'il me l'affiche sur cette forme là :

1) Quand dans une station on a les deux fichiers


2) Quand dans une station on a que le fichier "/etc/centrifydc/users.allow"


3) Quand dans une station on a que le fichier "/etc/sudoers"

[VinsS]

Qu'est-ce qui t'empêche de les traiter à la suite l'un de l'autre ?

Je veux dire comme ceci:

Code :

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

1
2
3
4
5
6
7
8
9
 
fl = fl1 = ""
for element in list_files:
    PATH = element["path"] 
    if PATH == "/etc/sudoers":
        fl = ": %s" % PATH
    elif PATH == "/etc/centrifydc/users.allow":
        fl1 = ": %s" % PATH
print "%s : %s %s %s" % (SYSNAME, id, fl, fl1)

[Taylor08]

VinsS comment te dire....

C'est parfait ! Merci à vous les gars !



A bientot

[e-ric]

Désolé pour tout à l'heure, j'ai été un peu vif mais je ne comprenais pas ton intention.
Bonne suite.

[wiztricks]

Salut,

ou encore comme ceci:

Code :

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

1
2
    paths = [ zot for zot in  ("/etc/sudoers", "/etc/centrifydc/users.allow") if zot in list_files ]
    print "%s : %s %s" % (SYSNAME, id, ':'.join(paths))

Il y a plein de façons d'écrire cela en Python.
listes et chaînes de caractères sont des objets de base.
Apprendre à les utiliser sont des constructions qu'on apprend dans des tutos.

Si vous ne maîtrisez pas encore çà, travailler avec une API comme celle de "Red Hat Satellite 5.5" devient galère.
Imaginer ce que vous pourriez en faire doit être une sorte de torture.

Ce pourrait être une opportunité pour vous motiver à (re-)lire le Swinnen.
Essayer de consolider/revoir les trucs qu'on a survolé lorsqu'on en a besoin est un bon truc pour progresser.

Faire marcher ce code pour qu'il réponde à vos besoins est un sujet, vous donner les moyens d'y arriver par vous même en est un autre.

A vous de voir.

- W

[Taylor08]

Wiztricks t'inquiète pas je suis qu'un débutant sur python, je ne cherchais pas non plus à faire un code en 3 lignes (un peu comme ton code). Mais plutot compréhensible à première vue comme celui de VinsS. Je fonctionne petit à petit en cherchant à comprendre ce que les autres font pour que je puisse le faire moi meme. L'optimiser n'est pas encore le but mais faire ce qui me parait le plus clair est mon BUT!

Chaque chose en son temps selon moi.

Cdt,

P.S: Merci pour l'aide j'essayerais de le tester ton bout de code et l'adapter en fonction de mes besoins

[wiztricks]

Wiztricks t'inquiète pas je suis qu'un débutant sur python, je ne cherchais pas non plus à faire un code en 3 lignes (un peu comme ton code). Mais plutot compréhensible à première vue comme celui de VinsS. Je fonctionne petit à petit en cherchant à comprendre ce que les autres font pour que je puisse le faire moi meme. L'optimiser n'est pas encore le but mais faire ce qui me parait le plus clair est mon BUT!

Inutile de dire que vous débutez: çà se voit.
Mon propos était de vous inciter à jeter un oeil (et même les deux) dans un tuto.

- W