> On Июнь 8, 2017, 4:47 п.п., Alejandro Fernandez wrote:
> > ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/resourcemanager.py
> > Lines 188 (patched)
> > <https://reviews.apache.org/r/59920/diff/1/?file=1744632#file1744632line189>
> >
> >     ZK Client is not guaranteed to be on this host.
> >     For that reason, we need an alternate solution as part of disabling 
> > security instead of code that runs on RM.

I missed this somehow. Fixed in new patch


- Dmytro


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59920/#review177326
-----------------------------------------------------------


On Июнь 9, 2017, 11:59 д.п., Dmytro Sen wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/59920/
> -----------------------------------------------------------
> 
> (Updated Июнь 9, 2017, 11:59 д.п.)
> 
> 
> Review request for Ambari, Alejandro Fernandez, Sumit Mohanty, Sid Wagle, and 
> Vitalyi Brodetskyi.
> 
> 
> Bugs: AMBARI-21204
>     https://issues.apache.org/jira/browse/AMBARI-21204
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> From RM logs :
> 2017-06-07 14:23:19,191 FATAL resourcemanager.ResourceManager 
> (ResourceManager.java:main(1240)) - Error starting ResourceManager
> org.apache.hadoop.service.ServiceStateException: java.io.IOException: 
> Couldn't set ACLs on parent ZNode: /yarn-leader-election
>         at 
> org.apache.hadoop.service.ServiceStateException.convert(ServiceStateException.java:59)
>         at 
> org.apache.hadoop.service.AbstractService.init(AbstractService.java:172)
>         at 
> org.apache.hadoop.service.CompositeService.serviceInit(CompositeService.java:107)
>         at 
> org.apache.hadoop.yarn.server.resourcemanager.AdminService.serviceInit(AdminService.java:152)
>         at 
> org.apache.hadoop.service.AbstractService.init(AbstractService.java:163)
>         at 
> org.apache.hadoop.service.CompositeService.serviceInit(CompositeService.java:107)
>         at 
> org.apache.hadoop.yarn.server.resourcemanager.ResourceManager.serviceInit(ResourceManager.java:281)
>         at 
> org.apache.hadoop.service.AbstractService.init(AbstractService.java:163)
>         at 
> org.apache.hadoop.yarn.server.resourcemanager.ResourceManager.main(ResourceManager.java:1236)
> Caused by: java.io.IOException: Couldn't set ACLs on parent ZNode: 
> /yarn-leader-election
>         at 
> org.apache.hadoop.ha.ActiveStandbyElector.ensureParentZNode(ActiveStandbyElector.java:351)
>         at 
> org.apache.hadoop.yarn.server.resourcemanager.EmbeddedElectorService.serviceInit(EmbeddedElectorService.java:103)
>         at 
> org.apache.hadoop.service.AbstractService.init(AbstractService.java:163)
>         ... 7 more
> Caused by: org.apache.zookeeper.KeeperException$BadVersionException: 
> KeeperErrorCode = BadVersion for /yarn-leader-election
> 
> 
> The problem is that disabling security changes zk ACL for resource manager as 
> part of AMBARI-19331. After the recent change in HDFS-11403, RM checks znode 
> version and fails if it's different than expected.
> The correct fix could be to remove znode during security disabling and do not 
> break election znode consistency by manually changing ACL to all. RM should 
> create it with proper ACL.
> 
> 
> Diffs
> -----
> 
>   ambari-agent/src/main/java/org/apache/ambari/tools/zk/ZkMigrator.java 
> c100b85 
>   ambari-agent/src/test/java/org/apache/ambari/tools/zk/ZkMigratorTest.java 
> b7e9c0c 
>   
> ambari-common/src/main/python/resource_management/core/resources/zkmigrator.py
>  344b9a4 
>   
> ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/resourcemanager.py
>  81b99e6 
>   
> ambari-server/src/main/resources/common-services/YARN/3.0.0.3.0/package/scripts/resourcemanager.py
>  78675bf 
> 
> 
> Diff: https://reviews.apache.org/r/59920/diff/2/
> 
> 
> Testing
> -------
> 
> Unit tests passed
> 
> 
> Thanks,
> 
> Dmytro Sen
> 
>

Reply via email to