Re: Review Request 47256: Atlas Integration : Atlas fails to come up with solr as indexing search when the zookeeper quorum has more than one host

2016-05-12 Thread Nate Cole

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


Ship it!




Ship It!

- Nate Cole


On May 11, 2016, 4:41 p.m., Tom Beerbower wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/47256/
> ---
> 
> (Updated May 11, 2016, 4:41 p.m.)
> 
> 
> Review request for Ambari, John Speidel and Nate Cole.
> 
> 
> Bugs: AMBARI-16464
> https://issues.apache.org/jira/browse/AMBARI-16464
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> Atlas fails to communicate with solr when the zookeeper quorum has more than 
> one host, as a result of this atlas fails to come up.
> 
> The config parameter "atlas.graph.index.search.solr.zookeeper-url" value is 
> set to "zookeeper_quorum/logsearch". Since there are more than one zookeeper 
> servers deployed, this value evaluates to 
> "[os-r6-apathan-atlas-erie-nosec-3.openstacklocal:2181, 
> os-r6-apathan-atlas-erie-nosec-2.openstacklocal:2181/logsearch]".
> 
> The value was set to match the value of the ZK connection string in the 
> logsearch solr-env.sh ...
> # Set the ZooKeeper connection string if using an external ZooKeeper 
> ensemble
> # e.g. host1:2181,host2:2181/chroot
> # Leave empty if not using SolrCloud
> 
> ZK_HOST="c6401.ambari.apache.org:2181,c6402.ambari.apache.org:2181,c6403.ambari.apache.org:2181/logsearch"
> 
> It looks like for atlas.graph.index.search.solr.zookeeper-url it should be ...
> 
> c6401.ambari.apache.org:2181/logsearch,c6402.ambari.apache.org:2181/logsearch,c6403.ambari.apache.org:2181/logsearch
> 
> Also needed to update the atlas user-credential.properites and 
> policy-store.txt to get Atlas to deploy (due to recent changes in Atlas).
> 
> 
> Diffs
> -
> 
>   
> ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/files/policy-store.txt
>  PRE-CREATION 
>   
> ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/files/users-credentials.properties
>  4d264a0 
>   
> ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/metadata.py
>  8e2fcc2 
>   
> ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/params.py
>  51fa039 
>   
> ambari-server/src/main/resources/stacks/HDP/2.5/services/ATLAS/configuration/application-properties.xml
>  78600ee 
>   ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py 
> 3d689d7 
>   ambari-server/src/test/python/stacks/2.3/ATLAS/test_metadata_server.py 
> 8ae2c11 
>   ambari-server/src/test/python/stacks/2.5/ATLAS/test_atlas_server.py 422b1c9 
>   ambari-server/src/test/python/stacks/2.5/common/test_stack_advisor.py 
> e9ddc25 
> 
> Diff: https://reviews.apache.org/r/47256/diff/
> 
> 
> Testing
> ---
> 
> Updated unit tests.
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Tom Beerbower
> 
>



Re: Review Request 47256: Atlas Integration : Atlas fails to come up with solr as indexing search when the zookeeper quorum has more than one host

2016-05-11 Thread Tom Beerbower

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

(Updated May 11, 2016, 8:41 p.m.)


Review request for Ambari, John Speidel and Nate Cole.


Bugs: AMBARI-16464
https://issues.apache.org/jira/browse/AMBARI-16464


Repository: ambari


Description (updated)
---

Atlas fails to communicate with solr when the zookeeper quorum has more than 
one host, as a result of this atlas fails to come up.

The config parameter "atlas.graph.index.search.solr.zookeeper-url" value is set 
to "zookeeper_quorum/logsearch". Since there are more than one zookeeper 
servers deployed, this value evaluates to 
"[os-r6-apathan-atlas-erie-nosec-3.openstacklocal:2181, 
os-r6-apathan-atlas-erie-nosec-2.openstacklocal:2181/logsearch]".

The value was set to match the value of the ZK connection string in the 
logsearch solr-env.sh ...
# Set the ZooKeeper connection string if using an external ZooKeeper 
ensemble
# e.g. host1:2181,host2:2181/chroot
# Leave empty if not using SolrCloud

ZK_HOST="c6401.ambari.apache.org:2181,c6402.ambari.apache.org:2181,c6403.ambari.apache.org:2181/logsearch"

It looks like for atlas.graph.index.search.solr.zookeeper-url it should be ...

c6401.ambari.apache.org:2181/logsearch,c6402.ambari.apache.org:2181/logsearch,c6403.ambari.apache.org:2181/logsearch

Also needed to update the atlas user-credential.properites and policy-store.txt 
to get Atlas to deploy (due to recent changes in Atlas).


Diffs
-

  
ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/files/policy-store.txt
 PRE-CREATION 
  
ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/files/users-credentials.properties
 4d264a0 
  
ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/metadata.py
 8e2fcc2 
  
ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/params.py
 51fa039 
  
ambari-server/src/main/resources/stacks/HDP/2.5/services/ATLAS/configuration/application-properties.xml
 78600ee 
  ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py 
3d689d7 
  ambari-server/src/test/python/stacks/2.3/ATLAS/test_metadata_server.py 
8ae2c11 
  ambari-server/src/test/python/stacks/2.5/ATLAS/test_atlas_server.py 422b1c9 
  ambari-server/src/test/python/stacks/2.5/common/test_stack_advisor.py e9ddc25 

Diff: https://reviews.apache.org/r/47256/diff/


Testing
---

Updated unit tests.

mvn clean test


Thanks,

Tom Beerbower



Review Request 47256: Atlas Integration : Atlas fails to come up with solr as indexing search when the zookeeper quorum has more than one host

2016-05-11 Thread Tom Beerbower

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

Review request for Ambari, John Speidel and Nate Cole.


Bugs: AMBARI-16464
https://issues.apache.org/jira/browse/AMBARI-16464


Repository: ambari


Description
---

Atlas fails to communicate with solr when the zookeeper quorum has more than 
one host, as a result of this atlas fails to come up.

The config parameter "atlas.graph.index.search.solr.zookeeper-url" value is set 
to "zookeeper_quorum/logsearch". Since there are more than one zookeeper 
servers deployed, this value evaluates to 
"[os-r6-apathan-atlas-erie-nosec-3.openstacklocal:2181, 
os-r6-apathan-atlas-erie-nosec-2.openstacklocal:2181/logsearch]".

The value was set to match the value of the ZK connection string in the 
logsearch solr-env.sh ...
# Set the ZooKeeper connection string if using an external ZooKeeper 
ensemble
# e.g. host1:2181,host2:2181/chroot
# Leave empty if not using SolrCloud

ZK_HOST="c6401.ambari.apache.org:2181,c6402.ambari.apache.org:2181,c6403.ambari.apache.org:2181/logsearch"

It looks like for atlas.graph.index.search.solr.zookeeper-url it should be ...

c6401.ambari.apache.org:2181/logsearch,c6402.ambari.apache.org:2181/logsearch,c6403.ambari.apache.org:2181/logsearch


Diffs
-

  
ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/files/policy-store.txt
 PRE-CREATION 
  
ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/files/users-credentials.properties
 4d264a0 
  
ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/metadata.py
 8e2fcc2 
  
ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/params.py
 51fa039 
  
ambari-server/src/main/resources/stacks/HDP/2.5/services/ATLAS/configuration/application-properties.xml
 78600ee 
  ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py 
3d689d7 
  ambari-server/src/test/python/stacks/2.3/ATLAS/test_metadata_server.py 
8ae2c11 
  ambari-server/src/test/python/stacks/2.5/ATLAS/test_atlas_server.py 422b1c9 
  ambari-server/src/test/python/stacks/2.5/common/test_stack_advisor.py e9ddc25 

Diff: https://reviews.apache.org/r/47256/diff/


Testing
---

Updated unit tests.

mvn clean test


Thanks,

Tom Beerbower