inventory/cleanup #144

Open
djerun wants to merge 3 commits from inventory/cleanup into main
Owner
  • manage ssh connectivity in shared ssh.cfg usable outside of ansible with optional local overrides in ssh.cfg.local
  • use ssh.cfg in ansible and eliminate ssh config bleed from ~/.ssh/config
  • collect host key fingerprints in shared known_hosts file populated with fingerprints of reachable hosts
  • add missing user for chaosknoten
  • sort hosts in inventories
- manage ssh connectivity in shared ssh.cfg usable outside of ansible with optional local overrides in ssh.cfg.local - use ssh.cfg in ansible and eliminate ssh config bleed from ~/.ssh/config - collect host key fingerprints in shared known_hosts file populated with fingerprints of reachable hosts - add missing user for chaosknoten - sort hosts in inventories
[inventory] specify ssh user for chaosknoten
All checks were successful
/ Ansible Lint (push) Successful in 2m30s
c610e8fb3e
[inventory] sort hosts
All checks were successful
/ Ansible Lint (push) Successful in 2m34s
/ build (pull_request) Successful in 28s
/ Ansible Lint (pull_request) Successful in 2m26s
1d6c131f79
You can view your changes at https://staging.infra-docs.hamburg.ccc.de/pr144/
june requested review from june 2026-09-22 02:14:26 +02:00
@ -7,1 +7,4 @@
interpreter_python = python3
[ssh_connection]
ssh_args = -F ssh.cfg -o StrictHostKeyChecking=accept-new -o UserKnownHostsFile=known_hosts
Owner

Setting -F fully changed the configuration and ignores the ~/.ssh/config, right? While I see the benefit of preventing config bleed, this would then also ignore custom global SSH configuration, which one would then need to replicate in the ssh.cfg.local, which just introduces another quirk to worry about when using this repo.

Setting `-F` fully changed the configuration and ignores the `~/.ssh/config`, right? While I see the benefit of preventing config bleed, this would then also ignore custom global SSH configuration, which one would then need to replicate in the `ssh.cfg.local`, which just introduces another quirk to worry about when using this repo.
Author
Owner

you can also just import your global config from ssh.cfg.local if that has no unwanted side effects

you can also just import your global config from ssh.cfg.local if that has no unwanted side effects
stb approved these changes 2026-10-08 11:26:29 +02:00
stb left a comment

Very nice, I like it!

Very nice, I like it!
Owner

For the commit I would suggest something like "Consolidate SSH config" as the headline

For the commit I would suggest something like "Consolidate SSH config" as the headline
All checks were successful
/ Ansible Lint (push) Successful in 2m34s
Required
Details
/ build (pull_request) Successful in 28s
Required
Details
/ Ansible Lint (pull_request) Successful in 2m26s
Required
Details
This pull request has changes conflicting with the target branch.
  • inventories/chaosknoten/hosts.yaml
  • inventories/z9/hosts.yaml
View command line instructions

Manual merge helper

Use this merge commit message when completing the merge manually.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin inventory/cleanup:inventory/cleanup
git switch inventory/cleanup
Sign in to join this conversation.
No description provided.