Closed
Bug 1003652
Opened 10 years ago
Closed 10 years ago
Always turn on ril debugging in telephony marionette test
Categories
(Firefox OS Graveyard :: RIL, defect)
Tracking
(feature-b2g:2.0, tracking-b2g:backlog)
RESOLVED
FIXED
2.0 S2 (23may)
People
(Reporter: aknow, Assigned: aknow)
References
Details
(Whiteboard: [p=1])
Attachments
(1 file, 1 obsolete file)
2.40 KB,
patch
|
aknow
:
review+
|
Details | Diff | Splinter Review |
Recently, we encounter several time out in telephony marionette tests executed on try server. However, it's difficult to dig into the problem without the ril debugging log. So I would like to always turn on the debugging in those test cases. Will it cause any problem?
Assignee | ||
Comment 1•10 years ago
|
||
Achieve by altering the pref value during tests.
Attachment #8414980 -
Flags: review?(htsai)
Comment 2•10 years ago
|
||
Comment on attachment 8414980 [details] [diff] [review] Turn on debugging in marionette test Review of attachment 8414980 [details] [diff] [review]: ----------------------------------------------------------------- I think Jonathan is a properer reviewer.
Attachment #8414980 -
Flags: review?(htsai) → review?(jgriffin)
Assignee | ||
Comment 3•10 years ago
|
||
(In reply to Hsin-Yi Tsai [:hsinyi] from comment #2) > Comment on attachment 8414980 [details] [diff] [review] > Turn on debugging in marionette test > > Review of attachment 8414980 [details] [diff] [review]: > ----------------------------------------------------------------- > > I think Jonathan is a properer reviewer. ... ? The modification is done in telephony's head.js
Comment 4•10 years ago
|
||
(In reply to Szu-Yu Chen [:aknow] from comment #3) > (In reply to Hsin-Yi Tsai [:hsinyi] from comment #2) > > Comment on attachment 8414980 [details] [diff] [review] > > Turn on debugging in marionette test > > > > Review of attachment 8414980 [details] [diff] [review]: > > ----------------------------------------------------------------- > > > > I think Jonathan is a properer reviewer. > > ... ? > The modification is done in telephony's head.js Aknow, sorry for not explaining myself. The change itself is fine for me, but I am not sure how it would afect the whole marrionet process. Vicamo told me that marionette process has been killed due to a large logging message. Is the situation still? This pref not only works for TelephonyProvider module but also for ril_worker, rilcontenthelper , data connection ... the message size is obvious. That's why I think I would like to have Jonathan's review. The other thing is turning on debugging might longer the test time. We shall be more careful to not lead to more timeout possibilities. Thank you.
Comment 5•10 years ago
|
||
According to timeout, we could solve it by splitting the cases or extand the time, so not that serious. Just to make sure we have enough tests on try before landing.
Comment 6•10 years ago
|
||
Comment on attachment 8414980 [details] [diff] [review] Turn on debugging in marionette test Review of attachment 8414980 [details] [diff] [review]: ----------------------------------------------------------------- Looks good to me, but I still like to see if Jonathan is okay with this (see comment 4), thanks.
Attachment #8414980 -
Flags: review+
Assignee | ||
Comment 7•10 years ago
|
||
(In reply to Hsin-Yi Tsai [:hsinyi] from comment #5) > According to timeout, we could solve it by splitting the cases or extand the > time, so not that serious. Just to make sure we have enough tests on try > before landing. It's time out, but not means the running time is not enough. What I found is that some events are missing and it cause our test script stuck at somewhere. We can not dig into the problem without the log. All we can do is to enable the log and retest it on the try server. It wastes the time and is not guaranteed reproducible.
Comment 8•10 years ago
|
||
(In reply to Szu-Yu Chen [:aknow] from comment #7) > (In reply to Hsin-Yi Tsai [:hsinyi] from comment #5) > > According to timeout, we could solve it by splitting the cases or extand the > > time, so not that serious. Just to make sure we have enough tests on try > > before landing. > > It's time out, but not means the running time is not enough. > > What I found is that some events are missing and it cause our test script > stuck at somewhere. We can not dig into the problem without the log. All we > can do is to enable the log and retest it on the try server. It wastes the > time and is not guaranteed reproducible. Totally agree!
Comment 9•10 years ago
|
||
Comment on attachment 8414980 [details] [diff] [review] Turn on debugging in marionette test Review of attachment 8414980 [details] [diff] [review]: ----------------------------------------------------------------- lgtm; I don't think this should cause any problems for our test infrastructure
Attachment #8414980 -
Flags: review?(jgriffin) → review+
Assignee | ||
Updated•10 years ago
|
Whiteboard: [p=1]
Target Milestone: --- → 2.0 S2 (23may)
Assignee | ||
Comment 10•10 years ago
|
||
Attachment #8414980 -
Attachment is obsolete: true
Attachment #8423032 -
Flags: review+
Assignee | ||
Comment 11•10 years ago
|
||
https://tbpl.mozilla.org/?tree=Try&rev=f4b175787a10
Keywords: checkin-needed
Comment 12•10 years ago
|
||
https://hg.mozilla.org/integration/b2g-inbound/rev/219cb1047f3b
Keywords: checkin-needed
Comment 13•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/219cb1047f3b
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Updated•10 years ago
|
blocking-b2g: --- → backlog
feature-b2g: --- → 2.0
Updated•9 years ago
|
blocking-b2g: backlog → ---
tracking-b2g:
--- → backlog
You need to log in
before you can comment on or make changes to this bug.
Description
•