527792 - push latest version of getpersonas.com to production (r56028)

Status: VERIFIED FIXED

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

Description

[Myk Melez [:myk] [@mykmelez]]

The latest version of the production branch of getpersonas.com should get pushed to production to pick up the fix for bug 527710, which adds a username property to the JSON feeds that supports a new feature in the upcoming version 1.4 of the Personas extension.

A number of other changes have been made on the getpersonas.com trunk, but only this one change is to be pushed in this particular push.  Ryan is going to manually apply the change to the production branch of the website repository before the push to make sure it's the only change that gets pushed to production (bug 527791).

If at all possible, please do this push during the regularly scheduled maintenance window this Thursday, November 12.

Comment 1

[Ryan Doherty (:rdoherty)]

r527710

Comment 2

[Ryan Doherty (:rdoherty)]

(In reply to comment #1)
> r527710

wrong. r55884

Comment 3

[Shyam Mani [:fox2mike]]

I can grab this.

Comment 4

[Toby Elliott [:telliott]]

Attachment: migration script to update personas

Stick it in the root personas directory (/var/www/personas on sm-personas01) and run php migration.php. Will rebuild all json files.

Comment 5

[Toby Elliott [:telliott]]

Added script to retrofit all the index_1.json files. Drop it into /var/www/personas (or whatever the svn root is on production) and run it as sudo.

Comment 6

[matthew zeier [:mrz]]

Are you asking for a prod push or a stage push (sm-personas01)?

Comment 7

[Toby Elliott [:telliott]]

prod push - my comment about sm-personas01 was just that that was where it would go on stage, so likely where it goes in prod.

Comment 8

[Ryan Doherty (:rdoherty)]

New svn revision, have another isolated page for mozilla.com that can go out with this push. 

r55963

Page is external/mozilla/students.php, self-contained like the others in external/mozilla, no apache config changes needed.

Comment 9

[Ryan Doherty (:rdoherty)]

Another svn revision (sorry for these changes coming in late)

r56028

Removes some functionality, the rest is l10n updates.

Comment 10

[Shyam Mani [:fox2mike]]

[root@mradm02 getpersonas.com]# svn up -r 56028
A    server/external/mozilla/students.php
U    server/external/mozilla/footer.php
 U   server/external/mozilla/index.php
U    server/external/mozilla/header.php
 U   server/external/mozilla
UU   server/admin/pending.php
UU   server/lib/personas_functions.php
Updated to revision 56028.

Updated fine. 

Running the migration script now, it's spitting out numbers.

Comment 11

[Shyam Mani [:fox2mike]]

Done.

67349 <- was the last one.

Comment 12

[Myk Melez [:myk] [@mykmelez]]

Looks great, thanks Shyam!

Comment 13

[Ryan Doherty (:rdoherty)]

re-opening, there seems to be some regressions. These urls used to work:

http://www.getpersonas.com/en-US/external/mozilla
http://www.getpersonas.com/en-US/external/mozilla/firstrun.php (loads incorrect page)

And this url was going live with the push:
http://www.getpersonas.com/en-US/external/mozilla/students.php

From what I can tell, it appears the apache config was modified/reverted to an earlier state.

In order for the above urls to work properly, this needs to be in the vhost file:

AliasMatch ^/([a-zA-Z-]+/)?external/(.*) /path/to/personas/server/external/$2

And make sure this is not in the vhost file:
RewriteRule ^/(?:\w{2,3}(?:-\w{2}(?:-mac)?)?/)?external/mozilla/?
/mnt/netapp/personas/server/external/mozilla/index.php

This was filed in bug 527120 previously.

Comment 14

[Ryan Doherty (:rdoherty)]

Also found a bug, will need a svn up, will get rev # in a min.

Comment 15

[Ryan Doherty (:rdoherty)]

(In reply to comment #14)
> Also found a bug, will need a svn up, will get rev # in a min.

r56061

Comment 16

[Shyam Mani [:fox2mike]]

[root@mradm02 getpersonas.com]# svn up -r 56061
UU   server/external/mozilla/students.php

And apache changes pushed out.

Comment 17

[Ryan Doherty (:rdoherty)]

Looks awesome, thanks Shyam!