[STORM-2093] Fix permissions in multi-tenant, secure mode

[ppoulosk]

Heap dumps created on OOM, when served through the logviewer in secure multi-tenant configurations have permissions set such that the logviewer cannot read them.

This change checks permissions in this case before serving and then runs the worker-launcher to enable the logviewer to serve them.

[HeartSaVioR]

@ppoulosk
There's compilation error on Travis build. Could you check?

[ppoulosk]

Thanks, @HeartSaVioR. Pushed fix.

[harshach]

+1

[revans2]

+1

[revans2]

@ppoulosk Sadly with the latest refactoring of the code SupervisorUtils.processLauncherAndWait is not on the classpath for storm-core any more. It is a part of storm-server. You may need to do some refactoring to make this work.

[HeartSaVioR]

storm-core depends on storm-server, so that would be fine for now. We may want to address this when we move logviewer into webservice.

[HeartSaVioR]

@ppoulosk@revans2
Ah I missed that processLauncherAndWait is moved to ClientSupervisorUtils, which resides on storm-client. Since storm-server depends on storm-client, changing SupervisorUtils to ClientSupervisorUtils would be good to go.

[revans2]

@HeartSaVioR I'm not sure if @ppoulosk is going to be able to finish this, so @Ethanlm has submitted #2187 to take this over.