1243768 - 404 download links on /android/all/ for certain locales

Status: RESOLVED FIXED

(www.mozilla.org :: Product Details, defect)

Description

[Alex Gibson [:agibson]]

On the /android/all page we have 404 download links for certain locales:

https://www.mozilla.org/en-US/firefox/android/all/

=========================== short test summary info ============================
FAIL tests/functional/test_download_l10n.py::test_localized_download_links[https://download.mozilla.org/?product=fennec-latest&os=android&lang=el]
FAIL tests/functional/test_download_l10n.py::test_localized_download_links[https://download.mozilla.org/?product=fennec-latest&os=android-api-9&lang=el]
FAIL tests/functional/test_download_l10n.py::test_localized_download_links[https://download.mozilla.org/?product=fennec-latest&os=android&lang=nn-NO]
FAIL tests/functional/test_download_l10n.py::test_localized_download_links[https://download.mozilla.org/?product=fennec-latest&os=android-api-9&lang=nn-NO]
FAIL tests/functional/test_download_l10n.py::test_localized_download_links[https://download.mozilla.org/?product=fennec-latest&os=android&lang=sr]
FAIL tests/functional/test_download_l10n.py::test_localized_download_links[https://download.mozilla.org/?product=fennec-latest&os=android-api-9&lang=sr]
FAIL tests/functional/test_download_l10n.py::test_localized_download_links[https://download.mozilla.org/?product=fennec-latest&os=android&lang=vi]
FAIL tests/functional/test_download_l10n.py::test_localized_download_links[https://download.mozilla.org/?product=fennec-latest&os=android-api-9&lang=vi]

I'm not entirely clear on where the issue lies currently, but asking Flod on IRC he could provide the following info that may lead to clues:

"So, we have two list of locales in Android. One is locales that go in the multi-locale apk, those failing links are for single-locale apks (i.e., the 'el' apk contains only Greek, etc.). It means that the build is failing, so there's on apl"

Comment 1

[Kohei Yoshino [:kohei]]

I guess the data from product-details is outdated:
http://viewvc.svn.mozilla.org/vc/libs/product-details/json/mobile_details.json?view=markup

Comment 2

[Josh Mize [:jgmize]]

(In reply to Kohei Yoshino [:kohei] from comment #1)
> I guess the data from product-details is outdated:
> http://viewvc.svn.mozilla.org/vc/libs/product-details/json/mobile_details.
> json?view=markup

Sylvestre will you please take a look at this?

Comment 3

[Sylvestre Ledru [:Sylvestre]]

Sorry about the delay. I think I fixed it in r1243768
(but not sure as it is something new to me)

Comment 4

[Alex Gibson [:agibson]]

Ran the test again and there seems to be one remaining locale that's still 404:

=========================== short test summary info ============================
FAIL tests/functional/test_download_l10n.py::test_localized_download_links[https://download.mozilla.org/?product=fennec-latest&os=android&lang=sr]
FAIL tests/functional/test_download_l10n.py::test_localized_download_links[https://download.mozilla.org/?product=fennec-latest&os=android-api-9&lang=sr]

Sylvestre, can you please take a look? 

Nearly there thanks!

Comment 5

[Sylvestre Ledru [:Sylvestre]]

Oups, sorry, fixed in r150157

Comment 6

[Alex Gibson [:agibson]]

Fixed on prod, thanks again Sylvestre!

============================= test session starts ==============================
platform darwin -- Python 2.7.10, pytest-2.8.5, py-1.4.31, pluggy-0.3.1
rootdir: /Users/agibson/Sites/bedrock, inifile: setup.cfg
plugins: django-2.8.0, html-1.7, rerunfailures-0.05, selenium-1.1, variables-1.2, xdist-1.13.1
collected 94 items 

tests/functional/test_download_l10n.py ..............................................................................................

 generated html file: /Users/agibson/Sites/bedrock/tests/functional/results.html 
========================== 94 passed in 18.32 seconds ==========================