Bug 1856002 - dwh setup automatically restarts ovirt-engine and still asks the user to do it manually
Summary: dwh setup automatically restarts ovirt-engine and still asks the user to do i...
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: ovirt-engine-dwh
Classification: oVirt
Component: Setup
Version: 4.4.1.2
Hardware: Unspecified
OS: Unspecified
medium
low
Target Milestone: ---
: ---
Assignee: Yedidyah Bar David
QA Contact: Lucie Leistnerova
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2020-07-11 17:27 UTC by Lucie Leistnerova
Modified: 2021-05-20 07:07 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2021-05-20 07:07:18 UTC
oVirt Team: Integration
Embargoed:


Attachments (Terms of Use)

Description Lucie Leistnerova 2020-07-11 17:27:54 UTC
Description of problem:
When installing dwn on separate machine, setup asks about restarting the ovirt-engine automatically. User says Yes, engine is restarted, but still in the end of the setup it asks to do it manually.

Version-Release number of selected component (if applicable):
ovirt-engine-dwh-setup-4.4.1.2-1.el8ev.noarch

How reproducible: always


Steps to Reproduce:
1. install dwh on separate machine
2. answer 'Yes' to 
The engine should be restarted for Single-Sign-On (SSO) to work. Do this as part of Setup? If not, you will have to do this later by yourself
3.

Actual results:
setup output contains
          Please restart the engine by running the following on <engine fqdn> :
          # service ovirt-engine restart
          This is required for the dashboard to work.


Expected results: no restart information


Additional info:
When user answers 'No', there is this restart info twice
+
          Please run the following command on the engine machine <engine fqdn>, for SSO to work:
          systemctl restart ovirt-engine

Comment 1 Yedidyah Bar David 2020-07-16 13:51:21 UTC
These are two separate issues - one is done for dwh (so that the dashboard works), other for grafana (so that SSO works, until we fix bug 1846294 - then we should remove this part).

The right solution is probably to add code to engine-common plugin to do a remote restart of the engine, conditionally, and allow other plugins to set the condition. I need to think a bit about this - especially about the texts - we probably want to show all of them, e.g.:

The remote engine should be restarted for these reasons:
- Grafana SSO
- Admin UI dashboard

And collect the actual reasons (the bulleted lines) from the plugins.

Comment 2 Yedidyah Bar David 2020-07-16 13:53:15 UTC
This means that it will require patches for both the engine and dwh.

BTW, not always you run into this - if you e.g. setup engine+dwh on one machine, and grafana on another (not sure we decided to support this officially, but it should work, more-or-less), you'll only see the text/prompt from grafana.


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