-
Notifications
You must be signed in to change notification settings - Fork 1.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix typo disable_looups in inventory/proxmox #5640
fix typo disable_looups in inventory/proxmox #5640
Conversation
- resolve issue with lookups in proxmox inventory config
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your contribution! Can you please add a changelog fragment? Thanks!
Docs Build 📝Thank you for contribution!✨ This PR has been merged and your docs changes will be incorporated when they are next published. |
Co-authored-by: Felix Fontein <felix@fontein.de>
Backport to stable-5: 💔 cherry-picking failed — conflicts found❌ Failed to cleanly apply fd436bd on top of patchback/backports/stable-5/fd436bdbc2917365580851a9a0cf940875633ea1/pr-5640 Backporting merged PR #5640 into main
🤖 @patchback |
Backport to stable-6: 💚 backport PR created✅ Backport PR branch: Backported as #5644 🤖 @patchback |
@tbjers thanks for fixing this! |
* fix typo disable_looups in inventory/proxmox - resolve issue with lookups in proxmox inventory config * add changelog fragment * Update changelogs/fragments/5640-fix-typo-proxmox-inventory.yml Co-authored-by: Felix Fontein <felix@fontein.de> Co-authored-by: Felix Fontein <felix@fontein.de> (cherry picked from commit fd436bd)
…entory/proxmox (#5644) fix typo disable_looups in inventory/proxmox (#5640) * fix typo disable_looups in inventory/proxmox - resolve issue with lookups in proxmox inventory config * add changelog fragment * Update changelogs/fragments/5640-fix-typo-proxmox-inventory.yml Co-authored-by: Felix Fontein <felix@fontein.de> Co-authored-by: Felix Fontein <felix@fontein.de> (cherry picked from commit fd436bd) Co-authored-by: Torgny Bjers <torgny@bjers.org>
Thank you so much for merging so quickly! |
SUMMARY
Resolve issue with lookups in proxmox inventory config.
ISSUE TYPE
COMPONENT NAME
proxmox
ADDITIONAL INFORMATION
Once typo is fixed, it correctly lists hosts.