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 <edwa...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Edward Haas <edwa...@redhat.com>
Gerrit-Reviewer: Freddy Rolland <froll...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com>
Gerrit-Reviewer: Ondřej Svoboda <osvob...@redhat.com>
Gerrit-Reviewer: Petr Horáček <phora...@redhat.com>
Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org>
Gerrit-HasComments: No
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to