1240182 - Move INFO log lines for established and closed connections to DEBUG level

Status: RESOLVED FIXED

(Remote Protocol :: Marionette, defect)

Description

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Running marionette with the latest logging changes gives me a lot of those lines for INFO level:

1452892967923	Marionette	INFO	Closed connection conn3
1452892967924	Marionette	INFO	Accepted connection conn5 from 127.0.0.1:35844
1452892967926	Marionette	INFO	Accepted connection conn6 from 127.0.0.1:35846
1452892967928	Marionette	INFO	Closed connection conn6

I feel that this is way too much output and should not belong to INFO. This might be perfect for DEBUG instead.

Comment 1

[Andreas Tolfsen ❲:ato❳]

Attachment: MozReview Request: Bug 1240182 - Use Log.Level.Debug for logging accepted and closed sockets; r?automatedtester

Review commit: https://reviewboard.mozilla.org/r/31293/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/31293/

Comment 2

[David Burns :automatedtester]

Comment on attachment 8709136 [details]
MozReview Request: Bug 1240182 - Use Log.Level.Debug for logging accepted and closed sockets; r?automatedtester

https://reviewboard.mozilla.org/r/31293/#review28061

Comment 3

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/8ca79644b089

Comment 4

[Nigel Babu [:nigelb]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/4724e966fc73c605cc4865cfb208d5e98c9319d8
Backed out changeset 8ca79644b089 (bug 1240182) for xpcshell bustage on b2g

Comment 5

[Nigel Babu [:nigelb]]

Backed out for b2g xpcshell bustages like these

https://treeherder.mozilla.org/logviewer.html#?job_id=20014951&repo=mozilla-inbound
https://treeherder.mozilla.org/logviewer.html#?job_id=20014476&repo=mozilla-inbound

Comment 6

[Nigel Babu [:nigelb]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/2007ddbc08c9acf8d032b62c2005b9d65ba8e31b
Bug 1240182 - Use Log.Level.Debug for logging accepted and closed sockets; r=automatedtester

Comment 7

[Nigel Babu [:nigelb]]

Apologies. This bug wasn't the cause of the issues and I've relanded it.

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/2007ddbc08c9