You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2021/09/30 03:43:43 UTC

[GitHub] [pulsar] Anonymitaet commented on a change in pull request #12238: [Doc] Add geo-replication authentication doc.

Anonymitaet commented on a change in pull request #12238:
URL: https://github.com/apache/pulsar/pull/12238#discussion_r719034271



##########
File path: site2/docs/administration-geo.md
##########
@@ -83,6 +83,12 @@ $ bin/pulsar-admin clusters create \
 
 3. Run similar commands on `us-east` and `us-cent` to create connections among clusters.
 
+> #### Tip

Review comment:
       Please update content as below:
   
   Line 69:
   
   ```suggestion
   
   > #### Tip
   >
   > - If you want to use a secure connection for a cluster, you can use the flags `--broker-url-secure` and `--url-secure`. For more information, see [pulsar-admin clusters create](https://pulsar.apache.org/tools/pulsar-admin/).
   > - Different clusters may have different authentications. You can use the authentication flag `--auth-plugin` and `--auth-parameters` together to set cluster authentication, which overrides `brokerClientAuthenticationPlugin` and `brokerClientAuthenticationParameters` if `authenticationEnabled` sets to `true` in `broker.conf` and `standalone.conf`. For more information, see [authentication and authorization](concepts-authentication.md).

##########
File path: site2/website-next/versioned_docs/version-2.7.3/administration-geo.md
##########
@@ -91,6 +91,14 @@ $ bin/pulsar-admin clusters create \
 
 3. Run similar commands on `us-east` and `us-cent` to create connections among clusters.
 
+:::tip
+
+Different clusters may have different authentications.
+You can add auth param `--auth-plugin` and `--auth-parameters` to set cluster authentication, which will override `brokerClientAuthenticationPlugin` and `brokerClientAuthenticationParameters` if `authenticationEnabled=true`.\
+For more information about authentication, see [here](concepts-authentication.md).
+
+:::
+

Review comment:
       No need to update docs in `website-next` folder.




-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org