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

Update consul_kv.py add token to lookup get kv #57341

Open
wants to merge 2 commits into
base: devel
from

Conversation

Projects
None yet
2 participants
@chris93111
Copy link
Contributor

commented Jun 4, 2019

add token for lookup get kv

SUMMARY

Token is not apply with lookup to get kv and needed to target host

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

consul_kv.py

ADDITIONAL INFORMATION

Error was rpc error making call: Permission denied"

{{ lookup('consul_kv', '{{item}}',token='xxxx-zzzz-xxxx-zzzz', host='xx.xx.xx.xx', port='8500') }}

Update consul_kv.py
add token for lookup get kv
@ansibot

This comment has been minimized.

Copy link
Contributor

commented Jun 4, 2019

The test ansible-test sanity --test pylint [explain] failed with 1 error:

lib/ansible/plugins/lookup/consul_kv.py:146:29: bad-whitespace Exactly one space required after assignment                     tokenapi =  kwargs.get('token', None)                              ^

The test ansible-test sanity --test pep8 [explain] failed with 1 error:

lib/ansible/plugins/lookup/consul_kv.py:146:31: E222 multiple spaces after operator

click here for bot help

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