[jira] [Commented] (YARN-1573) ZK store should pick a private password for root-node-acls

2014-01-22 Thread Sandy Ryza (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-1573?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13879142#comment-13879142
 ] 

Sandy Ryza commented on YARN-1573:
--

+1, LGTM

 ZK store should pick a private password for root-node-acls
 --

 Key: YARN-1573
 URL: https://issues.apache.org/jira/browse/YARN-1573
 Project: Hadoop YARN
  Issue Type: Sub-task
  Components: resourcemanager
Affects Versions: 2.4.0
Reporter: Karthik Kambatla
Assignee: Karthik Kambatla
 Attachments: yarn-1573-1.patch, yarn-1573-2.patch


 Currently, when HA is enabled, ZK store uses cluster-timestamp as the 
 password for root node ACLs to give the Active RM exclusive access to the 
 store. A more private value like a random number might be better. 



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)


[jira] [Commented] (YARN-1573) ZK store should pick a private password for root-node-acls

2014-01-20 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-1573?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13876919#comment-13876919
 ] 

Hadoop QA commented on YARN-1573:
-

{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12624015/yarn-1573-2.patch
  against trunk revision .

{color:green}+1 @author{color}.  The patch does not contain any @author 
tags.

{color:red}-1 tests included{color}.  The patch doesn't appear to include 
any new or modified tests.
Please justify why no new tests are needed for this 
patch.
Also please list what manual steps were performed to 
verify this patch.

{color:green}+1 javac{color}.  The applied patch does not increase the 
total number of javac compiler warnings.

{color:green}+1 javadoc{color}.  The javadoc tool did not generate any 
warning messages.

{color:green}+1 eclipse:eclipse{color}.  The patch built with 
eclipse:eclipse.

{color:green}+1 findbugs{color}.  The patch does not introduce any new 
Findbugs (version 1.3.9) warnings.

{color:green}+1 release audit{color}.  The applied patch does not increase 
the total number of release audit warnings.

{color:green}+1 core tests{color}.  The patch passed unit tests in 
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common 
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager.

{color:green}+1 contrib tests{color}.  The patch passed contrib unit tests.

Test results: 
https://builds.apache.org/job/PreCommit-YARN-Build/2917//testReport/
Console output: https://builds.apache.org/job/PreCommit-YARN-Build/2917//console

This message is automatically generated.

 ZK store should pick a private password for root-node-acls
 --

 Key: YARN-1573
 URL: https://issues.apache.org/jira/browse/YARN-1573
 Project: Hadoop YARN
  Issue Type: Sub-task
  Components: resourcemanager
Affects Versions: 2.4.0
Reporter: Karthik Kambatla
Assignee: Karthik Kambatla
 Attachments: yarn-1573-1.patch, yarn-1573-2.patch


 Currently, when HA is enabled, ZK store uses cluster-timestamp as the 
 password for root node ACLs to give the Active RM exclusive access to the 
 store. A more private value like a random number might be better. 



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)


[jira] [Commented] (YARN-1573) ZK store should pick a private password for root-node-acls

2014-01-20 Thread Karthik Kambatla (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-1573?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13877003#comment-13877003
 ] 

Karthik Kambatla commented on YARN-1573:


TestZKRMStateStore#testFencing validates that the private password works for 
fencing.

 ZK store should pick a private password for root-node-acls
 --

 Key: YARN-1573
 URL: https://issues.apache.org/jira/browse/YARN-1573
 Project: Hadoop YARN
  Issue Type: Sub-task
  Components: resourcemanager
Affects Versions: 2.4.0
Reporter: Karthik Kambatla
Assignee: Karthik Kambatla
 Attachments: yarn-1573-1.patch, yarn-1573-2.patch


 Currently, when HA is enabled, ZK store uses cluster-timestamp as the 
 password for root node ACLs to give the Active RM exclusive access to the 
 store. A more private value like a random number might be better. 



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)


[jira] [Commented] (YARN-1573) ZK store should pick a private password for root-node-acls

2014-01-17 Thread Sandy Ryza (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-1573?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13874602#comment-13874602
 ] 

Sandy Ryza commented on YARN-1573:
--

Do we ever want someone else (e.g. an administrator) to be able to access the 
store?  Also, not that we need to do the same, but how does HDFS handle this?

 ZK store should pick a private password for root-node-acls
 --

 Key: YARN-1573
 URL: https://issues.apache.org/jira/browse/YARN-1573
 Project: Hadoop YARN
  Issue Type: Sub-task
  Components: resourcemanager
Affects Versions: 2.4.0
Reporter: Karthik Kambatla
Assignee: Karthik Kambatla
 Attachments: yarn-1573-1.patch


 Currently, when HA is enabled, ZK store uses cluster-timestamp as the 
 password for root node ACLs to give the Active RM exclusive access to the 
 store. A more private value like a random number might be better. 



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)


[jira] [Commented] (YARN-1573) ZK store should pick a private password for root-node-acls

2014-01-17 Thread Karthik Kambatla (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-1573?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13875051#comment-13875051
 ] 

Karthik Kambatla commented on YARN-1573:


The private password is only for create-delete perms. The admin still has admin 
access and can reclaim all other kinds of accesses as well.

 ZK store should pick a private password for root-node-acls
 --

 Key: YARN-1573
 URL: https://issues.apache.org/jira/browse/YARN-1573
 Project: Hadoop YARN
  Issue Type: Sub-task
  Components: resourcemanager
Affects Versions: 2.4.0
Reporter: Karthik Kambatla
Assignee: Karthik Kambatla
 Attachments: yarn-1573-1.patch


 Currently, when HA is enabled, ZK store uses cluster-timestamp as the 
 password for root node ACLs to give the Active RM exclusive access to the 
 store. A more private value like a random number might be better. 



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)


[jira] [Commented] (YARN-1573) ZK store should pick a private password for root-node-acls

2014-01-15 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-1573?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13872769#comment-13872769
 ] 

Hadoop QA commented on YARN-1573:
-

{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12623242/yarn-1573-1.patch
  against trunk revision .

{color:green}+1 @author{color}.  The patch does not contain any @author 
tags.

{color:red}-1 tests included{color}.  The patch doesn't appear to include 
any new or modified tests.
Please justify why no new tests are needed for this 
patch.
Also please list what manual steps were performed to 
verify this patch.

{color:green}+1 javac{color}.  The applied patch does not increase the 
total number of javac compiler warnings.

{color:green}+1 javadoc{color}.  The javadoc tool did not generate any 
warning messages.

{color:green}+1 eclipse:eclipse{color}.  The patch built with 
eclipse:eclipse.

{color:red}-1 findbugs{color}.  The patch appears to introduce 1 new 
Findbugs (version 1.3.9) warnings.

{color:green}+1 release audit{color}.  The applied patch does not increase 
the total number of release audit warnings.

{color:red}-1 core tests{color}.  The patch failed these unit tests in 
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common 
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager:

  org.apache.hadoop.yarn.server.resourcemanager.TestRMRestart

{color:green}+1 contrib tests{color}.  The patch passed contrib unit tests.

Test results: 
https://builds.apache.org/job/PreCommit-YARN-Build/2891//testReport/
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-YARN-Build/2891//artifact/trunk/patchprocess/newPatchFindbugsWarningshadoop-yarn-server-resourcemanager.html
Console output: https://builds.apache.org/job/PreCommit-YARN-Build/2891//console

This message is automatically generated.

 ZK store should pick a private password for root-node-acls
 --

 Key: YARN-1573
 URL: https://issues.apache.org/jira/browse/YARN-1573
 Project: Hadoop YARN
  Issue Type: Sub-task
  Components: resourcemanager
Affects Versions: 2.4.0
Reporter: Karthik Kambatla
Assignee: Karthik Kambatla
 Attachments: yarn-1573-1.patch


 Currently, when HA is enabled, ZK store uses cluster-timestamp as the 
 password for root node ACLs to give the Active RM exclusive access to the 
 store. A more private value like a random number might be better. 



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)