with_consul_kv ANSIBLE_CONSUL_URL environment.

[Christopher Warner]

I want to resurrect a currently closed issue https://github.com/ansible/ansible/issues/16398. I'm just trying to understand why with_consul_kv looks for the env variable ANSIBLE_CONSUL_URL and we just don't make this an ansible variable. The problem i'm having is using it with tower as I can't control the environment var from there and because the way this is written it would need to happen in the ansible-playbook exec environment. All work arounds involve either writing my own module or adding the consul agent as a dep. Is there some reason we can't just modify with_consul_kv to look for a predefined var, or at the very least because that would be a breaking change to remove it, add some logic to also check for a predefined var? Anyone else see any problems with this?