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.
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
As they should and as this is preparation for a future change
reorganizing the file structure.
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.
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
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
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.