You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@storm.apache.org by tedxia <gi...@git.apache.org> on 2015/06/25 14:16:47 UTC

[GitHub] storm pull request: STORM-910: Submit job failed when submitter us...

GitHub user tedxia opened a pull request:

    https://github.com/apache/storm/pull/603

    STORM-910: Submit job failed when submitter user is also set in nimbu…

    …s.supervisor.users

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/tedxia/storm ted-fix--bug-with-SimpleACLAuthorizer

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/storm/pull/603.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #603
    
----
commit 08dcda5f82dcf0a5fb89c6aa9e70fc6b39168b53
Author: xiajun <xi...@xiaomi.com>
Date:   2015-06-25T12:15:10Z

    STORM-910: Submit job failed when submitter user is also set in nimbus.supervisor.users

----


---
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 is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] storm pull request: STORM-910: Submit job failed when submitter us...

Posted by tedxia <gi...@git.apache.org>.
Github user tedxia closed the pull request at:

    https://github.com/apache/storm/pull/603


---
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 is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] storm pull request: STORM-910: Submit job failed when submitter us...

Posted by revans2 <gi...@git.apache.org>.
Github user revans2 commented on the pull request:

    https://github.com/apache/storm/pull/603#issuecomment-115284737
  
    This was actually done on purpose.  If you have run-as-user enabled and you run a topology as the supervisor user, that topology now has the ability to launch things as other users.  We put it here so users would get an error early.  If you want to run this way I really would like to see an explicit configuration to enable it that is off by default.


---
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 is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] storm pull request: STORM-910: Submit job failed when submitter us...

Posted by tedxia <gi...@git.apache.org>.
Github user tedxia commented on the pull request:

    https://github.com/apache/storm/pull/603#issuecomment-115465234
  
    Thanks for the explain, I will close this patch.


---
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 is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---