Change in mom[master]: Create new evaluator for each evaluation run

alitke at redhat.com alitke at redhat.com
Tue Nov 26 16:08:17 UTC 2013


Adam Litke has submitted this change and it was merged.

Change subject: Create new evaluator for each evaluation run
......................................................................


Create new evaluator for each evaluation run

MoM used one evaluator for all the policy runs. That caused weird
side effects because the stack already contained old values from
previous runs.

Policies could not be evaluated properly because we changed defvar
semantic from set to setdefault in one of previous patches.

Change-Id: I3d5343e94d8ecddca0e0c9dd7fc893d94e751378
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1025845
Signed-off-by: Martin Sivak <msivak at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/21704
Reviewed-by: Adam Litke <alitke at redhat.com>
---
M mom/Policy/Policy.py
1 file changed, 6 insertions(+), 5 deletions(-)

Approvals:
  Adam Litke: Looks good to me, approved
  Martin Sivák: Verified



-- 
To view, visit http://gerrit.ovirt.org/21704
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I3d5343e94d8ecddca0e0c9dd7fc893d94e751378
Gerrit-PatchSet: 2
Gerrit-Project: mom
Gerrit-Branch: master
Gerrit-Owner: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>



More information about the Engine-commits mailing list