[API] Composed node /nodes didn't have show correct content

[chesterkuo]

with common/redfish/api.py, def nodes_list(),

bmc_ip is hard-code and should be displayed in the metadata property of nodes, same issue as bmcmac, nw.

more data need add into node detail.

[chesterkuo (chester-kuo)]

also , we should not use node id as below in code, the redfish id and will confuse with id of location,
suggest to use url_id as we had in API, make sense ?

nodeid = lnk.split("/")[-1]

[chesterkuo (chester-kuo)]

the nodeid may have duplicate entry if support to multiple POD environment, ex:

/redfish/v1/nodes/node1 show in PODM1 , and another PODM may have also same node url id.

[Ananth Narayan S (ananth-narayan)]

Not related to : 1636826 though the title is very similar.

[Lin Yang (lin-a-yang)]

I have marked https://bugs.launchpad.net/openstack-valence/+bug/1636826 is duplicated to this one.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/418657
Committed: https://git.openstack.org/cgit/openstack/valence/commit/?id=59d6a712a0d1768a8625b62cc319922f0b27fbf2
Submitter: Jenkins
Branch: master

commit 59d6a712a0d1768a8625b62cc319922f0b27fbf2
Author: Lin Yang <email address hidden>
Date: Tue Jan 10 19:17:08 2017 -0800

    Align api /nodes with api-ref spec

    1. Return briefly info of node after composed node
    2. Fixed hardcode fields in node details, like ip addr, mac, etc.
       Added more fields according to valence api spec.

    Change-Id: I9291b1245895fd7693f5e960559e9f29a314b712
    Closes-Bug: #1636826