Projet

Général

Profil

Anomalie #6854

CreoleService status ne renvoie pas toujours le code d'erreur attendu

Ajouté par Joël Cuissinat il y a plus de 10 ans. Mis à jour il y a environ 10 ans.

Statut:
Fermé
Priorité:
Bas
Assigné à:
Catégorie:
-
Début:
14/02/2014
Echéance:
21/02/2014
% réalisé:

100%

Temps estimé:
0.50 h
Temps passé:
Distribution:
EOLE 2.4

Description

root@amontoto:~# CreoleService ntp status
Service ntp in root is not running
root@amontoto:~# echo $?
0

Demandes liées

Lié à creole - Anomalie #7292: Impossible d’instancier sans sympa/roundcube Fermé 04/02/2014 21/02/2014
Lié à arv - Evolution #7449: Les éventuelles erreurs d'arrêt et démarrage des services dans posttemplate/00-arv Fermé 04/04/2014

Révisions associées

Révision b05a5755 (diff)
Ajouté par Bruno Boiget il y a environ 10 ans

Retour des codes de service par CreoleService 'status'

Dans le cas de manage_services:
- on retourne toujours 1 dès qu'un service est en erreur

Dans le cas de unamaged_service (non gérés par creole):
- On lève une exception si les fonctions retournent {'code':1, 'msg':...}
- Rqe: seuls stop/start/restart sont gérés actuellement

Fixes #6854

Historique

#1 Mis à jour par Daniel Dehennin il y a plus de 10 ans

On fait quoi lorsque le service est question est présent dans 3 conteneurs différents et qu’il ne fonctionne pas sur seulement sur l’un d’entre eux ?

#2 Mis à jour par Philippe Caseiro il y a plus de 10 ans

On retourne 1 car si le service est en erreur quelque part la commande "CreoleService" doit renvoyer un code d'erreur.
Je pense qu'a la moindre erreur il faut retourner 1

#3 Mis à jour par Joël Cuissinat il y a plus de 10 ans

  • Assigné à Daniel Dehennin supprimé
  • Priorité changé de Normal à Bas
  • Version cible changé de Eole 2.4-beta3 à 189

#4 Mis à jour par Joël Cuissinat il y a plus de 10 ans

  • Version cible changé de 189 à Eole 2.4-RC2

Dans le cas où cette demande serait finalement rejetée, il faudrait commenter le test unitaire suivant : http://dev-eole.ac-dijon.fr/projects/creole/repository/revisions/master/entry/tests/test_creoleservice.py#L12

#5 Mis à jour par Daniel Dehennin il y a environ 10 ans

  • Echéance mis à 21/02/2014
  • Assigné à mis à Bruno Boiget
  • Début mis à 14/02/2014

C’est Bruno qui a le code.

#6 Mis à jour par Bruno Boiget il y a environ 10 ans

  • Statut changé de Nouveau à Résolu
  • % réalisé changé de 0 à 100

#7 Mis à jour par Joël Cuissinat il y a environ 10 ans

  • Statut changé de Résolu à Fermé
root@horus:~# CreoleService ntp status
Service ntp in root is not running
root@horus:~# echo $?
1

root@horus:~# py.test /usr/share/creole/tests/test_creoleservice.py 
============================= test session starts =============================
platform linux2 -- Python 2.7.3 -- pytest-1.3.4
test path 1: /usr/share/creole/tests/test_creoleservice.py

../usr/share/creole/tests/test_creoleservice.py ..

========================== 2 passed in 1.18 seconds ===========================

#8 Mis à jour par Gilles Grandgérard il y a environ 10 ans

ok

Formats disponibles : Atom PDF