canonical/multipass

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

Open

#646 aberto em 28 de fev. de 2019

Ver no GitHub
 (4 comments) (0 reactions) (1 assignee)C++ (634 forks)batch import
enhancementgood first issue

Métricas do repositório

Stars
 (6.956 stars)
Métricas de merge de PR
 (Mesclagem média 16d 2h) (50 fundiu PRs em 30d)

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.

Guia do colaborador

Direção de pesquisa
Investigue o tratamento de erro atual no parsing da configuração cloud init no código do Multipass. Localize onde as exceções yaml cpp são lançadas e rastreie o caminho até onde são capturadas ou propagadas para o cliente. Em seguida, implemente um bloco catch para exceções yaml cpp e relance com uma mensagem de erro mais descritiva indicando qual parte da configuração está malformada.
Pilha de tecnologia
cpp
Domain
backendcli
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++CMake
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70