yahoo-eng-team team mailing list archive

[Chris Dent <cdent+lp@xxxxxxxxxxxxx>]

Public bug reported:

The resource tracker has a dict, named 'tracked_instances' which based
on the name is used to keep track of the instances that the resource
tracker is tracking.

However, on every run of the 'update_available_resource' method, the
'_update_usage_from_instances' method is called and in there the
tracked_instances dict is cleared. This means that the conditionals in
the '_update_usage_from_instance' (singular) method always indicate that
the current instance is considered new and the various update methods
for that instance will always be called.

In the case of the calls to the placement API, this means there are many
extra calls which could be avoided.

Removing the clear() call results in no unit or functional test
failures. A test run the gate will be tried.

** Affects: nova
     Importance: Undecided
         Status: New


** Tags: resource-tracker scheduler

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

Title:
  The resource tracker clears the tracked_instances dictionary on every
  periodic job

Status in OpenStack Compute (nova):
  New

Bug description:
  The resource tracker has a dict, named 'tracked_instances' which based
  on the name is used to keep track of the instances that the resource
  tracker is tracking.

  However, on every run of the 'update_available_resource' method, the
  '_update_usage_from_instances' method is called and in there the
  tracked_instances dict is cleared. This means that the conditionals in
  the '_update_usage_from_instance' (singular) method always indicate
  that the current instance is considered new and the various update
  methods for that instance will always be called.

  In the case of the calls to the placement API, this means there are
  many extra calls which could be avoided.

  Removing the clear() call results in no unit or functional test
  failures. A test run the gate will be tried.

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