89luca89/terrible

[IMPROVEMENT] Improve Asserts

Aperta

#87 aperta il 4 lug 2021

Vedi su GitHub
 (3 commenti) (0 reazioni) (1 assegnatario)YAML (29 fork)github user discovery
bugenhancementexperimentalhelp wanted

Metriche repository

Star
 (257 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Rivedi il codice delle asserzioni attuali e identifica le lacune nella validazione YAML. Considera l'aggiunta di controlli per indentazione e struttura. Valuta l'uso di linter esterni come yamllint.
Tech stack
Nessuno
Dominio
testingdevops
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
YAMLAnsible
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65