Project

General

Profile

Anomalie #6854

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

Added by Joël Cuissinat over 10 years ago. Updated about 10 years ago.

Status:
Fermé
Priority:
Bas
Assigned To:
Category:
-
Start date:
02/14/2014
Due date:
02/21/2014
% Done:

100%

Estimated time:
0.50 h
Spent time:
Distribution:
EOLE 2.4

Description

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

Related issues

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

Associated revisions

Revision b05a5755 (diff)
Added by Bruno Boiget over 10 years ago

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

History

#1 Updated by Daniel Dehennin over 10 years ago

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 Updated by Philippe Caseiro over 10 years ago

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 Updated by Joël Cuissinat over 10 years ago

  • Assigned To deleted (Daniel Dehennin)
  • Priority changed from Normal to Bas
  • Target version changed from Eole 2.4-beta3 to 189

#4 Updated by Joël Cuissinat over 10 years ago

  • Target version changed from 189 to 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 Updated by Daniel Dehennin over 10 years ago

  • Due date set to 02/21/2014
  • Assigned To set to Bruno Boiget
  • Start date set to 02/14/2014

C’est Bruno qui a le code.

#6 Updated by Bruno Boiget over 10 years ago

  • Status changed from Nouveau to Résolu
  • % Done changed from 0 to 100

#7 Updated by Joël Cuissinat about 10 years ago

  • Status changed from Résolu to 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 ===========================

Also available in: Atom PDF