Bug 1443296

Summary:	WEB Console defaults to the first IP Address when connecting to Cockpit with RHV VMs
Product:	Red Hat CloudForms Management Engine	Reporter:	Kevin Morey <kmorey>
Component:	UI - OPS	Assignee:	Brian McLaughlin <bmclaugh>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Niyaz Akhtar Ansari <nansari>
Severity:	high	Docs Contact:
Priority:	high
Version:	5.7.0	CC:	bmclaugh, cpelland, dajohnso, hkataria, jhardy, kmorey, mpovolny, nansari, nstephan, obarenbo, simaishi
Target Milestone:	GA	Keywords:	TestOnly, ZStream
Target Release:	5.9.0
Hardware:	All
OS:	All
Whiteboard:	cockpit
Fixed In Version:	5.9.0.1	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:
Clones:	1458925 (view as bug list)		Environment:
Last Closed:	2018-03-06 15:25:48 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	1458925

Description Kevin Morey 2017-04-19 05:32:19 UTC

Description of problem:
While with a customer this week we experienced an issue where the IPv6 address on their RHEL 7 VMs was the first in the array. This caused the Web Console button form the VM summary screen not to work. Looking in the logs we determined that it grabs the first address it finds rather than the IPv4 address. 

Version-Release number of selected component (if applicable):
5.7.2.1

How reproducible:
The majority of the RHV 4.0.7 VMs show an IPv6 address as the first address rather than the IPv4 address.

Steps to Reproduce:
1. Navigate to the VM summary page of a RHV VM with IPv6 and IPv4 enabled
2. Attempt to connect to the Access -> Web Console console
3. 

Actual results:
As per the logs CF attempts to connect to the IPv6 address rather than the IPv4 address

Expected results:
Expecting CF to be smart enough to connect to the IPv4 address:9090 (cockpit) of the RHV VM

Additional info:

Comment 3 Brian McLaughlin 2017-04-20 14:07:12 UTC

Kevin, what version of RHEL7 (ie: 7.2?)?

Comment 4 Kevin Morey 2017-04-20 14:08:57 UTC

RHEL 7.3

Comment 8 CFME Bot 2017-05-04 18:36:45 UTC

https://github.com/ManageIQ/manageiq/pull/15000

Comment 11 CFME Bot 2017-05-23 14:01:18 UTC

New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/2a1ae60cd29b5e941b87eb260f1110ac5085c768

commit 2a1ae60cd29b5e941b87eb260f1110ac5085c768
Author:     Brian McLaughlin <bmclaugh>
AuthorDate: Thu May 4 11:45:51 2017 -0400
Commit:     Brian McLaughlin <bmclaugh>
CommitDate: Wed May 17 14:25:10 2017 -0400

    Spec tests
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1443296

 spec/models/vm/operations_spec.rb | 30 ++++++++++++++++++++++++++++++
 1 file changed, 30 insertions(+)
 create mode 100644 spec/models/vm/operations_spec.rb

Comment 12 CFME Bot 2017-05-23 14:01:25 UTC

New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/3c0df1ae188f281994dcb3b332a2eec49878927c

commit 3c0df1ae188f281994dcb3b332a2eec49878927c
Author:     Brian McLaughlin <bmclaugh>
AuthorDate: Fri May 5 07:38:21 2017 -0400
Commit:     Brian McLaughlin <bmclaugh>
CommitDate: Wed May 17 14:25:49 2017 -0400

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

 app/models/vm/operations.rb | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

Comment 13 CFME Bot 2017-05-23 14:01:30 UTC

New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/8d6dbba4674de049f619231b92332a7b7b927f58

commit 8d6dbba4674de049f619231b92332a7b7b927f58
Author:     Brian McLaughlin <bmclaugh>
AuthorDate: Thu May 4 11:43:53 2017 -0400
Commit:     Brian McLaughlin <bmclaugh>
CommitDate: Wed May 17 14:25:10 2017 -0400

    Make Cockpit console default to IPV4
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1443296

 app/models/vm/operations.rb | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

Comment 14 CFME Bot 2017-05-23 14:01:36 UTC

New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/76007821310e1aa449e8ae5447c9f56e9eb49a5d

commit 76007821310e1aa449e8ae5447c9f56e9eb49a5d
Author:     Brian McLaughlin <bmclaugh>
AuthorDate: Tue May 9 14:43:03 2017 -0400
Commit:     Brian McLaughlin <bmclaugh>
CommitDate: Wed May 17 14:25:49 2017 -0400

    Update spec tests
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1443296

 spec/models/vm/operations_spec.rb | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

Comment 16 CFME Bot 2017-06-13 18:46:07 UTC

New commit detected on ManageIQ/manageiq/fine:
https://github.com/ManageIQ/manageiq/commit/3c4ce1ef27f12e8a5cfe5e5110bfb0c57c5ac9e6

commit 3c4ce1ef27f12e8a5cfe5e5110bfb0c57c5ac9e6
Author:     Brian McLaughlin <bmclaugh>
AuthorDate: Thu May 4 11:43:53 2017 -0400
Commit:     Brian McLaughlin <bmclaugh>
CommitDate: Tue Jun 6 15:44:15 2017 -0400

    Make Cockpit console default to IPV4
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1443296

 app/models/vm/operations.rb | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

Comment 17 CFME Bot 2017-06-13 18:46:13 UTC

New commit detected on ManageIQ/manageiq/fine:
https://github.com/ManageIQ/manageiq/commit/0efbb2900d8ead54b2f4753f5a4f293941a81cc4

commit 0efbb2900d8ead54b2f4753f5a4f293941a81cc4
Author:     Brian McLaughlin <bmclaugh>
AuthorDate: Thu May 4 11:45:51 2017 -0400
Commit:     Brian McLaughlin <bmclaugh>
CommitDate: Tue Jun 6 15:44:15 2017 -0400

    Spec tests
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1443296

 spec/models/vm/operations_spec.rb | 30 ++++++++++++++++++++++++++++++
 1 file changed, 30 insertions(+)
 create mode 100644 spec/models/vm/operations_spec.rb

Comment 18 CFME Bot 2017-06-13 18:46:19 UTC

New commit detected on ManageIQ/manageiq/fine:
https://github.com/ManageIQ/manageiq/commit/ee5a5e1e8f0805c9bc8aa07cb3c6ad53f6f823c1

commit ee5a5e1e8f0805c9bc8aa07cb3c6ad53f6f823c1
Author:     Brian McLaughlin <bmclaugh>
AuthorDate: Fri May 5 07:38:21 2017 -0400
Commit:     Brian McLaughlin <bmclaugh>
CommitDate: Tue Jun 6 15:44:15 2017 -0400

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

 app/models/vm/operations.rb | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

Comment 19 CFME Bot 2017-06-13 18:46:25 UTC

New commit detected on ManageIQ/manageiq/fine:
https://github.com/ManageIQ/manageiq/commit/140baa342dec934ec7819486ff1a6c581f56f0be

commit 140baa342dec934ec7819486ff1a6c581f56f0be
Author:     Brian McLaughlin <bmclaugh>
AuthorDate: Tue May 9 14:43:03 2017 -0400
Commit:     Brian McLaughlin <bmclaugh>
CommitDate: Tue Jun 6 15:44:15 2017 -0400

    Update spec tests
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1443296

 spec/models/vm/operations_spec.rb | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

Comment 20 Dave Johnson 2017-08-15 14:33:24 UTC

KK, I believe this is you know, right?  I am thinking cockpit should but feel free to kick it back to me if you don't agree.

Comment 21 Vatsal Parekh 2017-10-10 09:02:38 UTC

Able to verify this on Version 5.9.0.1.20171004215954_13427ce