yahoo-eng-team team mailing list archive

[Ed Leafe <ed@xxxxxxxxx>]

Public bug reported:

The _get_node() method in the Ironic virt driver has the following
signature and docstring:

    def _get_node(self, node_uuid):
        """Get a node by its UUID."""

However, there are 4 places in the code where this method is called that
passes the string for the nodename instead:

node_is_available()
_node_from_cache()
plug_vifs()
unplug_vifs()

I don't know how any of these are working, but I guess the periodic
update that correctly calls _get_node() with a UUID keeps the node cache
current.

** Affects: nova
     Importance: Low
         Status: New


** Tags: ironic virt

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1704231

Title:
  Ironic driver calling _get_node() with nodename

Status in OpenStack Compute (nova):
  New

Bug description:
  The _get_node() method in the Ironic virt driver has the following
  signature and docstring:

      def _get_node(self, node_uuid):
          """Get a node by its UUID."""

  However, there are 4 places in the code where this method is called
  that passes the string for the nodename instead:

  node_is_available()
  _node_from_cache()
  plug_vifs()
  unplug_vifs()

  I don't know how any of these are working, but I guess the periodic
  update that correctly calls _get_node() with a UUID keeps the node
  cache current.

To manage notifications about this bug go to:
https://bugs.launchpad.net/nova/+bug/1704231/+subscriptions