Skip to content

Commit c5be138

Browse files
committed
Adjust readme and tests to the new repo structure
Signed-off-by: Markus Katharina Brechtel <[email protected]>
1 parent 683d273 commit c5be138

File tree

3 files changed

+8
-15
lines changed

3 files changed

+8
-15
lines changed

.github/workflows/tests.yml

+2-4
Original file line numberDiff line numberDiff line change
@@ -64,15 +64,13 @@ jobs:
6464
6565
- name: Deploy the test VMs
6666
run: |
67-
cd ansible
68-
cp hosts.yaml.example hosts.yaml
6967
7068
if [ "${{ matrix.os }}" = "ubuntu/20.04" ]; then
7169
# Ubuntu 20.04's OVN is too old.
72-
sed -i "s/ovn_release:.*/ovn_release: \"ppa\"/g" hosts.yaml
70+
sed -i "s/ovn_release:.*/ovn_release: \"ppa\"/g" inventories/baremetal.yaml
7371
fi
7472
75-
ansible-playbook deploy.yaml
73+
ansible-playbook -i inventories/baremetal.yaml deploy.yaml
7674
7775
- name: Post deployment validation
7876
run: |

README.md

+6-10
Original file line numberDiff line numberDiff line change
@@ -34,9 +34,9 @@ tofu apply -target=module.baremetal
3434
```
3535

3636
### Run the Ansible Playbook
37-
Go to the ansible directory:
37+
Go back to the repo root directory:
3838
```
39-
cd ../ansible/
39+
cd ../
4040
```
4141

4242
NOTE: If you need the same version of Ansible this was tested with:
@@ -47,15 +47,11 @@ pipenv shell
4747
ansible-galaxy install -r ansible_requirements.yml
4848
```
4949

50-
Copy the example inventory file:
51-
```
52-
cp hosts.yaml.example hosts.yaml
53-
```
5450
NOTE: If you are connecting to a remote Incus host you will need to change the `ansible_incus_remote` variable to match the name of the Incus remote (see: `incus remote list` for a list of remote names to use).
5551

5652
Run the Playbooks:
5753
```
58-
ansible-playbook deploy.yaml
54+
ansible-playbook -i inventories/baremetal.yaml deploy.yaml
5955
```
6056

6157
NOTE: When re-deploying the same cluster (e.g. following a `terraform destroy`),
@@ -65,9 +61,9 @@ connection to the previously deployed systems which will cause the
6561
deployment to get stuck.
6662

6763
```
68-
rm ansible/data/ceph/*
69-
rm ansible/data/lvmcluster/*
70-
rm ansible/data/ovn/*
64+
rm data/ceph/*
65+
rm data/lvmcluster/*
66+
rm data/ovn/*
7167
```
7268

7369
### Test a VM and Container on the new Incus cluster

ansible.cfg

-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
[defaults]
2-
inventory = inventories/baremetal.yaml
32
error_on_undefined_vars = false
43
timeout = 30
54
forks = 10

0 commit comments

Comments
 (0)