diff options
author | Daniel P. Berrange <berrange@redhat.com> | 2012-12-17 20:45:03 +0000 |
---|---|---|
committer | Daniel P. Berrange <berrange@redhat.com> | 2013-01-03 13:46:26 +0000 |
commit | b1298cc990e80ec7b51f353e143fd15888be53b4 (patch) | |
tree | baa8276454de8e2163d2522fe1fbc5a1a6f4c0ab /nova/exception.py | |
parent | 7240f7d480a14e1dfde83400af2bb385b4e3b9e9 (diff) | |
download | nova-b1298cc990e80ec7b51f353e143fd15888be53b4.tar.gz nova-b1298cc990e80ec7b51f353e143fd15888be53b4.tar.xz nova-b1298cc990e80ec7b51f353e143fd15888be53b4.zip |
Delete pointless nova.virt.VIFDriver class
There is a nova.virt.VIFDriver class which the hypervisor
VIF drivers inherit from. None of the hypervisor subclasses
pay any attention to the API contract defined to the parent
class. Some of the hypervisors don't bother trying to use
the VIFDriver class at all (hyperv, baremetal).
It is not worth trying to make the hypervisor subclasses
follow the same API contract, since the base parent class
offers no re-usable functionality of its own. The hypervisor
base classs nova.virt.ComputeDriver also has no need for the
VIFDriver API to exist, since its integration point is the
'plug_vifs' method.
As such the nova.virt.VIFDriver class serves no purpose
other than to mislead reviewers into thinking there is a
common API where none actually exists. It should thus be
deleted
Change-Id: Id287f7e66143a248a5f106e010510a3f09d2b1e7
Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
Diffstat (limited to 'nova/exception.py')
0 files changed, 0 insertions, 0 deletions