migrate server-full to pyramid/cornice/mozsvc

VERIFIED WONTFIX

Status

VERIFIED WONTFIX
7 years ago
6 years ago

People

(Reporter: rfkelly, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [qa-])

Attachments

(1 attachment)

Created attachment 597740 [details] [diff] [review]
patch migrating server-full to pyramid

This patch updates server-full to use our new pyramid-based infrastructure.  Major changes:
  * merge .ini/.conf files into a single .ini file
  * replace wsgiapp.py with views.py
  * move controller code into views.py

Required to make easy use of the pyramid-based server-syncstorage code.
Attachment #597740 - Flags: review?(telliott)
Attachment #597740 - Flags: review?(tarek)
Comment on attachment 597740 [details] [diff] [review]
patch migrating server-full to pyramid

I did not see any change in setup.py or the spec file to add who.plugin.vepauth and PyVep as dependencies. 

Other than than looks good, while I am not sure we want to bother upgrading any old project other than storage
Attachment #597740 - Flags: review?(tarek) → review+
Sorry drop the remark I though I was reviewing server-reg ;) 

lgtm
ISTM that much like server-reg, server-full will not be needed in the new world or tokenservers and sagrada?  If so I will wontfix this for same reasons as Bug 727764.
We will need a server-full, but it's pretty much going to have to be redone from scratch, as it'll need to install token server, sync server, a browserid suite for generating accounts, etc, etc

Updated

7 years ago
Whiteboard: [qa-]
When the time comes to do server-full in the new model, we'll have a bunch of new bugs :P
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → WONTFIX
Attachment #597740 - Flags: review?(telliott)
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.