Ironic

Pull latest changes to UOM from Nova

Bug #1259527 reported by Mike Durnosvistov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic
Fix Released
Low
Unassigned
Ironic 2014.1 "icehouse"

Bug Description

Almost every object thus far has done this:

  updates = {}
  changes = self.obj_what_changed()
  for field in changes:
        updates[field] = self[field]

It's need to add that as part of the base object and makes every place that does the above to just use that.

Mike Durnosvistov (glacierrdev)
Changed in ironic:
assignee: nobody → Mikhail Durnosvistov (mdurnosvistov)
Mike Durnosvistov (glacierrdev)
Changed in ironic:
importance: Undecided → Low
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ironic (master)

Fix proposed to branch: master
Review: https://review.openstack.org/61143

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ironic (master)

Reviewed: https://review.openstack.org/61143
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=45406b013264540646429fa81eb758ed3ec79f15
Submitter: Jenkins
Branch: master

commit 45406b013264540646429fa81eb758ed3ec79f15
Author: Mikhail Durnosvistov <email address hidden>
Date: Tue Dec 10 09:52:12 2013 -0500

    Clean up duplicated change-building code in objects

    Almost every object thus far has done this:

      updates = {}
      changes = self.obj_what_changed()
      for field in changes:
            updates[field] = self[field]

    to get a dict of updates to apply to the database. This patch adds
    that as part of the base object and makes every place that does
    the above to just use that.

    It's a port from a patch in nova ,link to the origional patch:
    https://review.openstack.org/#/c/45353/

    Change-Id: I8d2616ecc2c88cf4b21e7e8193f5b985407ed869
    Closes-Bug: #1259527

Changed in ironic:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in ironic:
milestone: none → icehouse-2
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in ironic:
milestone: icehouse-2 → 2014.1
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.