524132 - Update getpersonas.com to latest production revision (r54078)

Status: VERIFIED FIXED

(Infrastructure & Operations Graveyard :: WebOps: Other, task)

Description

[Ryan Doherty (:rdoherty)]

Got a small but important update for getpersonas.com (bug 522754)

Note: stage did not allow .htaccess overrides, so this needs to be turned on in production if not already on.

Update adds 1 page and minor functionality. No changes to currently existing code.

Comment 1

[Ryan Doherty (:rdoherty)]

Can this go out today? It's blocking some 3.6 firstrun & other pages QA.

Comment 2

[Phong Tran [:phong]]

need to get approval for this since it's outside of downtime window.

Comment 3

[John Slater]

(In reply to comment #2)
> need to get approval for this since it's outside of downtime window.
approval from whom?

Comment 4

[Zandr Milewski [:zandr]]

Update pushed.

Comment 5

[Ryan Doherty (:rdoherty)]

http://www.getpersonas.com/en-US/external/mozilla is a 404, most likely due to .htaccess overrides not being allowed (comment #0)

Moving severity down, I don't think anyone needs this over the weekend.

Comment 6

[matthew zeier [:mrz]]

We -really- shouldn't be doing production pushes outside of maintenance windows.

Comment 7

[Ryan Doherty (:rdoherty)]

(In reply to comment #6)
> We -really- shouldn't be doing production pushes outside of maintenance
> windows.

Understandable, getting these pages QA'd for 3.6 is a high priority and previous pushes outside of maintenance window gave me the impression it was ok. 

Can the fix for the 404 page be part of tmrws push?

Comment 8

[matthew zeier [:mrz]]

ack. okay to push this whenever ryan is ready with the "second part".

Comment 9

[Ryan Doherty (:rdoherty)]

Other fix is bug 523222

Comment 10

[Ryan Doherty (:rdoherty)]

http://www.getpersonas.com/en-US/external/mozilla is still a 404, anything I can do to help? The .htaccess file here is supposed to point that url to server/external/mozilla/index.php

http://viewvc.svn.mozilla.org/vc/projects/getpersonas.com/production/.htaccess?revision=54075&view=markup

Upping priority because this blocks firstrun and whatsnew pages for 3.6.

Comment 11

[Dave Miller [:justdave]]

I think this breaks because of the locale code redirect when you try to use a path that doesn't have a locale in it yet.

Try adding either [L] or [PT] after the RewriteRule target (based on rules trying to do similar things in http://viewvc.svn.mozilla.org/vc/projects/mozilla.com/tags/production/.htaccess?view=markup )

Comment 12

[Ryan Doherty (:rdoherty)]

I'm confused, if this works on stage, why wouldn't it work in production?
(http://sm-personas01.mozilla.org/en-US/external/mozilla)

I can the [L] or [PT] after the rule, but I'd rather not spend days guessing and waiting for a push to production to find out I'm wrong.

Comment 13

[Ryan Doherty (:rdoherty)]

r54722 has [PT] after the rule.

The sooner this can be fixed the better, thanks!

Comment 14

[Ryan Doherty (:rdoherty)]

Actually, r54728 for prod branch.

Comment 15

[Ryan Doherty (:rdoherty)]

What's the status on this? It's been sitting for a while now and still blocking 3.6 pages. Need to get this resolved asap.

Comment 16

[Dave Miller [:justdave]]

Need someone else to take this on, I'm too involved in another project to track this down at the moment.

We could blindly push r54728 and hope it works, but it would probably be a good idea to compare the apache config between production and the slave and see what's different that would affect how the .htaccess file is interpreted first just to make sure.

Comment 17

[Tara (musingt)]

Thanks Ryan. and Dave.  Is there anything I can do to help find some resources to get this done?  I'd like to resolve these bugs asap, as they are blocking several pages right now.  I'd like to get this fixed in the next day or two if possible, so that we don't experience any further delays.  Can we do that, please? Let me know what I can do to push this along. Thanks.

Comment 18

[Jeremy Orem [:oremj]]

Production is AllowOverride None.

Comment 19

[John Slater]

(In reply to comment #18)
> Production is AllowOverride None.

So what does that mean in the context of getting this resolved so we can get the 3.6 pages done in time?

Comment 20

[Jeremy Orem [:oremj]]

It doesn't mean anything. I was just letting Ryan know.

Also, added the following to the vhost:
        RewriteRule ^/(?:\w{2,3}(?:-\w{2}(?:-mac)?)?/)?external/mozilla/? /mnt/netapp/personas/server/external/mozilla/index.php

The link is no longer a 404. Everything okay with this bug now?

Comment 21

[Ryan Doherty (:rdoherty)]

Page works, but http://www.getpersonas.com/static/img/persona-sample.jpg is a 404, which is part of the page. It's on the prod. branch, not sure why it doesn't show up.

Thanks Jeremy! I was hoping to use a .htaccess file so the vhost file wouldn't need to be touched by anyone, reducing the amount of work for IT. Can we update getpersonas.com in the near future to allow htaccess files?

Comment 22

[Ryan Doherty (:rdoherty)]

It also looks like bug 523681 is back, not sure if it's because of this.

Comment 23

[Ryan Doherty (:rdoherty)]

Pretty sure the image not showing up is due to bug 523681. 'persona' is in the url, so I think the regex breaks it.

Comment 24

[Jeremy Orem [:oremj]]

Think this bug is fixed, just forgot to mark it as such.