]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Fix backups.rpcapi to pass objects over RPC
authorMichal Dulko <michal.dulko@intel.com>
Tue, 23 Jun 2015 13:42:00 +0000 (15:42 +0200)
committerMichal Dulko <michal.dulko@intel.com>
Tue, 23 Jun 2015 13:42:00 +0000 (15:42 +0200)
commitaff4c847adb10552373055c1a0da0e27bd7984a4
tree811cb366cc5654a1cfff92c9d7bdd293dc84d8ab
parent5f6cbecc5ec5b6ca034290dc7bd04ab7d956cd15
Fix backups.rpcapi to pass objects over RPC

Commit Icff37261b367463b71a1268be16f9c97f595bf0c brought bugs in
backup.rpcapi. For export_record, import_record and reset_status
backup.id is passed instead of whole backup object - which
backup.manager expects. This causes severe failures. This commit
changes the rpcapi to pass objects and adds missing
backup.test_rpcapi unit tests that will prevent such failures in
the future.

Change-Id: I303b77214785165d37663fe8d502988e86a7950f
Closes-Bug: 1467841
cinder/backup/rpcapi.py
cinder/tests/unit/backup/test_rpcapi.py [new file with mode: 0644]
cinder/tests/unit/fake_backup.py [new file with mode: 0644]