89luca89/terrible

[IMPROVEMENT] Improve Asserts

Aberta

#87 aberto em 4 de jul. de 2021

Ver no GitHub
 (3 comentários) (0 reação) (1 responsável)YAML (29 forks)github user discovery
bugenhancementexperimentalhelp wanted

Métricas do repositório

Stars
 (257 estrelas)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

Some assertions should be improved, for example:

            host-vm-1:
              disk_source: "/mnt/iso/debian10-terraform.qcow2"
              os_family: Debian
              cpu: 1
              memory: 512
              hypervisor: hypervisor_1
              ansible_host: 192.168.122.9
              network_interfaces:
                iface_1:
                  name: default
                  type: nat
                  ip: 192.168.122.9
                  gw: 192.168.122.1
                  dns:
                    - 192.168.122.1
              default_route: True <<<--- note this is unindented

This snippets passes the assertions even if it's not valid We should review and evaluate more corner cases to include in the assertions

Also we could think of an external tool/linter as @alegrey91 suggests to validate inventories

Guia do colaborador

Direção de pesquisa
Revise o código de asserções atual e identifique lacunas na validação YAML. Considere adicionar verificações de indentação e estrutura. Avalie o uso de linters externos como yamllint.
Pilha de tecnologia
None
Domain
testingdevops
Tipo Issue
Teste
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
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
YAMLAnsible
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65