Bug 1623562

Summary: [RFE] Don't show allocated IPs in dropdown while assigning floating IPs via CloudForms
Product: Red Hat CloudForms Management Engine Reporter: Satoe Imaishi <simaishi>
Component: ProvidersAssignee: Gilles Dubreuil <gdubreui>
Status: CLOSED ERRATA QA Contact: Ido Ovadia <iovadia>
Severity: high Docs Contact:
Priority: high    
Version: 5.9.0CC: dkholodo, dmetzger, gblomqui, gdubreui, jfrey, jhardy, jocarter, jprause, lavenel, lmiccini, niroy, obarenbo, pmukhedk, slucidi, smallamp
Target Milestone: GAKeywords: FutureFeature, ZStream
Target Release: 5.9.6   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 5.9.6.3 Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of: 1595767 Environment:
Last Closed: 2018-12-13 15:15:24 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: Openstack Target Upstream Version:
Embargoed:
Bug Depends On: 1595767    
Bug Blocks:    
Attachments:
Description Flags
undefined none

Comment 2 CFME Bot 2018-09-17 16:53:08 UTC 
New commit detected on ManageIQ/manageiq-ui-classic/gaprindashvili:

https://github.com/ManageIQ/manageiq-ui-classic/commit/76a3e1def7cd2dbbce6e2e75ea23ba68b601448f
commit 76a3e1def7cd2dbbce6e2e75ea23ba68b601448f
Author:     Milan Zázrivec <mzazrivec>
AuthorDate: Fri Jul 27 02:57:15 2018 -0400
Commit:     Milan Zázrivec <mzazrivec>
CommitDate: Fri Jul 27 02:57:15 2018 -0400

    Merge pull request #4357 from gildub/no-allocated-IPs

    VM: associate only unused floating IPs
    (cherry picked from commit d777b49c83cb27bb288d80f227f71cdbf520beb9)

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

 app/controllers/mixins/actions/vm_actions/associate_floating_ip.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Comment 4 Ido Ovadia 2018-10-15 14:30:12 UTC 
Failed QA
=========

CFME 5.9.5.0

While trying to associate FIP to an instance, no FIP listed. Only  'undefined'

Screenshot is enclosed
Comment 5 Ido Ovadia 2018-10-15 14:33:28 UTC 
Created attachment 1494059 [details]
undefined
Comment 8 Gilles Dubreuil 2018-11-19 07:46:34 UTC 
I confirm the list of floating IPs is not displaying properly. Investigating.
Comment 9 Gilles Dubreuil 2018-11-20 23:05:59 UTC 
Upstream complementary patch has been merged. 

Waiting for backport to gaprindashvili.
Comment 10 CFME Bot 2018-11-21 14:28:51 UTC 
New commit detected on ManageIQ/manageiq-ui-classic/gaprindashvili:

https://github.com/ManageIQ/manageiq-ui-classic/commit/075812394abf5137e6c92f70b4fc3ac40b157c21
commit 075812394abf5137e6c92f70b4fc3ac40b157c21
Author:     Milan Zázrivec <mzazrivec>
AuthorDate: Tue Nov 20 06:37:41 2018 -0500
Commit:     Milan Zázrivec <mzazrivec>
CommitDate: Tue Nov 20 06:37:41 2018 -0500

    Merge pull request #4944 from gildub/vm_cloud_floating_ip

    VM CLoud: floating ips to associate must be a list

    (cherry picked from commit 911a289574b1d433c6a8fcde890ebb39d7f24da2)

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

 app/controllers/mixins/actions/vm_actions/associate_floating_ip.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Comment 11 Danylo Kholodov 2018-12-07 08:59:44 UTC 
Verified at CFME 5.9.6.5
Comment 13 errata-xmlrpc 2018-12-13 15:15:24 UTC 
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2018:3816