canonical/multipass

Handle yaml-cpp exceptions from a cloud-init config better

Open

#646 ouverte le 28 févr. 2019

Voir sur GitHub
 (4 commentaires) (0 réactions) (1 assigné)C++ (634 forks)batch import
enhancementgood first issue

Métriques du dépôt

Stars
 (6 956 stars)
Métriques de merge PR
 (Merge moyen 16j 2h) (50 PRs mergées en 30 j)

Description

Currently, we just bubble up any yaml-cpp exceptions back to the client, but the error message is out of context of what is really wrong. For example, if a user creates a cloud-init config with incorrect yaml, an error could look like: launch failed: operator[] call on a scalar As can be seen, it's not clear where the error is.

We should catch the yaml-cpp exceptions and make it a more meaningful message to pass to the client and user.

Guide contributeur

Direction de recherche
Examinez la gestion actuelle des erreurs dans l'analyse de la configuration cloud init dans le code de Multipass. Localisez où les exceptions yaml cpp sont levées et tracez le chemin jusqu'à leur capture ou propagation au client. Ensuite, implémentez un bloc catch pour les exceptions yaml cpp et relancez avec un message d'erreur plus descriptif indiquant quelle partie de la configuration est mal formée.
Stack technique
cpp
Domaine
backendcli
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++CMake
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70