977933 - Increase WebRTC logging temporarily to catch intermittent test failures

Status: RESOLVED FIXED

(Core :: WebRTC, defect)

Comment 1

[Nils Ohlmeier [:drno]]

https://tbpl.mozilla.org/?tree=Try&rev=3b9cb875944a

Comment 2

[Nils Ohlmeier [:drno]]

Attachment: increase_webrtc_test_log_level.patch

Comment 3

[Randell Jesup [:jesup] (needinfo me)]

Comment on attachment 8383430 [details] [diff] [review]
increase_webrtc_test_log_level.patch

Review of attachment 8383430 [details] [diff] [review]:
-----------------------------------------------------------------

Please run a Try before this lands to make sure the logging a) works, and b) doesn't generate so much logging as to choke tbpl/etc

Comment 4

[Randell Jesup [:jesup] (needinfo me)]

I see the Try is already running.  never mind :-)

Comment 5

[Nils Ohlmeier [:drno]]

I'm looking at the failures in test_dataChannel_basicAudioVideo.html...

Comment 6

[Nils Ohlmeier [:drno]]

Attachment: increase_webrtc_test_log_level.patch

Try again with lower log level to avoid log truncation.

Comment 7

[Nils Ohlmeier [:drno]]

Try: https://tbpl.mozilla.org/?tree=Try&rev=76134fb0833e

Comment 8

[Nils Ohlmeier [:drno]]

Either we just got lucky or it works better with log level 6.

Comment 9

[Nils Ohlmeier [:drno]]

I don't see any truncated log files any more. So I believe we are good.

Comment 10

[Nils Ohlmeier [:drno]]

Comment on attachment 8384780 [details] [diff] [review]
increase_webrtc_test_log_level.patch

Carrying forward r+=jesup

Comment 11

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/5c960b9a5273

Comment 12

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/5c960b9a5273