/juju

Better log message in case of TLS error #7416

Merged
merged 1 commit into from May 30, 2017

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@wupeka @rogpeppe @jujubot
@wupeka
Member

wupeka commented May 30, 2017

Description of change

Follow up from #7414 - lower error message level.

QA steps

Check debug level log messages on TestUpdateCert unit test.
@rogpeppe
Owner

rogpeppe commented May 30, 2017

LGTM, thanks
@wupeka
Member

wupeka commented May 30, 2017

$$merge$$
@jujubot
Contributor

jujubot commented May 30, 2017

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

jujubot commented May 30, 2017

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/11028
@wupeka
Member

wupeka commented May 30, 2017

$$merge$$
@jujubot
Contributor

jujubot commented May 30, 2017

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

@jujubot jujubot merged commit 19c2b32 into juju:develop May 30, 2017
1 check failed

1 check failed

github-check-merge-juju Built PR, ran unit tests, and tested LXD deploy. Use !!.*!! to request another build. IE, !!build!!, !!retry!!
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment