yahoo-eng-team team mailing list archive

["Dmitry \"Divius\" Tantsur" <divius.inside@xxxxxxxxx>]

Ironic driver now leaves in Nova tree, so we can't fix this bug on
Ironic side.

** Also affects: nova
   Importance: Undecided
       Status: New

** Changed in: ironic
       Status: In Progress => Invalid

-- 
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/1308171

Title:
  Ironic nova driver should be more efficient with ironic client calls

Status in OpenStack Bare Metal Provisioning Service (Ironic):
  Invalid
Status in OpenStack Compute (Nova):
  New

Bug description:
  When the nova ironic driver makes ironic client calls, it currently
  gets a brand new client for each call. This can be done more
  efficiently by caching the client and re-authenticating as needed
  (e.g., when the auth token expires). With recent changes to the driver
  to cleanup the retry logic, this should be easier to do. [1][2] The
  driver class can contain a single IronicClientWrapper reference to use
  for the client calls, and the IronicClientWrapper object can handle
  caching and reauthenticating as needed.

  [1] https://review.openstack.org/83105
  [2] https://review.openstack.org/86993

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