Bug 977440 - Share the engine's os info with DWH through a translation table
Summary: Share the engine's os info with DWH through a translation table
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Enterprise Virtualization Manager
Classification: Red Hat
Component: ovirt-engine
Version: 3.3.0
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: 3.3.0
Assignee: Roy Golan
QA Contact: Ilanit Stein
URL:
Whiteboard: virt
Depends On:
Blocks: 985346 1019461
TreeView+ depends on / blocked
 
Reported: 2013-06-24 14:37 UTC by Roy Golan
Modified: 2015-09-22 13:09 UTC (History)
12 users (show)

Fixed In Version: is15
Doc Type: Bug Fix
Doc Text:
Clone Of:
: 985346 (view as bug list)
Environment:
Last Closed:
oVirt Team: ---
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 17767 0 None None None Never

Description Roy Golan 2013-06-24 14:37:37 UTC
Description of problem:
in 3.3 we adding the ability to add user-defined OSs to the engine.
Reports on vm os type will end with up with new entries without the actual name of the OS, as kept in the engine.

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:
DWH keeps its own translation based on the VmOsType enum and won't have any info
on new OSs

Expected results:
DWH will be able to read the mapping from the engine db. The data will relay in 
a dedicated dwh_osinfo table.

Additional info:

Comment 1 Yaniv Lavi 2013-06-25 12:59:58 UTC
This is a documentation bug. We need to add a guide to adding new OSs to the history database. Zac, which component to move this to? 



Yaniv

Comment 2 Roy Golan 2013-07-17 10:32:31 UTC
resetting this bug to the engine
Yaniv I cloned one for the DWH

Comment 3 Itamar Heim 2013-07-18 04:37:21 UTC
what happens if user adds an OS, then removes it?
or adds an OS, then removes it, then adds another OS with same ID?
how will localization be handled?
do we even support using OsInfo by customers in 3.3 before we used it ourselves for a version and are sure we thought through all the upgrade issues on customer changes?

Comment 4 Barak 2013-08-01 13:03:42 UTC
A short explanation about this issue:

Since in 3.3 a customer can add a guest OS on his own,
And the list is currently is hardcoded in the history DB, than we need a mechanism to collect the other guest OSs.

It needs to be dynamic.

So there are 2 bugs:
- this virt bug - that is on the backend that should create a table (dwh_osinfo) that needs to be refreshed every engine restart.
- Bug  985346 (on dwh) - to collect data from the dwh_osinfo (only on refresh)

Hence moving this bug back to virt.

Comment 5 Yaniv Lavi 2013-09-09 20:31:03 UTC
Please provide status update.


Yaniv

Comment 6 Roy Golan 2013-09-11 06:34:03 UTC
Yaniv its pending long ago your review in gerrit, after the comment to add an entry in dwh_history_timekeeping

Comment 7 Zac Dover 2013-09-24 05:13:32 UTC
(In reply to Yaniv Dary from comment #1)
> This is a documentation bug. We need to add a guide to adding new OSs to the
> history database. Zac, which component to move this to? 
> 
> 
> 
> Yaniv

I am sorry about the extreme lateness of this reply; it bypassed my inbox entirely.

This bug impacts http://documentation-devel.engineering.redhat.com/docs/en-US/Red_Hat_Enterprise_Virtualization/3.2/html/Administration_Guide/chap-Reports_History_Database_Reports_and_Dashboards.html, to begin with.

If we want to add a section detailing how to add a new OS to the history database, I'll need more information about the feature before I can determine which book it should be added to.

Comment 8 Ilanit Stein 2013-09-24 08:57:31 UTC
Verified on is15.


Added this entry in a new .properties file under /etc/ovirt-engine/osinfo.conf.d:

os.ilanit.id.value = 2000
os.ilanit.name.value = Ilanit
os.ilanit.derivedFrom.value = other
os.ilanit.description.value = General user defined OS
os.ilanit.family.value = user defined
os.ilanit.devices.audio.value = ac97

Comment 9 Itamar Heim 2014-01-21 22:22:53 UTC
Closing - RHEV 3.3 Released

Comment 10 Itamar Heim 2014-01-21 22:24:02 UTC
Closing - RHEV 3.3 Released

Comment 11 Itamar Heim 2014-01-21 22:27:48 UTC
Closing - RHEV 3.3 Released


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