Code review comment for lp:~axwalk/juju-core/lp1225825-netlookupip-ip

Revision history for this message
Andrew Wilkins (axwalk) wrote :

https://codereview.appspot.com/13504044/diff/1/environs/manual/provisioner.go
File environs/manual/provisioner.go (right):

https://codereview.appspot.com/13504044/diff/1/environs/manual/provisioner.go#newcode80
environs/manual/provisioner.go:80: logger.Warningf("failed to resolve
%v: %v", ip, err)
On 2013/09/18 03:52:12, thumper wrote:
> Is it really a warning if we can't do a reverse lookup? Is it
something that we
> should be fixing later?

Hmm, yeah, it's not necessarily a problem.

> Perhaps just log with info or debug, certainly not really a big deal
is it?

I'll log it as INFO.

https://codereview.appspot.com/13504044/

« Back to merge proposal