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
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.
I suspect this is another problem so I changed the bug report to match this.