All users were logged out of Bugzilla on October 13th, 2018

Re-enable test_receive_call.py

RESOLVED FIXED

Status

RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: zcampbell, Assigned: bsilverberg)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
Re enable this test
I will try this one
(Assignee)

Updated

5 years ago
Depends on: 924835
(Assignee)

Comment 2

5 years ago
Thank you for your enthusiasm, Viorela, but I think this task is being blocked by bug 924831 and 924835, so it's not quite ready to be worked on.
(Assignee)

Comment 3

5 years ago
Created attachment 8396551 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/17593
Attachment #8396551 - Flags: review?(zcampbell)
Attachment #8396551 - Flags: review?(viorela.ioia)
Attachment #8396551 - Flags: review?(robert.chira)
Attachment #8396551 - Flags: review?(florin.strugariu)
Attachment #8396551 - Flags: review?(andrei.hutusoru)
(Assignee)

Updated

5 years ago
QA Contact: bob.silverberg
(Assignee)

Updated

5 years ago
Assignee: nobody → bob.silverberg
QA Contact: bob.silverberg
Comment on attachment 8396551 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/17593

The test failed once
Attachment #8396551 - Flags: review?(viorela.ioia) → review-
Attachment #8396551 - Flags: review?(robert.chira) → review-
Test started failing after running multiple times. See comment in PR.
(Assignee)

Comment 6

5 years ago
Comment on attachment 8396551 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/17593

This is ready to be re-reviewed.
Attachment #8396551 - Flags: review?(viorela.ioia)
Attachment #8396551 - Flags: review?(robert.chira)
Attachment #8396551 - Flags: review-
(Assignee)

Comment 7

5 years ago
Comment on attachment 8396551 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/17593

If you have time, davehunt, a review of this would be appreciated.
Attachment #8396551 - Flags: review?(dave.hunt)
Attachment #8396551 - Flags: review?(viorela.ioia) → review+
Comment on attachment 8396551 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/17593

Tested this and looks OK
Attachment #8396551 - Flags: review?(florin.strugariu) → review+
Attachment #8396551 - Flags: review?(robert.chira) → review+
(Assignee)

Comment 9

5 years ago
This test is doing well as an adhoc on CI, but still needs a review from Zac and/or Dave to be ready to merge.

Updated

5 years ago
Attachment #8396551 - Flags: review?(andrei.hutusoru)
Comment on attachment 8396551 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/17593

Thanks Bob, sorry for the delay. I'm going to r- for the comments mentioned in the pull request. If there's an urgent need to get this landed then I don't see any real blockers, but if we have time then I'd suggest taking a little longer on this.
Attachment #8396551 - Flags: review?(dave.hunt) → review-
(Assignee)

Updated

5 years ago
Attachment #8396551 - Flags: review- → review?(dave.hunt)
Comment on attachment 8396551 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/17593

I added a few comments that I think would be good to fix up, so r+ with these changes. Note that I haven't run the tests myself though, and I did notice that Travis failed.
Attachment #8396551 - Flags: review?(dave.hunt) → review+
(Assignee)

Updated

5 years ago
Attachment #8396551 - Flags: review+ → review?(dave.hunt)
Comment on attachment 8396551 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/17593

Looks great, thanks Bob! :-)
Attachment #8396551 - Flags: review?(dave.hunt) → review+
(Reporter)

Comment 13

5 years ago
Comment on attachment 8396551 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/17593

r+!!
Attachment #8396551 - Flags: review?(zcampbell) → review+
(Reporter)

Comment 14

5 years ago
Merged, yay!

It's a great test:
https://github.com/mozilla-b2g/gaia/commit/29e9731148ef3960485d2e4e232de5ef09cd4832
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Reverted for:
https://tbpl.mozilla.org/php/getParsedLog.php?id=38255056&tree=B2g-Inbound
{
09:09:49     INFO - Installing gaia-ui-tests into virtualenv /builds/slave/test/build/venv
09:09:49     INFO - Running command: ['/builds/slave/test/build/venv/bin/pip', 'install', '--download-cache', '/builds/slave/test/build/venv/cache', '-r', '/builds/slave/test/gaia/tests/python/gaia-ui-tests/tbpl_requirements.txt', '--no-index', '--find-links', 'http://pypi.pvt.build.mozilla.org/pub', '--find-links', 'http://pypi.pub.build.mozilla.org/pub', '-e', '/builds/slave/test/gaia/tests/python/gaia-ui-tests'] in /builds/slave/test/build
09:09:49     INFO - Copy/paste: /builds/slave/test/build/venv/bin/pip install --download-cache /builds/slave/test/build/venv/cache -r /builds/slave/test/gaia/tests/python/gaia-ui-tests/tbpl_requirements.txt --no-index --find-links http://pypi.pvt.build.mozilla.org/pub --find-links http://pypi.pub.build.mozilla.org/pub -e /builds/slave/test/gaia/tests/python/gaia-ui-tests
09:09:50     INFO -  Ignoring indexes: http://pypi.python.org/simple/
09:09:50     INFO -  Obtaining file:///builds/slave/test/gaia/tests/python/gaia-ui-tests
09:09:50     INFO -    Running setup.py egg_info for package from file:///builds/slave/test/gaia/tests/python/gaia-ui-tests
09:09:50     INFO -  Requirement already satisfied (use --upgrade to upgrade): marionette-client in ./venv/lib/python2.7/site-packages (from -r /builds/slave/test/gaia/tests/python/gaia-ui-tests/tbpl_requirements.txt (line 1))
09:09:50     INFO -  Requirement already satisfied (use --upgrade to upgrade): mozdevice>=0.31 in ./venv/lib/python2.7/site-packages (from gaiatest)
09:09:50     INFO -  Requirement already satisfied (use --upgrade to upgrade): moztest>=0.3 in ./venv/lib/python2.7/site-packages (from gaiatest)
09:09:50     INFO -  Downloading/unpacking plivo==0.9.6 (from gaiatest)
09:09:50     INFO -    Could not find any downloads that satisfy the requirement plivo==0.9.6 (from gaiatest)
09:09:50     INFO -  No distributions at all found for plivo==0.9.6 (from gaiatest)
09:09:50     INFO -  Storing complete log in /home/cltbld/.pip/pip.log
09:09:50    ERROR - Return code: 1
09:09:50    FATAL - Error running install of package, /builds/slave/test/build/venv/bin/pip install --download-cache /builds/slave/test/build/venv/cache -r /builds/slave/test/gaia/tests/python/gaia-ui-tests/tbpl_requirements.txt --no-index --find-links http://pypi.pvt.build.mozilla.org/pub --find-links http://pypi.pub.build.mozilla.org/pub -e /builds/slave/test/gaia/tests/python/gaia-ui-tests!
}

Please make sure the package is on the mirror before relanding, or else tweak the patch so TBPL jobs don't require it.

Revert commit:
https://github.com/mozilla-b2g/gaia/commit/381009095bc7d2b9aea26a641f5e8df7c8728efc
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Reporter)

Updated

5 years ago
Depends on: 1000133
(Reporter)

Comment 16

5 years ago
Re-landed, yay!

https://github.com/mozilla-b2g/gaia/commit/4b6c9d4929c57005ab142e8eb3dc6783d55c427c
Status: REOPENED → RESOLVED
Last Resolved: 5 years ago5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.