Skip to content

Conversation

@Machos65
Copy link

@Machos65 Machos65 commented May 8, 2025

Description

Please provide a detailed description of the changes you have made. Include the motivation for these changes, and any
additional context that may be important.

the PR should include helm chart changes and operator changes.

Related Issue(s)

Please list any related issues and link them here.

Checklist

For operator, please complete the following checklist:

  • run make generate to generate the code.
  • run golangci-lint run to check the code style.
  • run make test to run UT.
  • run make manifests to update the yaml files of CRD.

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).

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@yandongxiao
Copy link
Collaborator

Why do you want to modify the number of replicas and specifications ?

Copy link

@Gunnnn Gunnnn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great resource economy! Thanks a lot!

@yandongxiao
Copy link
Collaborator

I can understand the problem you encountered. In a personal testing environment, we rarely need resources of such a large scale. However, in the examples/starrocks directory, we use the minimum specifications that can ensure the online operation of StarRocks. For example, the minimum number of FEs is 3, and it must be an odd number.

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.

4 participants