Skip to content
This repository has been archived by the owner on Apr 1, 2023. It is now read-only.

Completed jobs in Sidekiq::Status::Web #44

Closed
marcboquet opened this issue Oct 22, 2014 · 5 comments
Closed

Completed jobs in Sidekiq::Status::Web #44

marcboquet opened this issue Oct 22, 2014 · 5 comments

Comments

@marcboquet
Copy link

I'm moving from resque to sidekiq, and noticed that with sidekiq-status I can't see the information for completed jobs as I could with resque-status.
So the jobs appear only on the 'Statuses' list when they are being processed, but disappear once they complete, no matter the expiration.
Is this normal behavior or am I doing something wrong?

@utgarda
Copy link
Owner

utgarda commented Feb 9, 2015

Yup, it's a feature, at least so far. I can see you started implementing showing completed jobs in your branch?

@marcboquet
Copy link
Author

Yep, have a first implementation but needs a bit more work.
Would you be open to merge it once it's done?

@utgarda
Copy link
Owner

utgarda commented Feb 11, 2015

Absolutely. That'd be fabulous. I don' use Sidekiq currently, so I can't point the right direction for new features, all I can do is answer people's questions about how it works, and that's it. And most of the time that's not required, cause the codebase is tiny and ( hopefully ) quite readable. Sharing a feature that is important enough for you to implement is very likely what other users need. Also, it's just a cool thing to do :)

@vanboom
Copy link

vanboom commented Oct 31, 2015

I think this is related to the job not hanging around until expiration...

Sidekiq::Status::status( job_id )

starts returning nil as soon as the job is completed, so I am not able to see that the job has been completed by polling status.

@pramodshinde
Copy link
Collaborator

@utgarda I drilled down this issue and observed following

In Sidekiq::Status::Web module

  queue = Sidekiq::Workers.new
  • Above API, only return jobs which are in progress i.e. with status :working
  • I also looked into @marcboquet's code on his feature branch but I'm not liking his hack to show up completed jobs on statuses tab.
    I started working on this feature at branch (not committed any code yet).

For this I am thinking of storing some additional info with status for a job instead of using above API.

utgarda added a commit that referenced this issue Dec 21, 2015
Feature: Added all job statuses to Sidekiq::Status::Web fixes #44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants