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
https:/ /codereview. appspot. com/13504044/ diff/1/ environs/ manual/ provisioner. go manual/ provisioner. go (right):
File environs/
https:/ /codereview. appspot. com/13504044/ diff/1/ environs/ manual/ provisioner. go#newcode80 manual/ provisioner. go:80: logger. Warningf( "failed to resolve
environs/
%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/