commands: Simplify _lookup_hostname() in ctdb module

[anoopcs9]

          @avanthakkar Why are we not returning IPv6 address from here? What special case are we dealing with additional `ipv6_address`?

Originally posted by @anoopcs9 in #146 (comment)

[avanthakkar]

LGTM

[phlogistonjohn]

I had assumed the way the code was previously was that it was designed to prefer any ipv4 address over an ipv6 address. @avanthakkar is that not the case? - I see you approved already, and so I guess that is not important - but I would like to confirm this clearly.

[anoopcs9]

I had assumed the way the code was previously was that it was designed to prefer any ipv4 address over an ipv6 address.

Ah..I see that this preference is clearly mentioned in the description for #137. I was searching for any indications in commit message and there wasn't any.

So this change is no longer relevant.

[avanthakkar]

I had assumed the way the code was previously was that it was designed to prefer any ipv4 address over an ipv6 address.

Ah..I see that this preference is clearly mentioned in the description for #137. I was searching for any indications in commit message and there wasn't any.

So this change is no longer relevant.

ah yes, my bad! I missed to take this point into consideration