Bug 1318665 - [RFE] - make DWH required for engine.
Summary: [RFE] - make DWH required for engine.
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: RFEs
Version: 4.0.0
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: ovirt-4.0.0-beta
: 4.0.0
Assignee: Yedidyah Bar David
QA Contact: Lukas Svaty
URL:
Whiteboard:
Depends On: 1350166
Blocks: ovirt_system_dashboards 1337280 1351864
TreeView+ depends on / blocked
 
Reported: 2016-03-17 13:23 UTC by Yaniv Lavi
Modified: 2016-07-05 07:55 UTC (History)
6 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2016-07-05 07:55:49 UTC
oVirt Team: Integration
Embargoed:
rule-engine: ovirt-4.0.0+
lsvaty: testing_plan_complete-
ylavi: planning_ack+
sbonazzo: devel_ack+
pstehlik: testing_ack+


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 56254 0 master MERGED pacakging: spec: Require dwh-setup 2020-08-31 12:20:08 UTC
oVirt gerrit 56255 0 master MERGED packaging: setup: Note that dwh is required 2020-08-31 12:20:07 UTC
oVirt gerrit 56296 0 master MERGED packaging: setup: add common function getDBConfig 2020-08-31 12:20:07 UTC
oVirt gerrit 56297 0 master MERGED packaging: setup: Configure access to dwh db 2020-08-31 12:20:07 UTC
oVirt gerrit 56321 0 master MERGED ovirt-engine_upgrade_to_master: add missing key 2020-08-31 12:20:07 UTC
oVirt gerrit 56341 0 master MERGED packaging: setup: initialize DWHCoreEnv.ENABLE 2020-08-31 12:20:06 UTC
oVirt gerrit 56462 0 master MERGED packaging: setup: getDBConfig: Allow overriding host 2020-08-31 12:20:06 UTC
oVirt gerrit 56465 0 master MERGED packaging: setup: Copy db creds to engine 2020-08-31 12:20:06 UTC
oVirt gerrit 57047 0 master MERGED packaging: setup: remote_engine: Allow chmod/chown 2020-08-31 12:20:05 UTC
oVirt gerrit 57048 0 master MERGED packaging: setup: remote_engine: Prompt after copy_to_engine 2020-08-31 12:20:06 UTC

Description Yaniv Lavi 2016-03-17 13:23:05 UTC
Description of problem:
Make DWH required for engine for dashboards usage.

1. DWH setup plugin will be mandatory. In engine-setup 'configure
dwh?' question, we'll add text saying that dwh is mandatory, and that
if user chooses 'no', remote dwh should be set up.

2. We'll add a question about "minimal or full" sampling/history/etc. Minimal will be the default.

3. If on same host, update engine conf to access dwh.

4. If dwh is remote, add text to inform user how to update engine conf
to access dwh.

We should probably also:

5. Document how to change from full to partial history collection and
vice-verse.

6. Update existing documentation for migrating dwh to a remote host to
also mention needing to update engine conf.

Comment 1 Yedidyah Bar David 2016-04-14 15:08:38 UTC
What is the format and location of the conf file we need to add to the engine so that it can access the DWH database? Perhaps simply the existing one dwh uses /etc/ovirt-engine-dwh/ovirt-engine-dwh.d/10-setup-database.conf ? Perhaps copy it to /etc/ovirt-engine/engine.conf.d ?

Comment 2 Yedidyah Bar David 2016-04-17 11:50:56 UTC
I guess I can find answers to most of my questions in:

https://gerrit.ovirt.org/#/c/54058/30/packaging/services/ovirt-engine/ovirt-engine.xml.in

Moving needinfo to Alexander, just in case.

Comment 3 Alexander Wels 2016-04-25 11:49:17 UTC
Yes, you can find the answers in ovirt-engine.xml.in but here they are just in case.

DWH_DB_HOST="localhost"
DWH_DB_PORT="5432"
DWH_DB_USER="ovirt_engine_dwh"
DWH_DB_PASSWORD="ovirt_engine_dwh"
DWH_DB_DATABASE="ovirt_engine_history"
DWH_DB_SECURED="False"
DWH_DB_SECURED_VALIDATION="False"
DWH_DB_DRIVER="org.postgresql.Driver"
DWH_DB_URL="jdbc:postgresql://localhost:5432/ovirt_engine_history?sslfactory=org.postgresql.ssl.NonValidatingFactory"

Comment 4 Yedidyah Bar David 2016-05-01 06:21:53 UTC
(In reply to Alexander Wels from comment #3)
> Yes, you can find the answers in ovirt-engine.xml.in but here they are just
> in case.
> 
> DWH_DB_HOST="localhost"
> DWH_DB_PORT="5432"
> DWH_DB_USER="ovirt_engine_dwh"
> DWH_DB_PASSWORD="ovirt_engine_dwh"
> DWH_DB_DATABASE="ovirt_engine_history"
> DWH_DB_SECURED="False"
> DWH_DB_SECURED_VALIDATION="False"
> DWH_DB_DRIVER="org.postgresql.Driver"
> DWH_DB_URL="jdbc:postgresql://localhost:5432/
> ovirt_engine_history?sslfactory=org.postgresql.ssl.NonValidatingFactory"

IOW, identical to what we already write in [1] for DWH's own use.

BUT, note that in the copy for the engine, we'll keep the DWH machine
FQDN, in both DWH_DB_HOST and DWH_DB_URL, IF db host is 'localhost' there,
so that the engine can access it even if it's remote. It's then up to the
user to make sure that this fqdn resolves correctly from the engine side
(and is accessible in the firewall etc).

[1] /etc/ovirt-engine-dwh/ovirt-engine-dwhd.conf.d/10-setup-database.conf

Comment 5 Sandro Bonazzola 2016-05-02 10:01:06 UTC
Moving from 4.0 alpha to 4.0 beta since 4.0 alpha has been already released and bug is not ON_QA.

Comment 6 Yaniv Lavi 2016-05-04 14:20:48 UTC
The user should have a option in setup to reconfigure DWH to move from full blown to minimal and vice versa. The default should be minimal with the engine on tha same machine and full blown on a separate installation.

Comment 7 Yedidyah Bar David 2016-05-10 10:15:27 UTC
Setting doc text -. I expect proper documentation in the main docs etc. to accompany the dashboards themselves.

Comment 9 Lukas Svaty 2016-06-15 09:16:00 UTC
verified in ovirt-engine-4.0.0.4-0.1.el7ev.noarch

Comment 10 Sandro Bonazzola 2016-07-05 07:55:49 UTC
oVirt 4.0.0 has been released, closing current release.


Note You need to log in before you can comment on or make changes to this bug.