Bug 1056577

Summary: osinfo - property inheritance doesn't cover minor version
Product: [Retired] oVirt Reporter: Roy Golan <rgolan>
Component: ovirt-engine-coreAssignee: Roy Golan <rgolan>
Status: CLOSED NOTABUG QA Contact: bugs <bugs>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 3.5CC: gklein, gustavo.pedrosa, leonardo.bianconi, michal.skrivanek, ofrenkel, rbalakri, rgolan, vdelima, vitor.lima, yeylon
Target Milestone: ---   
Target Release: 3.6.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: virt
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-03-30 10:10:44 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Virt RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Roy Golan 2014-01-22 13:50:27 UTC 
Description of problem:

the lookup algorithem for a derived value by version is naive and rely on making
the default version-less values. 
this doesn't scale when features update and have a default set in a later version

os.base.k.value = X
os.derived.k.value.3.2 = Y

os.derived.k.value.3.3 = ?   - this should be Y. instead it X

to overcome that we duplicate a property and bump the version value
os.derived.k.value.3.3 = Y

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

How reproducible:
100%

Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:

we be able to get Y for os.derived.k.value.3.3 without adding an entry

Additional info:
Comment 1 Itamar Heim 2014-01-26 08:11:54 UTC 
Setting target release to current version for consideration and review. please
do not push non-RFE bugs to an undefined target release to make sure bugs are
reviewed for relevancy, fix, closure, etc.
Comment 2 Michal Skrivanek 2014-08-14 13:37:53 UTC 
Roy, you need to revive and finish the patch I think
Comment 3 Michal Skrivanek 2014-10-31 08:38:55 UTC 
Roy? I think it's really worth finishing this one eventually, it's a great enhancement
Comment 4 Michal Skrivanek 2014-12-01 12:36:04 UTC 
nothing going on so far, pushing out of 3.5
Comment 5 Roy Golan 2015-03-30 10:10:44 UTC 
the out of the box default values is good enough so there is no need for that change. 

see the merged https://gerrit.ovirt.org/#/c/39077/ or https://gerrit.ovirt.org/gitweb?p=ovirt-engine.git;a=commit;h=829815c6a245c4ace32a248123c13fb37e8bb14e