Skip to content

[BugFix] Ensure query port can be configured in init-password job #455

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

Conversation

yandongxiao
Copy link
Collaborator

Description

A scenario that requires modifying the container listening query port:

  1. The source cluster is deployed on virtual machines, while the target cluster is deployed on k8s. External access to the internal IP of k8s is not possible, but pods within k8s can access external IPs.
  2. The migration tool is deployed outside of k8s. The StarRocks cluster on k8s is exposed through /etc/host (frontend domain name, node IP of k8s) + NodePort.
  3. In the cross-cluster migration tool, use the port obtained from Show Frontend for recording in Frontend. At the same time, use this port to access the target cluster (instead of configuring target_fe_query_port).
  4. The target cluster pulls data from the source cluster.

Checklist

For helm chart, please complete the following checklist:

  • make sure you have updated the values.yaml
    file of starrocks chart.
  • In scripts directory, run bash create-parent-chart-values.sh to update the values.yaml file of the parent
    chart( kube-starrocks chart).

@yandongxiao
Copy link
Collaborator Author

Tested three scenarios:

  1. Configuration does not include the query port setting.
    1. Configuration includes the query port setting.
  2. The query port setting in the configuration is commented out.
    All three scenarios meet expectations.

@yandongxiao yandongxiao merged commit 9aac3e3 into StarRocks:main Feb 23, 2024
@yandongxiao yandongxiao added v1.9.3 bugfix fix something that does not work labels Mar 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix fix something that does not work v1.9.3
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant