"nova hypervisor-show" should also show the ip address of the hypervisor

Registered by Guangya Liu (Jay Lau)

After blueprint of https://blueprints.launchpad.net/nova/+spec/no-compute-fanout-to-scheduler, host_ip was also stored in DB.

"nova hypervisor-show" will call hypervisor API to get compute node info and the info already include host_ip, it is better that this command can print host_ip out.

Blueprint information

Status:
Complete
Approver:
Russell Bryant
Priority:
Low
Drafter:
Guangya Liu (Jay Lau)
Direction:
Approved
Assignee:
Guangya Liu (Jay Lau)
Definition:
Approved
Series goal:
Accepted for icehouse
Implementation:
Implemented
Milestone target:
milestone icon 2014.1
Started by
Guangya Liu (Jay Lau)
Completed by
Guangya Liu (Jay Lau)

Whiteboard

Gerrit topic: https://review.openstack.org/#q,topic:bp/no-compute-fanout-to-scheduler,n,z

Addressed by: https://review.openstack.org/53798
    Add hypervisor extension for V2 API

Addressed by: https://review.openstack.org/52733
    hypervisor show should also include ip address

Gerrit topic: https://review.openstack.org/#q,topic:bp/hypervisor-show-ip,n,z

"The host_ip was moved to the database because it was being used already and needed a new home, while I agree with the idea behind this BP, if we store it, make it accessible via REST for admins, I think this is a good time to revisit how we store IP addresses and why we do etc" ~jogo 10/28/2013

Thanks Joe. I think storing host_ip in compute_nodes table is ok as it is an attribute of compute node. Also adding host_ip can increase debug ability for end users. ~jay-lau-513 10/28/2013

@Russell, can you give a help? I have uploaded my patch to Gerrit. But as this bp was not approved yet, so the patch in Gerrit was now marked as "Do not merge". Can you please help approve this? Per my previous append, this bp can help improve the debug ability for end user. Thanks. ~jay-lau-513 11/06/2013

"Jay, sounds reasonable enough, can you update the BP with the relevant docImpact information (http://justwriteclick.com/2013/09/17/openstack-docimpact-flag-walk-through/) then we can approve the BP.

Thanks Joe. OK, I will update commit log and upload a new patch again.

Yeah I would love to see some more information on the API that was added, and the likely nova-client patch, etc.

Seems we do not need to update nova client, as the API part already handle the output well. Thanks. --jay-lau-513

(?)

Work Items

This blueprint contains Public information 
Everyone can see this information.