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

Storage server breaks if no cache defined

RESOLVED FIXED

Status

Cloud Services
Server: Sync
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: philikon, Assigned: tarek)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Created attachment 499897 [details] [diff] [review]
fix, no tests

Trying to run the Python server (server-full) locally against sqlite breaks for me since syncstorage.wsgiapp.StorageServerApp assumes self.cache exists.

(development.ini no longer refers to sqlite.conf by default which is a change I personally don't agree with because it means the server is no longer zero-conf. It may also be part of the reason why this bug wasn't caught earlier.)
Attachment #499897 - Flags: review?(tarek)
(Assignee)

Comment 1

7 years ago
(In reply to comment #0)
> Created attachment 499897 [details] [diff] [review]
> fix, no tests
> 
> Trying to run the Python server (server-full) locally against sqlite breaks for
> me since syncstorage.wsgiapp.StorageServerApp assumes self.cache exists.

These two options are mutually inclusive. I'll also need to raise an error at initialization in that case (+test). Will do when I am back from vacation.

> 
> (development.ini no longer refers to sqlite.conf by default which is a change I
> personally don't agree with because it means the server is no longer zero-conf.
> It may also be part of the reason why this bug wasn't caught earlier.)

It's unrelated and was an unintended change. I accidentally committed my modified development file when I committed some doc. Will fix it as well.
Assignee: nobody → tarek
Status: NEW → ASSIGNED
(Assignee)

Comment 2

7 years ago
# Revert the config: http://hg.mozilla.org/services/server-full/rev/7d57979a3241
# Make sure the app cannot be initialized if Memcache is not present, and if check_blacklisted_nodes is set to true: http://hg.mozilla.org/services/server-storage/rev/c534b608ab51
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
(Reporter)

Updated

7 years ago
Attachment #499897 - Flags: review?(tarek)
You need to log in before you can comment on or make changes to this bug.