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

Show Network Port name in Floating IP list #14970

Merged

Conversation

Ladas
Copy link
Contributor

@Ladas Ladas commented May 3, 2017

Show Network Port name in Floating IP list, to identify the owner
in a case of missing VM. The owner can be e.g. Load Balancer.

Partially fix:
https://bugzilla.redhat.com/show_bug.cgi?id=1387616

Show Network Port name in Floating IP list, to identify the owner
in a case of missing VM. The owner can be e.g. Load Balancer.

Partially fix:
https://bugzilla.redhat.com/show_bug.cgi?id=1387616
@Ladas
Copy link
Contributor Author

Ladas commented May 3, 2017

@miq-bot assign @kbrock

@miq-bot
Copy link
Member

miq-bot commented May 3, 2017

Checked commit Ladas@a243850 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
0 files checked, 0 offenses detected
Everything looks fine. ⭐

@kbrock kbrock merged commit a0cd650 into ManageIQ:master May 8, 2017
@kbrock kbrock added this to the Sprint 60 Ending May 8, 2017 milestone May 8, 2017
simaishi pushed a commit that referenced this pull request Jun 5, 2017
…ng_ip_list

Show Network Port name in Floating IP list
(cherry picked from commit a0cd650)

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

simaishi commented Jun 5, 2017

Fine backport details:

$ git log -1
commit df4442c549cc1ec02f636a9d5d37926a1dd43e3f
Author: Keenan Brock <[email protected]>
Date:   Mon May 8 12:08:53 2017 -0400

    Merge pull request #14970 from Ladas/show_network_port_name_in_floating_ip_list
    
    Show Network Port name in Floating IP list
    (cherry picked from commit a0cd65074c12d7cd3a44cbf3d7da302b8cb34682)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1458943

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