Skip to content

fix: Use ansible_facts[] syntax for fact references#57

Merged
john-derose merged 1 commit intomasterfrom
fix/ansible-facts-syntax
Mar 1, 2026
Merged

fix: Use ansible_facts[] syntax for fact references#57
john-derose merged 1 commit intomasterfrom
fix/ansible-facts-syntax

Conversation

@homestak-bot
Copy link
Contributor

Summary

  • Replace deprecated top-level fact variables (ansible_distribution, ansible_distribution_release, etc.) with ansible_facts['...'] syntax
  • Ansible-core 2.20+ treats top-level fact injection as an error, breaking pve-setup on fresh hosts

Files changed

  • proxmox/roles/install/tasks/kernel.yml — assert conditions
  • proxmox/roles/install/tasks/packages.yml — repo URL and when condition
  • debian/roles/iac_tools/tasks/main.yml — HashiCorp repo URL

Test plan

  • homestak pve-setup succeeds on mother (Debian 13, ansible-core 2.20.3)

🤖 Generated with Claude Code

Ansible-core 2.20+ treats top-level fact injection (ansible_distribution,
ansible_distribution_release, etc.) as deprecated and errors out. Switch
to ansible_facts['...'] syntax in install and iac_tools roles.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@john-derose john-derose enabled auto-merge (squash) March 1, 2026 17:24
@john-derose john-derose merged commit 74e9a8a into master Mar 1, 2026
1 check failed
@john-derose john-derose deleted the fix/ansible-facts-syntax branch March 1, 2026 17:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants