Bug 790516 - aeolus should only return a valid ip or text indicating a valid ip was not found
Summary: aeolus should only return a valid ip or text indicating a valid ip was not found
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: CloudForms Cloud Engine
Classification: Retired
Component: aeolus-conductor
Version: 1.0.0
Hardware: Unspecified
OS: Unspecified
unspecified
high
Target Milestone: rc
Assignee: Matt Wagner
QA Contact: wes hayutin
URL: https://hp-sl2x170zg6-01.rhts.eng.bos...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-02-14 18:25 UTC by wes hayutin
Modified: 2014-08-17 22:27 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-08-30 17:15:21 UTC
Embargoed:


Attachments (Terms of Use)

Description wes hayutin 2012-02-14 18:25:41 UTC
Description of problem:

Currently depending on the client status.. deltacloud will report either

1. a valid ip
2. a mac addr
3. a vnc add (rhevm)

Deltacloud has fixed a bug that indicates and differentiates the type.

Aeolus should only return a valid ip 

or

some text indicating a valid ip is not reported from the instances


This will help reduce the confusion around ip addr and instances..
People sshing into rhevm clusters..etc..

Comment 1 Matt Wagner 2012-02-14 19:15:06 UTC
From https://issues.apache.org/jira/browse/DTACLOUD-143 --

  <public_addresses><address port='5902' type='vnc'>qeblade27.rhq.lab.eng.bos.redhat.com</address></public_addresses>
  <private_addresses></private_addresses> 

So the type='vnc' is key. I'll look at what we get for an instance IP, and I think we should discard anything that isn't that. (Unless we want to track MACs or VNC addresses separately, which might be a nice enhancement.)

Comment 3 Matt Wagner 2012-02-20 21:18:44 UTC
Pushed to master:

commit acd8031878cbabcef121646e61be6388c4e6d52f
Author: Matt Wagner <matt.wagner>
Date:   Tue Feb 14 16:03:24 2012 -0500

    BZ 790516 - Only accept ipv4, hostname addresses for instances
    
    Deltacloud labels 'vnc' and 'mac' addresses; don't set them as
    instance addresses! (There is no 'ipv6' type currently.)
    
    Resolves https://bugzilla.redhat.com/show_bug.cgi?id=790516

Comment 4 John Eckersberg 2012-02-20 21:45:04 UTC
acd8031 (maint 61e2cd2) in aeolus-conductor-0.8.0-31

Comment 5 wes hayutin 2012-02-24 03:15:16 UTC
verified in

 [root@qeblade31 ~]# rpm -qa | grep aeolus
aeolus-conductor-daemons-0.8.0-35.el6.noarch
rubygem-aeolus-image-0.3.0-9.el6.noarch
rubygem-aeolus-cli-0.3.0-10.el6.noarch
aeolus-all-0.8.0-35.el6.noarch
aeolus-conductor-0.8.0-35.el6.noarch
aeolus-conductor-doc-0.8.0-35.el6.noarch
aeolus-configure-2.5.0-15.el6.noarch


Note You need to log in before you can comment on or make changes to this bug.