Closed Bug 1207548 Opened 9 years ago Closed 9 years ago

Mochitest xulrunner-sdk-40.zip doesn't contain ssltunnel

Categories

(Testing :: Mochitest, defect, P1)

defect

Tracking

(firefox44 fixed)

RESOLVED FIXED
mozilla44
Tracking Status
firefox44 --- fixed

People

(Reporter: jdai, Assigned: jdai)

References

Details

Attachments

(2 files)

How to update a newer xulrunner-sdk-40.zip which contains ssltunnel file? Since I don't know how to update a new xulrunner-sdk-40.zip file to testcluster, I revert back to ICS Emulator xulrunner.zip file which is not working.

Try link:
https://treeherder.allizom.org/#/jobs?repo=try&revision=01002f47b0d2
Flags: needinfo?(jgriffin)
Flags: needinfo?(garndt)
There are instructions on how to do this, here:

https://wiki.mozilla.org/ReleaseEngineering/Buildduty/Other_Duties#How_to_upload_new_xre.zip_files_for_B2G_tests

The script will need some updates to work with current paths to build artifacts, but it should be relatively straightforward, I hope.  Let me know if you run into problems.
Flags: needinfo?(jgriffin)
After I add ssltunnel into xulrunner-sdk-40.zip[1], TC will not get the same error[2], but it comes out with a new error.

Hi :ahal,
Could you give me some help on this?

[1] https://treeherder.allizom.org/#/jobs?repo=try&revision=1b712b3f5ee2
[2] ERROR INFO | runtests.py | expected to find ssltunnel at /home/worker/build/xre/bin/ssltunnel

=====
06:31:55     INFO -  MochitestServer : launching [u'/home/worker/build/xre/bin/xpcshell', '-g', '/home/worker/build/xre/bin', '-v', '170', '-f', '/home/worker/build/xre/bin/components/httpd.js', '-e', "const _PROFILE_PATH = '/tmp/tmpnOlzv8'; const _SERVER_PORT = '8888'; const _SERVER_ADDR = '10.0.2.2'; const _TEST_PREFIX = undefined; const _DISPLAY_RESULTS = false;", '-f', '/home/worker/build/tests/mochitest/server.js']
06:31:55     INFO -  runtests.py | Server pid: 433
06:31:55     INFO -  runtests.py | Websocket server pid: 436
06:31:55     INFO -  runtests.py | SSL tunnel pid: 441
06:33:25  WARNING -  TEST-UNEXPECTED-FAIL | runtests.py | Timed out while waiting for server startup.
06:35:32     INFO -  Traceback (most recent call last):
06:35:32     INFO -    File "runtestsb2g.py", line 198, in run_tests
06:35:32     INFO -      self.startServers(options, None)
06:35:32     INFO -    File "runtestsb2g.py", line 374, in startServers
06:35:32     INFO -      MochitestUtilsMixin.startServers(self, options, debuggerInfo)
06:35:32     INFO -    File "/home/worker/build/tests/mochitest/runtests.py", line 824, in startServers
06:35:32     INFO -      self.server.ensureReady(self.SERVER_STARTUP_TIMEOUT)
06:35:32     INFO -    File "/home/worker/build/tests/mochitest/runtests.py", line 427, in ensureReady
06:35:32     INFO -      sys.exit(1)
06:35:32     INFO -  SystemExit: 1
06:35:32     INFO -  0 ERROR Automation Error: Received unexpected exception while running application
06:35:33     INFO -  Stopping web server
06:37:40     INFO -  Stopping web socket server
06:37:40     INFO -  Stopping ssltunnel
06:37:40     INFO -  runtestsb2g.py | Running tests: end.
06:37:43     INFO -  SUITE-END | took 515s
06:37:43    ERROR - Return code: 1
=====
Flags: needinfo?(ahalberstadt)
I find out the solution which is xulrunner-sdk-40.zip miss server.js to start server.

Try link:
https://treeherder.allizom.org/#/jobs?repo=try&revision=867703ec3278
Flags: needinfo?(ahalberstadt)
Attached patch WIP, patch, v1Splinter Review
Here is the WIP patch:
1.Update a new xulrunner-sdk-40.zip[1] which I don't have permission to push file to taskcluster artifact.
2.Use no-read-buildbot-config to get rid of read-buildbot-config step.

Try link:
https://treeherder.allizom.org/#/jobs?repo=try&revision=867703ec3278

[1] http://people.mozilla.org/~jdai/xulrunner-sdk-40.zip
Assignee: nobody → jdai
Attachment #8665429 - Flags: review?(wcosta)
Attachment #8665429 - Flags: review?(wcosta) → feedback?(wcosta)
(In reply to John Dai[:johnz][:jdai] from comment #1)
> How to update a newer xulrunner-sdk-40.zip which contains ssltunnel file?
> Since I don't know how to update a new xulrunner-sdk-40.zip file to
> testcluster, I revert back to ICS Emulator xulrunner.zip file which is not
> working.
> 
> Try link:
> https://treeherder.allizom.org/#/jobs?repo=try&revision=01002f47b0d2

Hrm, it looks like James took care of this before.  Basically he just created a taskcluster artifact for that file that we then reference the task definition linked to above.

That URL could be replaced with:
https://queue.taskcluster.net/v1/task/YNNF94qRSyCXiSU_AQgFOQ/runs/0/artifacts/public/cache/xulrunner-sdk-40.zip

I just created a task that uploaded the file on people.mozilla.org as a taskcluster artifact.
Flags: needinfo?(garndt)
Comment on attachment 8665429 [details] [diff] [review]
WIP, patch, v1

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

I will give f+, but I don't much experience on emulator tests.
Attachment #8665429 - Flags: feedback?(wcosta) → feedback+
Bug 1207548 - Update new xulrunner-sdk-40.zip with ssltunnel file. r?garndt, wcosta
Attachment #8665768 - Flags: review?(wcosta)
Attachment #8665768 - Flags: review?(garndt)
Comment on attachment 8665768 [details]
MozReview Request: Bug 1207548 - Update new xulrunner-sdk-40.zip with ssltunnel file. r?garndt, wcosta

https://reviewboard.mozilla.org/r/20325/#review18311

Looks good as long as we can get this pushed to try and make sure everything is green before landing this.

::: testing/taskcluster/tasks/tests/b2g_emulator_mochitest.yml:16
(Diff revision 1)
> +        --no-read-buildbot-config

I'm not entirely sure what this line does (I thought that it stopped mozharness from trying to read the build properties file and complaining when it's not found, but maybe I'm mistaken.
Attachment #8665768 - Flags: review?(garndt) → review+
Attachment #8665768 - Flags: review?(wcosta) → review+
Comment on attachment 8665768 [details]
MozReview Request: Bug 1207548 - Update new xulrunner-sdk-40.zip with ssltunnel file. r?garndt, wcosta

https://reviewboard.mozilla.org/r/20325/#review18323

+1 Greg comment
Priority: -- → P1
 B2G KK Emulator x86 opt  seems very broken in this try run. is this expected ?
Flags: needinfo?(jdai)
(In reply to Carsten Book [:Tomcat] from comment #13)
>  B2G KK Emulator x86 opt  seems very broken in this try run. is this
> expected ?

Yes, B2G KK Emulator x86 opt has other issues, it will fix in other bug, this bug provides correct xulrunner.zip for TC.
Flags: needinfo?(jdai)
(In reply to John Dai[:johnz][:jdai] from comment #14)
> (In reply to Carsten Book [:Tomcat] from comment #13)
> >  B2G KK Emulator x86 opt  seems very broken in this try run. is this
> > expected ?
> 
> Yes, B2G KK Emulator x86 opt has other issues, it will fix in other bug,
> this bug provides correct xulrunner.zip for TC.

Just to second John - This bug aims at fixing the missing ssltunnel issue so that mochitest could be *launched* successfully. From the latest try link, the error ||INFO - 0 ERROR INFO | runtests.py | expected to find ssltunnel at /home/worker/build/xre/bin/ssltunnel"|| is gone. That is, ssltunnel is detected as expected.
After I apply bug 1210370, emulator-x86 mochitest can pass most of tests.

Try link:
https://treeherder.allizom.org/#/jobs?repo=try&revision=e6dd43dda7cc&filter-searchStr=x86
Flags: needinfo?(cbook)
done!
Flags: needinfo?(cbook)
This bug is nothing relative with bug 1204800, please help to land it.
Flags: needinfo?(jdai) → needinfo?(wkocher)
See Also: → 1212243
Flags: needinfo?(cbook)
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/7ba3263d5483
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
Blocks: 1239261
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: