Alon Bar-Lev has posted comments on this change.

Change subject: packaging: setup: Allow dwh on separate host
......................................................................


Patch Set 2:

> I also moved the db creds questions, similar to the engine's common.

probably best is to move the cred questions to utility at setup lib and call it 
from all plugins, then the reports can just call it twice without duplicating 
logic. this will not require additional package, we already have too much of 
these already.

> I was actually pretty certain you'll like this one. This will allow in the 
> future to package more than one set of "top-level" packages - users that want 
> remote db servers for some/all components, will not be forced to install a 
> local db server.

When do we force install a local database server? (except of the ovirt-engine 
metapackage)

-- 
To view, visit http://gerrit.ovirt.org/27516
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I06fc960481af258b3954a8968be3439393d3ebdb
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-dwh
Gerrit-Branch: master
Gerrit-Owner: Yedidyah Bar David <d...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbona...@redhat.com>
Gerrit-Reviewer: Yaniv Dary <yd...@redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <d...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to