If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

[gaia] Need to add APN settings of CDMA carriers to the database for QA testing

RESOLVED FIXED

Status

Firefox OS
Gaia
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: arthurcc, Assigned: arthurcc)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Need to add APN settings of CDMA carriers to the database. For example Sprint in the US and in Asia Pacific Telecom in Taiwan.
Blocks: 906596
Unblock 906596 since this is not a gaia bug.
No longer blocks: 906596
Created attachment 792713 [details]
link to https://github.com/mozilla-b2g/gaia/pull/11627

The added settings was found on the internet. I am not able to test these settings because MMS is not supported in v1.2 and data connection is still not working for now [1].

[1] Bug 906587 - [Wasabi] Reference phone cannot camp on EVDO network.
Attachment #792713 - Flags: feedback?(josea.olivera)
Comment on attachment 792713 [details]
link to https://github.com/mozilla-b2g/gaia/pull/11627

Switching to Kaze since Antonio is on PTO.
Attachment #792713 - Flags: feedback?(josea.olivera) → review?(kaze)

Comment 4

4 years ago
Hi Arthur,

Actually only EVDO network is not working, you can still using 1x data call with APTG UIM card on wasabi. FYI.
Assignee: nobody → arthur.chen
Some feedback here with Kaze's permission. We have never added anything to the database (e.g. apns_conf.xml) files used to build our apn.json database so I recommend to continue doing the same for APN settings for CDMA carriers. Arthur, you are adding APN sets in those files. I'd recommend to add a new database file for CDMA carriers as CyanogenMod project does (see [1]).

[1] https://github.com/CyanogenMod/android_vendor_cm/blob/ics/prebuilt/common/etc/apns-conf-cdma.xml
Jose, thanks! Patch was updated based on the comments.
Comment on attachment 792713 [details]
link to https://github.com/mozilla-b2g/gaia/pull/11627

Jose, are you able to review this? This could save lots of time of QA inputing the  APN settings every time they reflash builds.
Attachment #792713 - Flags: review?(kaze) → review?(josea.olivera)
Comment on attachment 792713 [details]
link to https://github.com/mozilla-b2g/gaia/pull/11627

Kaze, I had a wrong impression of that Jose is able to review the patch. Could you help review the change? Thanks!
Attachment #792713 - Flags: review?(josea.olivera) → review?(kaze)
Comment on attachment 792713 [details]
link to https://github.com/mozilla-b2g/gaia/pull/11627

FTR, Jose is the most qualified developer to review APN patches.

r=me, but pinging Jose in case he has something to say before we merge this.
Attachment #792713 - Flags: review?(kaze)
Attachment #792713 - Flags: review+
Attachment #792713 - Flags: feedback?(josea.olivera)
Comment on attachment 792713 [details]
link to https://github.com/mozilla-b2g/gaia/pull/11627

Just noticed the APN database for the CDMA carriers in the PR is not the one I commented about in comment #5. We use open source APN databases and IMHO we should do the same for APN settings for CDMA carriers. If the CDMA carrier we want to test with is not in the APN database from comment #5 the best we can do is to add those APN settings to the apns_conf_local.xml database.
Attachment #792713 - Flags: feedback?(josea.olivera) → feedback-
It seems that I misunderstood comment #5. I thought we need to create our own database for cdma. I'll update the patch to move the newly added APN settings to apns_conf_local.xml.
Comment on attachment 792713 [details]
link to https://github.com/mozilla-b2g/gaia/pull/11627

Antonio, I've updated the patch. Would you mind take a look at it again? Thanks!
Attachment #792713 - Flags: feedback- → feedback?(josea.olivera)
Comment on attachment 792713 [details]
link to https://github.com/mozilla-b2g/gaia/pull/11627

Arthur, I left a comment in github. would you mind to take a look please? Thanks.
Attachment #792713 - Flags: feedback?(josea.olivera)
Comment on attachment 792713 [details]
link to https://github.com/mozilla-b2g/gaia/pull/11627

I left a comment in github, could you help check it? Thanks!
Attachment #792713 - Flags: feedback?(josea.olivera)
Comment on attachment 792713 [details]
link to https://github.com/mozilla-b2g/gaia/pull/11627

Sorry for the lag. Looks good to me. Try to rebuild the apn.json database again before landing this patch. Some changes has been landed around the apn.json builder script and we must be sure everything's ok before landing a change in the database. Thanks!
Attachment #792713 - Flags: feedback?(josea.olivera) → feedback+
Thanks!

master: https://github.com/mozilla-b2g/gaia/commit/a894561828a3a00ea54fe69fad26f0cadf6239fa
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED

Comment 17

4 years ago
Verified on Wasabi master build but still not having correct APN programmed first.

Gaia:     b2f3fc404a0e7c577c49093f8300ab7e73154de9
Gecko:    32c8237212fcf578be7c0e00bfeb9c0ba06c4816
BuildID   20131015121031
Version   27.0a1
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(In reply to Enpei from comment #17)
> Verified on Wasabi master build but still not having correct APN programmed
> first.
> 
> Gaia:     b2f3fc404a0e7c577c49093f8300ab7e73154de9
> Gecko:    32c8237212fcf578be7c0e00bfeb9c0ba06c4816
> BuildID   20131015121031
> Version   27.0a1

You should have the new APN added for the SIM card with mcc="466" mnc="05" values. Are you using that SIM card?
Flags: needinfo?(echu)

Comment 19

4 years ago
Yes, I've passed device and SIM card(APTG account) to Edgar as well who is checking the bug now. Same card on same model device but with Android OS can program APN automatically.
Flags: needinfo?(echu)
FYI in this case gecko returns incorrect mcc and mnc values.
Enpei - Can you file a followup bug here?
Status: REOPENED → RESOLVED
Last Resolved: 4 years ago4 years ago
Resolution: --- → FIXED

Updated

4 years ago
Flags: needinfo?(echu)

Updated

4 years ago
Blocks: 927234

Updated

4 years ago
Flags: needinfo?(echu)
Revert the patch as there is a typo in it.

master: a82759418d6d56f276d1e917fe95eb268e0470b4
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Resolved.

master: 62a84888f1c10556af8fbbd50e126bb1afe96276
Status: REOPENED → RESOLVED
Last Resolved: 4 years ago4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.