Yedidyah Bar David has posted comments on this change.

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


Patch Set 2:

> The constants of dwh within reports are on reports constants, these are 
> duplicated, no need to add new package.

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

> the engine common was design for this, no reason why not to use single common 
> component between all components.

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.

-- 
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 <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Sandro Bonazzola <[email protected]>
Gerrit-Reviewer: Yaniv Dary <[email protected]>
Gerrit-Reviewer: Yedidyah Bar David <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to