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

Do not log job removed when destroy_failed_jobs? is false #898

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

yvbeek
Copy link

@yvbeek yvbeek commented Mar 26, 2016

The worker incorrectly logs that the job is removed when destroy_failed_jobs? is false

The worker incorrectly logs that the job is removed when destroy_failed_jobs? is false
@coveralls
Copy link

Coverage Status

Coverage remained the same at 91.121% when pulling c0e821d on Zyphrax:master into 401f43e on collectiveidea:master.

@yvbeek
Copy link
Author

yvbeek commented Mar 26, 2016

Why did the check fail? This is quite a trivial change.

@albus522
Copy link
Member

Because you now have 2 failed tests.

@yvbeek
Copy link
Author

yvbeek commented Mar 28, 2016

How can I see which tests fail? I've only modified a tiny bit of code on one line (see my commit).
Build 534 doesn't seem to provide any useful information.

@albus522
Copy link
Member

Click through to any one of the individual parts of the build

@yvbeek
Copy link
Author

yvbeek commented Mar 28, 2016

Thanks. Could you perhaps give me a hand in fixing this?
I don't see how this could trigger a failed test in lib/delayed/backend/shared_spec.rb

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.

None yet

3 participants