[GitHub] storm issue #2032: [STORM-2093] Fix permissions in multi-tenant, secure mode

Github user revans2 commented on the issue: https://github.com/apache/storm/pull/2032 @HeartSaVioR I'm not sure if @ppoulosk is going to be able to finish this, so @Ethanlm has submitted #2187 to take this over. --- If your project is set up for it, you can reply to this email and ha

[GitHub] storm issue #2032: [STORM-2093] Fix permissions in multi-tenant, secure mode

Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/2032 @ppoulosk @revans2 Ah I missed that processLauncherAndWait is moved to ClientSupervisorUtils, which resides on storm-client. Since storm-server depends on storm-client, changing SupervisorUt

[GitHub] storm issue #2032: [STORM-2093] Fix permissions in multi-tenant, secure mode

Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/2032 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. --- If your project is set up for it, you can reply to this

[GitHub] storm issue #2032: [STORM-2093] Fix permissions in multi-tenant, secure mode

Github user revans2 commented on the issue: https://github.com/apache/storm/pull/2032 @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 so

[GitHub] storm issue #2032: [STORM-2093] Fix permissions in multi-tenant, secure mode

Github user revans2 commented on the issue: https://github.com/apache/storm/pull/2032 +1 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] storm issue #2032: [STORM-2093] Fix permissions in multi-tenant, secure mode

Github user harshach commented on the issue: https://github.com/apache/storm/pull/2032 +1 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] storm issue #2032: [STORM-2093] Fix permissions in multi-tenant, secure mode

Github user ppoulosk commented on the issue: https://github.com/apache/storm/pull/2032 Thanks, @HeartSaVioR. Pushed fix. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] storm issue #2032: [STORM-2093] Fix permissions in multi-tenant, secure mode

Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/2032 @ppoulosk There's compilation error on Travis build. Could you check? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your