Opened 12 years ago

Closed 12 years ago

#317 closed defect (fixed)

rpld drops queues on restore when the server is not available at time of restore

Reported by: stephanj Owned by: ph3-der-loewe
Priority: medium Milestone:
Component: RoarAudio PlayList Daemon Version: 0.1.5
Keywords: store restore Cc:
Architecture: Compiler:
Difficulty: hard Kernel:
Operating System: Parent Tickets:
Patch attached: no Protocol:
Sound driver: Topic: Bug Fix

Description (last modified by ph3-der-loewe)

If a server is not available at the time of restore rpld just drops the queue completely. This leads into confused users and the need to re-add the queues.

There should be some way to work around this. Possible options include:

  • Terminate rpld with error.
  • Return failure for the restore() caller and keep the old environment.
  • Write a warning and ignore the problem.
  • Mark the queue with a runtime flag "inactive".
  • A combination of the above.

Subtickets

Change History (4)

comment:1 Changed 12 years ago by ph3-der-loewe

  • Description modified (diff)
  • Difficulty set to hard
  • Keywords store restore added
  • Summary changed from RPLD drops the Queue from the MainQueue out of the Store to rpld drops queues on restore when the server is not available at time of restore
  • Topic set to Bug Fix
  • Version set to current

I suspect this is another problem so I changed the bug report to match this.

comment:2 Changed 12 years ago by ph3-der-loewe

  • Owner set to ph3-der-loewe
  • Status changed from new to accepted

I started to work on this.

comment:3 Changed 12 years ago by ph3-der-loewe

I wrote an experimental patch to allow 'stub' connections (backends not connected to a server). restore() writes a warning in case it can not connect to the listed servers.

An interesting question is left: should rpld terminate in case it can not connect to servers while in startup (default servers or initial restore)? Should it be configurable?

comment:4 Changed 12 years ago by ph3-der-loewe

  • Resolution set to fixed
  • Status changed from accepted to closed
  • Version changed from current to 0.1.5

I close this now with the patch from my comment:3.
Future discussion can still happen here or in separate tickets.

Note: See TracTickets for help on using tickets.