827601 - rename notoriousb2g.personatest.org to native-persona.org

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::System, defect)

Description

[Jed Parsons (use needinfo, please) [:jedp, :jparsons]]

For v1 launch, we want to rename notoriousb2g to native-persona.org

There's a DNS switcheroo that needs to happen, and a line or two of gaia code, I believe.

Comment 1

[Jason Smith [:jsmith]]

We need to do this. Is any of the work here in the core? Or is it all Gaia work? Or a mixture of both?

Comment 2

[Jed Parsons (use needinfo, please) [:jedp, :jparsons]]

(In reply to Jason Smith [:jsmith] from comment #1)
> We need to do this. Is any of the work here in the core? Or is it all Gaia
> work? Or a mixture of both?

Just gaia code.  Specifically, two lines in apps/system/js/identity.js.

Comment 3

[Jason Smith [:jsmith]]

Note to triagers - we pretty much have to do this. Otherwise, persona won't use the right server for v1.

Comment 4

[Jason Smith [:jsmith]]

Jed - ferjm & I are trying native-persona.org and some URL variants, but I'm getting server not found. Can you clarify what URL we should be testing off of?

Comment 5

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Blocks because it will cause breakage in persona.  (Original name is ssssoo good though, makes me sad)

Comment 6

[Ben Francis [:benfrancis]]

Attachment: https://github.com/mozilla-b2g/gaia/pull/7365

Is this what you meant Jed?

Please confirm domain change before landing.

Comment 7

[Jed Parsons (use needinfo, please) [:jedp, :jparsons]]

(In reply to Jason Smith [:jsmith] from comment #4)
> Jed - ferjm & I are trying native-persona.org and some URL variants, but I'm
> getting server not found. Can you clarify what URL we should be testing off
> of?

Yes, DNS hasn't been set up for native-persona.org yet.  Once ops has done that we can make the switch.

Comment 8

[Jed Parsons (use needinfo, please) [:jedp, :jparsons]]

(In reply to Ben Francis [:benfrancis] from comment #6)
> Created attachment 699092 [details] [review]
> https://github.com/mozilla-b2g/gaia/pull/7365
> 
> Is this what you meant Jed?
> 
> Please confirm domain change before landing.

Hi, Ben, thanks for your help!  Please see comments on the PR; and we'll wait for DNS before pulling.  Cheers!

Comment 9

[Jason Smith [:jsmith]]

Jed - Can you link to a bug that ops is tracking for the DNS switch? We need mark that blocking-basecamp as well.

Comment 10

[Jed Parsons (use needinfo, please) [:jedp, :jparsons]]

(In reply to Jason Smith [:jsmith] from comment #9)
> Jed - Can you link to a bug that ops is tracking for the DNS switch? We need
> mark that blocking-basecamp as well.

I just sent Gene this one.  We'll make it clear in the comments here when it's been done.

Comment 11

[Gene Wood [:gene]]

Jason, I've linked the ticket with IT requesting the DNS be setup ( Bug 827944 )

Comment 12

[Jed Parsons (use needinfo, please) [:jedp, :jparsons]]

That's awesome - Gene, you and cturra did that at an amazing speed.

Now we've got to get an SSL cert.  I'm on it.

Comment 13

[Ben Francis [:benfrancis]]

Do you have a bug number or an ETA for getting the SSL certificate set up Jed? Is that the only thing still blocking this basecamp blocker from landing?

Thanks

Comment 14

[Jed Parsons (use needinfo, please) [:jedp, :jparsons]]

Created bug 828486 to track SSL cert

Comment 15

[Gene Wood [:gene]]

We've got the cert and I'm requesting approval from OpSec on installing the private key for this cert on the box : Bug 828726

Comment 16

[Andreas Gal :gal]

a=me if this gets done in time, but we won't block basecamp on this. If this doesn't land in time, we will have to send a follow-on patch.

Comment 17

[Jason Smith [:jsmith]]

(In reply to Andreas Gal :gal from comment #16)
> a=me if this gets done in time, but we won't block basecamp on this. If this
> doesn't land in time, we will have to send a follow-on patch.

Then that's a tef+, blocking- then. Given how we're triaging right now, I'll renom to get this on Gaia triage radar to take the right steps here.

We *do* need to do this before 1.0 goes out the door, no question about it. But I agree this is probably something we can land after Jan 15th.

Comment 18

[Ben Adida [:benadida]]

(In reply to Andreas Gal :gal from comment #16)
> a=me if this gets done in time, but we won't block basecamp on this. If this
> doesn't land in time, we will have to send a follow-on patch.

I recommend we keep this blcoking+. This is not a cosmetic change, it's a necessary change to move the phone to a production hostname. We cannot guarantee uptime on the personatest.org hostname, because it is the root domain for all of our integration testing. That means QA will depend on this, for identity and marketplace.

Given how small a change this is on the client, we should put this back to basecamp+.

back to basecamp? so Andreas and Jason can consider this ASAP.

Comment 19

[Jason Smith [:jsmith]]

I spoke with Dietrich about this at breakfast - this is likely a tef+, but not a basecamp blocker. Configuration changes can land after Jan 15th.

Note - tef+ = we're blocking 1.0 for it, but it doesn't need to be done by Jan 15th.

Make sense?

Comment 20

[Ben Francis [:benfrancis]]

Is the new domain all set set up and ready for Gaia to switch over? Bug 828726 sounds resolved, is it?

Comment 21

[Ben Adida [:benadida]]

(In reply to Jason Smith [:jsmith] from comment #19)
> I spoke with Dietrich about this at breakfast - this is likely a tef+, but
> not a basecamp blocker. Configuration changes can land after Jan 15th.
> 
> Note - tef+ = we're blocking 1.0 for it, but it doesn't need to be done by
> Jan 15th.
> 
> Make sense?

Ah, maybe I don't understand the new tef+ tag.

The one thing I want to make clear is that we might be, at some point very soon, forced to switch very rapidly to native-persona.org to keep things working. If we don't have the ability to do that rapidly, if we're gated by some other administrative process, that could handicap identity & marketplace.

As long as that's clear, and the tef+ tag still makes sense to you Jason, then that's fine with me.

Comment 22

[Jed Parsons (use needinfo, please) [:jedp, :jparsons]]

I think I'd like to switch this over this weekend.  Since the url is hard-coded in gaia, I'm going to suggest we land the gaia patch today, and as soon as it shows up in the next latest build of b2g, I'll swap the certs.  That should result in minimal disruption for testers.

I'll coordinate this with marketplace and identity today (devs and QA)

Comment 23

[Jed Parsons (use needinfo, please) [:jedp, :jparsons]]

Actually I take that back.  Working with Lloyd and Gene, we've found a way to make the transition seamless.

Comment 24

[Jed Parsons (use needinfo, please) [:jedp, :jparsons]]

Once the DNS is reconfigured, this will be good to land.  notoriousb2g will redirect to native-persona as long as we need it, which i think will just be through the beginning of next week (however long it takes the gaia patch to make it into everyone's desktop/device/simulator build).

Comment 25

[Jed Parsons (use needinfo, please) [:jedp, :jparsons]]

Ok, thanks to Gene, Lloyd, and the amazing IT team for its super speedy action, we now have native-identity.org up, and notoriousb2g redirecting/proxying to it.

Kumar, Krupa, jrgm, and I have verified that thing are working for desktop and phone.  So I think we can land this.

Thanks to everyone who helped out on this!

Comment 26

[Jed Parsons (use needinfo, please) [:jedp, :jparsons]]

Comment on attachment 699092 [details] [review]
https://github.com/mozilla-b2g/gaia/pull/7365

Thanks for the patch, Ben!

r=me.  There's a nit in the comments that would be great to address, but it's not crucial; let's just get this landed :)

Comment 27

[Lukas Blakk [:lsblakk] use ?needinfo]

Landed on mozilla-b2g18/gaia master prior to the 1/25 branching to mozilla-b2g18_v1_0_0/v1.0.0, updating status-b2g-v1.0.0 to fixed.

Comment 28

[amiller]

Does not make sense to create a regression issue.

Comment 29

[Chris Kreinbring [:CKreinbring]]

Can you please provide steps to verify this fix as we will blackbox test from the UI?

Comment 30

[Jed Parsons (use needinfo, please) [:jedp, :jparsons]]

Sorry this has gone unanswered for so long.  The native-persona.org domain is not used on FirefoxOS.  The device is hard-coded to go to firefoxos.persona.org, which is a cname for login.persona.org.  The way to verify this is to toggle on the pref toolkit.identity.debug=true, which enables verbose logging of the persona flows, and look for 'persona uri' in the output.