Discussion :

[Migousto]

Bonjour,

Je bosse sur un script en Powershell qui vise à vérifier l’état d'un service windows et qui le redémarre quand il est arrêté. Étant un super débutant dans le monde du scripting, j'aurais besoin d'un coup de pouce SVP.

Je souhaiterais que mon script m'affiche le résultat de ma commande dans un fichier texte. Toutes propositions pour faire évoluer le script sont la bienvenue. D'avance MERCI


Ci-dessous le script:

Code powershell :

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

1
2
3
4
5
6
7
8
9
10
# watch-service.ps1
 
 
while($true)
{
    $name = Get-Content -Path C:\temp\test.txt | where { $_ -ne "$null" } | Select-Object -Index 0
    $service = get-service $name
    "{0} : le service {1} est '{2}'" -f $(Get-Date -Format "dd/MM/yy HH:mm:ss"), $service.DisplayName, $service.Status
    if($service.Status -eq "Stopped") {Start-Service -name $name}
 }

[6ratgus]

salut Migousto


d'abord la solution a ta question "add-Content"
utilisable comme ceci :

Code :

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

            "{0} : le service {1} est '{2}'" -f $(Get-Date -Format "dd/MM/yy HH:mm:ss"), $service.DisplayName, $service.Status | add-Content c:\temp\testresultat.txt

après pourquoi la boucle while ?
et pourquoi utilisé un fichier pour avoir le nom du service ? un paramètre a l'appel du script est possible !

enfin les améliorations, une gestion d'erreur :

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
while($true) {
    $name = Get-Content -Path C:\temp\test.txt | where { $_ -ne "$null" } | Select-Object -Index 0
    try {
        $service = get-service $name -ErrorAction Stop
        if ($service.Status -eq "Stopped") {
            try {
                Start-Service -name $name -ErrorAction Stop
                $demarrage = "reussie"
                }
            catch {
                $demarrage = "impossible"
                }
            "{0} : le service {1} est '{2}', le redemmarage est {3}" -f $(Get-Date -Format "dd/MM/yy HH:mm:ss"), $service.DisplayName, $service.Status, $demarrage | add-Content c:\temp\testresultat.txt
            }
        }
    catch {
        "{0} : le service {1} est 'inconue'" -f $(Get-Date -Format "dd/MM/yy HH:mm:ss"), $service.DisplayName | add-Content c:\temp\testresultat.txt
        sleep -Seconds 10
        }
    }

[Migousto]

Bonjour 6ratgus,

Déjà un grand MERCI pour ta réponse,

Plus concretement, je voudrais que mon script va lire une valeur dans un fichier car je n'ai pas encore trouver la solution pour lire une valeur dans l'outil de supervision qui me remonte l'alarme...

l'idée et je m'explique je voudrais que mon script va lire l'object de class d'alarming qui est remonté (le nom du service windows ko), qu'il le redemarre quand il est KO et qu'il log l'action

Concernant la boucle While c'est un collegue qui me l'ajouté au script... j'ignore l'importance de son rôle

MERCI

[6ratgus]

ok

je t'explique ce que fait le script :
1 - il lis le fichier avec le nom de service
2 - si il le service n'existe pas il écrit dans le "log résultat" "le service est inconnue" puis revient en 1 et boucle sans arrêt sur cette action jusqu'à l'arrêt du script (a cause du while) et remplie le fichier log et donc le disque
3 - si il le service existe - il vérifie si le service est en service et si il est éteint il tente de le démarrer et écrit le résultat dans le log et revient en 1 et boucle sans arrêt sur cette action jusqu'à l'arrêt du script (a cause du while) et idem ci-dessus

le while posse donc un problème
mais une amélioration pourrait être d'effacer le nom du service dans le fichier pour plus qu'il ne boucle plus sur la même action
il faut aussi modifier le script pour que si le fichier est vide, le script attende

par contre ce script n'autorise qu'un seul service a la fois a être redémarrer et donc il ne peut y avoir plusieurs services KO simultanément
une évolution de plus a regarder

[ericlm128]

Une variante

Code powershell :

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
$name = Get-Content -Path C:\temp\test.txt | where { $_ -ne "$null" } | Select-Object -First 1
while ($true)
{
    try
    {
        $service = get-service $name -ErrorAction Stop
        if ($service.Status -eq [System.ServiceProcess.ServiceControllerStatus]::Stopped)
        {
            try
            {
                Start-Service -name $name -ErrorAction Stop
                $demarrage = "reussie"
            }
            catch
            {
                $demarrage = "impossible"
            }
            "{0} : le service {1} est '{2}', le redemmarage est {3}" -f $(Get-Date -Format "dd/MM/yy HH:mm:ss"), $service.DisplayName, $service.Status, $demarrage | add-Content c:\temp\testresultat.txt
        }
    }
    catch
    {
        "{0} : le service {1} est 'inconue'" -f $(Get-Date -Format "dd/MM/yy HH:mm:ss"), $name | add-Content c:\temp\testresultat.txt
        continue
    }
    sleep -Seconds 10
}

$name semble pouvoir être un tableau de string.
Je pense qu'il faut effectuer des tests unitaires.

Si j'ai 3 services dans $name et que celui du milieu est introuvable ou impossible à démarrer, le log ne reflétera pas la réalité.