mneethiraj commented on code in PR #1037:
URL: https://github.com/apache/ranger/pull/1037#discussion_r3520737520


##########
dev-support/ranger-docker/scripts/tagsync/ranger-tagsync-install.properties:
##########
@@ -25,28 +25,25 @@ TAG_DEST_RANGER_ENDPOINT = http://ranger.rangernw:6080
 # SSL config file name for HTTPS messages to tag destination - Ranger
 TAG_DEST_RANGER_SSL_CONFIG_FILENAME =
 
+# Atlas Kafka source (bootstrap.servers only; no ZooKeeper). Off by default in 
ranger-docker.
 TAG_SOURCE_ATLAS_ENABLED = false
 
-# Endpoint specifications needed by Atlas
 TAG_SOURCE_ATLAS_KAFKA_OFFSETS_TOPIC_REPLICATION_FACTOR = 1
 TAG_SOURCE_ATLAS_KAFKA_BOOTSTRAP_SERVERS = ranger-kafka.rangernw:9092
-TAG_SOURCE_ATLAS_KAFKA_ZOOKEEPER_CONNECT = ranger-zk.rangernw:2181
 TAG_SOURCE_ATLAS_KAFKA_ENTITIES_GROUP_ID = ranger_entities_consumer
-
-TAG_SOURCE_ATLAS_KAFKA_SERVICE_NAME = kafka
 TAG_SOURCE_ATLAS_KAFKA_SECURITY_PROTOCOL = SASL_PLAINTEXT
+TAG_SOURCE_ATLAS_KAFKA_SERVICE_NAME = kafka
 TAG_SOURCE_ATLAS_KERBEROS_PRINCIPAL = 
rangertagsync/[email protected]
 TAG_SOURCE_ATLAS_KERBEROS_KEYTAB = /etc/keytabs/rangertagsync.keytab
 
-TAG_SOURCE_ATLASREST_ENABLED = false
-
+TAG_SOURCE_ATLASREST_ENABLED = true

Review Comment:
   In docker setup under dev-support, tagsync should not be configured to load 
from rest-source (`TAG_SOURCE_ATLASREST_ENABLED`) as Atlas server is not part 
of this setup. File-source ( `TAG_SOURCE_FILE_ENABLED`) should be enabled; and 
Atlas Kafka notification source (`TAG_SOURCE_ATLAS_ENABLED`) can also be 
enabled. Please review and update.
   
   



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to