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

Floating IPs: Adds missing route for wait_for_task #192

Merged
merged 1 commit into from
Jan 19, 2017
Merged

Floating IPs: Adds missing route for wait_for_task #192

merged 1 commit into from
Jan 19, 2017

Conversation

gildub
Copy link
Contributor

@gildub gildub commented Jan 19, 2017

No description provided.

@miq-bot
Copy link
Member

miq-bot commented Jan 19, 2017

Checked commit https://github.com/gildub/manageiq-ui-classic/commit/8cc0b82abfa6d1a5b169b779e635bbf5fb82e166 with ruby 2.2.6, rubocop 0.46.0, and haml-lint 0.19.0
1 file checked, 0 offenses detected
Everything looks good. 🏆

@martinpovolny
Copy link
Member

martinpovolny commented Jan 19, 2017

Euwe? BZ?

@martinpovolny martinpovolny added this to the Sprint 53 Ending Jan 30, 2017 milestone Jan 19, 2017
@martinpovolny martinpovolny merged commit 050b7a6 into ManageIQ:master Jan 19, 2017
@tzumainn
Copy link
Contributor

This is probably euwe/yes, as a follow up for https://bugzilla.redhat.com/show_bug.cgi?id=1394277

@tzumainn
Copy link
Contributor

@miq-bot add_label euwe/yes

@simaishi
Copy link
Contributor

Euwe backport (to manageiq repo) details:

$ git log -1
commit 979afe91aede69fadc2f005e33324e454a799dda
Author: Martin Povolny <[email protected]>
Date:   Thu Jan 19 09:17:16 2017 +0100

    Merge pull request #192 from gildub/openstack_floating_ip_wait_for_task_route
    
    Floating IPs: Adds missing route for wait_for_task
    (cherry picked from commit 050b7a65893b31eccef073909a6389407063bd3e)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1410588

@simaishi simaishi removed the euwe/yes label Jan 19, 2017
@gildub gildub deleted the openstack_floating_ip_wait_for_task_route branch January 20, 2017 00:36
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.

5 participants