Skip to content
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

Enable templating and inline vaults for inventory plugins #60715

Open
wants to merge 6 commits into
base: devel
from

Conversation

@s-hertel
Copy link
Contributor

commented Aug 17, 2019

SUMMARY

Fixes #58281
Fixes #57012

Based on #58288. I moved the templating and decryption into ensure_type() since now it fails if the value is an inline vault or a templated var for an option that is not a string.

ISSUE TYPE
  • Feature Pull Request
bcoca and others added 6 commits Jun 24, 2019
move templating and vault decrypting to config manager now that mis-t…
…ypes are fatal

in the config manager don't template:
  - non-inventory plugins
  - inventory plugins that haven't set TEMPLATE_OPTIONS to True
  - options that have been blacklisted

allow ansible-inventory to handle extra vars

@sivel sivel removed the needs_triage label Aug 20, 2019

@s-hertel s-hertel changed the title [WIP] enable templating and inline vaults for inventory plugins Enable templating and inline vaults for inventory plugins Aug 22, 2019

@ansibot ansibot added core_review and removed WIP labels Aug 22, 2019

@ansibot ansibot added the stale_ci label Aug 30, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.