Setup testing of Ansible components via QEMU #70

Open
gingermusketeer wants to merge 4 commits from setup-testing-for-ansible-roles into main

This will allow us to produce higher quality ansible componets and
potentially run parts of the system locally for development/testing.

To test this run .tests/qemu/run_test.sh. The actual tests will fail until #68 is merged.

This will allow us to produce higher quality ansible componets and potentially run parts of the system locally for development/testing. To test this run `.tests/qemu/run_test.sh`. The actual tests will fail until #68 is merged.
This will allow us to produce higher quality ansible componets and
potentially run parts of the system locally for development/testing.
@ -0,0 +1,352 @@
#!/usr/bin/env bash
set -euo pipefail
Author
Owner

I have tried to make this file as readable as possible. Let me know if something isn't clear!

I have tried to make this file as readable as possible. Let me know if something isn't clear!
Member

Syntes koden var ryddig og oversiktlig! Har ikke jobbet noe særlig i qemu selv, men klarte stort sett å henge med på hva som skjedde

Syntes koden var ryddig og oversiktlig! Har ikke jobbet noe særlig i qemu selv, men klarte stort sett å henge med på hva som skjedde
Tormod left a comment

Stort sett bra. Har bare en innvending på test 4.

Stort sett bra. Har bare en innvending på test 4.
@ -0,0 +322,4 @@
echo ""
log "================================================================"
log " TEST 1: Check mode on Clean Machine (Expecting failure reproduction)"
Member

Hva mener du med "Expecting failure reproduction"?

Hva mener du med "Expecting failure reproduction"?
Author
Owner

Dette er utdatert nå. Jeg skal oppdatere det ettersom problemet som var grunnen til at jeg så på dette er fikset på main.

Takk for review!

Dette er utdatert nå. Jeg skal oppdatere det ettersom problemet som var grunnen til at jeg så på dette er fikset på main. Takk for review!
gingermusketeer marked this conversation as resolved
@ -0,0 +342,4 @@
echo ""
log "================================================================"
log " TEST 4: Apply mode on Existing Machine (Idempotency)"
Member

Tester denne egentlig idempotens? I så fall burde den vel faile, siden ansible rapporterer changed på Ensure systemd-timesyncd/timedated service is enabled and running. For å teste idempotens kunne vi kanskje gjort | grep changed=0 etter kommandoen.

`[+00m15s] ================================================================
[+00m15s] TEST 4: Apply mode on Existing Machine (Idempotency)
[+00m15s] ================================================================

PLAY [Test NTP Role] ***********************************************************************************************************************************************************************************************

TASK [Gathering Facts] *********************************************************************************************************************************************************************************************
ok: [test_node]

TASK [ntp : Install systemd-timesyncd package] *********************************************************************************************************************************************************************
ok: [test_node]

TASK [ntp : Ensure systemd-timesyncd service is enabled and running] ***********************************************************************************************************************************************
changed: [test_node]

TASK [ntp : Ensure systemd-timedated service is enabled and running] ***********************************************************************************************************************************************
changed: [test_node]`

Tester denne egentlig idempotens? I så fall burde den vel faile, siden ansible rapporterer changed på `Ensure systemd-timesyncd/timedated service is enabled and running`. For å teste idempotens kunne vi kanskje gjort `| grep changed=0` etter kommandoen. `[+00m15s] ================================================================ [+00m15s] TEST 4: Apply mode on Existing Machine (Idempotency) [+00m15s] ================================================================ PLAY [Test NTP Role] *********************************************************************************************************************************************************************************************** TASK [Gathering Facts] ********************************************************************************************************************************************************************************************* ok: [test_node] TASK [ntp : Install systemd-timesyncd package] ********************************************************************************************************************************************************************* ok: [test_node] TASK [ntp : Ensure systemd-timesyncd service is enabled and running] *********************************************************************************************************************************************** changed: [test_node] TASK [ntp : Ensure systemd-timedated service is enabled and running] *********************************************************************************************************************************************** changed: [test_node]`
Author
Owner

Du har det helt rett. Jeg ønsket å teste idempotens men det er vansklig men en role som har noe med tid å gjøre. Jeg har endret beskrivelsen!

Du har det helt rett. Jeg ønsket å teste idempotens men det er vansklig men en role som har noe med tid å gjøre. Jeg har endret beskrivelsen!
This pull request doesn't have enough approvals yet. 0 of 1 approvals granted.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin setup-testing-for-ansible-roles:setup-testing-for-ansible-roles
git switch setup-testing-for-ansible-roles
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
datakollektivet/systemer!70
No description provided.