Opened 14 years ago

Closed 5 years ago

#1246 closed bug (Fixed)

With every restart deluge loses some torrent label's

Reported by: simonbcn Owned by:
Priority: major Milestone: 1.3.3
Component: Plugin Version: 1.3.6
Keywords: label Cc: lysias@welho.net, Chase

Description

With every restart of deluged daemon, most of my torrents lose the label.
Deluge have 96 torrents now:
The labels associated to one tracker still work (54).
6 torrents still conserve their correct label.
26 torrents lose the label.
Moreover, these torrents are old and new. There isn't a rule that explain this bug.
It was a very bad idea to use the version of deluge in development.

Change History (20)

comment:1 by simonbcn, 14 years ago

Is there any solution to this problem? How could I export easily 96 torrents from current configuration to the new to solve definitely this problem?

comment:2 by Damien Churchill, 14 years ago

Is there any possibility you could tar up your config directory so I can see if I can reproduce this?

comment:3 by simonbcn, 14 years ago

Ok, but not here, not as public. Please, give me an email address to send it.

comment:4 by Damien Churchill, 14 years ago

Of course, wasn't expecting you to post it to the world!

My email is damoxc@gmail.com.

comment:5 by Lysias, 14 years ago

Cc: lysias@welho.net added

I had a similar problem where all torrents' labels would get reset to a single label after restarting the daemon. After some digging I found out that somehow the settings for automatically assigning a label by the tracker name were messed up. Each letter of a tracker name in the edit field was on its own line. Let's say I had a line "mp3tracker". It was split to

m
p
3
t

and so on. Single letters of course match many tracker names... I wonder if it was the WebUI that did this? Now I'm using the GTK client to connect to the daemon and so far everything's fine.

comment:6 by simonbcn, 14 years ago

I have stopped deluge-daemon, I have deleted the section "label" from label.conf and I have started deluge-daemon.
Now the problem is worst: now I lose all labels in every restart.
Any help here?

comment:7 by simonbcn, 14 years ago

Sorry, I deleted the "torrent_labels" section from label.conf file

comment:8 by Calum, 13 years ago

Priority: majortrivial
Version: 1.3.0_dev

The OP issue has not been reported by anyone else and it was a dev build being used so I was considering closing this ticket.

However the issue with the tracker name splitting that lysias reported needs investigating.

Lysias could you confirm this still occurs and the version of deluge used?

comment:9 by Lysias, 13 years ago

I haven't seen the problem since.

Using version 1.3.0-0ubuntu1 in a daemon/client setup.

comment:10 by Calum, 13 years ago

Resolution: fixed
Status: newclosed

comment:11 by simonbcn, 13 years ago

Priority: trivialminor
Resolution: fixed
Status: closedreopened
Version: 1.3.1

It isn't fixed.
Deluge 1.3.1 installed from tarball.
Same problem.

comment:12 by Calum, 13 years ago

Please describe exactly how we can repeat this. Including the label names used and debug logs.

comment:13 by Calum, 13 years ago

Milestone: Future1.3.x

comment:14 by raxxy, 13 years ago

I've been able to consistantly cause torrents to lose labels:

100+ torrents (almost always minimum 100 in this client, theres a few of us using the one deluge instance) ~10 labels ~5-10 torrents/label, some with 20+.

Deluge begins to go unresponsive to our GTK ui's that are connected remotly so i kill the deluged instance on the server

kill -9 deluged-pid

deluged -q to restart deluged,

upon restart, ~80 - 95% of the torrents have been put back under "no label"

If you're still unable to get reproduce the problem with that information, I'll give you access to the installation of mine so you can see for yourself.

comment:15 by Calum, 13 years ago

Cc: Chase added
Milestone: 1.3.x1.3.3
Priority: minormajor

Thanks to the logs provided by raxxy I can now pinpoint the issue. Labels with autoadd enabled will trigger clean_config to be called before all the torrents have been loaded from the state file upon startup. This means any torrent not added to the session at that point will be removed from label.conf.

Label core.py

A possible solution: Upon enabling the plugin, register the handler for SessionStartedEvent and when received apply the labels to those loaded torrents then register the handler for TorrentAddedEvent and carry on as per usual.

in reply to:  14 comment:16 by andar, 13 years ago

Replying to raxxy:

I've been able to consistantly cause torrents to lose labels:

100+ torrents (almost always minimum 100 in this client, theres a few of us using the one deluge instance) ~10 labels ~5-10 torrents/label, some with 20+.

Deluge begins to go unresponsive to our GTK ui's that are connected remotly so i kill the deluged instance on the server

kill -9 deluged-pid

deluged -q to restart deluged,

upon restart, ~80 - 95% of the torrents have been put back under "no label"

If you're still unable to get reproduce the problem with that information, I'll give you access to the installation of mine so you can see for yourself.

Are you trying a TERM before the KILL? If you are doing a KILL then there is a very good chance your state won't be written properly and you'll lose label information. There shouldn't be any reason you need to do a KILL unless there is truly a badly hung deluged process.

comment:17 by Calum, 13 years ago

Resolution: fixed
Status: reopenedclosed

Fixed in 1.3-stable: f7c21fd8

comment:18 by carbncl, 13 years ago

For information, I had this bug, just installed the 1.3-stable version from source to check that fix, works perfectly, thanks. My server has been up for 5 months because of this one... ;) Btw, great stability, loving deluge!

comment:19 by gnimsh, 11 years ago

Resolution: fixed
Status: closedreopened
Version: 1.3.11.3.6

I do not believe that this bug has been fixed. I just updated to the newest version 1.3.6 after adding the PPA to ubuntu and then I restarted and my label locagions have been completely reset. It's a very frustrating bug that causes me to have to reset the label location every time I restart deluge.

I'll provide any information needed to get this fixed.

comment:20 by int3l, 5 years ago

Resolution: Fixed
Status: reopenedclosed
Note: See TracTickets for help on using tickets.