Nova driver does not read hosts table

Bug #1441778 reported by Alexander Yip on 2015-04-08
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
congress
Undecided
Zhenzan Zhou

Bug Description

In nova_driver.py::update_from_datasource(), there is a comment that disables the hosts table.

        # TODO(thinrichs): debug and re-enable
        # self._translate_hosts(self.nova_client.hosts.list())

Figure out why and uncomment this line.

Changed in congress:
assignee: nobody → Zhenzan Zhou (zhenzan-zhou)
Samta Rangare (srangare) wrote :

host translation works fine, so i am re-enabling it.

Here is the data being populated by CLI

openstack congress datasource table list nova

+--------------+
| id |
+--------------+
| flavors |
| hosts |
| floating_IPs |
| servers |
+--------------+

openstack congress datasource row list nova hosts

+-----------+-----------+----------+
| host_name | service | zone |
+-----------+-----------+----------+
| mvista | conductor | internal |
| mvista | network | internal |
| mvista | scheduler | internal |
| mvista | compute | nova |
+-----------+-----------+----------+

Alexander Yip (ayip) wrote :

Sounds good. I'll take a look at the code review.

Reviewed: https://review.openstack.org/172333
Committed: https://git.openstack.org/cgit/stackforge/congress/commit/?id=c6493d0a2f8d27947dfcf8161755ae3ea6d16f33
Submitter: Jenkins
Branch: master

commit c6493d0a2f8d27947dfcf8161755ae3ea6d16f33
Author: Samta Rangare <email address hidden>
Date: Fri Apr 10 13:47:31 2015 +0530

    Nova driver does not read hosts table

    host translation section was commented out, but it works fine.
    Re-enabling it again. Further issues can be rectified by follow-
    up patch.

    Change-Id: Iac4d89de94a1239cf0dbf9249813d8dffd5ce423
    closes-Bug: #1441778

Changed in congress:
status: New → Fix Committed
Tim Hinrichs (thinrichs) on 2015-04-21
Changed in congress:
milestone: none → kilo-3
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers