apiclient: don't log at error level when dialing #6926

Merged
merged 1 commit into from Feb 6, 2017

Conversation

Projects
None yet
4 participants
Owner

rogpeppe commented Feb 6, 2017

PR #6620 changed the client logging so that it logs an
error when it can't connect to an API address.
It is common for these errors to occur when connecting
even though the actual connect succeeds, so this
PR changes the logging to debug level - it's part
of the implementation but not something that users
should always see.

This is a backport of #6830 to Juju 2.1, as the errors
are confusing people and 2.2 won't be out for a while.

apiclient: don't log at error level when dialing
PR #6620 changed the client logging so that it logs an
error when it can't connect to an API address.
It is common for these errors to occur when connecting
even though the actual connect succeeds, so this
PR changes the logging to debug level - it's part
of the implementation but not something that users
should always see.

mhilton approved these changes Feb 6, 2017

👍

Owner

jameinel commented Feb 6, 2017

$$merge$$

Contributor

jujubot commented Feb 6, 2017

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

@jujubot jujubot merged commit 20b593c into juju:2.1 Feb 6, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment