[GitHub] storm pull request #2390: [STORM-2790] Add nimbus admins groups

[Ethanlm]

Github user Ethanlm commented on a diff in the pull request:

    https://github.com/apache/storm/pull/2390#discussion_r147775339
  
    --- Diff: 
external/storm-hdfs/src/test/java/org/apache/storm/hdfs/blobstore/BlobStoreTest.java
 ---
    @@ -92,6 +93,19 @@ public static void cleanupAfterClass() throws 
IOException {
       // Method which initializes nimbus admin
       public static void initializeConfigs() {
         conf.put(Config.NIMBUS_ADMINS,"admin");
    +    conf.put(Config.NIMBUS_ADMINS_GROUPS,"adminsGroup");
    +
    --- End diff --
    
    nit: space after comma?


---