Bug 1483889

Summary: Custom Id is not persisted to the database
Product: [oVirt] ovirt-engine Reporter: Eli Mesika <emesika>
Component: Backend.CoreAssignee: Eli Mesika <emesika>
Status: CLOSED CURRENTRELEASE QA Contact: Pavel Stehlik <pstehlik>
Severity: high Docs Contact:
Priority: unspecified    
Version: 4.2.0CC: bugs, lveyde, masayag, mperina
Target Milestone: ovirt-4.1.6Keywords: CodeChange, ZStream
Target Release: 4.1.6.1Flags: rule-engine: ovirt-4.1+
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-09-19 10:02:18 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Infra RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Eli Mesika 2017-08-22 08:36:57 UTC 
Description of problem:

When creating a audit log record, there is an option to assign a custom id string to the record, this is use for example when dealing with interfaces where the custom id value gets the interface name in order to handle event flooding correctly.
However, the custom is value is used when generating the key for the audit log cache but since it is not persisted to the database, each record in the cache that has a non-empty custom id value will not be cleaned from the cache since the key for searching this record will have always an empty custom id value.

This behavior may lead to cache flooding and memory leak.

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


How reproducible:

Internal to code, can be tested only in developer environment using the debugger.


Steps to Reproduce:
1.
2.
3.

Actual results:

custom id is not persisted to he database causing some events not to be cleared from cache


Expected results:

custom id should be persisted to the database in order that the key stored in the cache and the key generated to search the cache will have the same value.

Additional info: