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

Checking killed worker with is_alive? #16908

Merged
merged 1 commit into from
Jan 30, 2018
Merged

Conversation

jameswnl
Copy link
Contributor

Currently it's calling alive? which is wrong.

@jameswnl
Copy link
Contributor Author

@miq-bot add_labels bug, gaprindashvili/yes

@miq-bot
Copy link
Member

miq-bot commented Jan 29, 2018

Checked commit jameswnl@577ada7 with ruby 2.3.3, rubocop 0.52.0, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. 🏆

@agrare agrare self-assigned this Jan 30, 2018
@agrare
Copy link
Member

agrare commented Jan 30, 2018

@jameswnl this looks good to me, did you finish your testing?

@jameswnl
Copy link
Contributor Author

@agrare yes, it looks good

@agrare agrare merged commit c3b0376 into ManageIQ:master Jan 30, 2018
@agrare agrare added this to the Sprint 79 Ending Feb 12, 2018 milestone Jan 31, 2018
simaishi pushed a commit that referenced this pull request Feb 12, 2018
@simaishi
Copy link
Contributor

Gaprindashvili backport details:

$ git log -1
commit 7a8f59fedf3efb22a90d9c82e9c8402e9e10ded8
Author: Adam Grare <agrare@redhat.com>
Date:   Tue Jan 30 15:16:16 2018 -0500

    Merge pull request #16908 from jameswnl/killworker
    
    Checking killed worker with is_alive?
    (cherry picked from commit c3b0376813e2da0cb6561e34afa1c4fee4d2c412)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1536041

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants