Bug 1846375 - [RFE] Include a link to grafana on front page
Summary: [RFE] Include a link to grafana on front page
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: Frontend.Core
Version: 4.4.1
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ovirt-4.4.1
: 4.4.1.7
Assignee: Sharon Gratch
QA Contact: Pavel Novotny
URL:
Whiteboard:
Depends On:
Blocks: 1854563
TreeView+ depends on / blocked
 
Reported: 2020-06-11 13:11 UTC by Yedidyah Bar David
Modified: 2020-07-22 18:12 UTC (History)
2 users (show)

Fixed In Version: ovirt-engine-4.4.1.7
Clone Of:
: 1854563 (view as bug list)
Environment:
Last Closed: 2020-07-08 08:25:57 UTC
oVirt Team: UX
Embargoed:
pm-rhel: ovirt-4.4?
pm-rhel: planning_ack?
pm-rhel: devel_ack+
lleistne: testing_ack+


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 109706 0 master MERGED engine: add the grafana instance link label text for landing page 2021-02-02 17:43:47 UTC
oVirt gerrit 110050 0 master MERGED engine: Add a new link entry to Grafana on landing page 2021-02-02 17:43:02 UTC
oVirt gerrit 110370 0 master MERGED engine: avoid logging the stack trace in case of non installed Grafana 2021-02-02 17:43:00 UTC

Description Yedidyah Bar David 2020-06-11 13:11:16 UTC
Description of problem:

We now support setting up grafana against DWH's database.

In bug 1845049 we make a link to this grafana instance available to the engine in its conf, as item "ENGINE_GRAFANA_BASE_URL".

It would be nice for the engine to show on the front page a link there, if the item exists and is not empty.

"Workaround": This is also shown by engine-setup when it finishes, if grafana is configured, e.g.:

          Web access for grafana is enabled at:
              https://some.domain/ovirt-engine-grafana/

Comment 1 Sharon Gratch 2020-07-01 17:55:01 UTC
The new "Monitoring Portal" entry is added to the landing page with a link to the Grafana URL.
both u/s and d/s branding are supported.

In case the engine conf file 10-setup-grafana-access.conf or properties are
invalid/missing, this new link will not be displayed on the landing page.

In case the user wants to overwrite the Grafana link, he can manually
set a value for the "portal_monitoring_url" message in external_resources.properties branding file.

If 10-setup-grafana-access.conf is created after engine is
running, refreshing the browser will read the file and add the new link to the landing page,
 so no restart is needed on engine side.

Comment 2 Sharon Gratch 2020-07-02 12:26:48 UTC
(In reply to Sharon Gratch from comment #1)
> 
> If 10-setup-grafana-access.conf is created after engine is
> running, refreshing the browser will read the file and add the new link to
> the landing page,
>  so no restart is needed on engine side.
This is no longer true. The branding won't read new conf file till engine restart. Added a comment on the relevant bug: https://bugzilla.redhat.com/show_bug.cgi?id=1846294#c3

Comment 3 Pavel Novotny 2020-07-07 16:53:08 UTC
VERIFIED in
ovirt-engine-4.4.1.7-0.3.el8ev

New link "Monitoring Portal" is present only on the welcome page" (oVirt upstream only*):

The link URL is defined either as value of ENGINE_GRAFANA_BASE_URL variable in config file /etc/ovirt-engine/engine.conf.d/10-setup-grafana-access.conf (engine restart needed)
or
can be overridden as "obrand.common.portal_monitoring_url" variable in /etc/ovirt-engine/branding/00-ovirt.brand/external_resources.properties branding file (only page reload needed).


(*) The portal link is not present on the downstream RHV welcome page (rhvm-branding-rhv-4.4.3-1.el8ev), the HTML part is missing in the welcome_page.template file.
    Cloning to downstream.

Comment 4 Sandro Bonazzola 2020-07-08 08:25:57 UTC
This bugzilla is included in oVirt 4.4.1 release, published on July 8th 2020.

Since the problem described in this bug report should be resolved in oVirt 4.4.1 release, it has been closed with a resolution of CURRENT RELEASE.

If the solution does not work for you, please open a new bug report.


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