Utilisation simultanée d'un script ksh

Version imprimable

Bonjour,

J'ai créé dans le cadre de mes tâches quotidiennes au travail un script permettant de faire des exports de tables (.dmp).

Ma question est la suivante, comment est-il possible de l'améliorer afin que plusieurs personnes puissent l'utiliser en même temps?

Le script :

Code:

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
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
 
#!/bin/ksh
#==============================================================================
#---------------+--------------------------------------------------------------
#       HISTO   |   Date   | Vers | Auteur | Libelle
#               |----------+------+--------+-----------------------------------
#               | 08/03/13 | 1.00 |  MGO   | 
#               |          |      |    	   | 
#               |          |      |        |
#==============================================================================
#=== Pré-requis ===#
rm -f export_table.lst
touch export_table.lst 
 
echo "Entrer le nom de la demande de service ou du changement : "
read nom_dds
mkdir $nom_dds ; chmod 777 $nom_dds
 
 
#=== Récupération du schéma ===#
nom_schema=0
until ! [[ $nom_schema = ?([-+])+([0-9]) ]]
do
	echo "Entrer le nom du schéma : "
	read nom_schema
	if [[ $nom_schema = ?([-+])+([0-9]) ]]
		then
		echo "La saisie est incorrecte !!"
	fi
done
 
 
#=== Récupération de l'instance ===#	
nom_instance=0
until ! [[ $nom_instance = ?([-+])+([0-9]) ]]
do
	echo "Entrer le nom de l'instance : "
	read nom_instance
	if [[ $nom_instance = ?([-+])+([0-9]) ]]
		then
		echo "La saisie est incorrecte !!"
	fi
done
 
 
#=== Récupération du nombre de table & incrémentation des tables dans le fichier export_table.lst ===#	
nombre_table=0
until [[ "$nombre_table" > 0 ]]
do
	echo "Entrer le nombre de table(s) à exporter : "
	read nombre_table	
	if ! [[ "$nombre_table" > 0 ]]
		then
		echo "La saisie est incorrecte !!"	
	fi	
done
while [[ $nombre_table -ge 1 ]]
do
	echo "Entrer le nom de la table : "
	read nom_table_saisie
	echo "$nom_table_saisie" >> export_table.lst
	(( nombre_table=nombre_table-1 ))
done
 
 
#=== Extraction des tables & compression des données ===#
for nom_table in `cat export_table.lst`
do
	expdp system/manager@$nom_instance tables=$nom_schema.$nom_table DIRECTORY=DIR_DP DUMPFILE=expdp$nom_schema_$nom_table.dmp logfile=expdp$nom_schema_$nom_table.log
	gzip expdp$nom_schema_$nom_table.dmp
	mv expdp$nom_schema_$nom_table.dmp.gz /travail/$LT/exports/$nom_instance/export/$nom_dds/
	mv expdp$nom_schema_$nom_table.log /travail/$LT/exports/$nom_instance/export/$nom_dds/
	chmod 755 /travail/$LT/exports/$nom_instance/export/$nom_dds/*
	sleep 2
	echo "======================== Fin $nom_table ========================"
done

Merci par avance
Matthieu

27/05/2013, 13h00
N_BaH

Bonjour,

d'abord, je n'irai pas cherché un fichier zippé.
tu peux le poster ici entre balise codes (bouton #).
euh, il fait combien de lignes ?

ensuite, pourquoi cela poserait des problèmes que le script soit utilisé par plusieurs utilisateurs en même temps ?
27/05/2013, 14h27
matthieu44

Cela pose problème car le contenu des variables sont saisi par l'utilisateur lui-même et plusieurs utilisateurs saisissent en même temps les variable seront faussées et le résultat attendu incorrect.

Je précise que je débute en scripting. soyez indulgent:oops:
27/05/2013, 14h54
N_BaH

Citation:

des variables sont saisi par l'utilisateur lui-même et plusieurs utilisateurs saisissent en même temps les variable seront faussées et le résultat attendu incorrect.

normalement, non : le script est exécuté dans l'environnement de chaque utilisateur.
si, dans mon environnement, j'assigne "bla bla" à la variable var, alors, dans ton environnement, $var n'existe pas !
27/05/2013, 17h49
matthieu44

Citation:

Envoyé par N_BaH

normalement, non : le script est exécuté dans l'environnement de chaque utilisateur.
si, dans mon environnement, j'assigne "bla bla" à la variable var, alors, dans ton environnement, $var n'existe pas !

En fait pour que le script fonctionne, il faut le lancer en tant que "oradba". Ce user peut dans les faits être utilisé par plusieurs personnes en même temps.

Voila pourquoi cela pose problème.
27/05/2013, 18h15
jack-ft

Oui, mais quel problème cela pose-t-il exactement? 8O
28/05/2013, 09h02
Flodelarab
Bonjour

Citation:

Envoyé par matthieu44

En fait pour que le script fonctionne, il faut le lancer en tant que "oradba". Ce user peut dans les faits être utilisé par plusieurs personnes en même temps.

Pourquoi ne pas laisser les utilisateurs en tant qu'eux-même et les ajouter au groupe oradba (d'où ils pourraient lancer le script) ? Ainsi, en créant le répertoire dans ~/tmp/ , il n'y aura pas de chevauchement, ni écrasement.

Si tu tiens a ce que tout le monde se mette au même endroit et appelle le dossier de la même façon, il n'y a pas de miracle, cela ne marchera pas.

Le plus simple est de tester $nom_dds pour voir s'il existe et considérer la réponse positive comme une saisie incorrecte. Mais j'imagine que tu as déjà essayé...
Genre:
Code:

1 2 3 4 5 6 if [ -e ~/home ] then echo oui else echo non fi

Citation:
Envoyé par matthieu44
Ma question est la suivante, comment est-il possible de l'améliorer afin que plusieurs personnes puissent l'utiliser en même temps?

Le script :
Code:

1 2 3 4 5 6 #!/bin/ksh rm -f export_table.lst touch export_table.lst ... ... > export_table.lst ...
En fait, tout le code utilise un fichier temporaire unique nommé en l'occurrence "export_table.lst" (dans le répertoire où le script est lancé).

De cette manière, plusieurs utilisateurs peuvent lancer simultanément ce script sans collision à condition de le faire à partir de répertoires différents, ce qui n'est pas complètement satisfaisant.

La "bonne" manière de faire (à mon sens) est de créer un vrai fichier temporaire:
Code:

1 2 3 4 5 6 7 #!/bin/ksh tmp_file=$(mktemp "/tmp/export_table.lst_XXXXXX") if [ $? -ne 0 ] ; then echo "tmp impossible" 1>&2 ; exit 1 ; fi ... ... > "${tmp_file}" ... rm -rf "${tmp_file}" # à protéger par un trap...
Si la commande mktemp n'est pas disponible, on peut utiliser le classique:

Code:

tmp_file=/tmp/export_table.lst_$$

Bonjour,

Pour utiliser le script en parallèle sans problème, on peut éviter les fichiers qui sont communs à tous les utilisateurs et utiliser les variables en mémoire qui restent privées.

J'ai fait des modifications et le résultat me semble pas trop mal (juste mon avis).

Code:

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
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
#!/bin/ksh
 
################
# initialisation
################
typeset reponse
typeset -a tables
 
####################
function lire_nombre
####################
# $1: message à afficher
#-------------------
{
  nombre_ok=""
  while [[ $nombre_ok == "" ]]
  do
    read reponse?"$1"
    if [[ "$reponse" == ?([-+])+([0-9]) ]]
    then
      nombre_ok="1"
    else
      echo "*** ATTENTION: saisie incorrecte ***"
    fi
  done
}
 
###########
# programme
###########
# creation repertoire
read dds?"Entrer le nom de la demande de service ou du changement : "
mkdir $HOME/dds
 
# lire le nom du schema
lire_nombre "Entrer le nom du schéma : "
schema=$reponse
 
# lire le nom de l'instance
lire_nombre "Entrer le nom de l'instance : "
instance=$reponse
 
# lire le nombre de tables
taille=0
while [[ $taille -le 0 ]]
do
  lire_nombre "Entrer le nombre de tables : "
  taille=$reponse
done
 
# lire le nom des tables
i=1
while [[ $i -le $taille ]]
do
  read reponse?"Entrer le nom de la table $i : "
  tables[i]=$reponse
  ((i++))
done
 
# extraction - compression
for table in ${tables[*]}
do
  # ici le code pour extraire - comprimer
done

30/05/2013, 15h29
matthieu44

Merci pour vos réponses.

Entre temps, j'avais déjà trouvé une partie de la solution à savoir la création d'un fichier temporaire.

Je vais tester également les améliorations de la part de jmelyn.
30/05/2013, 16h47
matthieu44

Code:

typeset -a tables

J'ai beau chercher, j'ai l'impression que l'option -a n'existe pas ?!

Je pense que tu voulais plutôt dire -i.
30/05/2013, 17h17
jmelyn

En ksh, la commande

Code:

typeset -a var

déclare une variable de type tableau indexé. Après, ça dépend de l'implémentation de ksh. Ça marche dans Arch Linux (mksh) et dans RHEL et Fedora (ksh). Mais ce n'est peut-être pas général.

Cependant les deux lignes typeset sont optionnelles, je les ai mises pour une meilleure compréhension. Sans elles, la création des variables se fera à la première utilisation. Cela étant, Bash est tout de même plus répandu et plus puissant.