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

added unsafe toggle to vars_prompt #49219

Open
wants to merge 2 commits into
base: devel
from

Conversation

Projects
None yet
4 participants
@bcoca
Member

bcoca commented Nov 27, 2018

fixes #47534

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

playbook/vars_prompt

@ansibot

This comment has been minimized.

Contributor

ansibot commented Nov 27, 2018

Hi @bcoca, thank you for submitting this pull-request!

click here for bot help

@ansibot

This comment has been minimized.

Contributor

ansibot commented Nov 27, 2018

@Xaroth

Xaroth approved these changes Nov 28, 2018

@abadger

This comment has been minimized.

Member

abadger commented Nov 28, 2018

I think we should call this no_template, raw, or similar rather than unsafe. But otherwise, the idea seems sound.

@ansibot ansibot removed the needs_triage label Nov 28, 2018

@bcoca

This comment has been minimized.

Member

bcoca commented Nov 28, 2018

I used 'unsafe' to mirror the existing !unsafe YAML tag we already use for vars declarations

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment