Better Structure the Repo and Some Other Enhancements #15

Merged
june merged 8 commits from cleanup into main 2024-12-08 16:14:41 +01:00
Owner
No description provided.
june added 6 commits 2024-12-05 23:20:39 +01:00
This playbook is for checking various host parameters.
check.yaml: add logic for printing all .dpkg-* files
All checks were successful
/ Ansible Lint (push) Successful in 1m31s
fab4942852
move roles, files and templates dirs out of playbook dir into root dir
All checks were successful
/ Ansible Lint (push) Successful in 1m38s
5bb283d5e7
Because of how Ansible local relative search paths work, the global
"files" and "templates" directories need to be next to the playbooks.
However its not intuitive to look into the "playbooks" directory to find
the files and templates for a host.
Therefore move them out of the "playbooks" directory into the root
directory and add symlinks so everything still works.

Similarly for local roles, they also need to be next to the playbooks.
So for a nicer structure, move the "roles" directory out into the root
directory as well and add a symlink so everything still works.

Also see:
https://docs.ansible.com/ansible/latest/playbook_guide/playbook_pathing.html#resolving-local-relative-paths
https://docs.ansible.com/ansible/latest/playbook_guide/playbooks_reuse_roles.html#storing-and-finding-roles
ensure all template files have a .j2 file extension
All checks were successful
/ Ansible Lint (push) Successful in 1m39s
af4abdc50b
As they should and as this is preparation for a future change
reorganizing the file structure.
reorganize (config) files and templates into one "resources" dir
All checks were successful
/ Ansible Lint (push) Successful in 1m39s
d0a28589c6
This groups the files and templates for each host together and therefore
makes it easier to see all the (config) files for a host.

Also clean up incorrect, unused docker_compose config for mumble and
clean up unused engelsystem configs.
enable pipelining as it seems to work and be faster
All checks were successful
/ Ansible Lint (push) Successful in 1m37s
/ Ansible Lint (pull_request) Successful in 1m37s
fa22ce52a0
See:
https://docs.ansible.com/ansible/latest/reference_appendices/config.html#ansible-pipelining
june added 1 commit 2024-12-05 23:32:05 +01:00
exclude symlinked ("roles" and "resources") dirs from vscode search
All checks were successful
/ Ansible Lint (push) Successful in 1m41s
/ Ansible Lint (pull_request) Successful in 1m34s
e152143c41
Also remove the ".vscode" directory from the ".gitignore" as the repo
includes stuff in it.
jtbx reviewed 2024-12-06 00:16:16 +01:00
ansible.cfg Outdated
@ -1,5 +1,6 @@
[defaults]
inventory = ./inventories/z9/hosts.yaml
pipelining = True
Owner

You could also try to add SSH multiplexing ssh_args = -C -o ControlMaster=auto -o ControlPersist=300s or would this be redundant with pipelining already enabled?

You could also try to add SSH multiplexing `ssh_args = -C -o ControlMaster=auto -o ControlPersist=300s` or would this be redundant with pipelining already enabled?
Author
Owner

Hmm, in my testing adding this option either makes it slower or gives no benefit really. Tho also the margin of error is quite big.
Am happy to add it or not, but doesn't seem to do much.

Hmm, in my testing adding this option either makes it slower or gives no benefit really. Tho also the margin of error is quite big. Am happy to add it or not, but doesn't seem to do much.
june marked this conversation as resolved
Owner

You didn't update the paths in the readme

You didn't update the paths in the readme
june added 1 commit 2024-12-08 01:35:13 +01:00
reference new "resources" dir and generally fix up and improve README
Some checks failed
/ Ansible Lint (push) Failing after 1m33s
/ Ansible Lint (pull_request) Failing after 1m28s
e7467f0621
june force-pushed cleanup from e7467f0621 to 19abc5f585 2024-12-08 02:55:30 +01:00 Compare
june merged commit 19abc5f585 into main 2024-12-08 16:14:41 +01:00
june deleted branch cleanup 2024-12-08 16:14:41 +01:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 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: CCCHH/ansible-infra#15
No description provided.