Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade rq 1.3.0 -> 1.5.2 #363

Closed
wants to merge 1 commit into from
Closed

Conversation

petlack
Copy link

@petlack petlack commented Sep 14, 2020

Fixes #362

@waldner
Copy link

waldner commented Oct 3, 2020

While this fixes the original issue, I now get:

rq-dashboard    | [2020-10-03 14:47:35,257] ERROR in app: Exception on /0/data/jobs/default/failed/11/1.json [GET]
rq-dashboard    | Traceback (most recent call last):
rq-dashboard    |   File "/usr/local/lib/python3.8/site-packages/flask/app.py", line 2447, in wsgi_app
rq-dashboard    |     response = self.full_dispatch_request()
rq-dashboard    |   File "/usr/local/lib/python3.8/site-packages/flask/app.py", line 1952, in full_dispatch_request
rq-dashboard    |     rv = self.handle_user_exception(e)
rq-dashboard    |   File "/usr/local/lib/python3.8/site-packages/flask/app.py", line 1821, in handle_user_exception
rq-dashboard    |     reraise(exc_type, exc_value, tb)
rq-dashboard    |   File "/usr/local/lib/python3.8/site-packages/flask/_compat.py", line 39, in reraise
rq-dashboard    |     raise value
rq-dashboard    |   File "/usr/local/lib/python3.8/site-packages/flask/app.py", line 1950, in full_dispatch_request
rq-dashboard    |     rv = self.dispatch_request()
rq-dashboard    |   File "/usr/local/lib/python3.8/site-packages/flask/app.py", line 1936, in dispatch_request
rq-dashboard    |     return self.view_functions[rule.endpoint](**req.view_args)
rq-dashboard    |   File "/rq_dashboard/web.py", line 101, in _wrapped
rq-dashboard    |     result_dict = f(*args, **kwargs)
rq-dashboard    |   File "/rq_dashboard/web.py", line 427, in list_jobs
rq-dashboard    |     total_items, jobs = get_queue_registry_jobs_count(
rq-dashboard    |   File "/rq_dashboard/web.py", line 234, in get_queue_registry_jobs_count
rq-dashboard    |     jobs = [serialize_job(job) for job in current_queue_jobs]
rq-dashboard    |   File "/rq_dashboard/web.py", line 234, in <listcomp>
rq-dashboard    |     jobs = [serialize_job(job) for job in current_queue_jobs]
rq-dashboard    |   File "/rq_dashboard/web.py", line 169, in serialize_job
rq-dashboard    |     id=job.id,
rq-dashboard    | AttributeError: 'NoneType' object has no attribute 'id'

when I try to look at failed jobs.

@jmandt
Copy link

jmandt commented Oct 23, 2020

@mgax @nvie would one of you guys be able to have a look at the pull request and merge it if this update of rq version is reasonable?

@mgax
Copy link
Contributor

mgax commented Oct 28, 2020

@mgax @nvie would one of you guys be able to have a look at the pull request and merge it if this update of rq version is reasonable?

Hey @jmandt, I haven't used rq-dashboard in years, can't help you there, sorry :)

@Cybernisk
Copy link

Cybernisk commented Nov 2, 2020

Hi, @jmandt I've got the same problem as @waldner above with rq==1.5.2 installed inside but with Finished jobs list.

@Cybernisk
Copy link

I guess I found the reason for the bug. RQ registries are not cleaned.
My example is that I have 29 jobs in FinishedJobRegistry, but if I count all Redis keys rq:job:* it gives me only 8.
I'm not sure, but it could be because of the old rq package I use. Seems that it was fixed in this MR. I will update all rq and rq-related packages in my project and will see how it goes.

@cjlapao cjlapao closed this Jun 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ValueError: time data '' does not match format '%Y-%m-%dT%H:%M:%SZ'
6 participants