Skip to content

IncidentKey is in fact optional

Romain Hennuyer requested to merge feat-snow into develop

Merge Request Canopsis

Le Incident Key est en fait optionnel.

Pré-requis

  • Insérer une conf ticketapi de type snow, en laissant le paramètre incident_key à vide
  • Depuis l'UI, sur une alarme, faire un ack pouis un declareticket
  • Le ticket est bien créé (un tag avec le numéro de ticket apparait)

Pour la validation

  • Le code respecte la PEP8
  • Chaque classe/fonction/méthode contient une docstring complète
  • Le code est revu, les modifications demandées sont appliquées
  • Les messages de commits sont compatibles avec l' angular spec ?
  • Les modifications apportées contiennent des tests unitaires
  • Les tests unitaires passent
  • la documentation a été mise à jour (si changement de comportement)

Tests

  1. Installer la branche dans un environnement Canopsis
  2. se mettre dans l'environnement Canopsis :su - canopsis
  3. (re)démarrer Canopsis :hypcontrol restart
  4. lancer les tests unitaires : ut_runner
  5. Retourner dans le dossier où les sources de canopsis sont installées :cd /vagrant/canopsis
  6. Exécuter les tests fonctionnels : cd sources/functional_testing/ && python2 runner.py

@adewarumez @lseguinot

Edited by Theo Plockyn

Merge request reports