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.

Sign up for GitHub

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

Jump to bottom

update infra_migration_job when polling to prevent timing out #18557

Merged
merged 1 commit into from
Mar 18, 2019
Merged

update infra_migration_job when polling to prevent timing out #18557

merged 1 commit into from
Mar 18, 2019

Conversation

jameswnl
Copy link
Contributor

@jameswnl jameswnl commented Mar 14, 2019 •
edited
Loading

@miq-bot add_labels bug, transformation

Update message on the job during polling stages to keep the job from timing out.

fixes https://bugzilla.redhat.com/show_bug.cgi?id=1688992

@miq-bot
Copy link
Member

miq-bot commented Mar 14, 2019

Checked commit jameswnl@29ec2d3 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. ⭐

@ghost
Copy link

ghost commented Mar 18, 2019

@miq-bot add_label hammer/yes, blocker

@jameswnl jameswnl changed the title [WIP] update infra_migration_job when polling to prevent timing out update infra_migration_job when polling to prevent timing out Mar 18, 2019
@jameswnl
Copy link
Contributor Author

@miq_bot assign @agrare
Adam, please help look into this.
thanks

@miq-bot miq-bot removed the wip label Mar 18, 2019
@roliveri roliveri merged commit 084fd30 into ManageIQ:master Mar 18, 2019
@roliveri roliveri added this to the Sprint 107 Ending Mar 18, 2019 milestone Mar 18, 2019
simaishi pushed a commit that referenced this pull request Mar 18, 2019
@roliveri @simaishi
Merge pull request #18557 from jameswnl/jobtimeout
b8aa100 
update infra_migration_job when polling to prevent timing out

(cherry picked from commit 084fd30)

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1690076
@simaishi
Copy link
Contributor

Hammer backport details:

$ git log -1
commit b8aa10006c30a807116b86986b4fdb9b549ec271
Author: Richard Oliveri <oliveri.richard.github@gmail.com>
Date:   Mon Mar 18 13:59:07 2019 -0400

    Merge pull request #18557 from jameswnl/jobtimeout
    
    update infra_migration_job when polling to prevent timing out
    
    (cherry picked from commit 084fd30c901587f891fc0e59e73c558b61361989)
    
    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1690076

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@roliveri roliveri roliveri approved these changes

Assignees

@agrare agrare

Labels
blocker bug hammer/backported v2v
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@jameswnl @miq-bot @simaishi @roliveri @agrare