Change in ovirt-engine[ovirt-engine-4.1]: restapi: Reduce severity of 'path not found'

Code Review gerrit at ovirt.org
Thu Mar 9 16:05:44 UTC 2017


>From Tal Nisan <tnisan at redhat.com>:

Tal Nisan has submitted this change and it was merged.

Change subject: restapi: Reduce severity of 'path not found'
......................................................................


restapi: Reduce severity of 'path not found'

Currently when the API doesn't isn't able to calculate the URL path for
a class, it sends an ERROR message to the log, something like this:

  ERROR Can't find relative path for class "o.o.e.a.r.VmDisksResource", will return null

That isn't really an error, rather a hint for developers that something
may be missing in the 'LinkHelper' class. But the ERROR serverity alarms
users. This patch changes the severity to WARNING, which is closer to
the real meaning.

Change-Id: Ib65c904360787d50d22d21fe50747b6ed7453a01
Related-To: https://bugzilla.redhat.com/1410038
Signed-off-by: Juan Hernandez <juan.hernandez at redhat.com>
(cherry picked from commit da7e8b20a4a04e128993a91b23da057804585e45)
---
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/util/LinkHelper.java
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Juan Hernandez: Verified
  Jenkins CI: Passed CI tests
  Oved Ourfali: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib65c904360787d50d22d21fe50747b6ed7453a01
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.1
Gerrit-Owner: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list