fix datacenter value in case where the datacenter is defined in the conf file

[jkhelil]

ISSUE TYPE

• Bugfix Pull Request

COMPONENT NAME

contrib/inventory/consul_io.py

ANSIBLE VERSION

ansible 2.2.0.0
  config file = /etc/ansible/ansible.cfg
  configured module search path = Default w/o overrides

SUMMARY

When datacenter is defined in the consul.ini conf file, kv groups are not retrieved for nodes, we should set self.current_dc with the right value to fix it

[jctanner]

@sgargan can you review please?

[jctanner]

@johnnycarlin @maguec are either of you able to review this?

[sgargan]

looks good shipit

[mliljedahl]

Think it is enough to set self.current_dc on line 227:

if config.has_config('datacenter'):
self.current_dc = config.datacenter
if config.has_config('host'):
......

But your implementation works, so shipit :)

[calfonso]

@jkhelil would you mind rebasing and we'll get this merged, thanks!

[Akasurde]

@jkhelil Can you please rebase ?

[gundalow]

@jkhelil Thank you for your PR.
As part of reviewing the backlog of PRs we are looking at PRs older PRs that haven't been update

Given that:

• Last commits were a long time ago
• Rebase hasn't been done

I'm going to close this.
If you, or anyone else wants to continue with this work then please do feel free to create a fresh PR and @mention the previous reviewers in here.