Bug 1365808 - engine-backup command and generic messages in web admin events pane
Summary: engine-backup command and generic messages in web admin events pane
Keywords:
Status: CLOSED DUPLICATE of bug 1372320
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: Backup-Restore.Engine
Version: 4.0.0
Hardware: Unspecified
OS: Unspecified
high
medium
Target Milestone: ovirt-4.0.5
: ---
Assignee: bugs@ovirt.org
QA Contact: Lucie Leistnerova
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-08-10 09:06 UTC by Gianluca Cecchi
Modified: 2016-09-20 11:35 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-09-20 11:35:37 UTC
oVirt Team: Infra
Embargoed:
rule-engine: ovirt-4.0.z+
ylavi: planning_ack+
sbonazzo: devel_ack+
lsvaty: testing_ack+


Attachments (Terms of Use)
events in web admin gui when engine-backup command has been executed (53.67 KB, image/png)
2016-08-10 09:06 UTC, Gianluca Cecchi
no flags Details

Description Gianluca Cecchi 2016-08-10 09:06:14 UTC
Created attachment 1189516 [details]
events in web admin gui when engine-backup command has been executed

Description of problem:
When running engine-backup --mode=backup you get generic event messages in web admin gui, wthout being able to correlate with the action executed

Version-Release number of selected component (if applicable):
ovirt-engine-tools-backup-4.0.0.6-1.el7.centos.noarch

How reproducible:
always

Steps to Reproduce:
1. run command
engine-backup --mode=backup --scope=all --file=/root/engine-backup.archive_$(date +%d-%m-%Y) --log=/root/engine-backup_$(date +%d-%m-%Y).log
Backing up:
Notifying engine
- Files
- Engine database 'engine'
- DWH database 'ovirt_engine_history'
Packing into file '/root/engine-backup.archive_09-08-2016'
Notifying engine
Done.

2. you get several events in webadmin gui (see attachment) but without any information. Only rows of kind:
Finished 
Started

3.

Actual results:
No correlation between web admin gui events and backup action executed

Expected results:
Something informational in web admin gui events

Additional info:

Comment 1 Yedidyah Bar David 2016-08-10 09:42:55 UTC
Not sure about the best solution, moving to Integration for now.

Comment 2 Yedidyah Bar David 2016-08-11 08:02:40 UTC
engine-backup calls:

SELECT LogEngineBackupEvent('${scope}', now(), ${status}, '${message}', '${ENGINE_FQDN}', '${logpath}');

Where:
- scope is one of files, db, dwhdb, reportsdb
- status is one of -1, 0, 1
- message is one of 'Started', 'Finished', 'Failed', or an error message
- The rest are as expected

Since engine-backup is not localized, I didn't include longer messages, expecting the db or engine to show something nicer to the user somehow.

LogEngineBackupEvent inserts both into engine_backup_log and into audit_log.
IIUC current bug is about the ui view of audit_log.
Not sure if engine_backup_log is user-viewable anywhere, or only used for monitoring backup status.

I can think of several solutions:
1. Patch engine-backup to have longer messages, but not localized
2. Patch LogEngineBackupEvent or some other db function to do this, no idea if can be localized
3. Keep existing texts in the db and patch the engine to display nicer ones, either in existing audit_log view or in a new engine_backup_log view.

Eli, what do you think?

Comment 3 Eli Mesika 2016-08-14 07:06:01 UTC
(In reply to Yedidyah Bar David from comment #2)

> 1. Patch engine-backup to have longer messages, but not localized
> 2. Patch LogEngineBackupEvent or some other db function to do this, no idea
> if can be localized
> 3. Keep existing texts in the db and patch the engine to display nicer ones,
> either in existing audit_log view or in a new engine_backup_log view.
> 
> Eli, what do you think?

I would go for option #1 
You can simply check the SCOPE_* in packaging/bin/engine-backup.sh.in::dobackup() and according to that provide more detailed message

Comment 4 Yaniv Lavi 2016-09-08 08:53:47 UTC
Please add more meaningful text on the engine side with translations.

Comment 5 Sandro Bonazzola 2016-09-16 14:21:09 UTC
(In reply to Eli Mesika from comment #3)
> I would go for option #1 
> You can simply check the SCOPE_* in
> packaging/bin/engine-backup.sh.in::dobackup() and according to that provide
> more detailed message

Moving to integration going with option #1

Comment 6 Sandro Bonazzola 2016-09-16 14:22:23 UTC
(In reply to Sandro Bonazzola from comment #5)
> (In reply to Eli Mesika from comment #3)
> > I would go for option #1 
> > You can simply check the SCOPE_* in
> > packaging/bin/engine-backup.sh.in::dobackup() and according to that provide
> > more detailed message
> 
> Moving to integration going with option #1

Sorry, didn't see Yaniv request, back to infra.

Comment 7 Martin Perina 2016-09-20 11:35:37 UTC

*** This bug has been marked as a duplicate of bug 1372320 ***


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