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 <msi...@redhat.com>
Reviewed-on: http://gerrit.ovirt.org/21704
Reviewed-by: Adam Litke <ali...@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 <msi...@redhat.com>
Gerrit-Reviewer: Adam Litke <ali...@redhat.com>
Gerrit-Reviewer: Martin Sivák <msi...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to