Internal Jenkins has submitted this change and it was merged. Change subject: Leopard front end: protect cases where report and schedule paths are absent ......................................................................
Leopard front end: protect cases where report and schedule paths are absent If the leopard front end is run on a pristine system, the front end will raise exceptions, because the directories containing reports and schedules do not yet exist. Since it's the controller's responsibility to create and populate these filesystem artifacts, in the frontend, let's just protect against the exception and log warnings. Testing: Before the fix, the reload_reports thread would immediately raise an exception and the thread would die; loading the Web app in a browser would cause an internal server error. After the fix, the reload_reports thread logs a warning but continues to run, and loading the Web app in a browser presents an appropriately empty listing of run names and schedules. Change-Id: I186174db136694d3df04d9159362c6eeaa70b5b8 Reviewed-on: http://gerrit.cloudera.org:8080/2164 Reviewed-by: Michael Brown <[email protected]> Tested-by: Internal Jenkins --- M tests/comparison/leopard/front_end.py 1 file changed, 21 insertions(+), 5 deletions(-) Approvals: Michael Brown: Looks good to me, approved Internal Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/2164 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I186174db136694d3df04d9159362c6eeaa70b5b8 Gerrit-PatchSet: 3 Gerrit-Project: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: Michael Brown <[email protected]> Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Michael Brown <[email protected]> Gerrit-Reviewer: Taras Bobrovytsky <[email protected]>
