On Fri, Sep 27, 2013 at 3:35 PM, Raffa Santi <[email protected]> wrote:
> This patch set builds upon the work of Pulkit Singhal and adds userspace > RBD > support for KVM to Ganeti, while laying the foundation for further work of > this > kind. Main differences from the original patch set: > > * More, better sanity checks. Checks are added at cluster and group set > params > time and at instance creation time. This means we should catch > configuration > errors even if the config.data file is modified by hand. > * Attempt to abstract this feature as an "access protocol", in order to > facilitate future work on Gluster > * More work on test coverage, plus maintainance of existing tests > > Possible problems: > > * LDP_ACCESS vs RDP_ACCESS. I admit I'm currently not 100% certain of what > does > what. This was introduced by Pulkit. > LDP stands for 'logical disk parameters'. That's the parameters on individual disks, whereas RBD_ACCESS (I guess you meant that, right?) is a parameter on disk templates, thus all disks of an instance. Helga just recently removed the logical disk types in favor of disk templates on individual disks too, so maybe there's more stuff we could unify (or at least remove the LDP_* constants all together). Helga, what to you think? > > Raffa Santi (6): > Define 'access' disk-parameter type. > New method GetUserspaceAccessURI in storage class > Add Userspace RBD support in KVM > Check validity of the access protocol parameter > Add tests specific to the user/kernelspace access feature > Update NEWS and docs to document access switch > > NEWS | 8 ++++ > doc/install.rst | 48 > ++++++++++++++++++----- > lib/backend.py | 2 +- > lib/cmdlib/cluster.py | 3 +- > lib/cmdlib/common.py | 54 > ++++++++++++++++++++++++++ > lib/cmdlib/group.py | 5 ++- > lib/cmdlib/instance.py | 14 +++++++ > lib/config.py | 9 +++++ > lib/constants.py | 14 ++++++- > lib/hypervisor/hv_kvm.py | 9 +++-- > lib/hypervisor/hv_xen.py | 2 +- > lib/objects.py | 1 + > lib/storage/base.py | 12 ++++++ > lib/storage/bdev.py | 9 +++++ > man/gnt-cluster.rst | 15 +++++++ > src/Ganeti/HsConstants.hs | 23 +++++++++++ > src/Ganeti/Types.hs | 26 +++++++++++++ > test/py/cmdlib/cluster_unittest.py | 23 ++++++++++- > test/py/ganeti.hypervisor.hv_xen_unittest.py | 32 ++++++++++----- > 19 files changed, 279 insertions(+), 30 deletions(-) > > -- > 1.7.10.4 > > -- Thomas Thrainer | Software Engineer | [email protected] | Google Germany GmbH Dienerstr. 12 80331 München Registergericht und -nummer: Hamburg, HRB 86891 Sitz der Gesellschaft: Hamburg Geschäftsführer: Graham Law, Christine Elizabeth Flores
