-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Use multiple YAML documents for inventory plugin examples #9551
Conversation
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.
FWIW, +1 from me.
IMO this makes inventory examples easier to understand, and silences some of the ansible-lint complaints about duplicate keys.
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.
one tiny typo that was there before, other than that LGTM
Co-authored-by: Alexei Znamensky <[email protected]>
Backport to stable-10: 💚 backport PR created✅ Backport PR branch: Backported as #9559 🤖 @patchback |
* Use multiple YAML documents for inventory plugin examples. * Add ignore.txt entries. * Delete no longer needed ignore.txt files. * Fix typo. Co-authored-by: Alexei Znamensky <[email protected]> --------- Co-authored-by: Alexei Znamensky <[email protected]> (cherry picked from commit 3f7ea60)
…or inventory plugin examples (#9559) Use multiple YAML documents for inventory plugin examples (#9551) * Use multiple YAML documents for inventory plugin examples. * Add ignore.txt entries. * Delete no longer needed ignore.txt files. * Fix typo. Co-authored-by: Alexei Znamensky <[email protected]> --------- Co-authored-by: Alexei Znamensky <[email protected]> (cherry picked from commit 3f7ea60) Co-authored-by: Felix Fontein <[email protected]>
SUMMARY
I thought this still doesn't work, but apparently @tremble fixed it more than a year ago in ansible/ansible@5346009! Thanks a lot for that :)
This likely won't pass CI since ansible-core 2.16 and before won't accept this, I'll add ignore.txt entries in a follow-up commit once I know what exactly fails.
ISSUE TYPE
COMPONENT NAME
inventory plugins