Adds host_ip to hypervisor show API

[OpenStack Infra]

https://review.openstack.org/52733
commit e05566de71f39acad3566fc31ba1053d84130c03
Author: Jay Lau <email address hidden>
Date: Wed Feb 5 22:44:15 2014 +0800

    Adds host_ip to hypervisor show API

    After no-compute-fanout-to-scheduler, host_ip was stored in the table
    of compute_nodes. Host ip address should be considered as the hypervisor
    attribute similar to the hypervisor_type, hypervisor_version etc, and
    now those attributes such as hypervisor_type, hypervisor_version etc
    are all listed as the hypervisor attribute when calling "nova
    hypervisor-show host", so we can also set "host_ip" as a new attribute
    output for this command.

    DocImpact
    1) Only administrators can view hypervisor detail in nova.
    2) It can help improve debug capabilities for nova. For example, if
    admin using SimpleCIDRAffinityFilter, then after VM is deployed, admin
    can check if the VM was deployed successfully to the desired host by
    checking ip address of the host via "nova hypervisor-show host".
    3) Add host_ip to the output for "nova hypervisor-show"

    Implement bp hypervisor-show-ip
    Change-Id: I006a504d030be1f47beb68a844647026a6daf0ce

[Stephen Gordon (sgordon)]

Doesn't look like this impacts the manuals side as we don't actually reference/use this command outside of the cli-reference (which doesn't show output).

[Lucky samadhiya (lucky-samadhiya)]

i think in json response there is already present host_IP . please find reference url for same
http://developer.openstack.org/api-ref-compute-v2.1.html#showHypervisor

is it correct ?

[Atsushi SAKAI (sakaia)]

For #2, Yes.

json response on the document has host_ip.
but the response body's detail (like host_ip, hypervisor_type) is not documented on API ref.
Would you add these items to the response parameter table?

[Anne Gentle (annegentle)]

Updated JSON example:
{
   "hypervisors":[
      {
         "status":"enabled",
         "service":{
            "host":"openstack.net",
            "disabled_reason":null,
            "id":16
         },
         "vcpus_used":1,
         "hypervisor_type":"QEMU",
         "local_gb_used":3,
         "vcpus":8,
         "hypervisor_hostname":"openstack.net",
         "memory_mb_used":2560,
         "memory_mb":7980,
         "current_workload":0,
         "state":"up",
         "host_ip":"172.29.236.100",
         "cpu_info":
                {
   "vendor":"Intel",
   "model":"qemu32",
   "arch":"x86_64",
   "features":[
      "pge",
      "clflush",
      "sep",
      "syscall",
      "vme",
      "msr",
      "cmov",
      "fpu",
      "pat",
      "lm",
      "tsc",
      "nx",
      "fxsr",
      "sse4.1",
      "pae",
      "sse4.2",
      "pclmuldq",
      "tsc-deadline",
      "mmx",
      "cx8",
      "mce",
      "de",
      "rdtscp",
      "mca",
      "pse",
      "lahf_lm",
      "popcnt",
      "apic",
      "sse",
      "pni",
      "aes",
      "sse2",
      "hypervisor",
      "ssse3",
      "cx16",
      "mtrr",
      "x2apic"
   ],
   "topology":{
      "cores":1,
      "cells":1,
      "threads":1,
      "sockets":8
   }
}]
         "running_vms":1,
         "free_disk_gb":1004,
         "hypervisor_version":2000000,
         "disk_available_least":955,
         "local_gb":1007,
         "free_ram_mb":5420,
         "id":4
      }
   ]
}

[Sharat Sharma (sharat-sharma)]

Assigning to myself since there is no activity from a long time. if any one have issues please tell me.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/310096

[OpenStack Infra (hudson-openstack)]

Change abandoned by Michael Still (<email address hidden>) on branch: master
Review: https://review.openstack.org/310096
Reason: This code hasn't been updated in a long time, and is in merge conflict. I am going to abandon this review, but feel free to restore it if you're still working on this.