Change in mom[master]: Include line number in parsing and runtime policy error mess...

alitke at redhat.com alitke at redhat.com
Tue Jun 9 13:23:31 UTC 2015


Adam Litke has submitted this change and it was merged.

Change subject: Include line number in parsing and runtime policy error messages
......................................................................


Include line number in parsing and runtime policy error messages

The current code only returns and error without any localization.
Line number helps to identify the policy code error place.

Signed-off-by: Martin Sivak <msivak at redhat.com>
Change-Id: Ic4d0c51ad0b008c1439ca422c5032285adeb3f68
Reviewed-on: https://gerrit.ovirt.org/42043
Reviewed-by: Adam Litke <alitke at redhat.com>
Reviewed-by: Jenkins CI
Tested-by: Jenkins CI
---
M mom/Policy/Parser.py
M mom/Policy/spark.py
M tests/ParserTests.py
3 files changed, 106 insertions(+), 45 deletions(-)

Approvals:
  Adam Litke: Looks good to me, approved
  Martin Sivák: Verified
  Jenkins CI: Verified; Looks good to me, but someone else must approve



-- 
To view, visit https://gerrit.ovirt.org/42043
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic4d0c51ad0b008c1439ca422c5032285adeb3f68
Gerrit-PatchSet: 4
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: Jenkins CI
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list