Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add_be_host_mapping_list_config #114

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

ico01
Copy link

@ico01 ico01 commented Jun 12, 2024

What type of PR is this:

  • BugFix
  • Feature
  • Enhancement
  • Refactor
  • UT
  • Doc
  • Tool

Which issues of this PR fixes :

Fixes #

Problem Summary(Required) :

Usage Note for StarRocks BE Host Mapping Configuration on Kubernetes

Overview:

When running Starrocks on Kubernetes environments, the internal service names of StarRocks backend (BE) nodes might differ from their corresponding external IPs or domain names. To facilitate seamless connectivity, we support configuring a mapping between these internal service names and their external counterparts using the starrocks.be.host.mapping.list configuration.

Configuration:

The starrocks.be.host.mapping.list property allows users to specify a list of mappings between the external IPs or domain names and internal service names of StarRocks BE nodes.

Format:

<external_ip>:<port>,<internal_service_name>:<port>;...

Example:

For instance, suppose the following mappings are required:

  • External IP 20.11.12.13:9060 maps to internal service name starrocks-cdc-cn-0.starrocks-cdc-cn-search.starrocks.svc.cluster.local:9060.
  • External IP 21.11.12.13:9060 maps to internal service name starrocks-cdc-cn-1.starrocks-cdc-cn-search.starrocks.svc.cluster.local:9060.

You can specify these mappings as:

starrocks.be.host.mapping.list = "20.11.12.13:9060,starrocks-cdc-cn-0.starrocks-cdc-cn-search.starrocks.svc.cluster.local:9060;21.11.12.13:9060,starrocks-cdc-cn-1.starrocks-cdc-cn-search.starrocks.svc.cluster.local:9060"

Checklist:

  • I have added test cases for my bug fix or my new feature
  • This pr will affect users' behaviors
  • This pr needs user documentation (for new or modified features or behaviors)
  • I have added documentation for my new feature or new function

@CLAassistant
Copy link

CLAassistant commented Jun 12, 2024

CLA assistant check
All committers have signed the CLA.

@ico01 ico01 force-pushed the add_be_host_mapping_list_config branch 2 times, most recently from a0bef77 to ce86cc1 Compare June 12, 2024 10:30
@ico01 ico01 force-pushed the add_be_host_mapping_list_config branch from be265d9 to 93997f4 Compare June 12, 2024 10:42
@ico01
Copy link
Author

ico01 commented Jun 12, 2024

@banmoy @wyb i need review

@ico01
Copy link
Author

ico01 commented Jun 13, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants