Mentions légales du service

Skip to content
Snippets Groups Projects

Don't crash when kadeploy data is missing for clusters not in default or production queues.

Merged JACQUOT Pierre requested to merge kadeployg5k into master

This merge request is related to toulouse-estats installation.

Because estats is an experimental cluster, we need to perform a few tests with kadeploy-devel before allowing deployment on kadeploy. This wasn't possible, since the rake gen:puppet:kadeployg5k SITE=toulouse crashed when generating the configuration.

This merge request allows the generation of only the kadeploy-dev configuration in the following case:

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • assigned to @pijacquo

  • Alexandre MERLIN approved this merge request

    approved this merge request

  • C'est bon pour moi, la seul remarque ça serait peut etre d'avoir un warning si le site ne contiens aucun cluster ?

    Je crois qu'on peut le faire élégamment en ruby, quelque du genre:

    clusters.each do |x|
      #...
    end.empty? and begin
      puts "Empty loop: no cluster"
    end
  • Oh, je savais pas qu'on pouvait utiliser le .empty? comme ça.

    Je regarde pour rajouter ça. :smile:

  • J'ai regardé cette après-midi.

    Même s'il est possible d'afficher un warning lorsque le site ne contient aucun cluster, ça ne serait pas très utile de le faire ici.

    En effet, dans l'état actuel des choses, la tâche rake utilise les données provenant du ref-repo pour itérer sur les clusters et ensuite chercher ces cluster dans les fichiers du générateur de config kadeploy[-dev]. Le chargement des fichiers du ref-repo échouera si le site n'a pas de cluster déclaré, et on n'entrera jamais dans le code de la tache rake. Donc je pense que c'est plutôt quelque chose qu'on devrait corriger du coté de l'input-loader plutôt que des taches rake.

    Je merge ce bout de code, parce que l'on va en avoir besoin pour travailler sur le cluster estats, mais je reste ouvert à de futures remarques et améliorations. :wink:

  • merged

  • JACQUOT Pierre mentioned in commit c677feb4

    mentioned in commit c677feb4

Please register or sign in to reply
Loading