Change in vdsm[master]: net: IP address hostport split tool

nsoffer at redhat.com nsoffer at redhat.com
Thu Mar 10 09:01:12 UTC 2016


Nir Soffer has posted comments on this change.

Change subject: net: IP address hostport split tool
......................................................................


Patch Set 4:

Dan, it is not about verticals, but about minimizing dependencies between packages.

The storage package provides services via its public apis. storage/mount is *not* a storage service - this is a wrapper over the mount command, and should move to lib/vdsm/common.

utils is a good place for the split_host utility, or maybe netutils. I don't care much about the name, only about not adding dependencies on the network package from storage.

Other dependencies from virt to netinfo should be removed as well.

In the long term, I think we should have these packages:

- vdsm-storage
- vdsm-virt
- vdsm-network
- vdsm-common
- vdsm-fence
- vdsm-gluster

And it should be able to install any combination of these packages as needed.

Each one may be a separate process, minimizing downtime of other services when one service panics (e.g. virt loosing connection to libvirt, or storage not having trouble with the spm), overloading due to scale issues, or security (some services running too much code as root).

-- 
To view, visit https://gerrit.ovirt.org/54312
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib246c78987f9707674a1966df91120d4189b41fc
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Edward Haas <edwardh at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Edward Haas <edwardh at redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Ondřej Svoboda <osvoboda at redhat.com>
Gerrit-Reviewer: Petr Horáček <phoracek at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
Gerrit-HasComments: No


More information about the vdsm-patches mailing list