Unite the host key(host, host_name)
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenStack Compute (nova) |
Confirmed
|
Low
|
Unassigned | ||
tempest |
Invalid
|
Undecided
|
tinytmy |
Bug Description
It used host_name key in
https:/
used host key in
https:/
Why not unite the key.
And this lead to novaclient bug such as:
>>> from novaclient.v1_1 import client
>>> nc = client.
>>> nc.hosts.
[<Host: controller>, <Host: controller>, <Host: controller>, <Host: controller>]
>>> nc.hosts.list()
[Traceback (most recent call last):
File "<stdin>", line 1, in <module>
File "/usr/lib/
return "<Host: %s>" % self.host
File "/usr/lib/
raise AttributeError(k)
AttributeError: host
-------
Maybe some people think it is the novaclient bug.
In this patch https:/
it fix the hosts.get(), but lead to hosts.list() cann't work.
Maybe we can add some judge such as(if .eg) in the function of
https:/
but I think unite the key to 'host' in nova is much better, if such we don't need to change novaclient and nova code looks better.
Changed in nova: | |
assignee: | nobody → tinytmy (tangmeiyan77) |
description: | updated |
Changed in nova: | |
importance: | Undecided → Low |
status: | New → Confirmed |
Changed in nova: | |
status: | Confirmed → In Progress |
Changed in tempest: | |
assignee: | nobody → tinytmy (tangmeiyan77) |
Changed in nova: | |
assignee: | tinytmy (tangmeiyan77) → nobody |
status: | In Progress → Confirmed |
Fix proposed to branch: master /review. openstack. org/152416
Review: https:/